From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web12.10764.1589891534508744051 for ; Tue, 19 May 2020 05:32:14 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: arm.com, ip: 217.140.110.172, mailfrom: ard.biesheuvel@arm.com) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 29C0430E; Tue, 19 May 2020 05:32:14 -0700 (PDT) Received: from [192.168.1.81] (unknown [10.37.8.132]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 6A8723F52E; Tue, 19 May 2020 05:32:13 -0700 (PDT) Subject: Re: [PATCH edk2-platforms 0/3] Silicon/SynQuacer: preparatory ConnectAll fixes To: devel@edk2.groups.io Cc: leif@nuviainc.com References: <20200516175934.31148-1-ard.biesheuvel@arm.com> From: "Ard Biesheuvel" Message-ID: Date: Tue, 19 May 2020 14:32:10 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0 MIME-Version: 1.0 In-Reply-To: <20200516175934.31148-1-ard.biesheuvel@arm.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit 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(-) >