Yes that looks better.
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Samer
> El-Haj-Mahmoud
> Sent: Thursday, May 14, 2020 11:55 PM
> To: Ni, Ray <ray...@intel.com>; devel@edk2.groups.io; Kinney, Michael D
> <michael.d.kin...@intel.com>
> Cc: Leif Lindholm <l...@nuviainc.com>; Ard Biesheuvel <ard.biesheu...@arm.com>
> Subject: Re: [edk2-devel] [edk2-platform][PATCH v1 2/4] Drivers/OptionRomPkg:
> Move ASIX drivers to AsixPkg
>
> Any consensus on this? I can just rename AsixPkg.dsc/fdf to Asix.dsc/fdf
> (with a matching PLATFORM_NAME ) if that is
> more consistent
>
> > -----Original Message-----
> > From: Ni, Ray <ray...@intel.com>
> > Sent: Monday, May 11, 2020 1:18 AM
> > To: Samer El-Haj-Mahmoud <samer.el-haj-mahm...@arm.com>;
> > devel@edk2.groups.io; Kinney, Michael D <michael.d.kin...@intel.com>
> > Cc: Leif Lindholm <l...@nuviainc.com>; Ard Biesheuvel
> > <ard.biesheu...@arm.com>
> > Subject: RE: [edk2-platform][PATCH v1 2/4] Drivers/OptionRomPkg: Move
> > ASIX drivers to AsixPkg
> >
> > Mike,
> > Is there a rule that a package name should end with "Pkg"?
> > I thought we have but it sounds to me many packages in Silicon directory
> > don't follow.
> >
> > In this case:
> > 1. "ASIX" folder holds the package content.
> > 2. Pkg dsc and dec file are named as "AsixPkg".
> > 3. PLATFORM_NAME is defined as "AsixPkg" in DSC file .
> >
> > Thanks,
> > Ray
> >
> > > -----Original Message-----
> > > From: Samer El-Haj-Mahmoud <samer.el-haj-mahm...@arm.com>
> > > Sent: Sunday, May 10, 2020 7:15 PM
> > > To: devel@edk2.groups.io
> > > Cc: Ni, Ray <ray...@intel.com>; Leif Lindholm <l...@nuviainc.com>; Ard
> > > Biesheuvel <ard.biesheu...@arm.com>; Kinney, Michael D
> > > <michael.d.kin...@intel.com>
> > > Subject: [edk2-platform][PATCH v1 2/4] Drivers/OptionRomPkg: Move
> > ASIX
> > > drivers to AsixPkg
> > >
> > > Move ASIX Ax88772 and Ax88772b USB networking drivers to the newly
> > > created Silicon/AsixPkg/.
> > >
> > > Cc: Ray Ni <ray...@intel.com>
> > > Cc: Leif Lindholm <l...@nuviainc.com>
> > > Cc: Ard Biesheuvel <ard.biesheu...@arm.com>
> > > Cc: Michael D Kinney <michael.d.kin...@intel.com>
> > >
> > > Signed-off-by: Samer El-Haj-Mahmoud <samer.el-haj-
> > mahm...@arm.com>
> > > ---
> > > Drivers/OptionRomPkg/OptionRomPkg.dsc
> > > |
> > > 2 --
> > > Platform/Hisilicon/HiKey/HiKey.dsc
> > > | 2 +-
> > > Platform/RaspberryPi/RPi3/RPi3.dsc
> > > | 2 +-
> > > Platform/RaspberryPi/RPi4/RPi4.dsc
> > > | 2 +-
> > > Silicon/ASIX/AsixPkg.dsc
> > > | 3 ++-
> > > Platform/Hisilicon/HiKey/HiKey.fdf
> > > | 2 +-
> > > Platform/RaspberryPi/RPi3/RPi3.fdf
> > > | 2 +-
> > > Platform/RaspberryPi/RPi4/RPi4.fdf
> > > | 2 +-
> > > {Drivers/OptionRomPkg =>
> > > Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772/Ax88772.inf | 0
> > > {Drivers/OptionRomPkg =>
> > > Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772b/Ax88772b.inf |
> > 0
> > > {Drivers/OptionRomPkg =>
> > > Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772/Ax88772.h | 0
> > > {Drivers/OptionRomPkg =>
> > > Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772b/Ax88772.h | 0
> > > {Drivers/OptionRomPkg =>
> > > Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772/Ax88772.c | 0
> > > {Drivers/OptionRomPkg =>
> > >
> > Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772/ComponentName.c
> > |
> > > 0 {Drivers/OptionRomPkg =>
> > > Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772/DriverBinding.c |
> > > 0 {Drivers/OptionRomPkg =>
> > > Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772/SimpleNetwork.c
> > |
> > > 0 {Drivers/OptionRomPkg =>
> > > Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772b/Ax88772.c | 0
> > > {Drivers/OptionRomPkg =>
> > >
> > Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772b/ComponentName
> > .c |
> > > 0
> > > {Drivers/OptionRomPkg =>
> > > Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772b/DriverBinding.c |
> > > 0 {Drivers/OptionRomPkg =>
> > >
> > Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772b/SimpleNetwork.c
> > |
> > > 0
> > > 20 files changed, 8 insertions(+), 9 deletions(-)
> > >
> > > diff --git a/Drivers/OptionRomPkg/OptionRomPkg.dsc
> > > b/Drivers/OptionRomPkg/OptionRomPkg.dsc
> > > index bea64b585ec6..153f47de5be1 100644
> > > --- a/Drivers/OptionRomPkg/OptionRomPkg.dsc
> > > +++ b/Drivers/OptionRomPkg/OptionRomPkg.dsc
> > > @@ -106,8 +106,6 @@ [Components]
> > > OptionRomPkg/CirrusLogic5430Dxe/CirrusLogic5430Dxe.inf
> > > OptionRomPkg/UndiRuntimeDxe/UndiRuntimeDxe.inf
> > > OptionRomPkg/Bus/Usb/FtdiUsbSerialDxe/FtdiUsbSerialDxe.inf
> > > - OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772/Ax88772.inf
> > > - OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/Ax88772b.inf
> > >
> > > [Components.IA32, Components.X64]
> > > OptionRomPkg/Application/BltLibSample/BltLibSample.inf
> > > diff --git a/Platform/Hisilicon/HiKey/HiKey.dsc
> > > b/Platform/Hisilicon/HiKey/HiKey.dsc
> > > index 600f19a30762..88674d4564d0 100644
> > > --- a/Platform/Hisilicon/HiKey/HiKey.dsc
> > > +++ b/Platform/Hisilicon/HiKey/HiKey.dsc
> > > @@ -247,7 +247,7 @@ [Components.common]
> > > #
> > > # AX88772 Ethernet Driver
> > > #
> > > -
> > Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/Ax88772b.inf
> > > + Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772b/Ax88772b.inf
> > >
> > > #
> > > # FAT filesystem + GPT/MBR partitioning diff --git
> > > a/Platform/RaspberryPi/RPi3/RPi3.dsc
> > > b/Platform/RaspberryPi/RPi3/RPi3.dsc
> > > index bf645110eb61..0f736fa80e91 100644
> > > --- a/Platform/RaspberryPi/RPi3/RPi3.dsc
> > > +++ b/Platform/RaspberryPi/RPi3/RPi3.dsc
> > > @@ -648,7 +648,7 @@ [Components.common]
> > > MdeModulePkg/Bus/Usb/UsbBusDxe/UsbBusDxe.inf
> > > MdeModulePkg/Bus/Usb/UsbKbDxe/UsbKbDxe.inf
> > > MdeModulePkg/Bus/Usb/UsbMassStorageDxe/UsbMassStorageDxe.inf
> > > -
> > Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/Ax88772b.inf
> > > + Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772b/Ax88772b.inf
> > >
> > > #
> > > # SD/MMC support
> > > diff --git a/Platform/RaspberryPi/RPi4/RPi4.dsc
> > > b/Platform/RaspberryPi/RPi4/RPi4.dsc
> > > index 0d54fa477ca2..064ca762c650 100644
> > > --- a/Platform/RaspberryPi/RPi4/RPi4.dsc
> > > +++ b/Platform/RaspberryPi/RPi4/RPi4.dsc
> > > @@ -661,7 +661,7 @@ [Components.common]
> > > MdeModulePkg/Bus/Usb/UsbBusDxe/UsbBusDxe.inf
> > > MdeModulePkg/Bus/Usb/UsbKbDxe/UsbKbDxe.inf
> > > MdeModulePkg/Bus/Usb/UsbMassStorageDxe/UsbMassStorageDxe.inf
> > > -
> > Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/Ax88772b.inf
> > > + Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772b/Ax88772b.inf
> > >
> > > #
> > > # SD/MMC support
> > > diff --git a/Silicon/ASIX/AsixPkg.dsc b/Silicon/ASIX/AsixPkg.dsc index
> > > d200bb7848c6..0e4ffd0f1743 100644
> > > --- a/Silicon/ASIX/AsixPkg.dsc
> > > +++ b/Silicon/ASIX/AsixPkg.dsc
> > > @@ -64,4 +64,5 @@ [PcdsFixedAtBuild]
> > > #
> > >
> > >
> > ##########################################################
> > ######
> > > ################
> > > [Components]
> > > -
> > > +Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772/Ax88772.inf
> > > +Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772b/Ax88772b.inf
> > > diff --git a/Platform/Hisilicon/HiKey/HiKey.fdf
> > > b/Platform/Hisilicon/HiKey/HiKey.fdf
> > > index 657e6c8ef2ec..10bbe62ef121 100644
> > > --- a/Platform/Hisilicon/HiKey/HiKey.fdf
> > > +++ b/Platform/Hisilicon/HiKey/HiKey.fdf
> > > @@ -158,7 +158,7 @@ [FV.FvMain]
> > > #
> > > # AX88772 Ethernet Driver for Apple Ethernet Adapter
> > > #
> > > - INF
> > > Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/Ax88772b.inf
> > > + INF
> > > + Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772b/Ax88772b.inf
> > >
> > > #
> > > # FAT filesystem + GPT/MBR partitioning diff --git
> > > a/Platform/RaspberryPi/RPi3/RPi3.fdf
> > > b/Platform/RaspberryPi/RPi3/RPi3.fdf
> > > index e854cd214153..c2f60cf93022 100644
> > > --- a/Platform/RaspberryPi/RPi3/RPi3.fdf
> > > +++ b/Platform/RaspberryPi/RPi3/RPi3.fdf
> > > @@ -289,7 +289,7 @@ [FV.FvMain]
> > > INF MdeModulePkg/Bus/Usb/UsbBusDxe/UsbBusDxe.inf
> > > INF MdeModulePkg/Bus/Usb/UsbKbDxe/UsbKbDxe.inf
> > > INF
> > MdeModulePkg/Bus/Usb/UsbMassStorageDxe/UsbMassStorageDxe.inf
> > > - INF
> > > Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/Ax88772b.inf
> > > + INF
> > > + Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772b/Ax88772b.inf
> > >
> > > #
> > > # SD/MMC support
> > > diff --git a/Platform/RaspberryPi/RPi4/RPi4.fdf
> > > b/Platform/RaspberryPi/RPi4/RPi4.fdf
> > > index b1f7aa2387df..e6e039434569 100644
> > > --- a/Platform/RaspberryPi/RPi4/RPi4.fdf
> > > +++ b/Platform/RaspberryPi/RPi4/RPi4.fdf
> > > @@ -293,7 +293,7 @@ [FV.FvMain]
> > > INF MdeModulePkg/Bus/Usb/UsbBusDxe/UsbBusDxe.inf
> > > INF MdeModulePkg/Bus/Usb/UsbKbDxe/UsbKbDxe.inf
> > > INF
> > MdeModulePkg/Bus/Usb/UsbMassStorageDxe/UsbMassStorageDxe.inf
> > > - INF
> > > Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/Ax88772b.inf
> > > + INF
> > > + Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772b/Ax88772b.inf
> > >
> > > #
> > > # SD/MMC support
> > > diff --git
> > > a/Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772/Ax88772.inf
> > > b/Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772/Ax88772.inf
> > > similarity index 100%
> > > rename from
> > > Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772/Ax88772.inf
> > > rename to
> > > Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772/Ax88772.inf
> > > diff --git
> > >
> > a/Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/Ax88772b.inf
> > > b/Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772b/Ax88772b.inf
> > > similarity index 100%
> > > rename from
> > > Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/Ax88772b.inf
> > > rename to
> > > Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772b/Ax88772b.inf
> > > diff --git
> > > a/Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772/Ax88772.h
> > > b/Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772/Ax88772.h
> > > similarity index 100%
> > > rename from
> > > Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772/Ax88772.h
> > > rename to
> > Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772/Ax88772.h
> > > diff --git
> > > a/Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/Ax88772.h
> > > b/Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772b/Ax88772.h
> > > similarity index 100%
> > > rename from
> > > Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/Ax88772.h
> > > rename to
> > > Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772b/Ax88772.h
> > > diff --git
> > > a/Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772/Ax88772.c
> > > b/Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772/Ax88772.c
> > > similarity index 100%
> > > rename from
> > > Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772/Ax88772.c
> > > rename to
> > Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772/Ax88772.c
> > > diff --git
> > >
> > a/Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772/ComponentNa
> > me.
> > > c
> > b/Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772/ComponentName
> > .c
> > > similarity index 100%
> > > rename from
> > >
> > Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772/ComponentNam
> > e.c
> > > rename to
> > >
> > Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772/ComponentName.c
> > > diff --git
> > >
> > a/Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772/DriverBinding.c
> > > b/Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772/DriverBinding.c
> > > similarity index 100%
> > > rename from
> > > Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772/DriverBinding.c
> > > rename to
> > > Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772/DriverBinding.c
> > > diff --git
> > >
> > a/Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772/SimpleNetwor
> > k.c
> > >
> > b/Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772/SimpleNetwork.c
> > > similarity index 100%
> > > rename from
> > >
> > Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772/SimpleNetwork.c
> > > rename to
> > > Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772/SimpleNetwork.c
> > > diff --git
> > > a/Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/Ax88772.c
> > > b/Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772b/Ax88772.c
> > > similarity index 100%
> > > rename from
> > > Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/Ax88772.c
> > > rename to
> > > Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772b/Ax88772.c
> > > diff --git
> > >
> > a/Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/ComponentN
> > ame
> > > .c
> > >
> > b/Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772b/ComponentNam
> > e.c
> > > similarity index 100%
> > > rename from
> > >
> > Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/ComponentNa
> > me.c
> > > rename to
> > >
> > Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772b/ComponentName.
> > c
> > > diff --git
> > >
> > a/Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/DriverBinding
> > .c
> > > b/Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772b/DriverBinding.c
> > > similarity index 100%
> > > rename from
> > >
> > Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/DriverBinding.c
> > > rename to
> > > Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772b/DriverBinding.c
> > > diff --git
> > >
> > a/Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/SimpleNetwo
> > rk.c
> > >
> > b/Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772b/SimpleNetwork.
> > c
> > > similarity index 100%
> > > rename from
> > >
> > Drivers/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/SimpleNetwork
> > .c
> > > rename to
> > > Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772b/SimpleNetwork.c
> > > --
> > > 2.17.1
>
> IMPORTANT NOTICE: The contents of this email and any attachments are
> confidential and may also be privileged. If you
> are not the intended recipient, please notify the sender immediately and do
> not disclose the contents to any other person,
> use it for any purpose, or store or copy the information in any medium. Thank
> you.
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#59649): https://edk2.groups.io/g/devel/message/59649
Mute This Topic: https://groups.io/mt/74113378/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
