* [PATCH 0/2] dts/ls1043a: add the DTS node for QSPI support @ 2016-03-31 6:44 ` Yuan Yao 0 siblings, 0 replies; 18+ 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: linux-arm-kernel, linux-kernel, devicetree, 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] 18+ messages in thread
* [PATCH 0/2] dts/ls1043a: add the DTS node for QSPI support @ 2016-03-31 6:44 ` Yuan Yao 0 siblings, 0 replies; 18+ messages in thread From: Yuan Yao @ 2016-03-31 6:44 UTC (permalink / raw) To: linux-arm-kernel 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] 18+ messages in thread
* [PATCH 0/2] dts/ls1043a: add the DTS node for QSPI support @ 2016-03-31 6:44 ` Yuan Yao 0 siblings, 0 replies; 18+ 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] 18+ messages in thread
* [PATCH 1/2] Documentation: fsl-quadspi: Add fsl, ls1043a-qspi compatible string 2016-03-31 6:44 ` Yuan Yao (?) @ 2016-03-31 6:45 ` Yuan Yao -1 siblings, 0 replies; 18+ 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: linux-arm-kernel, linux-kernel, devicetree, 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] 18+ messages in thread
* [PATCH 1/2] Documentation: fsl-quadspi: Add fsl, ls1043a-qspi compatible string @ 2016-03-31 6:45 ` Yuan Yao 0 siblings, 0 replies; 18+ messages in thread From: Yuan Yao @ 2016-03-31 6:45 UTC (permalink / raw) To: linux-arm-kernel 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] 18+ messages in thread
* [PATCH 1/2] Documentation: fsl-quadspi: Add fsl, ls1043a-qspi compatible string @ 2016-03-31 6:45 ` Yuan Yao 0 siblings, 0 replies; 18+ 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] 18+ messages in thread
* Re: [PATCH 1/2] Documentation: fsl-quadspi: Add fsl, ls1043a-qspi compatible string @ 2016-04-01 17:14 ` Rob Herring 0 siblings, 0 replies; 18+ messages in thread From: Rob Herring @ 2016-04-01 17:14 UTC (permalink / raw) To: Yuan Yao Cc: mark.rutland, computersforpeace, pawel.moll, han.xu, shawnguo, leoyang.li, devicetree, linux-kernel, linux-arm-kernel, Yuan Yao On Thu, Mar 31, 2016 at 02:45:00PM +0800, Yuan Yao wrote: > 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(-) There's a typo in the compatible string in the subject, otherwise: Acked-by: Rob Herring <robh@kernel.org> ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH 1/2] Documentation: fsl-quadspi: Add fsl, ls1043a-qspi compatible string @ 2016-04-01 17:14 ` Rob Herring 0 siblings, 0 replies; 18+ messages in thread From: Rob Herring @ 2016-04-01 17:14 UTC (permalink / raw) To: linux-arm-kernel On Thu, Mar 31, 2016 at 02:45:00PM +0800, Yuan Yao wrote: > 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(-) There's a typo in the compatible string in the subject, otherwise: Acked-by: Rob Herring <robh@kernel.org> ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH 1/2] Documentation: fsl-quadspi: Add fsl, ls1043a-qspi compatible string @ 2016-04-01 17:14 ` Rob Herring 0 siblings, 0 replies; 18+ 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] 18+ messages in thread
* [PATCH 2/2] dts/ls1043a: add the DTS node for QSPI support 2016-03-31 6:44 ` Yuan Yao (?) @ 2016-03-31 6:45 ` Yuan Yao -1 siblings, 0 replies; 18+ 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: linux-arm-kernel, linux-kernel, devicetree, 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] 18+ messages in thread
* [PATCH 2/2] dts/ls1043a: add the DTS node for QSPI support @ 2016-03-31 6:45 ` Yuan Yao 0 siblings, 0 replies; 18+ messages in thread From: Yuan Yao @ 2016-03-31 6:45 UTC (permalink / raw) To: linux-arm-kernel 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 at 0 { + compatible = "spansion,m25p80"; + #address-cells = <1>; + #size-cells = <1>; + spi-max-frequency = <20000000>; + ddr-quad-read; + reg = <0>; + }; +}; + &i2c0 { status = "okay"; pca9547 at 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 at 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 at 2180000 { compatible = "fsl,vf610-i2c"; #address-cells = <1>; -- 2.1.0.27.g96db324 ^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH 2/2] dts/ls1043a: add the DTS node for QSPI support @ 2016-03-31 6:45 ` Yuan Yao 0 siblings, 0 replies; 18+ 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] 18+ messages in thread
* Re: [PATCH 2/2] dts/ls1043a: add the DTS node for QSPI support @ 2016-04-12 2:49 ` Shawn Guo 0 siblings, 0 replies; 18+ messages in thread From: Shawn Guo @ 2016-04-12 2:49 UTC (permalink / raw) To: Yuan Yao Cc: robh+dt, mark.rutland, computersforpeace, pawel.moll, han.xu, leoyang.li, devicetree, linux-kernel, linux-arm-kernel, Yuan Yao 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. > + 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@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH 2/2] dts/ls1043a: add the DTS node for QSPI support @ 2016-04-12 2:49 ` Shawn Guo 0 siblings, 0 replies; 18+ messages in thread From: Shawn Guo @ 2016-04-12 2:49 UTC (permalink / raw) To: linux-arm-kernel 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. > + 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 at 0 { > + compatible = "spansion,m25p80"; > + #address-cells = <1>; > + #size-cells = <1>; > + spi-max-frequency = <20000000>; > + ddr-quad-read; Ditto > + reg = <0>; > + }; > +}; > + > &i2c0 { > status = "okay"; > pca9547 at 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 at 1550000 { Please sort the node in <soc>.dtsi in order of unit-address. That said, the node should be added between ifc at 1530000 and esdhc at 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 at 2180000 { > compatible = "fsl,vf610-i2c"; > #address-cells = <1>; > -- > 2.1.0.27.g96db324 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel at lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH 2/2] dts/ls1043a: add the DTS node for QSPI support @ 2016-04-12 2:49 ` Shawn Guo 0 siblings, 0 replies; 18+ 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] 18+ 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 -1 siblings, 0 replies; 18+ messages in thread From: Yao Yuan @ 2016-04-13 2:41 UTC (permalink / raw) To: Shawn Guo, Yuan Yao Cc: robh+dt, mark.rutland, computersforpeace, pawel.moll, Han Xu, Yang-Leo Li, devicetree, linux-kernel, linux-arm-kernel 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] 18+ messages in thread
* [PATCH 2/2] dts/ls1043a: add the DTS node for QSPI support @ 2016-04-13 2:41 ` Yao Yuan 0 siblings, 0 replies; 18+ messages in thread From: Yao Yuan @ 2016-04-13 2:41 UTC (permalink / raw) To: linux-arm-kernel 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 at 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 at 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 at 1550000 { > > Please sort the node in <soc>.dtsi in order of unit-address. That said, the node > should be added between ifc at 1530000 and esdhc at 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 at 2180000 { > > compatible = "fsl,vf610-i2c"; > > #address-cells = <1>; > > -- > > 2.1.0.27.g96db324 > > > > > > _______________________________________________ > > linux-arm-kernel mailing list > > linux-arm-kernel at lists.infradead.org > > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > > ^ permalink raw reply [flat|nested] 18+ messages in thread
* RE: [PATCH 2/2] dts/ls1043a: add the DTS node for QSPI support @ 2016-04-13 2:41 ` Yao Yuan 0 siblings, 0 replies; 18+ messages in thread From: Yao Yuan @ 2016-04-13 2:41 UTC (permalink / raw) To: Shawn Guo, Yuan Yao Cc: robh+dt, mark.rutland, computersforpeace, pawel.moll, Han Xu, Yang-Leo Li, devicetree, linux-kernel, linux-arm-kernel 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] 18+ messages in thread
end of thread, other threads:[~2016-04-13 2:41 UTC | newest] Thread overview: 18+ 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:44 ` Yuan Yao 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 ` Yuan Yao 2016-03-31 6:45 ` Yuan Yao 2016-04-01 17:14 ` Rob Herring 2016-04-01 17:14 ` Rob Herring 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 2016-03-31 6:45 ` Yuan Yao 2016-03-31 6:45 ` Yuan Yao 2016-04-12 2:49 ` Shawn Guo 2016-04-12 2:49 ` Shawn Guo 2016-04-12 2:49 ` Shawn Guo 2016-04-13 2:41 ` Yao Yuan 2016-04-13 2:41 ` Yao Yuan 2016-04-13 2:41 ` Yao Yuan
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.