* [U-Boot] OMAP3 SPL Overflow @ 2016-08-21 11:29 Adam Ford 2016-08-21 12:45 ` Tom Rini 0 siblings, 1 reply; 6+ messages in thread From: Adam Ford @ 2016-08-21 11:29 UTC (permalink / raw) To: u-boot Using Git bisect, it appears to me that a78cd8613204188991c192b8dae2de0aae3b1722 makes "region `.sram' overflowed by 888 bytes" when I compile for omap3_logic. As I move forward through the git commits that overflow amount is continuing to grow. Does anyone else with an OMAP3 have this issue? adam ^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] OMAP3 SPL Overflow 2016-08-21 11:29 [U-Boot] OMAP3 SPL Overflow Adam Ford @ 2016-08-21 12:45 ` Tom Rini 2016-08-22 4:43 ` Lokesh Vutla 0 siblings, 1 reply; 6+ messages in thread From: Tom Rini @ 2016-08-21 12:45 UTC (permalink / raw) To: u-boot On Sun, Aug 21, 2016 at 06:29:42AM -0500, Adam Ford wrote: > Using Git bisect, it appears to me that > a78cd8613204188991c192b8dae2de0aae3b1722 makes "region `.sram' > overflowed by 888 bytes" when I compile for omap3_logic. As I move > forward through the git commits that overflow amount is continuing to > grow. > > Does anyone else with an OMAP3 have this issue? The best solution I have for this at current is a patch that re-works how we use the download image area and compacts the size we reserve for the scratch area to be still much larger than required. I'll cc you off-list so you can test it, I put aside testing this when I couldn't convince my beagleboard classic to boot from SD (but the xM is fine with it). -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160821/cd3678e7/attachment.sig> ^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] OMAP3 SPL Overflow 2016-08-21 12:45 ` Tom Rini @ 2016-08-22 4:43 ` Lokesh Vutla 2016-08-22 12:08 ` Tom Rini 0 siblings, 1 reply; 6+ messages in thread From: Lokesh Vutla @ 2016-08-22 4:43 UTC (permalink / raw) To: u-boot On Sunday 21 August 2016 06:15 PM, Tom Rini wrote: > On Sun, Aug 21, 2016 at 06:29:42AM -0500, Adam Ford wrote: > >> Using Git bisect, it appears to me that >> a78cd8613204188991c192b8dae2de0aae3b1722 makes "region `.sram' >> overflowed by 888 bytes" when I compile for omap3_logic. As I move >> forward through the git commits that overflow amount is continuing to >> grow. >> >> Does anyone else with an OMAP3 have this issue? > > The best solution I have for this at current is a patch that re-works > how we use the download image area and compacts the size we reserve for > the scratch area to be still much larger than required. I agree here. Right now the scratch addr space is 4KB. We can reduce it to 1KB. Below is the patch that is being used in ti-u-boot to solve this issue. -----------8<------------------8<------------------------------------- ^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] OMAP3 SPL Overflow 2016-08-22 4:43 ` Lokesh Vutla @ 2016-08-22 12:08 ` Tom Rini 2016-08-25 16:02 ` Adam Ford 0 siblings, 1 reply; 6+ messages in thread From: Tom Rini @ 2016-08-22 12:08 UTC (permalink / raw) To: u-boot On Mon, Aug 22, 2016 at 10:13:24AM +0530, Lokesh Vutla wrote: > > > On Sunday 21 August 2016 06:15 PM, Tom Rini wrote: > > On Sun, Aug 21, 2016 at 06:29:42AM -0500, Adam Ford wrote: > > > >> Using Git bisect, it appears to me that > >> a78cd8613204188991c192b8dae2de0aae3b1722 makes "region `.sram' > >> overflowed by 888 bytes" when I compile for omap3_logic. As I move > >> forward through the git commits that overflow amount is continuing to > >> grow. > >> > >> Does anyone else with an OMAP3 have this issue? > > > > The best solution I have for this at current is a patch that re-works > > how we use the download image area and compacts the size we reserve for > > the scratch area to be still much larger than required. > > I agree here. Right now the scratch addr space is 4KB. We can reduce it > to 1KB. Below is the patch that is being used in ti-u-boot to solve this > issue. Ah good. That's the same amount I also cut things down to, for all of the TI platforms that have this setup from ROM. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160822/ca62d4c0/attachment.sig> ^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] OMAP3 SPL Overflow 2016-08-22 12:08 ` Tom Rini @ 2016-08-25 16:02 ` Adam Ford 2016-08-25 23:32 ` Tom Rini 0 siblings, 1 reply; 6+ messages in thread From: Adam Ford @ 2016-08-25 16:02 UTC (permalink / raw) To: u-boot On Mon, Aug 22, 2016 at 7:08 AM, Tom Rini <trini@konsulko.com> wrote: > On Mon, Aug 22, 2016 at 10:13:24AM +0530, Lokesh Vutla wrote: >> >> >> On Sunday 21 August 2016 06:15 PM, Tom Rini wrote: >> > On Sun, Aug 21, 2016 at 06:29:42AM -0500, Adam Ford wrote: >> > >> >> Using Git bisect, it appears to me that >> >> a78cd8613204188991c192b8dae2de0aae3b1722 makes "region `.sram' >> >> overflowed by 888 bytes" when I compile for omap3_logic. As I move >> >> forward through the git commits that overflow amount is continuing to >> >> grow. >> >> >> >> Does anyone else with an OMAP3 have this issue? >> > >> > The best solution I have for this at current is a patch that re-works >> > how we use the download image area and compacts the size we reserve for >> > the scratch area to be still much larger than required. >> >> I agree here. Right now the scratch addr space is 4KB. We can reduce it >> to 1KB. Below is the patch that is being used in ti-u-boot to solve this >> issue. > > Ah good. That's the same amount I also cut things down to, for all > of the TI platforms that have this setup from ROM. Tom - Is there any way to get one of these patch options pushed before the next RC candidate? I have evaluated both, and I don't see any issues on the DM3730 that I have. > > -- > Tom ^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] OMAP3 SPL Overflow 2016-08-25 16:02 ` Adam Ford @ 2016-08-25 23:32 ` Tom Rini 0 siblings, 0 replies; 6+ messages in thread From: Tom Rini @ 2016-08-25 23:32 UTC (permalink / raw) To: u-boot On Thu, Aug 25, 2016 at 11:02:04AM -0500, Adam Ford wrote: > On Mon, Aug 22, 2016 at 7:08 AM, Tom Rini <trini@konsulko.com> wrote: > > On Mon, Aug 22, 2016 at 10:13:24AM +0530, Lokesh Vutla wrote: > >> > >> > >> On Sunday 21 August 2016 06:15 PM, Tom Rini wrote: > >> > On Sun, Aug 21, 2016 at 06:29:42AM -0500, Adam Ford wrote: > >> > > >> >> Using Git bisect, it appears to me that > >> >> a78cd8613204188991c192b8dae2de0aae3b1722 makes "region `.sram' > >> >> overflowed by 888 bytes" when I compile for omap3_logic. As I move > >> >> forward through the git commits that overflow amount is continuing to > >> >> grow. > >> >> > >> >> Does anyone else with an OMAP3 have this issue? > >> > > >> > The best solution I have for this at current is a patch that re-works > >> > how we use the download image area and compacts the size we reserve for > >> > the scratch area to be still much larger than required. > >> > >> I agree here. Right now the scratch addr space is 4KB. We can reduce it > >> to 1KB. Below is the patch that is being used in ti-u-boot to solve this > >> issue. > > > > Ah good. That's the same amount I also cut things down to, for all > > of the TI platforms that have this setup from ROM. > > Tom - > > Is there any way to get one of these patch options pushed before the > next RC candidate? I have evaluated both, and I don't see any issues > on the DM3730 that I have. I'm honestly debating it with myself still right now. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160825/27ab0cbd/attachment.sig> ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-08-25 23:32 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-08-21 11:29 [U-Boot] OMAP3 SPL Overflow Adam Ford 2016-08-21 12:45 ` Tom Rini 2016-08-22 4:43 ` Lokesh Vutla 2016-08-22 12:08 ` Tom Rini 2016-08-25 16:02 ` Adam Ford 2016-08-25 23:32 ` Tom Rini
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox