From mboxrd@z Thu Jan  1 00:00:00 1970
From: s.hauer@pengutronix.de (Sascha Hauer)
Date: Thu, 3 Apr 2014 08:49:59 +0200
Subject: [PATCH] ARM i.MX51: Add Digi ConnectCore devicetree
In-Reply-To: <533C4FF5.1010402@gmail.com>
References: <1396075706-25384-1-git-send-email-shc_work@mail.ru>
 <1396075706-25384-2-git-send-email-shc_work@mail.ru>
 <20140402113934.GH8753@dragon>
 <20140402125318.GZ17250@pengutronix.de>
 <20140402174458.GC27994@pengutronix.de>
 <533C4FF5.1010402@gmail.com>
Message-ID: <20140403064959.GD17250@pengutronix.de>
To: linux-arm-kernel@lists.infradead.org
List-Id: linux-arm-kernel.lists.infradead.org

On Wed, Apr 02, 2014 at 07:59:17PM +0200, Dirk Behme wrote:
> Am 02.04.2014 19:44, schrieb Robert Schwebel:
> >[adding Nicolas to Cc]
> >
> >On Wed, Apr 02, 2014 at 02:53:18PM +0200, Sascha Hauer wrote:
> >>On Wed, Apr 02, 2014 at 07:39:35PM +0800, Shawn Guo wrote:
> >>>On Sat, Mar 29, 2014 at 10:48:26AM +0400, Alexander Shiyan wrote:
> >>>>+&nfc {
> >>>>+	pinctrl-names = "default";
> >>>>+	pinctrl-0 = <&pinctrl_nfc>;
> >>>>+	nand-bus-width = <8>;
> >>>>+	nand-ecc-mode = "hw";
> >>>>+	nand-on-flash-bbt;
> >>>>+	#address-cells = <1>;
> >>>>+	#size-cells = <1>;
> >>>>+	status = "okay";
> >>>>+
> >>>>+	partition at 0 {
> >>>>+		label = "boot";
> >>>>+		reg = <0x00000 0x80000>;
> >>>>+	};
> >>>>+
> >>>>+	partition at 1 {
> >>>>+		label = "env";
> >>>>+		reg = <0x80000 0x40000>;
> >>>>+	};
> >>>
> >>>@Sascha, does the argument that we shouldn't have partition table in the
> >>>mainline dtb still stands?
> >>
> >>Yes, I still think the partitions shouldn't be in the Linux dts files.
> >
> >The dtb files for the Atmel Xplained board also has a partition
> >definition, so the same comment applies there, too.
> >
> >Besides being no hardware description, this implies a certain way of
> >using the system; for example, with the "kernel" plus "rootfs"
> >partitions, it suggests that you can't have the kernel in the standard
> >root filesystem (which is possible with barebox and the freedesktop.org
> >bootloader specification).
> 
> Just out of curiosity, because I never got a good answer to this
> question (but maybe I asked the wrong way):
> 
> If not in the dts, where should the partition table be put instead, then?

I think the bootloader should provide it. The bootloader has the chance
to fill in the correct data from its own configuration (which can be a
devicetree aswell). We should at least make sure that the bootloader and
the kernel use a consistent partition table.
(Which reminds me that barebox also doesn't do this correctly atm)

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |