[PATCH] doc: PCI: altera: Fix the 'ranges' property in example

[Marek Vasut]

On Monday, December 28, 2015 at 08:42:00 AM, Ley Foon Tan wrote:
> On Mon, Dec 28, 2015 at 3:34 PM, Marek Vasut <marex at denx.de> wrote:
> > On Monday, December 28, 2015 at 07:56:15 AM, Ley Foon Tan wrote:
> >> On Thu, 2015-12-24 at 10:45 +0100, Marek Vasut wrote:
> >> > The example does not work on real hardware with the PCIe HIP [1].
> >> > The problem is with incorrect "ranges" property in the example, so
> >> > one cannot just copy-paste the example into his DT and expect this
> >> > to work. This patches aligns the "ranges" in the example with the
> >> > reference FPGA design.
> >> 
> >> Hi Marek
> > 
> > Hi Ley,
> > 
> >> The original "ranges" is working for me. What error did you encounter?
> > 
> > Every time a driver accessed the Txs range, the system got stuck hard.
> > The Cra access always worked fine though, so the PCIe devices were always
> > detected.
> > 
> > I checked with signaltap and my impression is that the wrong address
> > propagated into the request passed to the Txs port of the HardIP block,
> > unless I change the range configuration.
> > 
> > Note that I tested Intel Centrino 6235 WiFi card and Atheros AR5006 WiFi
> > card. The intel in particular uses both the Txs and even MSI , so to use
> > the intel, the whole PCIe block has to work properly ; with this change,
> > it does and I can use the intel card just fine.
> > 
> >> Thanks.
> 
> Hi Marek
> 
> I tested two Ethernet adapters, one SSD NVMe and our custom endpoint
> without the problem.
> Can you please send me your dts file if possible?
> 
> Do you use the latest Altera pcie driver in v4.4?

I'm using next from 20151223, crude patch is attached.

Best regards,
Marek Vasut
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-ARM-socfpga-Enable-PCIe-support.patch
Type: text/x-patch
Size: 2018 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20151228/5a3c326f/attachment.bin>