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.80.1 #2 (Red Hat Linux)) id 1ZFd93-0003Zm-Vw for barebox@lists.infradead.org; Thu, 16 Jul 2015 06:59:50 +0000 Date: Thu, 16 Jul 2015 08:59:27 +0200 From: Sascha Hauer Message-ID: <20150716065927.GA18700@pengutronix.de> References: <1436890599-31412-1-git-send-email-s.trumtrar@pengutronix.de> <20150716060723.GX18700@pengutronix.de> <20150716064700.GC20836@pengutronix.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20150716064700.GC20836@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 1/5] ARM: socfpga: socrates: add qspi partitions To: Steffen Trumtrar Cc: barebox@lists.infradead.org On Thu, Jul 16, 2015 at 08:47:00AM +0200, Steffen Trumtrar wrote: > On Thu, Jul 16, 2015 at 08:07:23AM +0200, Sascha Hauer wrote: > > On Tue, Jul 14, 2015 at 06:16:35PM +0200, Steffen Trumtrar wrote: > > > Signed-off-by: Steffen Trumtrar > > > --- > > > arch/arm/dts/socfpga_cyclone5_socrates.dts | 15 +++++++++++++++ > > > 1 file changed, 15 insertions(+) > > > > > > diff --git a/arch/arm/dts/socfpga_cyclone5_socrates.dts b/arch/arm/dts/socfpga_cyclone5_socrates.dts > > > index 125ad1b8502d..ccb940584b5b 100644 > > > --- a/arch/arm/dts/socfpga_cyclone5_socrates.dts > > > +++ b/arch/arm/dts/socfpga_cyclone5_socrates.dts > > > @@ -80,5 +80,20 @@ > > > cdns,tsd2d-ns = <50>; > > > cdns,tchsh-ns = <4>; > > > cdns,tslch-ns = <4>; > > > + > > > + partition@0 { > > > + label = "prebootloader0"; > > > + reg = <0x00000 0x10000>; > > > + }; > > > > Why prebootloader0? There's only one or not? > > > > There can be up to four and the space for the other > three is left free here. I thought prebootloader1-3 might > be added later. Ah, I see, there currently is a hole between prebootloader0 and barebox0 which is big enough to hold three other prebootloaders. In which cases is one of the other slots used? Would it make sense to change the spinor handler to write the image there aswell? 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