[U-Boot] [PATCH 1/2] dm/pci: Change the first CFG read to Vendor ID in enumeration

sjg at google.com sjg at google.com
Thu Nov 22 20:22:01 UTC 2018


Hi Bin,

Thanks a lot for your review!

Thanks,
Zhiqiang

> -----Original Message-----
> From: Bin Meng <bmeng.cn at gmail.com>
> Sent: 2018年10月8日 16:53
> To: Z.q. Hou <zhiqiang.hou at nxp.com>
> Cc: U-Boot Mailing List <u-boot at lists.denx.de>; Simon Glass
> <sjg at chromium.org>; Christian Gmeiner <christian.gmeiner at gmail.com>;
> Tuomas Tynkkynen <tuomas.tynkkynen at iki.fi>; Bernhard Messerklinger
> <bernhard.messerklinger at br-automation.com>; Masahiro Yamada
> <yamada.masahiro at socionext.com>; Mingkai Hu <mingkai.hu at nxp.com>;
> M.h. Lian <minghuan.lian at nxp.com>
> Subject: Re: [PATCH 1/2] dm/pci: Change the first CFG read to Vendor ID in
> enumeration
>
> On Mon, Oct 8, 2018 at 4:36 PM Zhiqiang Hou <Zhiqiang.Hou at nxp.com>
> wrote:
> >
> > From: Hou Zhiqiang <Zhiqiang.Hou at nxp.com>
> >
> > As the PCIe specification recommend reading the Vendor ID register to
> > determine if a Function is present, read the Vendor ID of a
> > non-existent Function must not result in system error, so we'd better
> > make the first CFG read to Vendor ID instead of Header Type register
> > in the PCIe enumeration.
> >
> > Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou at nxp.com>
> > ---
> >  drivers/pci/pci-uclass.c | 11 +++++++----
> >  1 file changed, 7 insertions(+), 4 deletions(-)
> >
>
> Reviewed-by: Bin Meng <bmeng.cn at gmail.com>

Applied to u-boot-dm/master, thanks!


More information about the U-Boot mailing list