On 11 December 2015 at 11:59, Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx> wrote: > Hi, > > On Tue, Dec 08, 2015 at 07:29:03PM +0100, codekipper@xxxxxxxxx wrote: >> From: Marcus Cooper <codekipper@xxxxxxxxx> >> >> The Itead Ibox is a multi board device based on the Allwinner A20 SoC. >> It contains the A20 Itead Core module and a base board for the external >> interfaces. >> >> The core module comes with 4GB NAND and 1GB DDR RAM. >> >> The base board to which the core board is connected provides 3 USB 2.0 Host >> ports, 1 USB 2.0 OTG, 1 uSD slot, 10/100 Ethernet port, HDMI, IR receiver, >> SPDIF and a 32-pin GPIO header. This header expands the features of core >> board by exposing the VGA pins, audio In/Out pins, SATA, SPI, I2C, UARTS, >> USB-OTG and power. >> >> As there is an A10 version of the Itead Core Module and at least one other >> base board to support then this patch partitions the device tree files with >> some consideration that these variants may be added later. >> >> Signed-off-by: Marcus Cooper <codekipper@xxxxxxxxx> >> --- >> arch/arm/boot/dts/Makefile | 1 + >> arch/arm/boot/dts/sun7i-a20-itead-core.dtsi | 83 ++++++++++++++++++ >> arch/arm/boot/dts/sun7i-a20-itead-ibox.dts | 98 ++++++++++++++++++++++ >> arch/arm/boot/dts/sunxi-itead-core-common.dtsi | 111 +++++++++++++++++++++++++ >> 4 files changed, 293 insertions(+) >> create mode 100644 arch/arm/boot/dts/sun7i-a20-itead-core.dtsi >> create mode 100644 arch/arm/boot/dts/sun7i-a20-itead-ibox.dts >> create mode 100644 arch/arm/boot/dts/sunxi-itead-core-common.dtsi > > So, it looks like you're doing several things in this patch: > > 1) Create a common DTSI for all itead core variants > (sunxi-itead-core-common.dtsi) > > 2) Create an A20 variant of the itead core > 3) Add support for a board using the A20 itead core. > > Please make three different patches for this, and converting the A10 > itead board would be a good idea too. Sure, I will split this into four(or five) patches and include the A10 variant. I haven't got a A10 Itead board but I've seen that someone has just popped on irc asking questions about it. I will ping them to see if they can verify it for me. CK > > Thanks! > Maxime > > -- > Maxime Ripard, Free Electrons > Embedded Linux, Kernel and Android engineering > http://free-electrons.com -- 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