On Tue, 2014-03-25 at 14:41 +0100, Valentin Longchamp wrote:
> +     lbc: localbus@ffe124000 {
> +             reg = <0xf 0xfe124000 0 0x1000>;
> +             ranges = <0 0 0xf 0xffa00000 0x00040000         /* LB 0 */
> +                       1 0 0xf 0xfb000000 0x00010000         /* LB 1 */
> +                       2 0 0xf 0xd0000000 0x10000000         /* LB 2 */
> +                       3 0 0xf 0xe0000000 0x10000000>;       /* LB 3 */
> +
> +             nand@0,0 {
> +                     #address-cells = <1>;
> +                     #size-cells = <1>;
> +                     compatible = "fsl,elbc-fcm-nand";
> +                     reg = <0 0 0x40000>;
> +
> +                     partition@0 {
> +                             label = "ubi0";
> +                             reg = <0x0 0x10000000>;
> +                     };

Putting partition info in the dts file is a bad habit and (as I've told
others) I don't think we should continue doing so in new dts files.

In this case it looks like you're just making the entire chip into one
partition, so I'm not sure what the point is of partitioning at all.

-Scott


_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Reply via email to