On Wed, 29 May 2013, Gabriel Fernandez wrote: > From: Gabriel Fernandez <gabriel.fernan...@st.com> > > This board includes the U8540 platform and specifies only the > ST-Ericsson UIB version 3.
I'm confused by this. Is it a new board, or just a UIB? > --- > arch/arm/boot/dts/Makefile | 1 + > arch/arm/boot/dts/ccu8540-uib-v3.dts | 16 ++++++++++++++++ > 2 files changed, 17 insertions(+) > create mode 100644 arch/arm/boot/dts/ccu8540-uib-v3.dts > > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > index 1995988..29cb5db 100644 > --- a/arch/arm/boot/dts/Makefile > +++ b/arch/arm/boot/dts/Makefile > @@ -159,6 +159,7 @@ dtb-$(CONFIG_ARCH_U8500) += snowball.dtb \ > hrefprev60.dtb \ > hrefv60plus.dtb \ > ccu8540.dtb \ > + ccu8540-uib-v3.dtb \ > ccu9540.dtb > dtb-$(CONFIG_ARCH_SHMOBILE) += emev2-kzm9d.dtb \ > r8a7740-armadillo800eva.dtb \ > diff --git a/arch/arm/boot/dts/ccu8540-uib-v3.dts > b/arch/arm/boot/dts/ccu8540-uib-v3.dts > new file mode 100644 > index 0000000..4b4982e > --- /dev/null > +++ b/arch/arm/boot/dts/ccu8540-uib-v3.dts If this is a UIB, it should be an include file (*.dtsi). > @@ -0,0 +1,16 @@ > +/* > + * Copyright 2012 ST-Ericsson AB > + * > + * The code contained herein is licensed under the GNU General Public > + * License. You may obtain a copy of the GNU General Public License > + * Version 2 or later at the following locations: > + * > + * http://www.opensource.org/licenses/gpl-license.html > + * http://www.gnu.org/copyleft/gpl.html > + */ > + > +#include "ccu8540.dts" No, the other way round. ccu8540.dts should include this one. > +/ { > + model = "ST-Ericsson U8540 platform with device tree UIB v3"; > +}; This patch doesn't actually do anything. Fix it up, so it at least as some functionality. -- Lee Jones Linaro ST-Ericsson Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
