Silicon/SynQuacer: preparatory ConnectAll fixes

[edk2-devel] [PATCH edk2-platforms 0/3] Silicon/SynQuacer: preparatory ConnectAll fixes

Posted by Ard Biesheuvel 3 years, 11 months ago

The ArmPkg BDS platform lib still calls ConnectAll(), which is something
we should try to remove at some point.

This series does some preparatory work on SynQuacer so that everything works
as expected regardless of whether ConnectAll() is used: connect the I2C
buses, which carry devices that should be connected regardless (as they
are not boot targets). Also, let's fix the clunky PHY detect handling in
the NETSEC driver so that even if we connect the driver at boot, we only
delay the boot if the upper networking layers require it.


Ard Biesheuvel (3):
  Platform/96Boards/96BoardsI2cDxe: connect I2C controllers at EndOfDxe
  Silicon/SynQuacer/NetsecDxe: drop false dependency on device path
    protocol
  Silicon/SynQuacer/NetsecDxe: avoid media detection delay at boot

 .../Drivers/Net/NetsecDxe/NetsecDxe.dec       |   1 -
 .../Socionext/DeveloperBox/DeveloperBox.dsc   |   1 -
 .../96BoardsI2cDxe/96BoardsI2cDxe.inf         |   5 +-
 .../Drivers/Net/NetsecDxe/NetsecDxe.inf       |   4 +-
 .../Drivers/Net/NetsecDxe/NetsecDxe.h         |   5 +
 .../96Boards/96BoardsI2cDxe/96BoardsI2cDxe.c  |  18 +++
 .../Drivers/Net/NetsecDxe/NetsecDxe.c         | 112 +++++++++++++++---
 7 files changed, 125 insertions(+), 21 deletions(-)

-- 
2.17.1


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

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

Re: [edk2-devel] [PATCH edk2-platforms 0/3] Silicon/SynQuacer: preparatory ConnectAll fixes

Posted by Ard Biesheuvel 3 years, 11 months ago

On 5/16/20 7:59 PM, Ard Biesheuvel wrote:
> The ArmPkg BDS platform lib still calls ConnectAll(), which is something
> we should try to remove at some point.
> 
> This series does some preparatory work on SynQuacer so that everything works
> as expected regardless of whether ConnectAll() is used: connect the I2C
> buses, which carry devices that should be connected regardless (as they
> are not boot targets). Also, let's fix the clunky PHY detect handling in
> the NETSEC driver so that even if we connect the driver at boot, we only
> delay the boot if the upper networking layers require it.
> 
> 
> Ard Biesheuvel (3):
>    Platform/96Boards/96BoardsI2cDxe: connect I2C controllers at EndOfDxe
>    Silicon/SynQuacer/NetsecDxe: drop false dependency on device path
>      protocol
>    Silicon/SynQuacer/NetsecDxe: avoid media detection delay at boot
> 

Series pushed @ b29531f3186a0f65f7fe6870cc877dac5be7e7d3

thanks

>   .../Drivers/Net/NetsecDxe/NetsecDxe.dec       |   1 -
>   .../Socionext/DeveloperBox/DeveloperBox.dsc   |   1 -
>   .../96BoardsI2cDxe/96BoardsI2cDxe.inf         |   5 +-
>   .../Drivers/Net/NetsecDxe/NetsecDxe.inf       |   4 +-
>   .../Drivers/Net/NetsecDxe/NetsecDxe.h         |   5 +
>   .../96Boards/96BoardsI2cDxe/96BoardsI2cDxe.c  |  18 +++
>   .../Drivers/Net/NetsecDxe/NetsecDxe.c         | 112 +++++++++++++++---
>   7 files changed, 125 insertions(+), 21 deletions(-)
> 


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

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