* [U-Boot] [PATCH 01/2] powerpc/mpc85xx:Avoid fix address of bootpg section
@ 2014-03-31 10:01 Prabhakar Kushwaha
2014-04-03 4:02 ` Prabhakar Kushwaha
2014-04-23 21:51 ` York Sun
0 siblings, 2 replies; 6+ messages in thread
From: Prabhakar Kushwaha @ 2014-03-31 10:01 UTC (permalink / raw)
To: u-boot
It is not necessary for bootpg to be present at text + 512KB.
With increase of u-boot size (768KB), bootpg section's address
cannot be fixed.
Signed-off-by: Prabhakar Kushwaha <prabhakar@freescale.com>
---
arch/powerpc/cpu/mpc85xx/u-boot-nand.lds | 8 +++++++-
arch/powerpc/cpu/mpc85xx/u-boot.lds | 6 +++++-
2 files changed, 12 insertions(+), 2 deletions(-)
diff --git a/arch/powerpc/cpu/mpc85xx/u-boot-nand.lds b/arch/powerpc/cpu/mpc85xx/u-boot-nand.lds
index df3b0f9..d77a6dc 100644
--- a/arch/powerpc/cpu/mpc85xx/u-boot-nand.lds
+++ b/arch/powerpc/cpu/mpc85xx/u-boot-nand.lds
@@ -4,6 +4,12 @@
* SPDX-License-Identifier: GPL-2.0+
*/
+#include "config.h" /* CONFIG_BOARDDIR */
+
+#ifndef CONFIG_SYS_MONITOR_LEN
+#define CONFIG_SYS_MONITOR_LEN 0x80000
+#endif
+
OUTPUT_ARCH(powerpc)
/* Do we need any of these for elf?
__DYNAMIC = 0; */
@@ -76,7 +82,7 @@ SECTIONS
KEEP(arch/powerpc/cpu/mpc85xx/start.o (.bootpg))
} :text = 0xffff
- . = ADDR(.text) + 0x80000;
+ . = ADDR(.text) + CONFIG_SYS_MONITOR_LEN;
__bss_start = .;
.bss (NOLOAD) :
diff --git a/arch/powerpc/cpu/mpc85xx/u-boot.lds b/arch/powerpc/cpu/mpc85xx/u-boot.lds
index 2af4c80..99473dd 100644
--- a/arch/powerpc/cpu/mpc85xx/u-boot.lds
+++ b/arch/powerpc/cpu/mpc85xx/u-boot.lds
@@ -12,6 +12,10 @@
#define RESET_VECTOR_ADDRESS 0xfffffffc
#endif
+#ifndef CONFIG_SYS_MONITOR_LEN
+#define CONFIG_SYS_MONITOR_LEN 0x80000
+#endif
+
OUTPUT_ARCH(powerpc)
PHDRS
@@ -84,7 +88,7 @@ SECTIONS
{
KEEP(arch/powerpc/cpu/mpc85xx/start.o (.bootpg))
} :text = 0xffff
- . = ADDR(.text) + 0x80000;
+ . = ADDR(.text) + CONFIG_SYS_MONITOR_LEN;
#else
.bootpg RESET_VECTOR_ADDRESS - 0xffc :
{
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH 01/2] powerpc/mpc85xx:Avoid fix address of bootpg section
2014-03-31 10:01 [U-Boot] [PATCH 01/2] powerpc/mpc85xx:Avoid fix address of bootpg section Prabhakar Kushwaha
@ 2014-04-03 4:02 ` Prabhakar Kushwaha
2014-04-04 15:32 ` Prabhakar Kushwaha
2014-04-17 0:07 ` York Sun
2014-04-23 21:51 ` York Sun
1 sibling, 2 replies; 6+ messages in thread
From: Prabhakar Kushwaha @ 2014-04-03 4:02 UTC (permalink / raw)
To: u-boot
Hi All,
I just want to share my concern with respect to this patch.
This patch removed hard-coded value for 0x80000 and use
CONFIG_SYS_MONITOR_LEN.
There were many Freescale boards config which don't have correct
CONFIG_SYS_MONITOR_LEN(due to size increase from 512KB to 768KB). I
fixed them in subsequent patch.
This linker file is being used by many other powerpc boards/platforms.
Because of hard-coded value of 0x80000 in linker file, there boards are
booting till now.
but after this patch, there is a possibility there boards may not boot
because of wrong value defined for CONFIG_SYS_MONITOR_LEN by board
maintainers.
I will request powerpc board maintainers to check config file for
correct value of CONFIG_SYS_MONITOR_LEN.
Regards,
Prabhakar
On 3/31/2014 3:31 PM, Prabhakar Kushwaha wrote:
> It is not necessary for bootpg to be present at text + 512KB.
> With increase of u-boot size (768KB), bootpg section's address
> cannot be fixed.
>
> Signed-off-by: Prabhakar Kushwaha <prabhakar@freescale.com>
> ---
> arch/powerpc/cpu/mpc85xx/u-boot-nand.lds | 8 +++++++-
> arch/powerpc/cpu/mpc85xx/u-boot.lds | 6 +++++-
> 2 files changed, 12 insertions(+), 2 deletions(-)
>
> diff --git a/arch/powerpc/cpu/mpc85xx/u-boot-nand.lds b/arch/powerpc/cpu/mpc85xx/u-boot-nand.lds
> index df3b0f9..d77a6dc 100644
> --- a/arch/powerpc/cpu/mpc85xx/u-boot-nand.lds
> +++ b/arch/powerpc/cpu/mpc85xx/u-boot-nand.lds
> @@ -4,6 +4,12 @@
> * SPDX-License-Identifier: GPL-2.0+
> */
>
> +#include "config.h" /* CONFIG_BOARDDIR */
> +
> +#ifndef CONFIG_SYS_MONITOR_LEN
> +#define CONFIG_SYS_MONITOR_LEN 0x80000
> +#endif
> +
> OUTPUT_ARCH(powerpc)
> /* Do we need any of these for elf?
> __DYNAMIC = 0; */
> @@ -76,7 +82,7 @@ SECTIONS
> KEEP(arch/powerpc/cpu/mpc85xx/start.o (.bootpg))
> } :text = 0xffff
>
> - . = ADDR(.text) + 0x80000;
> + . = ADDR(.text) + CONFIG_SYS_MONITOR_LEN;
>
> __bss_start = .;
> .bss (NOLOAD) :
> diff --git a/arch/powerpc/cpu/mpc85xx/u-boot.lds b/arch/powerpc/cpu/mpc85xx/u-boot.lds
> index 2af4c80..99473dd 100644
> --- a/arch/powerpc/cpu/mpc85xx/u-boot.lds
> +++ b/arch/powerpc/cpu/mpc85xx/u-boot.lds
> @@ -12,6 +12,10 @@
> #define RESET_VECTOR_ADDRESS 0xfffffffc
> #endif
>
> +#ifndef CONFIG_SYS_MONITOR_LEN
> +#define CONFIG_SYS_MONITOR_LEN 0x80000
> +#endif
> +
> OUTPUT_ARCH(powerpc)
>
> PHDRS
> @@ -84,7 +88,7 @@ SECTIONS
> {
> KEEP(arch/powerpc/cpu/mpc85xx/start.o (.bootpg))
> } :text = 0xffff
> - . = ADDR(.text) + 0x80000;
> + . = ADDR(.text) + CONFIG_SYS_MONITOR_LEN;
> #else
> .bootpg RESET_VECTOR_ADDRESS - 0xffc :
> {
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH 01/2] powerpc/mpc85xx:Avoid fix address of bootpg section
2014-04-03 4:02 ` Prabhakar Kushwaha
@ 2014-04-04 15:32 ` Prabhakar Kushwaha
2014-04-17 0:07 ` York Sun
1 sibling, 0 replies; 6+ messages in thread
From: Prabhakar Kushwaha @ 2014-04-04 15:32 UTC (permalink / raw)
To: u-boot
Dear Wolfgang,
With this patch I removed hard-coding(0x80000) present in
arch/powerpc/cpu/mpc85xx/u-boot-nand.lds with CONFIG_SYS_MONITOR_LEN.
I also updated Freescale boards with correct CONFIG_SYS_MONITOR_LEN in
subsequent patch.
There is a possibility of other powerpc boards to have incorrect
CONFIG_SYS_MONITOR_LEN.
As I am not aware of actual size of the binary. May I update
CONFIG_SYS_MONITOR_LEN for all powerpc platforms as 0x80000 or leave it
up to the platform user to update with correct binary size. I will
check the build of these updated boards.
Thanks,
Prabhakar
On 4/3/2014 9:32 AM, Prabhakar Kushwaha wrote:
> Hi All,
>
> I just want to share my concern with respect to this patch.
> This patch removed hard-coded value for 0x80000 and use
> CONFIG_SYS_MONITOR_LEN.
>
> There were many Freescale boards config which don't have correct
> CONFIG_SYS_MONITOR_LEN(due to size increase from 512KB to 768KB). I
> fixed them in subsequent patch.
>
> This linker file is being used by many other powerpc boards/platforms.
> Because of hard-coded value of 0x80000 in linker file, there boards
> are booting till now.
> but after this patch, there is a possibility there boards may not
> boot because of wrong value defined for CONFIG_SYS_MONITOR_LEN by
> board maintainers.
>
> I will request powerpc board maintainers to check config file for
> correct value of CONFIG_SYS_MONITOR_LEN.
>
> Regards,
> Prabhakar
>
> On 3/31/2014 3:31 PM, Prabhakar Kushwaha wrote:
>> It is not necessary for bootpg to be present at text + 512KB.
>> With increase of u-boot size (768KB), bootpg section's address
>> cannot be fixed.
>>
>> Signed-off-by: Prabhakar Kushwaha <prabhakar@freescale.com>
>> ---
>> arch/powerpc/cpu/mpc85xx/u-boot-nand.lds | 8 +++++++-
>> arch/powerpc/cpu/mpc85xx/u-boot.lds | 6 +++++-
>> 2 files changed, 12 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/powerpc/cpu/mpc85xx/u-boot-nand.lds
>> b/arch/powerpc/cpu/mpc85xx/u-boot-nand.lds
>> index df3b0f9..d77a6dc 100644
>> --- a/arch/powerpc/cpu/mpc85xx/u-boot-nand.lds
>> +++ b/arch/powerpc/cpu/mpc85xx/u-boot-nand.lds
>> @@ -4,6 +4,12 @@
>> * SPDX-License-Identifier: GPL-2.0+
>> */
>> +#include "config.h" /* CONFIG_BOARDDIR */
>> +
>> +#ifndef CONFIG_SYS_MONITOR_LEN
>> +#define CONFIG_SYS_MONITOR_LEN 0x80000
>> +#endif
>> +
>> OUTPUT_ARCH(powerpc)
>> /* Do we need any of these for elf?
>> __DYNAMIC = 0; */
>> @@ -76,7 +82,7 @@ SECTIONS
>> KEEP(arch/powerpc/cpu/mpc85xx/start.o (.bootpg))
>> } :text = 0xffff
>> - . = ADDR(.text) + 0x80000;
>> + . = ADDR(.text) + CONFIG_SYS_MONITOR_LEN;
>> __bss_start = .;
>> .bss (NOLOAD) :
>> diff --git a/arch/powerpc/cpu/mpc85xx/u-boot.lds
>> b/arch/powerpc/cpu/mpc85xx/u-boot.lds
>> index 2af4c80..99473dd 100644
>> --- a/arch/powerpc/cpu/mpc85xx/u-boot.lds
>> +++ b/arch/powerpc/cpu/mpc85xx/u-boot.lds
>> @@ -12,6 +12,10 @@
>> #define RESET_VECTOR_ADDRESS 0xfffffffc
>> #endif
>> +#ifndef CONFIG_SYS_MONITOR_LEN
>> +#define CONFIG_SYS_MONITOR_LEN 0x80000
>> +#endif
>> +
>> OUTPUT_ARCH(powerpc)
>> PHDRS
>> @@ -84,7 +88,7 @@ SECTIONS
>> {
>> KEEP(arch/powerpc/cpu/mpc85xx/start.o (.bootpg))
>> } :text = 0xffff
>> - . = ADDR(.text) + 0x80000;
>> + . = ADDR(.text) + CONFIG_SYS_MONITOR_LEN;
>> #else
>> .bootpg RESET_VECTOR_ADDRESS - 0xffc :
>> {
>
>
>
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH 01/2] powerpc/mpc85xx:Avoid fix address of bootpg section
2014-04-03 4:02 ` Prabhakar Kushwaha
2014-04-04 15:32 ` Prabhakar Kushwaha
@ 2014-04-17 0:07 ` York Sun
2014-04-17 3:41 ` Prabhakar Kushwaha
1 sibling, 1 reply; 6+ messages in thread
From: York Sun @ 2014-04-17 0:07 UTC (permalink / raw)
To: u-boot
On 04/02/2014 09:02 PM, Prabhakar Kushwaha wrote:
> Hi All,
>
> I just want to share my concern with respect to this patch.
> This patch removed hard-coded value for 0x80000 and use CONFIG_SYS_MONITOR_LEN.
>
> There were many Freescale boards config which don?t have correct
> CONFIG_SYS_MONITOR_LEN(due to size increase from 512KB to 768KB). I fixed them
> in subsequent patch.
>
> This linker file is being used by many other powerpc boards/platforms. Because
> of hard-coded value of 0x80000 in linker file, there boards are booting till now.
> but after this patch, there is a possibility there boards may not boot because
> of wrong value defined for CONFIG_SYS_MONITOR_LEN by board maintainers.
>
> I will request powerpc board maintainers to check config file for correct value
> of CONFIG_SYS_MONITOR_LEN.
>
>
Prabhakar,
Where are we for this patch? I don't see any comment.
York
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH 01/2] powerpc/mpc85xx:Avoid fix address of bootpg section
2014-04-17 0:07 ` York Sun
@ 2014-04-17 3:41 ` Prabhakar Kushwaha
0 siblings, 0 replies; 6+ messages in thread
From: Prabhakar Kushwaha @ 2014-04-17 3:41 UTC (permalink / raw)
To: u-boot
On 4/17/2014 5:37 AM, York Sun wrote:
> On 04/02/2014 09:02 PM, Prabhakar Kushwaha wrote:
>> Hi All,
>>
>> I just want to share my concern with respect to this patch.
>> This patch removed hard-coded value for 0x80000 and use CONFIG_SYS_MONITOR_LEN.
>>
>> There were many Freescale boards config which don?t have correct
>> CONFIG_SYS_MONITOR_LEN(due to size increase from 512KB to 768KB). I fixed them
>> in subsequent patch.
>>
>> This linker file is being used by many other powerpc boards/platforms. Because
>> of hard-coded value of 0x80000 in linker file, there boards are booting till now.
>> but after this patch, there is a possibility there boards may not boot because
>> of wrong value defined for CONFIG_SYS_MONITOR_LEN by board maintainers.
>>
>> I will request powerpc board maintainers to check config file for correct value
>> of CONFIG_SYS_MONITOR_LEN.
>>
>>
> Prabhakar,
>
> Where are we for this patch? I don't see any comment.
>
>
I did not get any comments :(
Looks like every one is OK with it.
regards,
Prabhakar
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH 01/2] powerpc/mpc85xx:Avoid fix address of bootpg section
2014-03-31 10:01 [U-Boot] [PATCH 01/2] powerpc/mpc85xx:Avoid fix address of bootpg section Prabhakar Kushwaha
2014-04-03 4:02 ` Prabhakar Kushwaha
@ 2014-04-23 21:51 ` York Sun
1 sibling, 0 replies; 6+ messages in thread
From: York Sun @ 2014-04-23 21:51 UTC (permalink / raw)
To: u-boot
On 03/31/2014 03:01 AM, Prabhakar Kushwaha wrote:
> It is not necessary for bootpg to be present at text + 512KB.
> With increase of u-boot size (768KB), bootpg section's address
> cannot be fixed.
>
> Signed-off-by: Prabhakar Kushwaha <prabhakar@freescale.com>
> ---
Applied to u-boot-mpc85xx/master, thanks.
York
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-04-23 21:51 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-03-31 10:01 [U-Boot] [PATCH 01/2] powerpc/mpc85xx:Avoid fix address of bootpg section Prabhakar Kushwaha
2014-04-03 4:02 ` Prabhakar Kushwaha
2014-04-04 15:32 ` Prabhakar Kushwaha
2014-04-17 0:07 ` York Sun
2014-04-17 3:41 ` Prabhakar Kushwaha
2014-04-23 21:51 ` York Sun
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).