| Message ID | 1393410438-28205-1-git-send-email-l.stach@pengutronix.de (mailing list archive) |
|---|---|
| State | New, archived |
| Delegated to: | Bjorn Helgaas |
| Headers | show |
On Wed, Feb 26, 2014 at 11:27:17AM +0100, Lucas Stach wrote: > From: Juergen Beisert <jbe@pengutronix.de> > > Signed-off-by: Juergen Beisert <jbe@pengutronix.de> > Signed-off-by: Lucas Stach <l.stach@pengutronix.de> > --- > arch/arm/boot/dts/imx6qdl.dtsi | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi > index fb28b2ecb1db..aeccbb114fc5 100644 > --- a/arch/arm/boot/dts/imx6qdl.dtsi > +++ b/arch/arm/boot/dts/imx6qdl.dtsi > @@ -126,7 +126,8 @@ > 0x81000000 0 0 0x01f80000 0 0x00010000 /* downstream I/O */ > 0x82000000 0 0x01000000 0x01000000 0 0x00f00000>; /* non-prefetchable memory */ > num-lanes = <1>; > - interrupts = <0 123 0x04>; > + /* inta intb intc intd/msi */ What about replacing the comment with an 'interrupt-names' property? Shawn > + interrupts = <0 123 0x04>, <0 122 0x04>, <0 121 0x04>, <0 120 0x04>; > clocks = <&clks 189>, <&clks 187>, <&clks 206>, <&clks 144>; > clock-names = "pcie_ref_125m", "sata_ref_100m", "lvds_gate", "pcie_axi"; > status = "disabled"; > -- > 1.8.5.3 > -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wed, Feb 26, 2014 at 10:27:17AM +0000, Lucas Stach wrote: > From: Juergen Beisert <jbe@pengutronix.de> > > Signed-off-by: Juergen Beisert <jbe@pengutronix.de> > Signed-off-by: Lucas Stach <l.stach@pengutronix.de> > --- > arch/arm/boot/dts/imx6qdl.dtsi | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi > index fb28b2ecb1db..aeccbb114fc5 100644 > --- a/arch/arm/boot/dts/imx6qdl.dtsi > +++ b/arch/arm/boot/dts/imx6qdl.dtsi > @@ -126,7 +126,8 @@ > 0x81000000 0 0 0x01f80000 0 0x00010000 /* downstream I/O */ > 0x82000000 0 0x01000000 0x01000000 0 0x00f00000>; /* non-prefetchable memory */ > num-lanes = <1>; > - interrupts = <0 123 0x04>; > + /* inta intb intc intd/msi */ > + interrupts = <0 123 0x04>, <0 122 0x04>, <0 121 0x04>, <0 120 0x04>; This seems to be a "fsl,imx6q-pcie", "snps,dw-pcie" device. In Documentation/devicetree/bindings/pci/designware-pcie.txt I only see 3 interrupts listed. The fourth interrupt must be documented. Do the first three correspond to those in the documentation? Cheers, Mark. -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Am Mittwoch, den 26.02.2014, 16:34 +0000 schrieb Mark Rutland: > On Wed, Feb 26, 2014 at 10:27:17AM +0000, Lucas Stach wrote: > > From: Juergen Beisert <jbe@pengutronix.de> > > > > Signed-off-by: Juergen Beisert <jbe@pengutronix.de> > > Signed-off-by: Lucas Stach <l.stach@pengutronix.de> > > --- > > arch/arm/boot/dts/imx6qdl.dtsi | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi > > index fb28b2ecb1db..aeccbb114fc5 100644 > > --- a/arch/arm/boot/dts/imx6qdl.dtsi > > +++ b/arch/arm/boot/dts/imx6qdl.dtsi > > @@ -126,7 +126,8 @@ > > 0x81000000 0 0 0x01f80000 0 0x00010000 /* downstream I/O */ > > 0x82000000 0 0x01000000 0x01000000 0 0x00f00000>; /* non-prefetchable memory */ > > num-lanes = <1>; > > - interrupts = <0 123 0x04>; > > + /* inta intb intc intd/msi */ > > + interrupts = <0 123 0x04>, <0 122 0x04>, <0 121 0x04>, <0 120 0x04>; > > This seems to be a "fsl,imx6q-pcie", "snps,dw-pcie" device. > > In Documentation/devicetree/bindings/pci/designware-pcie.txt I only see 3 > interrupts listed. > > The fourth interrupt must be documented. > > Do the first three correspond to those in the documentation? > Urgh, seems Samsung and FSL do all the glue a lot different. The IRQs seem to be different and the same holds true for the clocks. I think we should split the Documentation here to avoid further confusion. Will send a patch. Regards, Lucas
diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi index fb28b2ecb1db..aeccbb114fc5 100644 --- a/arch/arm/boot/dts/imx6qdl.dtsi +++ b/arch/arm/boot/dts/imx6qdl.dtsi @@ -126,7 +126,8 @@ 0x81000000 0 0 0x01f80000 0 0x00010000 /* downstream I/O */ 0x82000000 0 0x01000000 0x01000000 0 0x00f00000>; /* non-prefetchable memory */ num-lanes = <1>; - interrupts = <0 123 0x04>; + /* inta intb intc intd/msi */ + interrupts = <0 123 0x04>, <0 122 0x04>, <0 121 0x04>, <0 120 0x04>; clocks = <&clks 189>, <&clks 187>, <&clks 206>, <&clks 144>; clock-names = "pcie_ref_125m", "sata_ref_100m", "lvds_gate", "pcie_axi"; status = "disabled";