Re: [U-Boot] [PATCH] arch/arm/lib/board.c: fix build error

2011-10-24 Thread Heiko Schocher
Hello Wolfgang,

Wolfgang Denk wrote:
 Commit dc8bbea arm: Use getenv_ulong() in place of getenv(), strtoul
 introduced a build error for all ARM boards with network support:
 
 board.c: In function 'board_init_r':
 board.c:569: error: 's' undeclared (first use in this function)
 board.c:569: error: (Each undeclared identifier is reported only once
 board.c:569: error: for each function it appears in.)
 
 Fix it.
 
 Signed-off-by: Wolfgang Denk w...@denx.de

posted a patch too, see your fix too late, sorry, so
forget my patch:

Acked-by: Heiko Schocher h...@denx.de

for this.

Thanks for fixing.

bye,
Heiko
-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] arch/arm/lib/board.c: fix build error

2011-10-23 Thread Wolfgang Denk
Commit dc8bbea arm: Use getenv_ulong() in place of getenv(), strtoul
introduced a build error for all ARM boards with network support:

board.c: In function 'board_init_r':
board.c:569: error: 's' undeclared (first use in this function)
board.c:569: error: (Each undeclared identifier is reported only once
board.c:569: error: for each function it appears in.)

Fix it.

Signed-off-by: Wolfgang Denk w...@denx.de
---
 arch/arm/lib/board.c |3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/arch/arm/lib/board.c b/arch/arm/lib/board.c
index c764844..367cf6b 100644
--- a/arch/arm/lib/board.c
+++ b/arch/arm/lib/board.c
@@ -441,6 +441,9 @@ void board_init_r(gd_t *id, ulong dest_addr)
 #if !defined(CONFIG_SYS_NO_FLASH)
ulong flash_size;
 #endif
+#if defined(CONFIG_CMD_NET)
+   char *s;
+#endif
 
gd = id;
 
-- 
1.7.6.2

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] arch/arm/lib/board.c: fix build error

2011-10-23 Thread Simon Glass
Hi Wolfgang,

On Sun, Oct 23, 2011 at 2:51 PM, Wolfgang Denk w...@denx.de wrote:
 Commit dc8bbea arm: Use getenv_ulong() in place of getenv(), strtoul
 introduced a build error for all ARM boards with network support:

 board.c: In function 'board_init_r':
 board.c:569: error: 's' undeclared (first use in this function)
 board.c:569: error: (Each undeclared identifier is reported only once
 board.c:569: error: for each function it appears in.)

 Fix it.

This might fix all boards if they have both net and flash, or neither.
But I sent a patch which tries to handle the case where we have one
but not the other.

(I suspect the PPC warning might be similar, but will wait until I
have done a MAKEALL before commenting on that)

Regards,
Simon


 Signed-off-by: Wolfgang Denk w...@denx.de
 ---
  arch/arm/lib/board.c |    3 +++
  1 files changed, 3 insertions(+), 0 deletions(-)

 diff --git a/arch/arm/lib/board.c b/arch/arm/lib/board.c
 index c764844..367cf6b 100644
 --- a/arch/arm/lib/board.c
 +++ b/arch/arm/lib/board.c
 @@ -441,6 +441,9 @@ void board_init_r(gd_t *id, ulong dest_addr)
  #if !defined(CONFIG_SYS_NO_FLASH)
        ulong flash_size;
  #endif
 +#if defined(CONFIG_CMD_NET)
 +       char *s;
 +#endif

        gd = id;

 --
 1.7.6.2

 ___
 U-Boot mailing list
 U-Boot@lists.denx.de
 http://lists.denx.de/mailman/listinfo/u-boot

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot