* [PATCH 0/2] dts/ls1043a: add the DTS node for QSPI support @ 2016-03-31 6:44 Yuan Yao 2016-03-31 6:45 ` [PATCH 1/2] Documentation: fsl-quadspi: Add fsl, ls1043a-qspi compatible string Yuan Yao 2016-03-31 6:45 ` [PATCH 2/2] dts/ls1043a: add the DTS node for QSPI support Yuan Yao 0 siblings, 2 replies; 6+ messages in thread From: Yuan Yao @ 2016-03-31 6:44 UTC (permalink / raw) To: robh+dt, mark.rutland, computersforpeace, pawel.moll, han.xu, shawnguo, leoyang.li Cc: devicetree, linux-kernel, linux-arm-kernel, Yuan Yao From: Yuan Yao <yao.yuan@nxp.com> The QSPI controller on LS1043A is the same with the QSPI on LS1021A. We shuld add the QSPI support on LS1043A. So this patch try to add the dts node and binding Documentation. Depend on the patch: arm64: dts: add LS1043a-QDS board support https://patchwork.kernel.org/patch/8528821/ Yuan Yao (2): Documentation: fsl-quadspi: Add fsl, ls1043a-qspi compatible string dts/ls1043a: add the DTS node for QSPI support Documentation/devicetree/bindings/mtd/fsl-quadspi.txt | 3 ++- arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts | 16 ++++++++++++++++ arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 14 ++++++++++++++ 3 files changed, 32 insertions(+), 1 deletion(-) -- 2.1.0.27.g96db324 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/2] Documentation: fsl-quadspi: Add fsl, ls1043a-qspi compatible string 2016-03-31 6:44 [PATCH 0/2] dts/ls1043a: add the DTS node for QSPI support Yuan Yao @ 2016-03-31 6:45 ` Yuan Yao [not found] ` <1459406701-40993-2-git-send-email-yao.yuan-KZfg59tc24xl57MIdRCFDg@public.gmane.org> 2016-03-31 6:45 ` [PATCH 2/2] dts/ls1043a: add the DTS node for QSPI support Yuan Yao 1 sibling, 1 reply; 6+ messages in thread From: Yuan Yao @ 2016-03-31 6:45 UTC (permalink / raw) To: robh+dt, mark.rutland, computersforpeace, pawel.moll, han.xu, shawnguo, leoyang.li Cc: devicetree, linux-kernel, linux-arm-kernel, Yuan Yao From: Yuan Yao <yao.yuan@nxp.com> new compatible string: "fsl,ls1043a-qspi". Signed-off-by: Yuan Yao <yao.yuan@nxp.com> --- Documentation/devicetree/bindings/mtd/fsl-quadspi.txt | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/mtd/fsl-quadspi.txt b/Documentation/devicetree/bindings/mtd/fsl-quadspi.txt index 0333ec8..c34aa6f 100644 --- a/Documentation/devicetree/bindings/mtd/fsl-quadspi.txt +++ b/Documentation/devicetree/bindings/mtd/fsl-quadspi.txt @@ -5,7 +5,8 @@ Required properties: "fsl,imx7d-qspi", "fsl,imx6ul-qspi", "fsl,ls1021a-qspi" or - "fsl,ls2080a-qspi" followed by "fsl,ls1021a-qspi" + "fsl,ls2080a-qspi" followed by "fsl,ls1021a-qspi", + "fsl,ls1043a-qspi" followed by "fsl,ls1021a-qspi" - reg : the first contains the register location and length, the second contains the memory mapping address and length - reg-names: Should contain the reg names "QuadSPI" and "QuadSPI-memory" -- 2.1.0.27.g96db324 ^ permalink raw reply related [flat|nested] 6+ messages in thread
[parent not found: <1459406701-40993-2-git-send-email-yao.yuan-KZfg59tc24xl57MIdRCFDg@public.gmane.org>]
* Re: [PATCH 1/2] Documentation: fsl-quadspi: Add fsl, ls1043a-qspi compatible string [not found] ` <1459406701-40993-2-git-send-email-yao.yuan-KZfg59tc24xl57MIdRCFDg@public.gmane.org> @ 2016-04-01 17:14 ` Rob Herring 0 siblings, 0 replies; 6+ messages in thread From: Rob Herring @ 2016-04-01 17:14 UTC (permalink / raw) To: Yuan Yao Cc: mark.rutland-5wv7dgnIgG8, computersforpeace-Re5JQEeQqe8AvxtiuMwx3w, pawel.moll-5wv7dgnIgG8, han.xu-3arQi8VN3Tc, shawnguo-DgEjT+Ai2ygdnm+yROfE0A, leoyang.li-3arQi8VN3Tc, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Yuan Yao On Thu, Mar 31, 2016 at 02:45:00PM +0800, Yuan Yao wrote: > From: Yuan Yao <yao.yuan-3arQi8VN3Tc@public.gmane.org> > > new compatible string: "fsl,ls1043a-qspi". > > Signed-off-by: Yuan Yao <yao.yuan-3arQi8VN3Tc@public.gmane.org> > --- > Documentation/devicetree/bindings/mtd/fsl-quadspi.txt | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) There's a typo in the compatible string in the subject, otherwise: Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 2/2] dts/ls1043a: add the DTS node for QSPI support 2016-03-31 6:44 [PATCH 0/2] dts/ls1043a: add the DTS node for QSPI support Yuan Yao 2016-03-31 6:45 ` [PATCH 1/2] Documentation: fsl-quadspi: Add fsl, ls1043a-qspi compatible string Yuan Yao @ 2016-03-31 6:45 ` Yuan Yao [not found] ` <1459406701-40993-3-git-send-email-yao.yuan-KZfg59tc24xl57MIdRCFDg@public.gmane.org> 1 sibling, 1 reply; 6+ messages in thread From: Yuan Yao @ 2016-03-31 6:45 UTC (permalink / raw) To: robh+dt, mark.rutland, computersforpeace, pawel.moll, han.xu, shawnguo, leoyang.li Cc: devicetree, linux-kernel, linux-arm-kernel, Yuan Yao From: Yuan Yao <yao.yuan@nxp.com> Signed-off-by: Yuan Yao <yao.yuan@nxp.com> --- arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts | 16 ++++++++++++++++ arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 14 ++++++++++++++ 2 files changed, 30 insertions(+) diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts b/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts index 97e9906..c8303a3 100644 --- a/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts +++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts @@ -100,6 +100,22 @@ }; }; +&qspi { + num-cs = <2>; + bus-num = <0>; + status = "okay"; + fsl,ddr-sampling-point = <4>; + + qflash0: s25fl128s@0 { + compatible = "spansion,m25p80"; + #address-cells = <1>; + #size-cells = <1>; + spi-max-frequency = <20000000>; + ddr-quad-read; + reg = <0>; + }; +}; + &i2c0 { status = "okay"; pca9547@77 { diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi index be72bf5..49b1aeb 100644 --- a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi +++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi @@ -208,6 +208,20 @@ status = "disabled"; }; + qspi: quadspi@1550000 { + compatible = "fsl,ls1043a-qspi", "fsl,ls1021a-qspi"; + #address-cells = <1>; + #size-cells = <0>; + reg = <0x0 0x1550000 0x0 0x10000>, + <0x0 0x40000000 0x0 0x4000000>; + reg-names = "QuadSPI", "QuadSPI-memory"; + interrupts = <0 99 0x4>; + clock-names = "qspi_en", "qspi"; + clocks = <&clockgen 4 0>, <&clockgen 4 0>; + big-endian; + status = "disabled"; + }; + i2c0: i2c@2180000 { compatible = "fsl,vf610-i2c"; #address-cells = <1>; -- 2.1.0.27.g96db324 ^ permalink raw reply related [flat|nested] 6+ messages in thread
[parent not found: <1459406701-40993-3-git-send-email-yao.yuan-KZfg59tc24xl57MIdRCFDg@public.gmane.org>]
* Re: [PATCH 2/2] dts/ls1043a: add the DTS node for QSPI support [not found] ` <1459406701-40993-3-git-send-email-yao.yuan-KZfg59tc24xl57MIdRCFDg@public.gmane.org> @ 2016-04-12 2:49 ` Shawn Guo 2016-04-13 2:41 ` Yao Yuan 0 siblings, 1 reply; 6+ messages in thread From: Shawn Guo @ 2016-04-12 2:49 UTC (permalink / raw) To: Yuan Yao Cc: robh+dt-DgEjT+Ai2ygdnm+yROfE0A, mark.rutland-5wv7dgnIgG8, computersforpeace-Re5JQEeQqe8AvxtiuMwx3w, pawel.moll-5wv7dgnIgG8, han.xu-3arQi8VN3Tc, leoyang.li-3arQi8VN3Tc, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Yuan Yao On Thu, Mar 31, 2016 at 02:45:01PM +0800, Yuan Yao wrote: > From: Yuan Yao <yao.yuan-3arQi8VN3Tc@public.gmane.org> > > Signed-off-by: Yuan Yao <yao.yuan-3arQi8VN3Tc@public.gmane.org> Please style of 'arm64: dts: ls1043a: ' for subject prefix. > --- > arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts | 16 ++++++++++++++++ > arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 14 ++++++++++++++ > 2 files changed, 30 insertions(+) > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts b/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts > index 97e9906..c8303a3 100644 > --- a/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts > @@ -100,6 +100,22 @@ > }; > }; > > +&qspi { Please sort such labeled node alphabetically in label name. That said, it should go after &lpuart0. > + num-cs = <2>; I do not see this property in bindings doc. > + bus-num = <0>; > + status = "okay"; Please let 'status' line be the last of property list. > + fsl,ddr-sampling-point = <4>; I do not see this one in bindings definition either. > + > + qflash0: s25fl128s@0 { > + compatible = "spansion,m25p80"; > + #address-cells = <1>; > + #size-cells = <1>; > + spi-max-frequency = <20000000>; > + ddr-quad-read; Ditto > + reg = <0>; > + }; > +}; > + > &i2c0 { > status = "okay"; > pca9547@77 { > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi > index be72bf5..49b1aeb 100644 > --- a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi > @@ -208,6 +208,20 @@ > status = "disabled"; > }; > > + qspi: quadspi@1550000 { Please sort the node in <soc>.dtsi in order of unit-address. That said, the node should be added between ifc@1530000 and esdhc@1560000. Shawn > + compatible = "fsl,ls1043a-qspi", "fsl,ls1021a-qspi"; > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x0 0x1550000 0x0 0x10000>, > + <0x0 0x40000000 0x0 0x4000000>; > + reg-names = "QuadSPI", "QuadSPI-memory"; > + interrupts = <0 99 0x4>; > + clock-names = "qspi_en", "qspi"; > + clocks = <&clockgen 4 0>, <&clockgen 4 0>; > + big-endian; > + status = "disabled"; > + }; > + > i2c0: i2c@2180000 { > compatible = "fsl,vf610-i2c"; > #address-cells = <1>; > -- > 2.1.0.27.g96db324 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH 2/2] dts/ls1043a: add the DTS node for QSPI support 2016-04-12 2:49 ` Shawn Guo @ 2016-04-13 2:41 ` Yao Yuan 0 siblings, 0 replies; 6+ messages in thread From: Yao Yuan @ 2016-04-13 2:41 UTC (permalink / raw) To: Shawn Guo, Yuan Yao Cc: robh+dt@kernel.org, mark.rutland@arm.com, computersforpeace@gmail.com, pawel.moll@arm.com, Han Xu, Yang-Leo Li, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org On Thu, Apr 12, 2016 at 10:50:01AM +0800, Shawn Guo wrote: > On Thu, Mar 31, 2016 at 02:45:01PM +0800, Yuan Yao wrote: > > From: Yuan Yao <yao.yuan@nxp.com> > > > > Signed-off-by: Yuan Yao <yao.yuan@nxp.com> > > Please style of 'arm64: dts: ls1043a: ' for subject prefix. > > > --- > > arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts | 16 ++++++++++++++++ > > arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 14 ++++++++++++++ > > 2 files changed, 30 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts > > b/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts > > index 97e9906..c8303a3 100644 > > --- a/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts > > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts > > @@ -100,6 +100,22 @@ > > }; > > }; > > > > +&qspi { > > Please sort such labeled node alphabetically in label name. That said, it should > go after &lpuart0. Ok, Thanks. > > + num-cs = <2>; > > I do not see this property in bindings doc. We can find this property in ./Documentation/devicetree/bindings/spi/spi-bus.txt > > > + bus-num = <0>; > > + status = "okay"; > > Please let 'status' line be the last of property list. Ok, Thanks. > > + fsl,ddr-sampling-point = <4>; > > I do not see this one in bindings definition either. It's used for DDR mode; The DDR mode will send to upstream late. So I will remove this, once the DDR mode patch is merged, I will send patch to add the property. > > > + > > + qflash0: s25fl128s@0 { > > + compatible = "spansion,m25p80"; > > + #address-cells = <1>; > > + #size-cells = <1>; > > + spi-max-frequency = <20000000>; > > + ddr-quad-read; > > Ditto Ditto, Thanks. > > > + reg = <0>; > > + }; > > +}; > > + > > &i2c0 { > > status = "okay"; > > pca9547@77 { > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi > > b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi > > index be72bf5..49b1aeb 100644 > > --- a/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi > > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi > > @@ -208,6 +208,20 @@ > > status = "disabled"; > > }; > > > > + qspi: quadspi@1550000 { > > Please sort the node in <soc>.dtsi in order of unit-address. That said, the node > should be added between ifc@1530000 and esdhc@1560000. Thanks. I will update it and resend it soon. > Shawn > > > + compatible = "fsl,ls1043a-qspi", "fsl,ls1021a-qspi"; > > + #address-cells = <1>; > > + #size-cells = <0>; > > + reg = <0x0 0x1550000 0x0 0x10000>, > > + <0x0 0x40000000 0x0 0x4000000>; > > + reg-names = "QuadSPI", "QuadSPI-memory"; > > + interrupts = <0 99 0x4>; > > + clock-names = "qspi_en", "qspi"; > > + clocks = <&clockgen 4 0>, <&clockgen 4 0>; > > + big-endian; > > + status = "disabled"; > > + }; > > + > > i2c0: i2c@2180000 { > > compatible = "fsl,vf610-i2c"; > > #address-cells = <1>; > > -- > > 2.1.0.27.g96db324 > > > > > > _______________________________________________ > > linux-arm-kernel mailing list > > linux-arm-kernel@lists.infradead.org > > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > > ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-04-13 2:41 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-03-31 6:44 [PATCH 0/2] dts/ls1043a: add the DTS node for QSPI support Yuan Yao
2016-03-31 6:45 ` [PATCH 1/2] Documentation: fsl-quadspi: Add fsl, ls1043a-qspi compatible string Yuan Yao
[not found] ` <1459406701-40993-2-git-send-email-yao.yuan-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
2016-04-01 17:14 ` Rob Herring
2016-03-31 6:45 ` [PATCH 2/2] dts/ls1043a: add the DTS node for QSPI support Yuan Yao
[not found] ` <1459406701-40993-3-git-send-email-yao.yuan-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
2016-04-12 2:49 ` Shawn Guo
2016-04-13 2:41 ` Yao Yuan
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).