From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jY2Pq-0002Dx-Fa for barebox@lists.infradead.org; Mon, 11 May 2020 06:59:55 +0000 Date: Mon, 11 May 2020 08:59:52 +0200 From: Sascha Hauer Message-ID: <20200511065952.GA15934@pengutronix.de> References: <20200429153559.15086-1-rhi@pengutronix.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20200429153559.15086-1-rhi@pengutronix.de> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH] ARM: dts: vexpress-v2p-ca9: adapt fixed NOR flash partition setup To: Roland Hieber Cc: Barebox Mailing List On Wed, Apr 29, 2020 at 05:35:59PM +0200, Roland Hieber wrote: > Upstream DTS commit 62a5017bf825c9e4d317 ("ARM: dts: vexpress: specify > AFS partition") [1] introduced an empty node at /smb@4000000 > /motherboard/flash@0,00000000/partitions, which is preferred by the OF > partitions parser over the single partition nodes. In the same commit, > upstream set the compatible to "arm,arm-firmware-suite", which barebox > does not know about. Adapt our fixed partition setup accordingly by > wrapping all partitions in an extra "partitions" node with the correct > compatible. I wonder what happens when we run this code on a flash that uses the arm-firmware-suite partitioning. We may end up with inconsistent partitioning then. In the end the barebox vexpress code only ever runs on qemu and not on the real hardware, so this is not relevant. Either way the patch seems like a good start, so applied. Sascha > > [1]: https://git.kernel.org/linus/62a5017bf825c9e4d3176eb975a01c329a9f36 > > Fixes: 6940ba22c66ac1c71350 ("dts: update to v5.3-rc1") > Signed-off-by: Roland Hieber > --- > arch/arm/dts/vexpress-v2p-ca9.dts | 26 +++++++++++++++----------- > 1 file changed, 15 insertions(+), 11 deletions(-) > > diff --git a/arch/arm/dts/vexpress-v2p-ca9.dts b/arch/arm/dts/vexpress-v2p-ca9.dts > index d0975330f205..b13c114f0cb0 100644 > --- a/arch/arm/dts/vexpress-v2p-ca9.dts > +++ b/arch/arm/dts/vexpress-v2p-ca9.dts > @@ -16,19 +16,23 @@ flash@0,00000000 { > #address-cells = <1>; > #size-cells = <1>; > > - partition@0 { > - label = "barebox"; > - reg = <0x0 0x80000>; > - }; > + partitions { > + compatible = "fixed-partitions"; > > - barebox_env: partition@80000 { > - label = "barebox-environment"; > - reg = <0x80000 0x80000>; > - }; > + partition@0 { > + label = "barebox"; > + reg = <0x0 0x80000>; > + }; > + > + barebox_env: partition@80000 { > + label = "barebox-environment"; > + reg = <0x80000 0x80000>; > + }; > > - state_storage: partition@100000 { > - label = "barebox-state"; > - reg = <0x100000 0x100000>; > + state_storage: partition@100000 { > + label = "barebox-state"; > + reg = <0x100000 0x100000>; > + }; > }; > }; > }; > -- > 2.26.2 > > > _______________________________________________ > barebox mailing list > barebox@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/barebox > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox