public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] ppc4xx: Remove some features from ALPR to fit into 256k again
@ 2008-12-09 19:09 Stefan Roese
  2008-12-10  9:23 ` Pieter Voorthuijsen
  0 siblings, 1 reply; 3+ messages in thread
From: Stefan Roese @ 2008-12-09 19:09 UTC (permalink / raw)
  To: u-boot

Signed-off-by: Stefan Roese <sr@denx.de>
---
 include/configs/alpr.h |    5 +----
 1 files changed, 1 insertions(+), 4 deletions(-)

diff --git a/include/configs/alpr.h b/include/configs/alpr.h
index 6e9f5e5..7ce8205 100644
--- a/include/configs/alpr.h
+++ b/include/configs/alpr.h
@@ -235,18 +235,15 @@
 #define CONFIG_CMD_DHCP
 #define CONFIG_CMD_DIAG
 #define CONFIG_CMD_EEPROM
-#define CONFIG_CMD_ELF
 #define CONFIG_CMD_FPGA
 #define CONFIG_CMD_I2C
 #define CONFIG_CMD_IRQ
 #define CONFIG_CMD_MII
 #define CONFIG_CMD_NAND
 #define CONFIG_CMD_NET
-#define CONFIG_CMD_NFS
 #define CONFIG_CMD_PCI
 #define CONFIG_CMD_PING
-#define CONFIG_CMD_REGINFO
-
+#undef CONFIG_CMD_NFS
 
 #undef CONFIG_WATCHDOG			/* watchdog disabled		*/
 
-- 
1.6.0.5

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [U-Boot] [PATCH] ppc4xx: Remove some features from ALPR to fit into 256k again
  2008-12-09 19:09 [U-Boot] [PATCH] ppc4xx: Remove some features from ALPR to fit into 256k again Stefan Roese
@ 2008-12-10  9:23 ` Pieter Voorthuijsen
  2008-12-10  9:27   ` Stefan Roese
  0 siblings, 1 reply; 3+ messages in thread
From: Pieter Voorthuijsen @ 2008-12-10  9:23 UTC (permalink / raw)
  To: u-boot

Hi Stefan,

This patch looks good to me. In my opinion we don't need PCI either, when we need it for debugging, we can enable it...

Best regards, Pieter

> -----Original Message-----
> From: Stefan Roese [mailto:sr at denx.de] 
> Sent: dinsdag 9 december 2008 20:09
> To: u-boot at lists.denx.de
> Cc: Pieter Voorthuijsen
> Subject: [PATCH] ppc4xx: Remove some features from ALPR to 
> fit into 256k again
> 
> Signed-off-by: Stefan Roese <sr@denx.de>
> ---
>  include/configs/alpr.h |    5 +----
>  1 files changed, 1 insertions(+), 4 deletions(-)
> 
> diff --git a/include/configs/alpr.h b/include/configs/alpr.h 
> index 6e9f5e5..7ce8205 100644
> --- a/include/configs/alpr.h
> +++ b/include/configs/alpr.h
> @@ -235,18 +235,15 @@
>  #define CONFIG_CMD_DHCP
>  #define CONFIG_CMD_DIAG
>  #define CONFIG_CMD_EEPROM
> -#define CONFIG_CMD_ELF
>  #define CONFIG_CMD_FPGA
>  #define CONFIG_CMD_I2C
>  #define CONFIG_CMD_IRQ
>  #define CONFIG_CMD_MII
>  #define CONFIG_CMD_NAND
>  #define CONFIG_CMD_NET
> -#define CONFIG_CMD_NFS
>  #define CONFIG_CMD_PCI
>  #define CONFIG_CMD_PING
> -#define CONFIG_CMD_REGINFO
> -
> +#undef CONFIG_CMD_NFS
>  
>  #undef CONFIG_WATCHDOG			/* watchdog 
> disabled		*/
>  
> --
> 1.6.0.5
> 
> 

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.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [U-Boot] [PATCH] ppc4xx: Remove some features from ALPR to fit into 256k again
  2008-12-10  9:23 ` Pieter Voorthuijsen
@ 2008-12-10  9:27   ` Stefan Roese
  0 siblings, 0 replies; 3+ messages in thread
From: Stefan Roese @ 2008-12-10  9:27 UTC (permalink / raw)
  To: u-boot

Hi Pieter,

On Wednesday 10 December 2008, Pieter Voorthuijsen wrote:
> This patch looks good to me.

OK, I'll push it upstream then.

> In my opinion we don't need PCI either, when 
> we need it for debugging, we can enable it...

And then you will find out, that it probably won't fit into the 256k anymore. 
No, let's keep it in for now.

Best regards,
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
=====================================================================

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2008-12-10  9:27 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-12-09 19:09 [U-Boot] [PATCH] ppc4xx: Remove some features from ALPR to fit into 256k again Stefan Roese
2008-12-10  9:23 ` Pieter Voorthuijsen
2008-12-10  9:27   ` Stefan Roese

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox