diff mbox

[U-Boot] powerpc: Increase u-boot size for selected boards

Message ID 1398288973-7973-1-git-send-email-yorksun@freescale.com
State Superseded
Delegated to: York Sun
Headers show

Commit Message

York Sun April 23, 2014, 9:36 p.m. UTC
These boards have compiling error with latest change in u-boot. The image
grows and exceeds the preconfigured size. This patch only address the
compiling error. It is not verified on the boards. The affected boards are

MPC8572DS, dlvision, io, iocon, neo

Signed-off-by: York Sun <yorksun@freescale.com>
CC: Dirk Eibach <eibach@gdsys.de>
CC: Kumar Gala <galak@kernel.crashing.org>
---
 include/configs/MPC8572DS.h    |    2 +-
 include/configs/dlvision-10g.h |    2 +-
 include/configs/io.h           |    2 +-
 include/configs/iocon.h        |    2 +-
 include/configs/neo.h          |    2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

Comments

Dirk Eibach April 24, 2014, 6:22 a.m. UTC | #1
NAK for dlvision, io, iocon and neo.
Flash partition layout does not allow changing CONFIG_SYS_TEXT_BASE.
I will try to come up with a different solution for these boards.

Cheers
Dirk

2014-04-23 23:36 GMT+02:00 York Sun <yorksun@freescale.com>:
> These boards have compiling error with latest change in u-boot. The image
> grows and exceeds the preconfigured size. This patch only address the
> compiling error. It is not verified on the boards. The affected boards are
>
> MPC8572DS, dlvision, io, iocon, neo
>
> Signed-off-by: York Sun <yorksun@freescale.com>
> CC: Dirk Eibach <eibach@gdsys.de>
> CC: Kumar Gala <galak@kernel.crashing.org>
> ---
>  include/configs/MPC8572DS.h    |    2 +-
>  include/configs/dlvision-10g.h |    2 +-
>  include/configs/io.h           |    2 +-
>  include/configs/iocon.h        |    2 +-
>  include/configs/neo.h          |    2 +-
>  5 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/include/configs/MPC8572DS.h b/include/configs/MPC8572DS.h
> index 7b63945..3a30feb 100644
> --- a/include/configs/MPC8572DS.h
> +++ b/include/configs/MPC8572DS.h
> @@ -30,7 +30,7 @@
>  #endif
>
>  #ifndef CONFIG_SYS_TEXT_BASE
> -#define CONFIG_SYS_TEXT_BASE   0xeff80000
> +#define CONFIG_SYS_TEXT_BASE   0xeff40000
>  #endif
>
>  #ifndef CONFIG_RESET_VECTOR_ADDRESS
> diff --git a/include/configs/dlvision-10g.h b/include/configs/dlvision-10g.h
> index 7877897..47484d8 100644
> --- a/include/configs/dlvision-10g.h
> +++ b/include/configs/dlvision-10g.h
> @@ -11,7 +11,7 @@
>  #define CONFIG_405EP           1       /* this is a PPC405 CPU */
>  #define CONFIG_DLVISION_10G    1       /*  on a DLVision-10G board */
>
> -#define        CONFIG_SYS_TEXT_BASE    0xFFFC0000
> +#define        CONFIG_SYS_TEXT_BASE    0xFFF80000
>
>  /*
>   * Include common defines/options for all AMCC eval boards
> diff --git a/include/configs/io.h b/include/configs/io.h
> index 9da6cc6..c7f0ded 100644
> --- a/include/configs/io.h
> +++ b/include/configs/io.h
> @@ -11,7 +11,7 @@
>  #define CONFIG_405EP           1       /* this is a PPC405 CPU */
>  #define CONFIG_IO              1       /*  on a Io board */
>
> -#define        CONFIG_SYS_TEXT_BASE    0xFFFC0000
> +#define        CONFIG_SYS_TEXT_BASE    0xFFF80000
>
>  /*
>   * Include common defines/options for all AMCC eval boards
> diff --git a/include/configs/iocon.h b/include/configs/iocon.h
> index f36c2a3..09fa1fb 100644
> --- a/include/configs/iocon.h
> +++ b/include/configs/iocon.h
> @@ -11,7 +11,7 @@
>  #define CONFIG_405EP           1       /* this is a PPC405 CPU */
>  #define CONFIG_IOCON           1       /*  on a IoCon board */
>
> -#define        CONFIG_SYS_TEXT_BASE    0xFFFC0000
> +#define        CONFIG_SYS_TEXT_BASE    0xFFF80000
>
>  /*
>   * Include common defines/options for all AMCC eval boards
> diff --git a/include/configs/neo.h b/include/configs/neo.h
> index d549985..c23eafe 100644
> --- a/include/configs/neo.h
> +++ b/include/configs/neo.h
> @@ -12,7 +12,7 @@
>  #define CONFIG_405EP           1       /* this is a PPC405 CPU */
>  #define CONFIG_NEO             1       /*  on a Neo board */
>
> -#define        CONFIG_SYS_TEXT_BASE    0xFFFC0000
> +#define        CONFIG_SYS_TEXT_BASE    0xFFF80000
>
>  /*
>   * Include common defines/options for all AMCC eval boards
> --
> 1.7.9.5
>
> _______________________________________________
> U-Boot mailing list
> U-Boot@lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
York Sun April 24, 2014, 5:02 p.m. UTC | #2
On 04/23/2014 11:22 PM, Dirk Eibach wrote:
> NAK for dlvision, io, iocon and neo.
> Flash partition layout does not allow changing CONFIG_SYS_TEXT_BASE.
> I will try to come up with a different solution for these boards.
> 

Thanks. I will resend the patch to deal with MPC8572DS only.

York
Heiko Schocher April 25, 2014, 5:19 a.m. UTC | #3
Hello York,

Am 24.04.2014 19:02, schrieb York Sun:
> On 04/23/2014 11:22 PM, Dirk Eibach wrote:
>> NAK for dlvision, io, iocon and neo.
>> Flash partition layout does not allow changing CONFIG_SYS_TEXT_BASE.
>> I will try to come up with a different solution for these boards.
>>
>
> Thanks. I will resend the patch to deal with MPC8572DS only.

Thanks! Hmm, as a maintainer is missing for this board, do you want
to volunteer?

bye,
Heiko
York Sun April 25, 2014, 5:49 a.m. UTC | #4
On Apr 24, 2014, at 10:19 PM, Heiko Schocher <hs@denx.de> wrote:

> Hello York,
> 
> Am 24.04.2014 19:02, schrieb York Sun:
>> On 04/23/2014 11:22 PM, Dirk Eibach wrote:
>>> NAK for dlvision, io, iocon and neo.
>>> Flash partition layout does not allow changing CONFIG_SYS_TEXT_BASE.
>>> I will try to come up with a different solution for these boards.
>>> 
>> 
>> Thanks. I will resend the patch to deal with MPC8572DS only.
> 
> Thanks! Hmm, as a maintainer is missing for this board, do you want
> to volunteer?
> 

Let me do some house cleaning to make sure I still have one of these boards laying around. I can’t be a maintainer without having it. Giving the age of these boards (and some other boards as well), I don’t know if we still have users.

York
diff mbox

Patch

diff --git a/include/configs/MPC8572DS.h b/include/configs/MPC8572DS.h
index 7b63945..3a30feb 100644
--- a/include/configs/MPC8572DS.h
+++ b/include/configs/MPC8572DS.h
@@ -30,7 +30,7 @@ 
 #endif
 
 #ifndef CONFIG_SYS_TEXT_BASE
-#define CONFIG_SYS_TEXT_BASE	0xeff80000
+#define CONFIG_SYS_TEXT_BASE	0xeff40000
 #endif
 
 #ifndef CONFIG_RESET_VECTOR_ADDRESS
diff --git a/include/configs/dlvision-10g.h b/include/configs/dlvision-10g.h
index 7877897..47484d8 100644
--- a/include/configs/dlvision-10g.h
+++ b/include/configs/dlvision-10g.h
@@ -11,7 +11,7 @@ 
 #define CONFIG_405EP		1	/* this is a PPC405 CPU */
 #define CONFIG_DLVISION_10G	1	/*  on a DLVision-10G board */
 
-#define	CONFIG_SYS_TEXT_BASE	0xFFFC0000
+#define	CONFIG_SYS_TEXT_BASE	0xFFF80000
 
 /*
  * Include common defines/options for all AMCC eval boards
diff --git a/include/configs/io.h b/include/configs/io.h
index 9da6cc6..c7f0ded 100644
--- a/include/configs/io.h
+++ b/include/configs/io.h
@@ -11,7 +11,7 @@ 
 #define CONFIG_405EP		1	/* this is a PPC405 CPU */
 #define CONFIG_IO	        1	/*  on a Io board */
 
-#define	CONFIG_SYS_TEXT_BASE	0xFFFC0000
+#define	CONFIG_SYS_TEXT_BASE	0xFFF80000
 
 /*
  * Include common defines/options for all AMCC eval boards
diff --git a/include/configs/iocon.h b/include/configs/iocon.h
index f36c2a3..09fa1fb 100644
--- a/include/configs/iocon.h
+++ b/include/configs/iocon.h
@@ -11,7 +11,7 @@ 
 #define CONFIG_405EP		1	/* this is a PPC405 CPU */
 #define CONFIG_IOCON		1	/*  on a IoCon board */
 
-#define	CONFIG_SYS_TEXT_BASE	0xFFFC0000
+#define	CONFIG_SYS_TEXT_BASE	0xFFF80000
 
 /*
  * Include common defines/options for all AMCC eval boards
diff --git a/include/configs/neo.h b/include/configs/neo.h
index d549985..c23eafe 100644
--- a/include/configs/neo.h
+++ b/include/configs/neo.h
@@ -12,7 +12,7 @@ 
 #define CONFIG_405EP		1	/* this is a PPC405 CPU */
 #define CONFIG_NEO	        1	/*  on a Neo board */
 
-#define	CONFIG_SYS_TEXT_BASE	0xFFFC0000
+#define	CONFIG_SYS_TEXT_BASE	0xFFF80000
 
 /*
  * Include common defines/options for all AMCC eval boards