From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Glass Date: Sun, 23 Aug 2015 15:21:35 -0600 Subject: [U-Boot] [PATCH v2 07/12] x86: fsp: Call fsp_init_phase_pci() in pci_uclass_post_probe() In-Reply-To: References: <1440078028-29464-1-git-send-email-bmeng.cn@gmail.com> <1440078028-29464-8-git-send-email-bmeng.cn@gmail.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Bin, On 22 August 2015 at 03:05, Bin Meng wrote: > Hi Simon, > > On Sat, Aug 22, 2015 at 7:27 AM, Simon Glass wrote: >> Hi Bin, >> >> On 20 August 2015 at 07:40, Bin Meng wrote: >>> Per Intel FSP specification, we should call FSP notify API to >>> inform FSP that PCI enumeration has been done so that FSP will >>> do any necessary initialization as required by the chipset's >>> BIOS Writer's Guide (BWG). >>> >>> Unfortunately we have to put this call here as with driver model, >>> the enumeration is all done on a lazy basis as needed, so until >>> something is touched on PCI it won't happen. >>> >>> Note we only call this after U-Boot is relocated and root bus has >>> finished probing. >>> >>> Signed-off-by: Bin Meng >>> --- >>> >>> Changes in v2: None >>> >>> drivers/pci/pci-uclass.c | 21 +++++++++++++++++++++ >>> 1 file changed, 21 insertions(+) >>> >> >> Acked-by: Simon Glass >> >> Please see below. >> >>> diff --git a/drivers/pci/pci-uclass.c b/drivers/pci/pci-uclass.c >>> index 4160274..c90e7ac 100644 >>> --- a/drivers/pci/pci-uclass.c >>> +++ b/drivers/pci/pci-uclass.c >>> @@ -14,6 +14,9 @@ >>> #include >>> #include >>> #include >>> +#if defined(CONFIG_X86) && defined(CONFIG_HAVE_FSP) >> >> Do we need CONFIG_X86 here? Do you think it is better to have it to be clearer? > > Technically it is not needed as CONFIG_HAVE_FSP is dependent on > CONFIG_X86 which is defined by Kconfig dependency. However I > intentionally put it here, as I thought this is a common driver code > for all architectures, and someone who is not familiar with x86 may > wonder what CONFIG_HAVE_FSP is, so I put CONFIG_X86 at first to > indicate this is x86-specific thing. What do you think? Yes I thought that might be it. I agree. Applied to u-boot-x86, thanks! [snip] Regards, Simon