Hi Bin, Thanks a lot for your review!
Thanks, Zhiqiang > -----Original Message----- > From: Bin Meng <bmeng...@gmail.com> > Sent: 2018年10月8日 16:53 > To: Z.q. Hou <zhiqiang....@nxp.com> > Cc: U-Boot Mailing List <u-boot@lists.denx.de>; Simon Glass > <s...@chromium.org>; Christian Gmeiner <christian.gmei...@gmail.com>; > Tuomas Tynkkynen <tuomas.tynkky...@iki.fi>; Bernhard Messerklinger > <bernhard.messerklin...@br-automation.com>; Masahiro Yamada > <yamada.masah...@socionext.com>; Mingkai Hu <mingkai...@nxp.com>; > M.h. Lian <minghuan.l...@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....@nxp.com> > wrote: > > > > From: Hou Zhiqiang <zhiqiang....@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....@nxp.com> > > --- > > drivers/pci/pci-uclass.c | 11 +++++++---- > > 1 file changed, 7 insertions(+), 4 deletions(-) > > > > Reviewed-by: Bin Meng <bmeng...@gmail.com> Applied to u-boot-dm/master, thanks! _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
