* [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