[U-Boot] [PATCH] ppc4xx: alpr: Remove some not needed commands to make image fit again

[U-Boot] [PATCH] ppc4xx: alpr: Remove some not needed commands to make image fit again

[Stefan Roese]

The latest changes increased the size of the alpr image a bit more.
Now it doesn't fit into the 256k reserved for it. This patch now removes
the commands "ping" and "diag" which are not needed in the production
systems.

Signed-off-by: Stefan Roese <sr@denx.de>
Cc: Pieter Voorthuijsen <pieter.voorthuijsen@prodrive.nl>
---
 include/configs/alpr.h |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/include/configs/alpr.h b/include/configs/alpr.h
index 964630f..0fed9ad 100644
--- a/include/configs/alpr.h
+++ b/include/configs/alpr.h
@@ -232,7 +232,6 @@
 #include <config_cmd_default.h>
 
 #define CONFIG_CMD_DHCP
-#define CONFIG_CMD_DIAG
 #define CONFIG_CMD_EEPROM
 #define CONFIG_CMD_FPGA
 #define CONFIG_CMD_I2C
@@ -240,7 +239,6 @@
 #define CONFIG_CMD_NAND
 #define CONFIG_CMD_NET
 #define CONFIG_CMD_PCI
-#define CONFIG_CMD_PING
 #undef CONFIG_CMD_NFS
 
 #undef CONFIG_WATCHDOG			/* watchdog disabled		*/
-- 
1.6.6.rc2

[U-Boot] [PATCH] ppc4xx: alpr: Remove some not needed commands to make image fit again

[Stefan Roese]

On Wednesday 16 December 2009 09:27:31 Stefan Roese wrote:
> The latest changes increased the size of the alpr image a bit more.
> Now it doesn't fit into the 256k reserved for it. This patch now removes
> the commands "ping" and "diag" which are not needed in the production
> systems.

Applied to u-boot-ppc4xx. Thanks.

Cheers,
Stefan

--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: office at denx.de

[U-Boot] [PATCH] ppc4xx: alpr: Remove some not needed commands to make image fit again

[Stefan Roese]

The latest changes increased the size of the alpr image a bit more.
Now it doesn't fit into the 256k reserved for it. This patch now removes
the commands "loads" and "loadb" which are not needed in the production
systems.

Signed-off-by: Stefan Roese <sr@denx.de>
Cc: Pieter Voorthuijsen <pieter.voorthuijsen@prodrive.nl>
---
 include/configs/alpr.h |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/include/configs/alpr.h b/include/configs/alpr.h
index f2392f6..ee0c14d 100644
--- a/include/configs/alpr.h
+++ b/include/configs/alpr.h
@@ -236,11 +236,13 @@
 #define CONFIG_CMD_EEPROM
 #define CONFIG_CMD_FPGA
 #define CONFIG_CMD_I2C
+#undef CONFIG_CMD_LOADB
+#undef CONFIG_CMD_LOADS
 #define CONFIG_CMD_MII
 #define CONFIG_CMD_NAND
 #define CONFIG_CMD_NET
-#define CONFIG_CMD_PCI
 #undef CONFIG_CMD_NFS
+#define CONFIG_CMD_PCI
 
 #undef CONFIG_WATCHDOG			/* watchdog disabled		*/
 
-- 
1.7.0.4

[U-Boot] [PATCH] ppc4xx: alpr: Remove some not needed commands to make image fit again

[Pieter Voorthuijsen]

Hi Stefan,

> The latest changes increased the size of the alpr image a bit more.
> Now it doesn't fit into the 256k reserved for it. This patch 
> now removes the commands "loads" and "loadb" which are not 
> needed in the production systems.

I see that PCI is also enabled again. This is not used and can be turned off. Otherwise I think we'll see this 'won't fit' message again in a few months. :)

> Signed-off-by: Stefan Roese <sr@denx.de>
> Cc: Pieter Voorthuijsen <pieter.voorthuijsen@prodrive.nl>
> ---
>  include/configs/alpr.h |    4 +++-
>  1 files changed, 3 insertions(+), 1 deletions(-)
> 
> diff --git a/include/configs/alpr.h b/include/configs/alpr.h 
> index f2392f6..ee0c14d 100644
> --- a/include/configs/alpr.h
> +++ b/include/configs/alpr.h
> @@ -236,11 +236,13 @@
>  #define CONFIG_CMD_EEPROM
>  #define CONFIG_CMD_FPGA
>  #define CONFIG_CMD_I2C
> +#undef CONFIG_CMD_LOADB
> +#undef CONFIG_CMD_LOADS
>  #define CONFIG_CMD_MII
>  #define CONFIG_CMD_NAND
>  #define CONFIG_CMD_NET
> -#define CONFIG_CMD_PCI
>  #undef CONFIG_CMD_NFS
> +#define CONFIG_CMD_PCI
>  
>  #undef CONFIG_WATCHDOG			/* watchdog 
> disabled		*/
>  
> --
> 1.7.0.4
> 
> 

Disclaimer: The information contained in this email, including any attachments is 
confidential and is for the sole use of the intended recipient(s). Any unauthorized 
review, use, disclosure or distribution is prohibited. If you are not the intended 
recipient, please notify the sender immediately by replying to this message and 
destroy all copies of this message and any attachments.

[U-Boot] [PATCH] ppc4xx: alpr: Remove some not needed commands to make image fit again

[Stefan Roese]

Hi Pieter,

I didn't hear from you for a long time. Hope you are doing well and business 
is good for Prodrive as always.

On Thursday 08 April 2010 09:37:05 Pieter Voorthuijsen wrote:
> > The latest changes increased the size of the alpr image a bit more.
> > Now it doesn't fit into the 256k reserved for it. This patch
> > now removes the commands "loads" and "loadb" which are not
> > needed in the production systems.
> 
> I see that PCI is also enabled again. This is not used and can be turned
> off. Otherwise I think we'll see this 'won't fit' message again in a few
> months. :)

Has PCI support been disabled at some time? Not sure. I'm hesitating to make 
such a change without testing it on real hardware. Please feel free to test it 
and send an addon patch to the list.

Thanks.
 
Cheers,
Stefan

--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: office at denx.de

[U-Boot] [PATCH] ppc4xx: alpr: Remove some not needed commands to make image fit again

[Stefan Roese]

On Thursday 08 April 2010 09:34:05 Stefan Roese wrote:
> The latest changes increased the size of the alpr image a bit more.
> Now it doesn't fit into the 256k reserved for it. This patch now removes
> the commands "loads" and "loadb" which are not needed in the production
> systems.

Applied to u-boot-ppc4xx/master. Thanks.
 
Cheers,
Stefan

--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: office at denx.de

[U-Boot] [PATCH] ppc4xx: alpr: Remove some not needed commands to make image fit again

[Stefan Roese]

The latest changes in the u-boot/next branch increased the size of the
alpr image a bit more. Now it doesn't fit into the 256k reserved for it.
This patch now removes the commands "askenv" and "irq" which are not
needed in the production systems.

Signed-off-by: Stefan Roese <sr@denx.de>
Cc: Pieter Voorthuijsen <pieter.voorthuijsen@prodrive.nl>
---
 include/configs/alpr.h |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/include/configs/alpr.h b/include/configs/alpr.h
index e6248e9..964630f 100644
--- a/include/configs/alpr.h
+++ b/include/configs/alpr.h
@@ -231,13 +231,11 @@
  */
 #include <config_cmd_default.h>
 
-#define CONFIG_CMD_ASKENV
 #define CONFIG_CMD_DHCP
 #define CONFIG_CMD_DIAG
 #define CONFIG_CMD_EEPROM
 #define CONFIG_CMD_FPGA
 #define CONFIG_CMD_I2C
-#define CONFIG_CMD_IRQ
 #define CONFIG_CMD_MII
 #define CONFIG_CMD_NAND
 #define CONFIG_CMD_NET
-- 
1.6.5.2

[U-Boot] [PATCH] ppc4xx: alpr: Remove some not needed commands to make image fit again

[Stefan Roese]

On Tuesday 17 November 2009 15:54:03 Stefan Roese wrote:
> The latest changes in the u-boot/next branch increased the size of the
> alpr image a bit more. Now it doesn't fit into the 256k reserved for it.
> This patch now removes the commands "askenv" and "irq" which are not
> needed in the production systems.

Applied to u-boot-ppc4xx/next. Thanks.
 
Cheers,
Stefan

--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: office at denx.de