Subject: Re: [PATCH] powerpc/4xx: Add GPIO, SPI and MMC suport to the Arches

From: Stefan Roese <[email-masked]>
Date: Fri, 31 Oct 2008 15:31:05 +0100

Hi Anton,

thanks for your comments.

On Friday 31 October 2008, Anton Vorontsov wrote:
> > diff --git a/arch/powerpc/boot/dts/arches.dts
> > b/arch/powerpc/boot/dts/arches.dts index d9113b1..c568095 100644
> > --- a/arch/powerpc/boot/dts/arches.dts
> > +++ b/arch/powerpc/boot/dts/arches.dts
> > @@ -195,6 +195,38 @@
> >  				interrupts = <0x3 0x4>;
> >  			};
> >
> > +			SPI0: spi@ef600900 {
> > +				#address-cells = <1>;
> > +				#size-cells = <0>;
> > +				device_type = "spi";
>
> device_type isn't needed.

OK, will remove.

> > +				compatible = "ibm,spi-460gt", "ibm,ppc4xx-spi";
> > +				reg = <0xef600900 7>;
> > +				interrupt-parent = <&UIC1>;
> > +				interrupts = <2 4>;
> > +
> > +				/* gpio(s) representing chip select(s) */
> > +				gpios = <&GPIO1 11 0>;
> > +
> > +				mmc-slot@0 {
> > +					compatible = "linux,mmc-spi", "mmc-spi";
>
> According to new (though still not approved) bindings, this should
> be "<board-specific>-mmc-slot", "mmc-spi-slot".

So I will wait a while with re-sending until this is "approved".

> > +					reg = <0>;
> > +					spi-max-frequency = <10000000>;
> > +					voltage-range = <3300 3300>;
>
> For new bindings I changed this to "voltage-ranges", so that we
> could specify discontinuous range.

OK, will change in next version.

Thanks.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: [email-masked]
=====================================================================