[PATCH v3 3/3] powerpc/mpc85xx: add support for Keymile's kmcoge4 board

Scott Wood scottwood at freescale.com
Wed Apr 9 10:47:32 EST 2014


On Tue, 2014-03-25 at 14:41 +0100, Valentin Longchamp wrote:
> +	lbc: localbus at 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 at 0,0 {
> +			#address-cells = <1>;
> +			#size-cells = <1>;
> +			compatible = "fsl,elbc-fcm-nand";
> +			reg = <0 0 0x40000>;
> +
> +			partition at 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




More information about the Linuxppc-dev mailing list