Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1234316

Re: [PATCH v7 5/6] Documentation: dt-bindings: pci: altera pcie device tree binding

Show all headers | View raw

On Mon, Sep 28, 2015 at 12:38 AM, Ley Foon Tan <lftan@altera.com> wrote:
> On Sat, Sep 26, 2015 at 11:55 AM, Rob Herring <robh@kernel.org> wrote:
>>
>> On 09/20/2015 09:13 PM, Ley Foon Tan wrote:
>> > This patch adds the bindings for Altera PCIe host controller driver and
>> > Altera PCIe MSI driver.

[...]

>> > +     num-vectors = <32>;
>> > +};
>> > diff --git a/Documentation/devicetree/bindings/pci/altera-pcie.txt b/Documentation/devicetree/bindings/pci/altera-pcie.txt
>> > new file mode 100644
>> > index 0000000..4440db1
>> > --- /dev/null
>> > +++ b/Documentation/devicetree/bindings/pci/altera-pcie.txt
>> > @@ -0,0 +1,49 @@
>> > +* Altera PCIe controller
>> > +
>> > +Required properties:
>> > +- compatible :       should contain "altr,pcie-root-port-1.0"
>> > +- reg:               a list of physical base address and length for TXS and CRA.
>> > +- reg-names: must include the following entries:
>> > +             "Txs" or "txs": TX slave port region
>> > +             "Cra" or "cra": Control register access region
>>
>> Why both cases? Can we please just have one (or none is better IMO).
> The PCIe IP on different device families use different register names.
> And our device tree generator will auto generate the register names
> based on the hardware description name. Too bad we can't change the
> hardware description names now.

Okay, your problem to maintain. Hopefully the driver just goes by index then.

Strictly speaking, if you have undocumented bindings downstream that
is your problem and we don't have to accept them as-is upstream. I'm
not going to worry about that here.

>> txs contains the config space?
> It is not the config space, but a memory slave port.

Then where is the config space? It should not be part of "ranges" is
all I care about.

Rob
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH v7 5/6] Documentation: dt-bindings: pci: altera pcie  device tree binding Rob Herring <robh@kernel.org> - 2015-09-26 20:20 +0200
  Re: [PATCH v7 5/6] Documentation: dt-bindings: pci: altera pcie  device tree binding Ley Foon Tan <lftan@altera.com> - 2015-09-28 07:50 +0200
    Re: [PATCH v7 5/6] Documentation: dt-bindings: pci: altera pcie  device tree binding Rob Herring <robh@kernel.org> - 2015-09-28 19:40 +0200
      Re: [PATCH v7 5/6] Documentation: dt-bindings: pci: altera pcie device tree binding Arnd Bergmann <arnd@arndb.de> - 2015-09-28 21:40 +0200
        Re: [PATCH v7 5/6] Documentation: dt-bindings: pci: altera pcie  device tree binding Ley Foon Tan <lftan@altera.com> - 2015-10-02 10:00 +0200
      Re: [PATCH v7 5/6] Documentation: dt-bindings: pci: altera pcie  device tree binding Ley Foon Tan <lftan@altera.com> - 2015-10-02 10:00 +0200
        Re: [PATCH v7 5/6] Documentation: dt-bindings: pci: altera pcie device tree binding Arnd Bergmann <arnd@arndb.de> - 2015-10-03 00:00 +0200
          Re: [PATCH v7 5/6] Documentation: dt-bindings: pci: altera pcie  device tree binding Marc Zyngier <marc.zyngier@arm.com> - 2015-10-03 15:20 +0200
          Re: [PATCH v7 5/6] Documentation: dt-bindings: pci: altera pcie  device tree binding Ley Foon Tan <lftan@altera.com> - 2015-10-05 05:20 +0200

csiph-web