* [U-Boot] [PATCH] configs: vf610: increase maximum size and enforce correct limit @ 2017-10-17 11:59 Stefan Agner 2017-10-30 8:59 ` Stefan Agner 0 siblings, 1 reply; 4+ messages in thread From: Stefan Agner @ 2017-10-17 11:59 UTC (permalink / raw) To: u-boot From: Stefan Agner <stefan.agner@toradex.com> On Vybrid SoCs U-Boot gets loaded into GFX SRAM which is 512KiB. Currently 32KiB is reserved for the IMX header. However, this is not reflected in the size limit. In v2017.11-rc2 the actual size limit (512KiB-32KiB) has been reached for Colibri VF61, which lead to a successful build of U-Boot but not a working binary. The IMX header is much smaller than 32KiB, typically around 1KiB. Decrease the reserved size to 4KiB and specify the correct U-Boot size limit. Apply this new base address and limit for all Vybrid based boards. Signed-off-by: Stefan Agner <stefan.agner@toradex.com> --- include/configs/colibri_vf.h | 4 ++-- include/configs/pcm052.h | 4 ++-- include/configs/vf610twr.h | 4 ++-- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/include/configs/colibri_vf.h b/include/configs/colibri_vf.h index 8166aa4afb..bf3bbffe24 100644 --- a/include/configs/colibri_vf.h +++ b/include/configs/colibri_vf.h @@ -75,8 +75,8 @@ #define CONFIG_FDTADDR 0x84000000 /* We boot from the gfxRAM area of the OCRAM. */ -#define CONFIG_SYS_TEXT_BASE 0x3f408000 -#define CONFIG_BOARD_SIZE_LIMIT 524288 +#define CONFIG_SYS_TEXT_BASE 0x3f401000 +#define CONFIG_BOARD_SIZE_LIMIT 520192 #define SD_BOOTCMD \ "sdargs=root=/dev/mmcblk0p2 rw rootwait\0" \ diff --git a/include/configs/pcm052.h b/include/configs/pcm052.h index 8144a83db2..060928bd30 100644 --- a/include/configs/pcm052.h +++ b/include/configs/pcm052.h @@ -89,8 +89,8 @@ #define CONFIG_LOADADDR 0x82000000 /* We boot from the gfxRAM area of the OCRAM. */ -#define CONFIG_SYS_TEXT_BASE 0x3f408000 -#define CONFIG_BOARD_SIZE_LIMIT 524288 +#define CONFIG_SYS_TEXT_BASE 0x3f401000 +#define CONFIG_BOARD_SIZE_LIMIT 520192 /* if no target-specific extra environment settings were defined by the target, define an empty one */ diff --git a/include/configs/vf610twr.h b/include/configs/vf610twr.h index 3430f27c40..ddfcd6f618 100644 --- a/include/configs/vf610twr.h +++ b/include/configs/vf610twr.h @@ -79,8 +79,8 @@ #define CONFIG_SYS_LOAD_ADDR 0x82000000 /* We boot from the gfxRAM area of the OCRAM. */ -#define CONFIG_SYS_TEXT_BASE 0x3f408000 -#define CONFIG_BOARD_SIZE_LIMIT 524288 +#define CONFIG_SYS_TEXT_BASE 0x3f401000 +#define CONFIG_BOARD_SIZE_LIMIT 520192 /* * We do have 128MB of memory on the Vybrid Tower board. Leave the last -- 2.14.2 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] configs: vf610: increase maximum size and enforce correct limit 2017-10-17 11:59 [U-Boot] [PATCH] configs: vf610: increase maximum size and enforce correct limit Stefan Agner @ 2017-10-30 8:59 ` Stefan Agner 2017-10-31 2:47 ` Tom Rini 0 siblings, 1 reply; 4+ messages in thread From: Stefan Agner @ 2017-10-30 8:59 UTC (permalink / raw) To: u-boot Hi Tom, Stefano, Any chance to get this still into 2017.11? It allows to use 2017.11 on my board... -- Stefan On 2017-10-17 13:59, Stefan Agner wrote: > From: Stefan Agner <stefan.agner@toradex.com> > > On Vybrid SoCs U-Boot gets loaded into GFX SRAM which is 512KiB. > Currently 32KiB is reserved for the IMX header. However, this is > not reflected in the size limit. In v2017.11-rc2 the actual size > limit (512KiB-32KiB) has been reached for Colibri VF61, which > lead to a successful build of U-Boot but not a working binary. > > The IMX header is much smaller than 32KiB, typically around 1KiB. > Decrease the reserved size to 4KiB and specify the correct U-Boot > size limit. Apply this new base address and limit for all Vybrid > based boards. > > Signed-off-by: Stefan Agner <stefan.agner@toradex.com> > --- > > include/configs/colibri_vf.h | 4 ++-- > include/configs/pcm052.h | 4 ++-- > include/configs/vf610twr.h | 4 ++-- > 3 files changed, 6 insertions(+), 6 deletions(-) > > diff --git a/include/configs/colibri_vf.h b/include/configs/colibri_vf.h > index 8166aa4afb..bf3bbffe24 100644 > --- a/include/configs/colibri_vf.h > +++ b/include/configs/colibri_vf.h > @@ -75,8 +75,8 @@ > #define CONFIG_FDTADDR 0x84000000 > > /* We boot from the gfxRAM area of the OCRAM. */ > -#define CONFIG_SYS_TEXT_BASE 0x3f408000 > -#define CONFIG_BOARD_SIZE_LIMIT 524288 > +#define CONFIG_SYS_TEXT_BASE 0x3f401000 > +#define CONFIG_BOARD_SIZE_LIMIT 520192 > > #define SD_BOOTCMD \ > "sdargs=root=/dev/mmcblk0p2 rw rootwait\0" \ > diff --git a/include/configs/pcm052.h b/include/configs/pcm052.h > index 8144a83db2..060928bd30 100644 > --- a/include/configs/pcm052.h > +++ b/include/configs/pcm052.h > @@ -89,8 +89,8 @@ > #define CONFIG_LOADADDR 0x82000000 > > /* We boot from the gfxRAM area of the OCRAM. */ > -#define CONFIG_SYS_TEXT_BASE 0x3f408000 > -#define CONFIG_BOARD_SIZE_LIMIT 524288 > +#define CONFIG_SYS_TEXT_BASE 0x3f401000 > +#define CONFIG_BOARD_SIZE_LIMIT 520192 > > /* if no target-specific extra environment settings were defined by the > target, define an empty one */ > diff --git a/include/configs/vf610twr.h b/include/configs/vf610twr.h > index 3430f27c40..ddfcd6f618 100644 > --- a/include/configs/vf610twr.h > +++ b/include/configs/vf610twr.h > @@ -79,8 +79,8 @@ > #define CONFIG_SYS_LOAD_ADDR 0x82000000 > > /* We boot from the gfxRAM area of the OCRAM. */ > -#define CONFIG_SYS_TEXT_BASE 0x3f408000 > -#define CONFIG_BOARD_SIZE_LIMIT 524288 > +#define CONFIG_SYS_TEXT_BASE 0x3f401000 > +#define CONFIG_BOARD_SIZE_LIMIT 520192 > > /* > * We do have 128MB of memory on the Vybrid Tower board. Leave the last ^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] configs: vf610: increase maximum size and enforce correct limit 2017-10-30 8:59 ` Stefan Agner @ 2017-10-31 2:47 ` Tom Rini 2017-10-31 9:59 ` Stefano Babic 0 siblings, 1 reply; 4+ messages in thread From: Tom Rini @ 2017-10-31 2:47 UTC (permalink / raw) To: u-boot On Mon, Oct 30, 2017 at 09:59:04AM +0100, Stefan Agner wrote: > Hi Tom, Stefano, > > Any chance to get this still into 2017.11? It allows to use 2017.11 on > my board... Stefano, this is your call, thanks! > > -- > Stefan > > On 2017-10-17 13:59, Stefan Agner wrote: > > From: Stefan Agner <stefan.agner@toradex.com> > > > > On Vybrid SoCs U-Boot gets loaded into GFX SRAM which is 512KiB. > > Currently 32KiB is reserved for the IMX header. However, this is > > not reflected in the size limit. In v2017.11-rc2 the actual size > > limit (512KiB-32KiB) has been reached for Colibri VF61, which > > lead to a successful build of U-Boot but not a working binary. > > > > The IMX header is much smaller than 32KiB, typically around 1KiB. > > Decrease the reserved size to 4KiB and specify the correct U-Boot > > size limit. Apply this new base address and limit for all Vybrid > > based boards. > > > > Signed-off-by: Stefan Agner <stefan.agner@toradex.com> > > --- > > > > include/configs/colibri_vf.h | 4 ++-- > > include/configs/pcm052.h | 4 ++-- > > include/configs/vf610twr.h | 4 ++-- > > 3 files changed, 6 insertions(+), 6 deletions(-) > > > > diff --git a/include/configs/colibri_vf.h b/include/configs/colibri_vf.h > > index 8166aa4afb..bf3bbffe24 100644 > > --- a/include/configs/colibri_vf.h > > +++ b/include/configs/colibri_vf.h > > @@ -75,8 +75,8 @@ > > #define CONFIG_FDTADDR 0x84000000 > > > > /* We boot from the gfxRAM area of the OCRAM. */ > > -#define CONFIG_SYS_TEXT_BASE 0x3f408000 > > -#define CONFIG_BOARD_SIZE_LIMIT 524288 > > +#define CONFIG_SYS_TEXT_BASE 0x3f401000 > > +#define CONFIG_BOARD_SIZE_LIMIT 520192 > > > > #define SD_BOOTCMD \ > > "sdargs=root=/dev/mmcblk0p2 rw rootwait\0" \ > > diff --git a/include/configs/pcm052.h b/include/configs/pcm052.h > > index 8144a83db2..060928bd30 100644 > > --- a/include/configs/pcm052.h > > +++ b/include/configs/pcm052.h > > @@ -89,8 +89,8 @@ > > #define CONFIG_LOADADDR 0x82000000 > > > > /* We boot from the gfxRAM area of the OCRAM. */ > > -#define CONFIG_SYS_TEXT_BASE 0x3f408000 > > -#define CONFIG_BOARD_SIZE_LIMIT 524288 > > +#define CONFIG_SYS_TEXT_BASE 0x3f401000 > > +#define CONFIG_BOARD_SIZE_LIMIT 520192 > > > > /* if no target-specific extra environment settings were defined by the > > target, define an empty one */ > > diff --git a/include/configs/vf610twr.h b/include/configs/vf610twr.h > > index 3430f27c40..ddfcd6f618 100644 > > --- a/include/configs/vf610twr.h > > +++ b/include/configs/vf610twr.h > > @@ -79,8 +79,8 @@ > > #define CONFIG_SYS_LOAD_ADDR 0x82000000 > > > > /* We boot from the gfxRAM area of the OCRAM. */ > > -#define CONFIG_SYS_TEXT_BASE 0x3f408000 > > -#define CONFIG_BOARD_SIZE_LIMIT 524288 > > +#define CONFIG_SYS_TEXT_BASE 0x3f401000 > > +#define CONFIG_BOARD_SIZE_LIMIT 520192 > > > > /* > > * We do have 128MB of memory on the Vybrid Tower board. Leave the last > _______________________________________________ > U-Boot mailing list > U-Boot at lists.denx.de > https://lists.denx.de/listinfo/u-boot -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: <http://lists.denx.de/pipermail/u-boot/attachments/20171030/72b37f2f/attachment.sig> ^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] configs: vf610: increase maximum size and enforce correct limit 2017-10-31 2:47 ` Tom Rini @ 2017-10-31 9:59 ` Stefano Babic 0 siblings, 0 replies; 4+ messages in thread From: Stefano Babic @ 2017-10-31 9:59 UTC (permalink / raw) To: u-boot Hi Tom, Stefan, On 31/10/2017 03:47, Tom Rini wrote: > On Mon, Oct 30, 2017 at 09:59:04AM +0100, Stefan Agner wrote: >> Hi Tom, Stefano, >> >> Any chance to get this still into 2017.11? It allows to use 2017.11 on >> my board... > > Stefano, this is your call, thanks! I looking into the list to pick up all remaining patches - this go in ! Regards, Stefano > >> >> -- >> Stefan >> >> On 2017-10-17 13:59, Stefan Agner wrote: >>> From: Stefan Agner <stefan.agner@toradex.com> >>> >>> On Vybrid SoCs U-Boot gets loaded into GFX SRAM which is 512KiB. >>> Currently 32KiB is reserved for the IMX header. However, this is >>> not reflected in the size limit. In v2017.11-rc2 the actual size >>> limit (512KiB-32KiB) has been reached for Colibri VF61, which >>> lead to a successful build of U-Boot but not a working binary. >>> >>> The IMX header is much smaller than 32KiB, typically around 1KiB. >>> Decrease the reserved size to 4KiB and specify the correct U-Boot >>> size limit. Apply this new base address and limit for all Vybrid >>> based boards. >>> >>> Signed-off-by: Stefan Agner <stefan.agner@toradex.com> >>> --- >>> >>> include/configs/colibri_vf.h | 4 ++-- >>> include/configs/pcm052.h | 4 ++-- >>> include/configs/vf610twr.h | 4 ++-- >>> 3 files changed, 6 insertions(+), 6 deletions(-) >>> >>> diff --git a/include/configs/colibri_vf.h b/include/configs/colibri_vf.h >>> index 8166aa4afb..bf3bbffe24 100644 >>> --- a/include/configs/colibri_vf.h >>> +++ b/include/configs/colibri_vf.h >>> @@ -75,8 +75,8 @@ >>> #define CONFIG_FDTADDR 0x84000000 >>> >>> /* We boot from the gfxRAM area of the OCRAM. */ >>> -#define CONFIG_SYS_TEXT_BASE 0x3f408000 >>> -#define CONFIG_BOARD_SIZE_LIMIT 524288 >>> +#define CONFIG_SYS_TEXT_BASE 0x3f401000 >>> +#define CONFIG_BOARD_SIZE_LIMIT 520192 >>> >>> #define SD_BOOTCMD \ >>> "sdargs=root=/dev/mmcblk0p2 rw rootwait\0" \ >>> diff --git a/include/configs/pcm052.h b/include/configs/pcm052.h >>> index 8144a83db2..060928bd30 100644 >>> --- a/include/configs/pcm052.h >>> +++ b/include/configs/pcm052.h >>> @@ -89,8 +89,8 @@ >>> #define CONFIG_LOADADDR 0x82000000 >>> >>> /* We boot from the gfxRAM area of the OCRAM. */ >>> -#define CONFIG_SYS_TEXT_BASE 0x3f408000 >>> -#define CONFIG_BOARD_SIZE_LIMIT 524288 >>> +#define CONFIG_SYS_TEXT_BASE 0x3f401000 >>> +#define CONFIG_BOARD_SIZE_LIMIT 520192 >>> >>> /* if no target-specific extra environment settings were defined by the >>> target, define an empty one */ >>> diff --git a/include/configs/vf610twr.h b/include/configs/vf610twr.h >>> index 3430f27c40..ddfcd6f618 100644 >>> --- a/include/configs/vf610twr.h >>> +++ b/include/configs/vf610twr.h >>> @@ -79,8 +79,8 @@ >>> #define CONFIG_SYS_LOAD_ADDR 0x82000000 >>> >>> /* We boot from the gfxRAM area of the OCRAM. */ >>> -#define CONFIG_SYS_TEXT_BASE 0x3f408000 >>> -#define CONFIG_BOARD_SIZE_LIMIT 524288 >>> +#define CONFIG_SYS_TEXT_BASE 0x3f401000 >>> +#define CONFIG_BOARD_SIZE_LIMIT 520192 >>> >>> /* >>> * We do have 128MB of memory on the Vybrid Tower board. Leave the last >> _______________________________________________ >> U-Boot mailing list >> U-Boot at lists.denx.de >> https://lists.denx.de/listinfo/u-boot > -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de ===================================================================== ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-10-31 9:59 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2017-10-17 11:59 [U-Boot] [PATCH] configs: vf610: increase maximum size and enforce correct limit Stefan Agner 2017-10-30 8:59 ` Stefan Agner 2017-10-31 2:47 ` Tom Rini 2017-10-31 9:59 ` Stefano Babic
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox