On Wed, Dec 17, 2008 at 11:56:07AM -0800, Grant Erickson wrote:
> Added additional information for type and compatibility strings and
> interrupt information to the SDRAM0 memory-controller device tree
> nodes for AMCC PowerPC 405EX[r]-based boards to facilitate binding
> with the new "ibm,sdram-4xx-ddr2" EDAC memory controller adapter driver.
> 
> Signed-off-by: Grant Erickson <[email protected]>
> ---
> As support in the associated EDAC adapter driver is added over time,
> similar changes will/should be made to the DTS files for boards
> leveraging realizations of this "ibm,sdram-4xx-ddr2" controller,
> including the 440SP, 440SPe, 460EX, 460GT and 460SX.
> 
>  arch/powerpc/boot/dts/haleakala.dts |   11 ++++++++++-
>  arch/powerpc/boot/dts/kilauea.dts   |   11 ++++++++++-
>  arch/powerpc/boot/dts/makalu.dts    |   11 ++++++++++-
>  3 files changed, 30 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/powerpc/boot/dts/haleakala.dts 
> b/arch/powerpc/boot/dts/haleakala.dts
> index 513bc43..e45ce7e 100644
> --- a/arch/powerpc/boot/dts/haleakala.dts
> +++ b/arch/powerpc/boot/dts/haleakala.dts
> @@ -89,8 +89,17 @@
>               clock-frequency = <0>; /* Filled in by U-Boot */
>  
>               SDRAM0: memory-controller {
> -                     compatible = "ibm,sdram-405exr";
> +                     device_type = "memory-controller";

This should not have a device_type.

> +                     compatible = "ibm,sdram-405exr", "ibm,sdram-4xx-ddr2";
>                       dcr-reg = <0x010 0x002>;
> +                     #address-cells = <0>;
> +                     #size-cells = <0>;

This seems odd.  These should only be present if the node does, or at
least can, have subnodes - I don't see that it would.

> +                     #interrupt-cells = <1>;
> +                     interrupt-parent = <&SDRAM0>;
> +                     interrupts = <0x0 0x1>;
> +                     interrupt-map = </* ECCDED Error */ 0x0 &UIC2 0x5 0x4
> +                                      /* ECCSEC Error */ 0x1 &UIC2 0x6 0x4>;
> +                     interrupt-map-mask = <0xffffffff>;
>               };

-- 
David Gibson                    | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au  | minimalist, thank you.  NOT _the_ _other_
                                | _way_ _around_!
http://www.ozlabs.org/~dgibson
_______________________________________________
Linuxppc-dev mailing list
[email protected]
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Reply via email to