On Thu, Mar 27, 2014 at 04:56:34PM +0100, Hans de Goede wrote:
> Hi,
> 
> On 03/27/2014 10:59 AM, Maxime Ripard wrote:
> > On Wed, Mar 26, 2014 at 09:18:00PM +0100, Hans de Goede wrote:
> >> Add a new sun6i-a31-m9 dts file for the Mele M9 / Mele A1000G Quad. These
> >> HTPCs use the same board in a different case, for more details see:
> >> http://linux-sunxi.org/Mele_M9
> >>
> >> Signed-off-by: Hans de Goede <hdego...@redhat.com>
> >> ---
> >>  arch/arm/boot/dts/Makefile         |  1 +
> >>  arch/arm/boot/dts/sun6i-a31-m9.dts | 30 ++++++++++++++++++++++++++++++
> >>  2 files changed, 31 insertions(+)
> >>  create mode 100644 arch/arm/boot/dts/sun6i-a31-m9.dts
> >>
> >> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> >> index aaa130e..5c02aac 100644
> >> --- a/arch/arm/boot/dts/Makefile
> >> +++ b/arch/arm/boot/dts/Makefile
> >> @@ -291,6 +291,7 @@ dtb-$(CONFIG_ARCH_SUNXI) += \
> >>    sun5i-a13-olinuxino.dtb \
> >>    sun5i-a13-olinuxino-micro.dtb \
> >>    sun6i-a31-colombus.dtb \
> >> +  sun6i-a31-m9.dtb \
> >>    sun7i-a20-cubieboard2.dtb \
> >>    sun7i-a20-cubietruck.dtb \
> >>    sun7i-a20-olinuxino-micro.dtb
> >> diff --git a/arch/arm/boot/dts/sun6i-a31-m9.dts 
> >> b/arch/arm/boot/dts/sun6i-a31-m9.dts
> >> new file mode 100644
> >> index 0000000..c95ee77
> >> --- /dev/null
> >> +++ b/arch/arm/boot/dts/sun6i-a31-m9.dts
> >> @@ -0,0 +1,30 @@
> >> +/*
> >> + * Copyright 2014 Hans de Goede <hdego...@redhat.com>
> >> + *
> >> + * 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
> >> + */
> >> +
> >> +/dts-v1/;
> >> +/include/ "sun6i-a31.dtsi"
> >> +
> >> +/ {
> >> +  model = "Mele M9 / A1000G Quad top set box";
> >> +  compatible = "mele,m9", "allwinner,sun6i-a31";
> >> +
> >> +  chosen {
> >> +          bootargs = "earlyprintk console=ttyS0,115200";
> >> +  };
> > 
> > How much memory does it have? You could probably add a memory node here.
> 
> It has 8x hynix h5tq2g83efr, so 2 GiB.
> 
> I don't see a memory node in any of the other sunxi dts files either,
> isn't that supposed to get dynamically added by u-boot ?

It does, but not on the A31, since we don't have any DT-enabled u-boot
(yet). So providing a default for the A31 boards is a good idea for
the time being.

> We've some boards, ie the cubieboard were there are both 512MB and
> 1G versions, and u-boot does some sort of memtest to figure out
> which is which (assuming that lower caps use lower caps ram chips,
> not half the chips then this won't work). So I don't think that
> hardcoding this in the dts is a good idea.

Maybe we need to enable ARM_ATAG_DTB_COMPAT for the A31 case then.

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

Attachment: signature.asc
Description: Digital signature

Reply via email to