From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thierry Reding Date: Wed, 21 Jan 2015 10:50:06 +0100 Subject: [U-Boot] [PATCH] pci: tegra: Fix port information parsing In-Reply-To: <1421773613-25138-1-git-send-email-sjoerd.simons@collabora.co.uk> References: <1421773613-25138-1-git-send-email-sjoerd.simons@collabora.co.uk> Message-ID: <20150121095005.GD26340@ulmo.nvidia.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 > --- > 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. So this patch: Tested-by: Thierry Reding Acked-by: Thierry Reding -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 819 bytes Desc: not available URL: