On Jan 29, 2009, at 1:20 PM, Anton Vorontsov wrote:

On Thu, Jan 29, 2009 at 06:37:15PM +0100, Reynes Philippe wrote:

quoted

Signed-off-by: Philippe Reynes <redacted>
---

diff --git a/arch/powerpc/boot/dts/mpc8377_rdb.dts

b/arch/powerpc/boot/dts/mpc8377_rdb.dts
index 165463f..45c9f03 100644

--- a/arch/powerpc/boot/dts/mpc8377_rdb.dts
+++ b/arch/powerpc/boot/dts/mpc8377_rdb.dts

@@ -109,6 +109,24 @@

			reg = <0x200 0x100>;
		};

+		gpio1: gpio-controller@c00 {
+			#gpio-cells = <2>;
+			compatible = "fsl,mpc8347-gpio",

I think this should be "fsl,mpc8377-gpio", "fsl,mpc8349-gpio".

[...]

quoted

--- a/arch/powerpc/boot/dts/mpc8378_rdb.dts
+++ b/arch/powerpc/boot/dts/mpc8378_rdb.dts

@@ -109,6 +109,24 @@

			reg = <0x200 0x100>;
		};

+		gpio1: gpio-controller@c00 {
+			#gpio-cells = <2>;
+			compatible = "fsl,mpc8347-gpio",
"fsl,mpc8349-gpio";

"fsl,mpc8378-gpio", "fsl,mpc8349-gpio".

[...]

quoted

--- a/arch/powerpc/boot/dts/mpc8379_rdb.dts
+++ b/arch/powerpc/boot/dts/mpc8379_rdb.dts

@@ -107,6 +107,24 @@

			reg = <0x200 0x100>;
		};

+		gpio1: gpio-controller@c00 {
+			#gpio-cells = <2>;
+			compatible = "fsl,mpc8347-gpio",
"fsl,mpc8349-gpio";

"fsl,mpc8379-gpio", "fsl,mpc8349-gpio".

Agreed w/Anton here.

- k