Subject: Re: [PATCH] powerpc/83xx: Add power management support for MPC837x boards

From: Scott Wood <[email-masked]>
Date: Fri, 20 Feb 2009 11:31:09 -0600

Anton Vorontsov wrote:
> diff --git a/arch/powerpc/boot/dts/mpc8377_mds.dts b/arch/powerpc/boot/dts/mpc8377_mds.dts
> index 3e3ec8f..c54b90d 100644
> --- a/arch/powerpc/boot/dts/mpc8377_mds.dts
> +++ b/arch/powerpc/boot/dts/mpc8377_mds.dts
> @@ -137,6 +137,7 @@
>  			reg = <0x3000 0x100>;
>  			interrupts = <14 0x8>;
>  			interrupt-parent = <&ipic>;
> +			sleep = <&pmc 0x0c000000>;
>  			dfsrr;

[snip]

> @@ -318,6 +325,7 @@
>  			reg = <0x2e000 0x1000>;
>  			interrupts = <42 0x8>;
>  			interrupt-parent = <&ipic>;
> +			sleep = <&pmc 0x0c000000>;

You have two different nodes with the same bits referenced.  These nodes 
cannot be put to sleep independently, and as such need to be under a 
common sleep-nexus node.

-Scott