From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Wed, 10 Dec 2014 21:20:07 -0500 Subject: [U-Boot] Use .data section in SPL ? In-Reply-To: References: <20141210163938.02D8.AA925319@jp.panasonic.com> <20141210140602.GQ20704@bill-the-cat> <20141211020447.GU20704@bill-the-cat> Message-ID: <20141211021917.GV20704@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Wed, Dec 10, 2014 at 07:13:51PM -0700, Simon Glass wrote: > Hi Tom, > > On 10 December 2014 at 19:04, Tom Rini wrote: > > On Wed, Dec 10, 2014 at 06:56:02PM -0700, Simon Glass wrote: > >> Hi Tom, > >> > >> On 10 December 2014 at 07:06, Tom Rini wrote: > >> > On Wed, Dec 10, 2014 at 04:39:39PM +0900, Masahiro Yamada wrote: > >> >> Hi. > >> >> > >> >> > >> >> I notice SPL is using .data section > >> >> > >> >> > >> >> > >> >> u32 *boot_params_ptr = NULL; > >> >> struct spl_image_info spl_image; > >> >> > >> >> /* Define board data structure */ > >> >> static bd_t bdata __attribute__ ((section(".data"))); > >> >> > >> >> > >> >> > >> >> If SPL is running on an execute-in-place device such NOR flash, > >> >> .data section is not writable. > >> >> > >> >> common/spl/spl_nor.c does not work. > >> > > >> > spl_nor.c only works today on PowerPC where there's nothing shoved into > >> > the .data section. > >> > > >> > And all of this is related to what's needed to enable DM in SPL on ARM > >> > as well, iirc anyhow. > >> > >> Can you please explain the SPL problem on ARM a bit more? If It could > >> understand what is going on, or how to repeat it, I might be able to > >> help. I'd really like to enable this DM SPL support... > > > > Did you see the WIP patch I sent out in reply to the SPL DM series? > > Not that I recall - you mean this one? > > http://patchwork.ozlabs.org/patch/417792/ > > I might have been asleep. Will take a look. Yup, that's the one. Fixing that (and the similar constructs the other TI platforms have) and then a few other similar constructs elsewhere is why we have to throw things into .data but write to them today in ARM. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: