diff mbox

[5/6] powerpc/of: add eSPI controller dts bindings

Message ID 1279591705-7574-6-git-send-email-Mingkai.hu@freescale.com (mailing list archive)
State Changes Requested
Delegated to: Grant Likely
Headers show

Commit Message

Mingkai Hu July 20, 2010, 2:08 a.m. UTC
Signed-off-by: Mingkai Hu <Mingkai.hu@freescale.com>
---
 Documentation/powerpc/dts-bindings/fsl/spi.txt |   20 ++++++++++++++++++++
 1 files changed, 20 insertions(+), 0 deletions(-)

Comments

Grant Likely July 26, 2010, 12:33 a.m. UTC | #1
On Tue, Jul 20, 2010 at 10:08:24AM +0800, Mingkai Hu wrote:
> Signed-off-by: Mingkai Hu <Mingkai.hu@freescale.com>
> ---
>  Documentation/powerpc/dts-bindings/fsl/spi.txt |   20 ++++++++++++++++++++
>  1 files changed, 20 insertions(+), 0 deletions(-)
> 
> diff --git a/Documentation/powerpc/dts-bindings/fsl/spi.txt b/Documentation/powerpc/dts-bindings/fsl/spi.txt
> index 80510c0..b360bf9 100644
> --- a/Documentation/powerpc/dts-bindings/fsl/spi.txt
> +++ b/Documentation/powerpc/dts-bindings/fsl/spi.txt
> @@ -29,3 +29,23 @@ Example:
>  		gpios = <&gpio 18 1	// device reg=<0>
>  			 &gpio 19 1>;	// device reg=<1>
>  	};
> +
> +
> +* eSPI (Enhanced Serial Peripheral Interface)
> +
> +Required properties:
> +- compatible : should be "fsl,espi".

Good practice is to always fully identify the SoC in the compatible values, followed by an optional list of other specific chips it is compatible with.  Generic compatibles like "fsl,espi" are not a good idea.

+- compatible: should be "fsl,<chip>-espi".

> +- reg : Offset and length of the register set for the device.
> +- interrupts : should contain eSPI interrupt, the device has one interrupt.
> +- fsl,espi-num-chipselects : the number of the chipselect signals.
> +
> +Example:
> +	spi@110000 {
> +		#address-cells = <1>;
> +		#size-cells = <0>;
> +		compatible = "fsl,espi";
> +		reg = <0x110000 0x1000>;
> +		interrupts = <53 0x2>;
> +		interrupt-parent = <&mpic>;
> +		fsl,espi-num-chipselects = <4>;
> +	};
> -- 
> 1.6.4
> 
>
Hu Mingkai-B21284 July 26, 2010, 7:35 a.m. UTC | #2
> -----Original Message-----
> From: Grant Likely [mailto:glikely@secretlab.ca] On Behalf Of 
> Grant Likely
> Sent: Monday, July 26, 2010 8:34 AM
> To: Hu Mingkai-B21284
> Cc: linuxppc-dev@ozlabs.org; galak@kernel.crashing.org; Zang 
> Roy-R61911
> Subject: Re: [PATCH 5/6] powerpc/of: add eSPI controller dts bindings
> 
> On Tue, Jul 20, 2010 at 10:08:24AM +0800, Mingkai Hu wrote:
> > Signed-off-by: Mingkai Hu <Mingkai.hu@freescale.com>
> > ---
> >  Documentation/powerpc/dts-bindings/fsl/spi.txt |   20 
> ++++++++++++++++++++
> >  1 files changed, 20 insertions(+), 0 deletions(-)
> > 
> > diff --git a/Documentation/powerpc/dts-bindings/fsl/spi.txt 
> > b/Documentation/powerpc/dts-bindings/fsl/spi.txt
> > index 80510c0..b360bf9 100644
> > --- a/Documentation/powerpc/dts-bindings/fsl/spi.txt
> > +++ b/Documentation/powerpc/dts-bindings/fsl/spi.txt
> > @@ -29,3 +29,23 @@ Example:
> >  		gpios = <&gpio 18 1	// device reg=<0>
> >  			 &gpio 19 1>;	// device reg=<1>
> >  	};
> > +
> > +
> > +* eSPI (Enhanced Serial Peripheral Interface)
> > +
> > +Required properties:
> > +- compatible : should be "fsl,espi".
> 
> Good practice is to always fully identify the SoC in the 
> compatible values, followed by an optional list of other 
> specific chips it is compatible with.  Generic compatibles 
> like "fsl,espi" are not a good idea.
> 
> +- compatible: should be "fsl,<chip>-espi".
> 

Ok, the mpc8536 is the first chip to develop this driver, so will use
"fsl,mpc8536-espi".

Thanks,
Mingkai
diff mbox

Patch

diff --git a/Documentation/powerpc/dts-bindings/fsl/spi.txt b/Documentation/powerpc/dts-bindings/fsl/spi.txt
index 80510c0..b360bf9 100644
--- a/Documentation/powerpc/dts-bindings/fsl/spi.txt
+++ b/Documentation/powerpc/dts-bindings/fsl/spi.txt
@@ -29,3 +29,23 @@  Example:
 		gpios = <&gpio 18 1	// device reg=<0>
 			 &gpio 19 1>;	// device reg=<1>
 	};
+
+
+* eSPI (Enhanced Serial Peripheral Interface)
+
+Required properties:
+- compatible : should be "fsl,espi".
+- reg : Offset and length of the register set for the device.
+- interrupts : should contain eSPI interrupt, the device has one interrupt.
+- fsl,espi-num-chipselects : the number of the chipselect signals.
+
+Example:
+	spi@110000 {
+		#address-cells = <1>;
+		#size-cells = <0>;
+		compatible = "fsl,espi";
+		reg = <0x110000 0x1000>;
+		interrupts = <53 0x2>;
+		interrupt-parent = <&mpic>;
+		fsl,espi-num-chipselects = <4>;
+	};