On 7/10/2019 7:45 PM, Jonathan Chocron wrote:
> Document Amazon's Annapurna Labs PCIe host bridge.
That is the way! (best to keep same wordings (Amazon's)
>
> Signed-off-by: Jonathan Chocron <[email protected]>
> ---
> .../devicetree/bindings/pci/pcie-al.txt | 45 +++++++++++++++++++
> MAINTAINERS | 1 +
> 2 files changed, 46 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/pci/pcie-al.txt
>
> diff --git a/Documentation/devicetree/bindings/pci/pcie-al.txt b/Documentation/devicetree/bindings/pci/pcie-al.txt
> new file mode 100644
> index 000000000000..d92cc529490e
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/pci/pcie-al.txt
> @@ -0,0 +1,45 @@
> +* Amazon Annapurna Labs PCIe host bridge
> +
> +Annapurna Labs' PCIe Host Controller is based on the Synopsys DesignWare
Amazon's
and what is the s' ? should it be for all
> +Example:
> +
> + pcie-external0 {
probably should have a reference with the address
> + compatible = "amazon,al-pcie";
> + reg = <0x0 0xfb600000 0x0 0x00100000
> + 0x0 0xfd800000 0x0 0x00010000
> + 0x0 0xfd810000 0x0 0x00001000>;
> + reg-names = "config", "controller", "dbi";
> + bus-range = <0 255>;
> + device_type = "pci";
> + #address-cells = <3>;
> + #size-cells = <2>;
> + #interrupt-cells = <1>;
> + interrupts = <GIC_SPI 49 IRQ_TYPE_LEVEL_HIGH>;
> + interrupt-map-mask = <0x00 0 0 7>;
> + interrupt-map = <0x0000 0 0 1 &gic_main GIC_SPI 41 IRQ_TYPE_LEVEL_HIGH>; /* INTa */
gic_main->gic
> + ranges = <0x02000000 0x0 0xc0010000 0x0 0xc0010000 0x0 0x07ff0000>;
> + };
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 5a6137df3f0e..d555beb794bb 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -12205,6 +12205,7 @@ PCIE DRIVER FOR ANNAPURNA LABS
AMAZON!
> M: Jonathan Chocron <[email protected]>
> L: [email protected]
> S: Maintained
> +F: Documentation/devicetree/bindings/pci/pcie-al.txt
> F: drivers/pci/controller/dwc/pcie-al.c
>
> PCIE DRIVER FOR AMLOGIC MESON
On Thu, Jul 11, 2019 at 10:12:35AM +0300, Shenhar, Talel wrote:
>
> On 7/10/2019 7:45 PM, Jonathan Chocron wrote:
> > Document Amazon's Annapurna Labs PCIe host bridge.
>
> That is the way! (best to keep same wordings (Amazon's)
Guys,
as a heads-up, the original posting, for whatever reason, did not hit
linux-pci@vger, which means that from a PCI patches queue review point
of view it does not exist.
It ought to be fixed otherwise we won't be able to review the code.
Lorenzo
> >
> > Signed-off-by: Jonathan Chocron <[email protected]>
> > ---
> > .../devicetree/bindings/pci/pcie-al.txt | 45 +++++++++++++++++++
> > MAINTAINERS | 1 +
> > 2 files changed, 46 insertions(+)
> > create mode 100644 Documentation/devicetree/bindings/pci/pcie-al.txt
> >
> > diff --git a/Documentation/devicetree/bindings/pci/pcie-al.txt b/Documentation/devicetree/bindings/pci/pcie-al.txt
> > new file mode 100644
> > index 000000000000..d92cc529490e
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/pci/pcie-al.txt
> > @@ -0,0 +1,45 @@
> > +* Amazon Annapurna Labs PCIe host bridge
> > +
> > +Annapurna Labs' PCIe Host Controller is based on the Synopsys DesignWare
>
> Amazon's
>
>
> and what is the s' ? should it be for all
>
> > +Example:
> > +
> > + pcie-external0 {
> probably should have a reference with the address
> > + compatible = "amazon,al-pcie";
> > + reg = <0x0 0xfb600000 0x0 0x00100000
> > + 0x0 0xfd800000 0x0 0x00010000
> > + 0x0 0xfd810000 0x0 0x00001000>;
> > + reg-names = "config", "controller", "dbi";
> > + bus-range = <0 255>;
> > + device_type = "pci";
> > + #address-cells = <3>;
> > + #size-cells = <2>;
> > + #interrupt-cells = <1>;
> > + interrupts = <GIC_SPI 49 IRQ_TYPE_LEVEL_HIGH>;
> > + interrupt-map-mask = <0x00 0 0 7>;
> > + interrupt-map = <0x0000 0 0 1 &gic_main GIC_SPI 41 IRQ_TYPE_LEVEL_HIGH>; /* INTa */
> gic_main->gic
> > + ranges = <0x02000000 0x0 0xc0010000 0x0 0xc0010000 0x0 0x07ff0000>;
> > + };
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 5a6137df3f0e..d555beb794bb 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -12205,6 +12205,7 @@ PCIE DRIVER FOR ANNAPURNA LABS
> AMAZON!
> > M: Jonathan Chocron <[email protected]>
> > L: [email protected]
> > S: Maintained
> > +F: Documentation/devicetree/bindings/pci/pcie-al.txt
> > F: drivers/pci/controller/dwc/pcie-al.c
> > PCIE DRIVER FOR AMLOGIC MESON
On Thu, 2019-07-11 at 10:32 +0100, Lorenzo Pieralisi wrote:
> On Thu, Jul 11, 2019 at 10:12:35AM +0300, Shenhar, Talel wrote:
> >
> > On 7/10/2019 7:45 PM, Jonathan Chocron wrote:
> > > Document Amazon's Annapurna Labs PCIe host bridge.
> >
> > That is the way! (best to keep same wordings (Amazon's)
>
> Guys,
>
> as a heads-up, the original posting, for whatever reason, did not hit
> linux-pci@vger, which means that from a PCI patches queue review
> point
> of view it does not exist.
>
> It ought to be fixed otherwise we won't be able to review the code.
>
> Lorenzo
>
I've succeeded to redirect the original emails to linux-pci@vger, so
please proceed with reviewing the patches.
Thanks,
Jonathan