[edk2-devel] [edk2-platform][PATCH v1 0/4] Update ASIX USB Networking drivers

Samer El-Haj-Mahmoud posted 4 patches 3 years, 10 months ago
Failed in applying to current master (apply log)
There is a newer version of this series
Silicon/ASIX/AsixPkg.dec                      |   34 +
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                      |   70 +
Platform/Hisilicon/HiKey/HiKey.fdf            |    2 +-
Platform/RaspberryPi/RPi3/RPi3.fdf            |    2 +-
Platform/RaspberryPi/RPi4/RPi4.fdf            |    2 +-
.../Bus/Usb/UsbNetworking/Ax88179/Ax88179.inf |   66 +
.../Bus/Usb/UsbNetworking/Ax88772/Ax88772.inf |    0
.../Usb/UsbNetworking/Ax88772b/Ax88772b.inf   |    0
.../Usb/UsbNetworking/Ax88772c/Ax88772c.inf   |   65 +
.../Bus/Usb/UsbNetworking/Ax88179/Ax88179.h   | 1127 ++++++++++++
.../Bus/Usb/UsbNetworking/Ax88772/Ax88772.h   |    0
.../Bus/Usb/UsbNetworking/Ax88772b/Ax88772.h  |    0
.../Bus/Usb/UsbNetworking/Ax88772c/Ax88772.h  | 1097 +++++++++++
.../Bus/Usb/UsbNetworking/Ax88179/Ax88179.c   | 1026 +++++++++++
.../Usb/UsbNetworking/Ax88179/ComponentName.c |  290 +++
.../Usb/UsbNetworking/Ax88179/DriverBinding.c |  620 +++++++
.../Usb/UsbNetworking/Ax88179/SimpleNetwork.c | 1585 ++++++++++++++++
.../Bus/Usb/UsbNetworking/Ax88772/Ax88772.c   |    0
.../Usb/UsbNetworking/Ax88772/ComponentName.c |    0
.../Usb/UsbNetworking/Ax88772/DriverBinding.c |    0
.../Usb/UsbNetworking/Ax88772/SimpleNetwork.c |    0
.../Bus/Usb/UsbNetworking/Ax88772b/Ax88772.c  |    0
.../UsbNetworking/Ax88772b/ComponentName.c    |    0
.../UsbNetworking/Ax88772b/DriverBinding.c    |    0
.../UsbNetworking/Ax88772b/SimpleNetwork.c    |    0
.../Bus/Usb/UsbNetworking/Ax88772c/Ax88772.c  | 1296 +++++++++++++
.../UsbNetworking/Ax88772c/ComponentName.c    |  246 +++
.../UsbNetworking/Ax88772c/DriverBinding.c    |  637 +++++++
.../UsbNetworking/Ax88772c/SimpleNetwork.c    | 1612 +++++++++++++++++
33 files changed, 9777 insertions(+), 8 deletions(-)
create mode 100644 Silicon/ASIX/AsixPkg.dec
create mode 100644 Silicon/ASIX/AsixPkg.dsc
create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88179/Ax88179.inf
rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772/Ax88772.inf (100%)
rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772b/Ax88772b.inf (100%)
create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772c/Ax88772c.inf
create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88179/Ax88179.h
rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772/Ax88772.h (100%)
rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772b/Ax88772.h (100%)
create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772c/Ax88772.h
create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88179/Ax88179.c
create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88179/ComponentName.c
create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88179/DriverBinding.c
create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88179/SimpleNetwork.c
rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772/Ax88772.c (100%)
rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772/ComponentName.c (100%)
rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772/DriverBinding.c (100%)
rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772/SimpleNetwork.c (100%)
rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772b/Ax88772.c (100%)
rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772b/ComponentName.c (100%)
rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772b/DriverBinding.c (100%)
rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772b/SimpleNetwork.c (100%)
create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772c/Ax88772.c
create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772c/ComponentName.c
create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772c/DriverBinding.c
create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772c/SimpleNetwork.c
[edk2-devel] [edk2-platform][PATCH v1 0/4] Update ASIX USB Networking drivers
Posted by Samer El-Haj-Mahmoud 3 years, 10 months ago
This series updates the ASIX USB networking drivers:

 - Create new Silicon/ASIX/ package
 - Move existing Ax88772 and Ax88772b drivers from OptionRomPkg
   to the newly created AsixPkg
 - Add new ASIX drivers Ax88179 and Ax88772c

Code pushed to https://github.com/samerhaj/edk2-platforms/tree/asix_usb_nic_drivers_v2

version 1: https://edk2.groups.io/g/devel/message/58771
version 2 changes:
 - Create new ASIX Package to host existing and new ASIX drivers[Ray]

Cc: Ray Ni <ray.ni@intel.com>
Cc: Leif Lindholm <leif@nuviainc.com>
Cc: Ard Biesheuvel <ard.biesheuvel@arm.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>

Signed-off-by: Samer El-Haj-Mahmoud <samer.el-haj-mahmoud@arm.com>

Samer El-Haj-Mahmoud (4):
  Silicon/AsixPkg: Create AsixPkg
  Drivers/OptionRomPkg: Move ASIX drivers to AsixPkg
  Silicon/AsixPkg: Add ASIX Ax88179 driver
  Silicon/AsixPkg: Add ASIX Ax88772c driver

 Silicon/ASIX/AsixPkg.dec                      |   34 +
 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                      |   70 +
 Platform/Hisilicon/HiKey/HiKey.fdf            |    2 +-
 Platform/RaspberryPi/RPi3/RPi3.fdf            |    2 +-
 Platform/RaspberryPi/RPi4/RPi4.fdf            |    2 +-
 .../Bus/Usb/UsbNetworking/Ax88179/Ax88179.inf |   66 +
 .../Bus/Usb/UsbNetworking/Ax88772/Ax88772.inf |    0
 .../Usb/UsbNetworking/Ax88772b/Ax88772b.inf   |    0
 .../Usb/UsbNetworking/Ax88772c/Ax88772c.inf   |   65 +
 .../Bus/Usb/UsbNetworking/Ax88179/Ax88179.h   | 1127 ++++++++++++
 .../Bus/Usb/UsbNetworking/Ax88772/Ax88772.h   |    0
 .../Bus/Usb/UsbNetworking/Ax88772b/Ax88772.h  |    0
 .../Bus/Usb/UsbNetworking/Ax88772c/Ax88772.h  | 1097 +++++++++++
 .../Bus/Usb/UsbNetworking/Ax88179/Ax88179.c   | 1026 +++++++++++
 .../Usb/UsbNetworking/Ax88179/ComponentName.c |  290 +++
 .../Usb/UsbNetworking/Ax88179/DriverBinding.c |  620 +++++++
 .../Usb/UsbNetworking/Ax88179/SimpleNetwork.c | 1585 ++++++++++++++++
 .../Bus/Usb/UsbNetworking/Ax88772/Ax88772.c   |    0
 .../Usb/UsbNetworking/Ax88772/ComponentName.c |    0
 .../Usb/UsbNetworking/Ax88772/DriverBinding.c |    0
 .../Usb/UsbNetworking/Ax88772/SimpleNetwork.c |    0
 .../Bus/Usb/UsbNetworking/Ax88772b/Ax88772.c  |    0
 .../UsbNetworking/Ax88772b/ComponentName.c    |    0
 .../UsbNetworking/Ax88772b/DriverBinding.c    |    0
 .../UsbNetworking/Ax88772b/SimpleNetwork.c    |    0
 .../Bus/Usb/UsbNetworking/Ax88772c/Ax88772.c  | 1296 +++++++++++++
 .../UsbNetworking/Ax88772c/ComponentName.c    |  246 +++
 .../UsbNetworking/Ax88772c/DriverBinding.c    |  637 +++++++
 .../UsbNetworking/Ax88772c/SimpleNetwork.c    | 1612 +++++++++++++++++
 33 files changed, 9777 insertions(+), 8 deletions(-)
 create mode 100644 Silicon/ASIX/AsixPkg.dec
 create mode 100644 Silicon/ASIX/AsixPkg.dsc
 create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88179/Ax88179.inf
 rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772/Ax88772.inf (100%)
 rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772b/Ax88772b.inf (100%)
 create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772c/Ax88772c.inf
 create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88179/Ax88179.h
 rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772/Ax88772.h (100%)
 rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772b/Ax88772.h (100%)
 create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772c/Ax88772.h
 create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88179/Ax88179.c
 create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88179/ComponentName.c
 create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88179/DriverBinding.c
 create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88179/SimpleNetwork.c
 rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772/Ax88772.c (100%)
 rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772/ComponentName.c (100%)
 rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772/DriverBinding.c (100%)
 rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772/SimpleNetwork.c (100%)
 rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772b/Ax88772.c (100%)
 rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772b/ComponentName.c (100%)
 rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772b/DriverBinding.c (100%)
 rename {Drivers/OptionRomPkg => Silicon/ASIX/Drivers}/Bus/Usb/UsbNetworking/Ax88772b/SimpleNetwork.c (100%)
 create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772c/Ax88772.c
 create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772c/ComponentName.c
 create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772c/DriverBinding.c
 create mode 100644 Silicon/ASIX/Drivers/Bus/Usb/UsbNetworking/Ax88772c/SimpleNetwork.c

-- 
2.17.1


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#58986): https://edk2.groups.io/g/devel/message/58986
Mute This Topic: https://groups.io/mt/74113376/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [edk2-platform][PATCH v1 0/4] Update ASIX USB Networking drivers
Posted by Ard Biesheuvel 3 years, 10 months ago
On 5/10/20 1:15 PM, Samer El-Haj-Mahmoud wrote:
> This series updates the ASIX USB networking drivers:
> 
>   - Create new Silicon/ASIX/ package
>   - Move existing Ax88772 and Ax88772b drivers from OptionRomPkg
>     to the newly created AsixPkg
>   - Add new ASIX drivers Ax88179 and Ax88772c
> 
> Code pushed to https://github.com/samerhaj/edk2-platforms/tree/asix_usb_nic_drivers_v2
> 
> version 1: https://edk2.groups.io/g/devel/message/58771
> version 2 changes:
>   - Create new ASIX Package to host existing and new ASIX drivers[Ray]
> 
> Cc: Ray Ni <ray.ni@intel.com>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Cc: Ard Biesheuvel <ard.biesheuvel@arm.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> 
> Signed-off-by: Samer El-Haj-Mahmoud <samer.el-haj-mahmoud@arm.com>
> 
> Samer El-Haj-Mahmoud (4):
>    Silicon/AsixPkg: Create AsixPkg
>    Drivers/OptionRomPkg: Move ASIX drivers to AsixPkg
>    Silicon/AsixPkg: Add ASIX Ax88179 driver
>    Silicon/AsixPkg: Add ASIX Ax88772c driver
> 

I happened to have one of each of these dongles in my drawer, so I 
managed to test them both.

Both work fine, although the 88179 one was rather slow.

In any case, the 10/100 one I tried is an Apple one (05ac:1402) which is 
also supported by the 88772b driver. The only other vid/pid pair which 
that old driver supports is also supported by the new one, so I think we 
can drop the old one altogether. The non-b 7720 driver is also supported 
by the new 10/100 driver, so we can drop the old 88772 as well, afaict.






-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#59128): https://edk2.groups.io/g/devel/message/59128
Mute This Topic: https://groups.io/mt/74113376/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [edk2-platform][PATCH v1 0/4] Update ASIX USB Networking drivers
Posted by Leif Lindholm 3 years, 10 months ago
On Mon, May 11, 2020 at 17:50:54 +0200, Ard Biesheuvel wrote:
> On 5/10/20 1:15 PM, Samer El-Haj-Mahmoud wrote:
> > This series updates the ASIX USB networking drivers:
> > 
> >   - Create new Silicon/ASIX/ package
> >   - Move existing Ax88772 and Ax88772b drivers from OptionRomPkg
> >     to the newly created AsixPkg
> >   - Add new ASIX drivers Ax88179 and Ax88772c
> > 
> > Code pushed to https://github.com/samerhaj/edk2-platforms/tree/asix_usb_nic_drivers_v2
> > 
> > version 1: https://edk2.groups.io/g/devel/message/58771
> > version 2 changes:
> >   - Create new ASIX Package to host existing and new ASIX drivers[Ray]
> > 
> > Cc: Ray Ni <ray.ni@intel.com>
> > Cc: Leif Lindholm <leif@nuviainc.com>
> > Cc: Ard Biesheuvel <ard.biesheuvel@arm.com>
> > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > 
> > Signed-off-by: Samer El-Haj-Mahmoud <samer.el-haj-mahmoud@arm.com>
> > 
> > Samer El-Haj-Mahmoud (4):
> >    Silicon/AsixPkg: Create AsixPkg
> >    Drivers/OptionRomPkg: Move ASIX drivers to AsixPkg
> >    Silicon/AsixPkg: Add ASIX Ax88179 driver
> >    Silicon/AsixPkg: Add ASIX Ax88772c driver
> > 
> 
> I happened to have one of each of these dongles in my drawer, so I managed
> to test them both.
> 
> Both work fine, although the 88179 one was rather slow.
> 
> In any case, the 10/100 one I tried is an Apple one (05ac:1402) which is
> also supported by the 88772b driver. The only other vid/pid pair which that
> old driver supports is also supported by the new one, so I think we can drop
> the old one altogether. The non-b 7720 driver is also supported by the new
> 10/100 driver, so we can drop the old 88772 as well, afaict.

If we can drop the old ones, I will be very happy.
I was just about to start providing some feedback on those (beyond
what you have) including bits like "drop the Hungarian notation".

Fixing those would be a fair amount of effort, and moving out of their
legacy home before that would set a bad example.

/
    Leif

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#59262): https://edk2.groups.io/g/devel/message/59262
Mute This Topic: https://groups.io/mt/74113376/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [edk2-platform][PATCH v1 0/4] Update ASIX USB Networking drivers
Posted by Samer El-Haj-Mahmoud 3 years, 10 months ago

> -----Original Message-----
> From: Leif Lindholm <leif@nuviainc.com>
> Sent: Tuesday, May 12, 2020 7:15 AM
> To: Ard Biesheuvel <Ard.Biesheuvel@arm.com>
> Cc: Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>;
> devel@edk2.groups.io; Ray Ni <ray.ni@intel.com>; Michael D Kinney
> <michael.d.kinney@intel.com>
> Subject: Re: [edk2-platform][PATCH v1 0/4] Update ASIX USB Networking
> drivers
>
> On Mon, May 11, 2020 at 17:50:54 +0200, Ard Biesheuvel wrote:
> > On 5/10/20 1:15 PM, Samer El-Haj-Mahmoud wrote:
> > > This series updates the ASIX USB networking drivers:
> > >
> > >   - Create new Silicon/ASIX/ package
> > >   - Move existing Ax88772 and Ax88772b drivers from OptionRomPkg
> > >     to the newly created AsixPkg
> > >   - Add new ASIX drivers Ax88179 and Ax88772c
> > >
> > > Code pushed to
> > > https://github.com/samerhaj/edk2-platforms/tree/asix_usb_nic_drivers
> > > _v2
> > >
> > > version 1: https://edk2.groups.io/g/devel/message/58771
> > > version 2 changes:
> > >   - Create new ASIX Package to host existing and new ASIX
> > > drivers[Ray]
> > >
> > > Cc: Ray Ni <ray.ni@intel.com>
> > > Cc: Leif Lindholm <leif@nuviainc.com>
> > > Cc: Ard Biesheuvel <ard.biesheuvel@arm.com>
> > > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > >
> > > Signed-off-by: Samer El-Haj-Mahmoud <samer.el-haj-
> mahmoud@arm.com>
> > >
> > > Samer El-Haj-Mahmoud (4):
> > >    Silicon/AsixPkg: Create AsixPkg
> > >    Drivers/OptionRomPkg: Move ASIX drivers to AsixPkg
> > >    Silicon/AsixPkg: Add ASIX Ax88179 driver
> > >    Silicon/AsixPkg: Add ASIX Ax88772c driver
> > >
> >
> > I happened to have one of each of these dongles in my drawer, so I
> > managed to test them both.
> >
> > Both work fine, although the 88179 one was rather slow.
> >
> > In any case, the 10/100 one I tried is an Apple one (05ac:1402) which
> > is also supported by the 88772b driver. The only other vid/pid pair
> > which that old driver supports is also supported by the new one, so I
> > think we can drop the old one altogether. The non-b 7720 driver is
> > also supported by the new
> > 10/100 driver, so we can drop the old 88772 as well, afaict.
>
> If we can drop the old ones, I will be very happy.
> I was just about to start providing some feedback on those (beyond what
> you have) including bits like "drop the Hungarian notation".
>

Sure, I can drop the old 8872/8872b drivers if we can replace with 8872c

> Fixing those would be a fair amount of effort, and moving out of their legacy
> home before that would set a bad example.
>
> /
>     Leif
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 (#59558): https://edk2.groups.io/g/devel/message/59558
Mute This Topic: https://groups.io/mt/74113376/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-