On Fri, 2013-11-01 at 15:58 -0700, Mark Rutland wrote: > > +#include "imx53.dtsi" > > + > > +/ { > > + model = "Voipac i.MX53 X53-DMM-668"; > > + compatible = "vp,imx53-dmm-668", "fsl,imx53"; > > + > > + memory@0 { > > + device_type = "memory"; > > + reg = <0x70000000 0x20000000>; > > + }; > > + > > + memory@1 { > > + device_type = "memory"; > > + reg = <0xb0000000 0x20000000>; > > + }; > > This looks wrong. Any unit-address should match the first reg entry, > and I've > not seen memory nodes with unit addresses anywhere else. > > Why not: > > memory { > device_type = "memory"; > reg = <0x70000000 0x20000000>, > <0xb0000000 0x20000000>; > }; This has already been fixed in the newer version of the patch. Regards; Rostislav Lisovy; -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html