Hi, On 21 January 2015 at 09:07, Bin Meng <bmeng...@gmail.com> wrote: > Hi Thierry, > > On Wed, Jan 21, 2015 at 5:50 PM, Thierry Reding <tred...@nvidia.com> wrote: >> On Tue, Jan 20, 2015 at 06:06:53PM +0100, Sjoerd Simons wrote: >>> commit a62e84d7b1824a202dd incorrectly changed the tegra pci code to the >>> new fdtdec pci helpers. To get the device index of the root port, the >>> "reg" property should be parsed from the dtb (as was previously the >>> case). >>> >>> With this patch i can successfully network boot my jetson tk1 >>> >>> Signed-off-by: Sjoerd Simons <sjoerd.sim...@collabora.co.uk> >>> --- >>> drivers/pci/pci_tegra.c | 5 ++--- >>> 1 file changed, 2 insertions(+), 3 deletions(-) >> >> Given the discussion here and in >> >> http://patchwork.ozlabs.org/patch/430815/ >> >> I agree with Bin that this is a more appropriate fix. The documentation >> of the fdtdec_get_pci_bdf() function could be improved, in my opinion, >> to mention how the compatible property is involved. That should clarify >> that any value in "reg" can be overridden by looking up the allocated >> bus number after enumeration. > > I can prepare a patch to improve the doc. > >> >> So this patch: >> >> Tested-by: Thierry Reding <tred...@nvidia.com> >> Acked-by: Thierry Reding <tred...@nvidia.com>
Since this patch seems OK, I'd like to pick it up for the x86 tree (where the breakage happens). Any objections? Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot