From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from metis.ext.pengutronix.de ([2001:6f8:1178:4:290:27ff:fe1d:cc33]) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1XyIPQ-0002lY-LA for barebox@lists.infradead.org; Tue, 09 Dec 2014 10:52:49 +0000 Date: Tue, 9 Dec 2014 11:52:26 +0100 From: Sascha Hauer Message-ID: <20141209105226.GW30369@pengutronix.de> References: <1418032791-35329-1-git-send-email-t.gamez@phytec.de> <20141209091302.GP30369@pengutronix.de> <1418120207.5704.8.camel@lws-gamez.phytec.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1418120207.5704.8.camel@lws-gamez.phytec.de> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="iso-8859-15" Content-Transfer-Encoding: quoted-printable Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH 1/3] ARM: dts: am335x-phytec-phyflex: Increase kernel partition on SPI NOR To: Teresa =?iso-8859-15?Q?G=E1mez?= Cc: barebox@lists.infradead.org On Tue, Dec 09, 2014 at 11:16:47AM +0100, Teresa G=E1mez wrote: > Hello Sascha, > = > Am Dienstag, den 09.12.2014, 10:13 +0100 schrieb Sascha Hauer: > > Hi Teresa, > > = > > On Mon, Dec 08, 2014 at 10:59:49AM +0100, Teresa G=E1mez wrote: > > > Increase kernel partition to end of SPI NOR. > > > = > > > Signed-off-by: Teresa G=E1mez > > > --- > > > arch/arm/dts/am335x-phytec-phyflex.dts | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > = > > > diff --git a/arch/arm/dts/am335x-phytec-phyflex.dts b/arch/arm/dts/am= 335x-phytec-phyflex.dts > > > index c4481f5..0de7e87 100644 > > > --- a/arch/arm/dts/am335x-phytec-phyflex.dts > > > +++ b/arch/arm/dts/am335x-phytec-phyflex.dts > > > @@ -187,7 +187,7 @@ > > > = > > > partition@4 { > > > label =3D "kernel"; > > > - reg =3D <0xe0000 0x400000>; > > > + reg =3D <0xe0000 0x0>; > > > }; > > = > > I wasn't aware barebox handles this case like you intend here. What > > about the kernel? Won't you end up with a zero sized partition there? > = > have a look at include/linux/mtd/partitions.h in the linux kernel. > = > There is a macro defined = > #define MTDPART_SIZ_FULL (0) > = > And the comment above says: = > size: the partition size; if defined as MTDPART_SIZ_FULL, the partition > will extend to the end of the master MTD device. > = > This works fine for kernel and barebox. Oh nice, didn't know that. In this case consider your patches applied. 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 | _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox