lkml.org 
[lkml]   [2015]   [Sep]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [PATCH v7 5/6] Documentation: dt-bindings: pci: altera pcie device tree binding
    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


    \
     
     \ /
      Last update: 2015-09-28 19:41    [W:5.992 / U:0.068 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site