> -----Original Message----- > From: Bin Meng [mailto:bmeng...@gmail.com] > Sent: Thursday, January 28, 2016 4:02 PM > To: Qianyu Gong <qianyu.g...@nxp.com> > Cc: u-boot@lists.denx.de > Subject: Re: [U-Boot] [PATCH] pci: gate print info of reading vender id with > CONFIG_DM_PCI > > On Thu, Jan 28, 2016 at 3:43 PM, Qianyu Gong <qianyu.g...@nxp.com> wrote: > > Hi, > > > >> -----Original Message----- > >> From: Bin Meng [mailto:bmeng...@gmail.com] > >> Sent: Thursday, January 28, 2016 3:30 PM > >> To: Qianyu Gong <qianyu.g...@nxp.com> > >> Cc: U-Boot Mailing List <u-boot@lists.denx.de> > >> Subject: Re: [U-Boot] [PATCH] pci: gate print info of reading vender > >> id with CONFIG_DM_PCI > >> > >> On Thu, Jan 28, 2016 at 3:15 PM, Gong Qianyu <qianyu.g...@nxp.com> wrote: > >> > From: Mingkai Hu <mingkai...@freescale.com> > >> > > >> > Referring to 'commit ff3e077bd23c ("dm: pci: Add a uclass for PCI")'. > >> > > >> > For legacy PCIe driver, it needs loop to read the vender_id from > >> > devie > >> > 0 to devie 255 to check if there is device available. > >> > Reading non-existen device will trigger the "Cannot read bus > >> > configuration: -1" information. > >> > > >> > Signed-off-by: Mingkai Hu <mingkai...@freescale.com> > >> > Signed-off-by: Gong Qianyu <qianyu.g...@nxp.com> > >> > --- > >> > >> Which pci controller are you testing? > >> > >> [snip] > >> > >> Regards, > >> Bin > > > > Designware PCIe controller on LS2085A. > > > > Please check commit 7ba34ff09f1ef105521f914e4ad4e4ac19975bac "pci: > layerscape: Adjust the return value when ls_pcie_addr_valid() fails" > which is already in mainline. > > Regards, > Bin
Ok. That's nice of you. And we also find another commit cab24b3407189a12 "dm: pci: Convert 'pci' command to driver model" submitted by Simon has fixed the issue. So no need of this patch. Thanks a lot. Regards, Qianyu _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
