[PATCH v2 07/10] ARM: tegra: pcie: Add device tree support
swarren at wwwdotorg.org
Tue Jun 26 13:22:32 EDT 2012
On 06/25/2012 12:34 AM, Thierry Reding wrote:
> On Fri, Jun 22, 2012 at 11:28:15AM -0600, Stephen Warren wrote:
>> On 06/22/2012 11:00 AM, Stephen Warren wrote:
>>> On 06/22/2012 05:00 AM, Thierry Reding wrote: ...
>>>> Stephen: can you try to find out whether the Tegra PCIe
>>>> controller indeed implements ECAM, or if this scheme is
>>>> actually just a proprietary variant?
>>> Sure. I have added this request to the bug I filed requesting
>>> more complete PCIe host documentation.
>> I've received unofficial confirmation that we do indeed implement
>> a non-standard/non-ECAM mapping, and what's in our driver matches
>> our HW.
> What I don't quite see yet is how the extended configuration space
> is supposed to work with the current driver. The PCIE_CONF_* macros
> don't provide for registers >= 256. Passing a value higher than
> that will mess with the device function field.
The current downstream driver is incorrect for that case. We should be
updating it (at least, I filed a bug to do this).
Here's how the HW works (I believe this information should be in some
future version of the TRM):
There are 3 address spaces:
* The CPU bus.
* An internal (to the PCIe controller) 40-bit address space.
Apparently the layout is HyperTransport-based. Whether HT defines
this, or whether our HW engineers are referring to our particular
implementation of HT, I'm not sure.
* The PCIe external bus.
Accesses from the CPU to the PCIe controller's 1GB aperture are mapped
into the 40-bit bus using the BAR configurations in the PCIe
controller; I believe this is what tegra_pcie_setup_translations() is
configuring in our downstream driver.
Accesses are then mapped from this 40-bit bus to the external 32-bit
bus, I believe using a hard-coded mapping, which I believe may be
inherited from (our implementation of?) HyperTransport
For config and extended config accesses the mapping from the internal
to external bus is as follows:
In the case of PCICFG space,
addr[27:24]=4'hE means type 0 and addr[27:24]=4'hF means type 1,
In the case of EXTCFG space,
addr[31:28]=4'h0 means type 0 and addr[31:28]=4'h1 means type 1,
addr[27:24]=upper register number (i.e. register number[11:8])
addr[7:0]=register number (i.e. register number[7:0])
(in actual fact, the HW matches the top 16 or 12 bits to determine
config/ext-config and transaction type, so the top two fields in the
lists above should really be considered merged)
I hope this helps!
More information about the linux-arm-kernel