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


--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to