diff mbox

[09/10,RESEND] spi: Add SSP/SPI device tree documentation

Message ID 1343076052-27312-10-git-send-email-marex@denx.de (mailing list archive)
State Superseded, archived
Headers show

Commit Message

Marek Vasut July 23, 2012, 8:40 p.m. UTC
Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Attila Kinali <attila@kinali.ch>
Cc: Chris Ball <cjb@laptop.org>
CC: Dong Aisheng <b29396@freescale.com>
Cc: Fabio Estevam <fabio.estevam@freescale.com>
Cc: Grant Likely <grant.likely@secretlab.ca>
Cc: Linux ARM kernel <linux-arm-kernel@lists.infradead.org>
Cc: Mark Brown <broonie@opensource.wolfsonmicro.com>
CC: Shawn Guo <shawn.guo@linaro.org>
---
 Documentation/devicetree/bindings/spi/mxs-spi.txt |   18 ++++++++++++++++++
 1 file changed, 18 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/spi/mxs-spi.txt

Comments

Sergei Shtylyov July 24, 2012, 6:15 p.m. UTC | #1
Hello.

On 07/24/2012 12:40 AM, Marek Vasut wrote:

> Signed-off-by: Marek Vasut <marex@denx.de>
> Cc: Attila Kinali <attila@kinali.ch>
> Cc: Chris Ball <cjb@laptop.org>
> CC: Dong Aisheng <b29396@freescale.com>
> Cc: Fabio Estevam <fabio.estevam@freescale.com>
> Cc: Grant Likely <grant.likely@secretlab.ca>
> Cc: Linux ARM kernel <linux-arm-kernel@lists.infradead.org>
> Cc: Mark Brown <broonie@opensource.wolfsonmicro.com>
> CC: Shawn Guo <shawn.guo@linaro.org>
> ---
>  Documentation/devicetree/bindings/spi/mxs-spi.txt |   18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/spi/mxs-spi.txt

> diff --git a/Documentation/devicetree/bindings/spi/mxs-spi.txt b/Documentation/devicetree/bindings/spi/mxs-spi.txt
> new file mode 100644
> index 0000000..f1263a9
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/spi/mxs-spi.txt
> @@ -0,0 +1,18 @@
> +* Freescale MX233/MX28 SSP/SPI
> +
> +Required properties:
> +- compatible: Should be "fsl,<soc>-spi", where soc is "imx23" or "imx28"
> +- reg: Offset and length of the register set for the device
> +- interrupts: Should contain SSP interrupts (error irq first, dma irq second)
> +- fsl,ssp-dma-channel: APBX DMA channel for the SSP
> +
> +Example:
> +
> +ssp0: ssp@80010000 {
> +	#address-cells = <1>;
> +	#size-cells = <0>;
> +	compatible = "fsl,imx28-spi";
> +	reg = <0x80010000 2000>;

   Not 0x2000?

WBR, Sergei


------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
Marek Vasut July 24, 2012, 7:43 p.m. UTC | #2
Dear Sergei Shtylyov,

> Hello.
> 
> On 07/24/2012 12:40 AM, Marek Vasut wrote:
> > Signed-off-by: Marek Vasut <marex@denx.de>
> > Cc: Attila Kinali <attila@kinali.ch>
> > Cc: Chris Ball <cjb@laptop.org>
> > CC: Dong Aisheng <b29396@freescale.com>
> > Cc: Fabio Estevam <fabio.estevam@freescale.com>
> > Cc: Grant Likely <grant.likely@secretlab.ca>
> > Cc: Linux ARM kernel <linux-arm-kernel@lists.infradead.org>
> > Cc: Mark Brown <broonie@opensource.wolfsonmicro.com>
> > CC: Shawn Guo <shawn.guo@linaro.org>
> > ---
> > 
> >  Documentation/devicetree/bindings/spi/mxs-spi.txt |   18
> >  ++++++++++++++++++ 1 file changed, 18 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/spi/mxs-spi.txt
> > 
> > diff --git a/Documentation/devicetree/bindings/spi/mxs-spi.txt
> > b/Documentation/devicetree/bindings/spi/mxs-spi.txt new file mode 100644
> > index 0000000..f1263a9
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/spi/mxs-spi.txt
> > @@ -0,0 +1,18 @@
> > +* Freescale MX233/MX28 SSP/SPI
> > +
> > +Required properties:
> > +- compatible: Should be "fsl,<soc>-spi", where soc is "imx23" or "imx28"
> > +- reg: Offset and length of the register set for the device
> > +- interrupts: Should contain SSP interrupts (error irq first, dma irq
> > second) +- fsl,ssp-dma-channel: APBX DMA channel for the SSP
> > +
> > +Example:
> > +
> > +ssp0: ssp@80010000 {
> > +	#address-cells = <1>;
> > +	#size-cells = <0>;
> > +	compatible = "fsl,imx28-spi";
> > +	reg = <0x80010000 2000>;
> 
>    Not 0x2000?

This is actually not the first place I was curious about in the imx28 DTS ... 
yes, the size is 0x2000, but "2000" as is is used in multiple places across 
imx28. Shawn?

> WBR, Sergei

Best regards,
Marek Vasut

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
Shawn Guo July 28, 2012, 11:40 a.m. UTC | #3
On Tue, Jul 24, 2012 at 09:43:48PM +0200, Marek Vasut wrote:
> This is actually not the first place I was curious about in the imx28 DTS ... 
> yes, the size is 0x2000, but "2000" as is is used in multiple places across 
> imx28. Shawn?
> 
It seems that I overlooked the problem when it got introduced at the
first place.  I will send a patch to fix it.  Thanks for spotting it.
Marek Vasut July 28, 2012, 11:42 a.m. UTC | #4
Dear Shawn Guo,

> On Tue, Jul 24, 2012 at 09:43:48PM +0200, Marek Vasut wrote:
> > This is actually not the first place I was curious about in the imx28 DTS
> > ... yes, the size is 0x2000, but "2000" as is is used in multiple places
> > across imx28. Shawn?
> 
> It seems that I overlooked the problem when it got introduced at the
> first place.  I will send a patch to fix it.  Thanks for spotting it.

Ok good.

Mark, do you want me to send a V2 of only this particular patch or whole set (I 
think that's overkill)?

Best regards,
Marek Vasut

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/spi/mxs-spi.txt b/Documentation/devicetree/bindings/spi/mxs-spi.txt
new file mode 100644
index 0000000..f1263a9
--- /dev/null
+++ b/Documentation/devicetree/bindings/spi/mxs-spi.txt
@@ -0,0 +1,18 @@ 
+* Freescale MX233/MX28 SSP/SPI
+
+Required properties:
+- compatible: Should be "fsl,<soc>-spi", where soc is "imx23" or "imx28"
+- reg: Offset and length of the register set for the device
+- interrupts: Should contain SSP interrupts (error irq first, dma irq second)
+- fsl,ssp-dma-channel: APBX DMA channel for the SSP
+
+Example:
+
+ssp0: ssp@80010000 {
+	#address-cells = <1>;
+	#size-cells = <0>;
+	compatible = "fsl,imx28-spi";
+	reg = <0x80010000 2000>;
+	interrupts = <96 82>;
+	fsl,ssp-dma-channel = <0>;
+};