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