[PATCH 4/7] Device tree for MPC5121 ADS

[Grant Likely]

Comments below.

On 1/8/08, John Rigby <jrigby at freescale.com> wrote:
> Bare minimum tree containing only
> what is currently supported.
>
> Signed-off-by: John Rigby <jrigby at freescale.com>
> ---
>  arch/powerpc/boot/dts/mpc5121ads.dts |  102 ++++++++++++++++++++++++++++++++++
>  1 files changed, 102 insertions(+), 0 deletions(-)
>  create mode 100644 arch/powerpc/boot/dts/mpc5121ads.dts
>
> diff --git a/arch/powerpc/boot/dts/mpc5121ads.dts b/arch/powerpc/boot/dts/mpc5121ads.dts
> new file mode 100644
> index 0000000..26471ff
> --- /dev/null
> +++ b/arch/powerpc/boot/dts/mpc5121ads.dts
> @@ -0,0 +1,102 @@
> +/*
> + * MPC5121E MDS Device Tree Source
> + *
> + * Copyright 2007 Freescale Semiconductor Inc.
> + *
> + * This program is free software; you can redistribute  it and/or modify it
> + * under  the terms of  the GNU General  Public License as published by the
> + * Free Software Foundation;  either version 2 of the  License, or (at your
> + * option) any later version.
> + */
> +
> +/ {
> +       model = "MPC5121ADS";
> +       compatible = "MPC5121ADS";

"fsl, mpc5121ads" (drop the caps and use the fsl, prefix)

> +       #address-cells = <1>;
> +       #size-cells = <1>;
> +
> +       cpus {
> +               #address-cells = <1>;
> +               #size-cells = <0>;
> +
> +               PowerPC,5121 at 0 {
> +                       device_type = "cpu";
> +                       reg = <0>;
> +                       d-cache-line-size = <20>;       // 32 bytes
> +                       i-cache-line-size = <20>;       // 32 bytes
> +                       d-cache-size = <8000>;          // L1, 32K
> +                       i-cache-size = <8000>;          // L1, 32K
> +                       ref-frequency = <3ef1480>;      // 66MHz ref clock
> +                       timebase-frequency = <2f34f60>; // 49.5MHz (396MHz/8) makes time tick correctly
> +                       bus-frequency = <bcd3d80>;      // 198MHz csb bus
> +                       clock-frequency = <179a7b00>;   // 396MHz ppc core ??
> +                       32-bit;
> +               };
> +       };
> +
> +       memory {
> +               device_type = "memory";
> +               reg = <00000000 10000000>;      // 256MB at 0
> +       };
> +
> +       cpld at 82000000 {
> +               device_type = "board-control";
> +               reg = <82000000 8000>;
> +       };
> +
> +       soc5121 at 80000000 {

soc at 80000000

"soc<cpu>@<addr>" is no longer recommended.

> +               #address-cells = <1>;
> +               #size-cells = <1>;
> +               #interrupt-cells = <2>;
> +               device_type = "soc";

- Drop device_type property
- add: compatible = "fsl,mpc5121-immr";

> +               ranges = <0 80000000 400000>;
> +               reg = <80000000 400000>;
> +               ref-frequency = <3ef1480>;      // 66MHz ref
> +               bus-frequency = <5e69ec0>;      // 99MHz ips ref
> +
> +               // IPIC
> +               // interrupts cell = <intr #, sense>
> +               // sense values match linux IORESOURCE_IRQ_* defines:
> +               // sense == 8: Level, low assertion
> +               // sense == 2: Edge, high-to-low change
> +               //
> +               ipic: pic at c00 {
> +                       interrupt-controller;
> +                       #address-cells = <0>;
> +                       #interrupt-cells = <2>;
> +                       reg = <c00 100>;
> +                       built-in;
> +                       device_type = "ipic";
> +               };
> +
> +               // 512x PSCs are not 52xx PSCs compatible
> +               // PSC3 serial port A aka ttyPSC0
> +               serial at 11300 {
> +                       device_type = "serial";
> +                       compatible = "mpc512x-psc-uart";

Specify the *exact* version in compatible first.  You can add generic
forms after that if you like, but the first entry should always be
exact (as per the generic names recommended practice).  It should be:
compatible = "fsl,mpc5121-psc-uart".

Also, make sure all compatible properties have the 'fsl,' prefix.

> +                       port-number = <0>;  // Logical port assignment
> +                       cell-index = <3>;
> +                       reg = <11300 100>;
> +                       interrupts = <28 8>; // actually the fifo irq
> +                       interrupt-parent = < &ipic >;
> +               };
> +
> +               // PSC4 serial port B aka ttyPSC1
> +               serial at 11400 {
> +                       device_type = "serial";
> +                       compatible = "mpc512x-psc-uart";
> +                       port-number = <1>;  // Logical port assignment

Drop port-number.  Aliases will be used instead.

> +                       cell-index = <4>;
> +                       reg = <11400 100>;
> +                       interrupts = <28 8>; // actually the fifo irq
> +                       interrupt-parent = < &ipic >;
> +               };
> +
> +               pscsfifo at 11f00 {
> +                       compatible = "mpc512x-pscsfifo";
> +                       reg = <11f00 100>;
> +                       interrupts = <28 8>;
> +                       interrupt-parent = < &ipic >;
> +               };
> +       };
> +};
> --
> 1.5.3.5.726.g41a7a
>
> _______________________________________________
> Linuxppc-dev mailing list
> Linuxppc-dev at ozlabs.org
> https://ozlabs.org/mailman/listinfo/linuxppc-dev
>


-- 
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.