[PATCH] lite5200b: flash definition in dts

[Grant Likely]

On 8/1/07, Domen Puncer <domen.puncer at telargo.com> wrote:
> Add flash definition for to lite5200b dts, and while at it
> fix "ranges" for soc node.
>
>
> Signed-off-by: Domen Puncer <domen.puncer at telargo.com>
>
> ---
> Hi!
>
> Sylvain, it would be nice to have this merged.
>
> # cat /proc/mtd
> dev:    size   erasesize  name
> mtd0: 01000000 00020000 "data0"
> mtd1: 00f00000 00020000 "data1"
> mtd2: 00100000 00020000 "u-boot"
>
> Some benchmarks:
> read:  2.3 MB/s
> erase: 168 kB/s
> write: 7.3 kB/s
>
>
>  arch/powerpc/boot/dts/lite5200b.dts |   24 +++++++++++++++++++++++-
>  1 file changed, 23 insertions(+), 1 deletion(-)
>
> Index: work-powerpc.git/arch/powerpc/boot/dts/lite5200b.dts
> ===================================================================
> --- work-powerpc.git.orig/arch/powerpc/boot/dts/lite5200b.dts
> +++ work-powerpc.git/arch/powerpc/boot/dts/lite5200b.dts
> @@ -52,7 +52,8 @@
>                 revision = "";                  // from bootloader
>                 #interrupt-cells = <3>;
>                 device_type = "soc";
> -               ranges = <0 f0000000 f0010000>;
> +               ranges = <00000000 f0000000 00010000
> +                         fe000000 fe000000 02000000>;

I don't think this is the right approach.  I think the SoC node is
intended for describing the on-chip devices, and the ranges property
reflects that.  Shouldn't flash nodes be up 1 level?

Cheers,
g.

>                 reg = <f0000000 00010000>;
>                 bus-frequency = <0>;            // from bootloader
>                 system-frequency = <0>;         // from bootloader
> @@ -403,5 +404,26 @@
>                         compatible = "mpc5200b-sram\0mpc5200-sram\0sram";
>                         reg = <8000 4000>;
>                 };
> +
> +               flash at fe000000 {
> +                       device_type = "rom";
> +                       compatible = "direct-mapped";
> +                       probe-type = "CFI";
> +                       reg = <fe000000 01000000>;
> +                       bank-width = <1>;
> +                       partitions = <00000000 01000000>;
> +                       partition-names = "data0";
> +               };
> +
> +               flash at ff000000 {
> +                       device_type = "rom";
> +                       compatible = "direct-mapped";
> +                       probe-type = "CFI";
> +                       reg = <ff000000 01000000>;
> +                       bank-width = <1>;
> +                       partitions = <00000000 00f00000
> +                                     00f00000 00100000>;
> +                       partition-names = "data1", "u-boot";
> +               };
>         };
>  };
> _______________________________________________
> Linuxppc-embedded mailing list
> Linuxppc-embedded at ozlabs.org
> https://ozlabs.org/mailman/listinfo/linuxppc-embedded
>



-- 
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.
grant.likely at secretlab.ca
(403) 399-0195