On 12/15/11 02:53, Ilya Yanok wrote: > Very basic support for HTKW mcx board. Able to boot via board-generic > and ramdisk/initramfs, however most of peripherals is unsupported. > Produces tons of twl4030 related errors as this board doesn't have > twl4030 installed. > > Signed-off-by: Ilya Yanok <yanok@xxxxxxxxxxx> > > --- > Changes from V1: > > - device tree moved to the separate patch > - iva node is disabled instead of using custom includes > - removed bootargs entry > > arch/arm/boot/dts/mcx.dts | 27 +++++++++++++++++++++++++++ > 1 files changed, 27 insertions(+), 0 deletions(-) > create mode 100644 arch/arm/boot/dts/mcx.dts > > diff --git a/arch/arm/boot/dts/mcx.dts b/arch/arm/boot/dts/mcx.dts > new file mode 100644 > index 0000000..66b81bd > --- /dev/null > +++ b/arch/arm/boot/dts/mcx.dts > @@ -0,0 +1,27 @@ > +/* > + * Copyright (C) 2011 Ilya Yanok, EmCraft Systems > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 as > + * published by the Free Software Foundation. > + */ > +/dts-v1/; > + > +/include/ "omap3.dtsi" > + > +/ { > + model = "HTKW mcx"; > + compatible = "htkw,mcx", "ti,omap3"; > + > + memory { > + device_type = "memory"; > + reg = <0x80000000 0x10000000>; /* 256 MB */ > + }; > + > + /* AM35xx doesn't have IVA */ > + soc { > + iva { > + status = "disabled"; > + }; > + }; I don't get it... Why SoCs that do not have those IP blocks should poke their configuration inside the h/w description (e.g. disable/enable/workaround/hack)? This way, why don't we also disable the PCIe which this SoC does not have? Of course, I'm exaggerating, but this just does not scale... Soon you will have a bunch of boards disabling stuff, that they *do not have natively*... Why don't generic OMAP3 DT file disable the EMAC? If we will go this way, we will find ourself fixing it later and producing the renaming/moving "churn", won't we? -- Regards, Igor. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html