public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot-Users] 1.3.3-rc2 - release status
@ 2008-04-29 21:49 Wolfgang Denk
  2008-04-30  7:36 ` Matthias Fuchs
  2008-04-30 20:29 ` [U-Boot-Users] [PATCH] env_nand: fix incompatible pointer type on nand_{read, write} Jean-Christophe PLAGNIOL-VILLARD
  0 siblings, 2 replies; 12+ messages in thread
From: Wolfgang Denk @ 2008-04-29 21:49 UTC (permalink / raw)
  To: u-boot

Hello,

U-Boot v1.3.3-rc2 has just been released.

PowerPC seems to be in a pretty good state now;  other  architectures
still  need a lot of testing / fixing. For example, on ARM we see the
following conflicts:

Configuring for kb9202 board...
start.S: Assembler messages:
start.S:205: Error: invalid constant -- `sub r0,r0,#(0x0200+128*1024)'
start.S:378: Error: invalid constant -- `sub r13,r13,#((32*1024)+(0x0200+128*1024))'
start.S:379: Error: invalid constant -- `sub r2,r2,#((32*1024)+(0x0200+128*1024))'
start.S:384: Error: invalid constant -- `sub r13,r13,#((32*1024)+(0x0200+128*1024))'
start.S:385: Error: invalid constant -- `sub r2,r2,#((32*1024)+(0x0200+128*1024))'
start.S:390: Error: invalid constant -- `sub r13,r13,#((32*1024)+(0x0200+128*1024))'
start.S:391: Error: invalid constant -- `sub r2,r2,#((32*1024)+(0x0200+128*1024))'
start.S:396: Error: invalid constant -- `sub r13,r13,#((32*1024)+(0x0200+128*1024))'
start.S:397: Error: invalid constant -- `sub r2,r2,#((32*1024)+(0x0200+128*1024))'
start.S:402: Error: invalid constant -- `sub r13,r13,#((32*1024)+(0x0200+128*1024))'
start.S:403: Error: invalid constant -- `sub r2,r2,#((32*1024)+(0x0200+128*1024))'
start.S:427: Error: invalid constant -- `sub r13,r13,#((32*1024)+(0x0200+128*1024))'
start.S:428: Error: invalid constant -- `sub r2,r2,#((32*1024)+(0x0200+128*1024))'
start.S:433: Error: invalid constant -- `sub r13,r13,#((32*1024)+(0x0200+128*1024))'
start.S:434: Error: invalid constant -- `sub r2,r2,#((32*1024)+(0x0200+128*1024))'
make[1]: *** [start.o] Error 1


Configuring for netstar board...
arm-linux-ld: cannot find -lgeneric
make[1]: *** [eeprom.srec] Error 1


Configuring for voiceblue board...
arm-linux-ld: cannot find -lgeneric
make[1]: *** [eeprom.srec] Error 1


Configuring for apollon board...
onenand_base.c:25: error: static declaration of 'memcpy' follows non-static declaration
/home/wd/git/u-boot/master/include/linux/string.h:70: error: previous declaration of 'memcpy' was here
make[1]: *** [onenand_base.o] Error 1


Configuring for adsvix board...
start.S:183:1: warning: "ICMR" redefined
In file included from start.S:33:
/home/wd/git/u-boot/master/include/asm/arch/pxa-regs.h:935:1: warning: this is the location of the previous definition
start.S:187:1: warning: "RCSR" redefined
/home/wd/git/u-boot/master/include/asm/arch/pxa-regs.h:1602:1: warning: this is the location of the previous definition
start.S:191:1: warning: "OSMR3" redefined
/home/wd/git/u-boot/master/include/asm/arch/pxa-regs.h:869:1: warning: this is the location of the previous definition
start.S:192:1: warning: "OSCR" redefined
/home/wd/git/u-boot/master/include/asm/arch/pxa-regs.h:870:1: warning: this is the location of the previous definition
start.S:193:1: warning: "OWER" redefined
/home/wd/git/u-boot/master/include/asm/arch/pxa-regs.h:872:1: warning: this is the location of the previous definition
start.S:194:1: warning: "OIER" redefined
/home/wd/git/u-boot/master/include/asm/arch/pxa-regs.h:873:1: warning: this is the location of the previous definition
start.S:207:1: warning: "CCCR" redefined
/home/wd/git/u-boot/master/include/asm/arch/pxa-regs.h:1770:1: warning: this is the location of the previous definition
pcmcia.c:39:2: warning: #warning "Board will only supply 5V, wait for next HW spin for selectable power"
pxa_pcmcia.c: In function 'pcmcia_on':
pxa_pcmcia.c:39: warning: operation on 'i' may be undefined
cmd_ide.c: In function '__ide_outb':
cmd_ide.c:828: warning: implicit declaration of function 'outb'
cmd_ide.c: In function '__ide_inb':
cmd_ide.c:837: warning: implicit declaration of function 'inb'
common/libcommon.a(cmd_ide.o): In function `__ide_outb':
/home/wd/git/u-boot/master/common/cmd_ide.c:828: undefined reference to `outb'
common/libcommon.a(cmd_ide.o): In function `__ide_inb':
/home/wd/git/u-boot/master/common/cmd_ide.c:837: undefined reference to `inb'
make: *** [u-boot] Error 1


Configuring for actux4 board...
actux4.c: In function 'board_init':
actux4.c:83: warning: left shift count >= width of type
actux4.c:83: warning: left shift count >= width of type


Configuring for davinci_schmoogie board...
env_nand.c: In function 'saveenv':
env_nand.c:200: warning: passing argument 3 of 'nand_write' from incompatible pointer type
env_nand.c: In function 'env_relocate_spec':
env_nand.c:275: warning: passing argument 3 of 'nand_read' from incompatible pointer type



Board maintainers / custodians - please help!


Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
If programming was easy, they wouldn't need something as  complicated
as a human being to do it, now would they?
                       - L. Wall & R. L. Schwartz, _Programming Perl_

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

* [U-Boot-Users] 1.3.3-rc2 - release status
  2008-04-29 21:49 [U-Boot-Users] 1.3.3-rc2 - release status Wolfgang Denk
@ 2008-04-30  7:36 ` Matthias Fuchs
  2008-04-30  7:49   ` Wolfgang Denk
  2008-04-30 20:29 ` [U-Boot-Users] [PATCH] env_nand: fix incompatible pointer type on nand_{read, write} Jean-Christophe PLAGNIOL-VILLARD
  1 sibling, 1 reply; 12+ messages in thread
From: Matthias Fuchs @ 2008-04-30  7:36 UTC (permalink / raw)
  To: u-boot

Hi, 

I noticed some warnings when compiling recent U-Boot source for some 4xx boards 
with gcc 4.0.0 or 4.2.2. Perhaps somebody has a clue on how to get rid of these:

gcc -g -Wall -pedantic -idirafter /data/home/matthias/git/u-boot/u-boot/include -idirafter /data/home/matthias/git/u-boot/u-boot/include2 -idirafter /data/home/matthias/git/u-boot/u-boot/include -DTEXT_BASE=0xFFFA0000 -DUSE_HOSTCC -O -c -o md5.o md5.c
md5.c: In function ?MD5Update?:
md5.c:95: warning: implicit declaration of function ?memmove?
md5.c:95: warning: incompatible implicit declaration of built-in function ?memmove?
md5.c:98: warning: incompatible implicit declaration of built-in function ?memmove?
md5.c:107: warning: incompatible implicit declaration of built-in function ?memmove?
md5.c:116: warning: incompatible implicit declaration of built-in function ?memmove?
md5.c: In function ?MD5Final?:
md5.c:143: warning: implicit declaration of function ?memset?
md5.c:143: warning: incompatible implicit declaration of built-in function ?memset?
md5.c:151: warning: incompatible implicit declaration of built-in function ?memset?
md5.c:161: warning: incompatible implicit declaration of built-in function ?memmove?
md5.c:162: warning: incompatible implicit declaration of built-in function ?memset?
gcc -g -Wall -pedantic -idirafter /data/home/matthias/git/u-boot/u-boot/include -idirafter /data/home/matthias/git/u-boot/u-boot/include2 -idirafter /data/home/matthias/git/u-boot/u-boot/include -DTEXT_BASE=0xFFFA0000 -DUSE_HOSTCC -O -c -o sha1.o sha1.c
sha1.c: In function ?sha1_update?:
sha1.c:249: warning: implicit declaration of function ?memcpy?
sha1.c:249: warning: incompatible implicit declaration of built-in function ?memcpy?
sha1.c:263: warning: incompatible implicit declaration of built-in function ?memcpy?
sha1.c: In function ?sha1_hmac?:
sha1.c:360: warning: implicit declaration of function ?memset?
sha1.c:360: warning: incompatible implicit declaration of built-in function ?memset?


Matthias

On Tuesday 29 April 2008 23:49, Wolfgang Denk wrote:
> Hello,
> 
> U-Boot v1.3.3-rc2 has just been released.
> 
> PowerPC seems to be in a pretty good state now;  other  architectures
> still  need a lot of testing / fixing. For example, on ARM we see the
> following conflicts:
> 
> Configuring for kb9202 board...
> start.S: Assembler messages:
> start.S:205: Error: invalid constant -- `sub r0,r0,#(0x0200+128*1024)'
> start.S:378: Error: invalid constant -- `sub r13,r13,#((32*1024)+(0x0200+128*1024))'
> start.S:379: Error: invalid constant -- `sub r2,r2,#((32*1024)+(0x0200+128*1024))'
> start.S:384: Error: invalid constant -- `sub r13,r13,#((32*1024)+(0x0200+128*1024))'
> start.S:385: Error: invalid constant -- `sub r2,r2,#((32*1024)+(0x0200+128*1024))'
> start.S:390: Error: invalid constant -- `sub r13,r13,#((32*1024)+(0x0200+128*1024))'
> start.S:391: Error: invalid constant -- `sub r2,r2,#((32*1024)+(0x0200+128*1024))'
> start.S:396: Error: invalid constant -- `sub r13,r13,#((32*1024)+(0x0200+128*1024))'
> start.S:397: Error: invalid constant -- `sub r2,r2,#((32*1024)+(0x0200+128*1024))'
> start.S:402: Error: invalid constant -- `sub r13,r13,#((32*1024)+(0x0200+128*1024))'
> start.S:403: Error: invalid constant -- `sub r2,r2,#((32*1024)+(0x0200+128*1024))'
> start.S:427: Error: invalid constant -- `sub r13,r13,#((32*1024)+(0x0200+128*1024))'
> start.S:428: Error: invalid constant -- `sub r2,r2,#((32*1024)+(0x0200+128*1024))'
> start.S:433: Error: invalid constant -- `sub r13,r13,#((32*1024)+(0x0200+128*1024))'
> start.S:434: Error: invalid constant -- `sub r2,r2,#((32*1024)+(0x0200+128*1024))'
> make[1]: *** [start.o] Error 1
> 
> 
> Configuring for netstar board...
> arm-linux-ld: cannot find -lgeneric
> make[1]: *** [eeprom.srec] Error 1
> 
> 
> Configuring for voiceblue board...
> arm-linux-ld: cannot find -lgeneric
> make[1]: *** [eeprom.srec] Error 1
> 
> 
> Configuring for apollon board...
> onenand_base.c:25: error: static declaration of 'memcpy' follows non-static declaration
> /home/wd/git/u-boot/master/include/linux/string.h:70: error: previous declaration of 'memcpy' was here
> make[1]: *** [onenand_base.o] Error 1
> 
> 
> Configuring for adsvix board...
> start.S:183:1: warning: "ICMR" redefined
> In file included from start.S:33:
> /home/wd/git/u-boot/master/include/asm/arch/pxa-regs.h:935:1: warning: this is the location of the previous definition
> start.S:187:1: warning: "RCSR" redefined
> /home/wd/git/u-boot/master/include/asm/arch/pxa-regs.h:1602:1: warning: this is the location of the previous definition
> start.S:191:1: warning: "OSMR3" redefined
> /home/wd/git/u-boot/master/include/asm/arch/pxa-regs.h:869:1: warning: this is the location of the previous definition
> start.S:192:1: warning: "OSCR" redefined
> /home/wd/git/u-boot/master/include/asm/arch/pxa-regs.h:870:1: warning: this is the location of the previous definition
> start.S:193:1: warning: "OWER" redefined
> /home/wd/git/u-boot/master/include/asm/arch/pxa-regs.h:872:1: warning: this is the location of the previous definition
> start.S:194:1: warning: "OIER" redefined
> /home/wd/git/u-boot/master/include/asm/arch/pxa-regs.h:873:1: warning: this is the location of the previous definition
> start.S:207:1: warning: "CCCR" redefined
> /home/wd/git/u-boot/master/include/asm/arch/pxa-regs.h:1770:1: warning: this is the location of the previous definition
> pcmcia.c:39:2: warning: #warning "Board will only supply 5V, wait for next HW spin for selectable power"
> pxa_pcmcia.c: In function 'pcmcia_on':
> pxa_pcmcia.c:39: warning: operation on 'i' may be undefined
> cmd_ide.c: In function '__ide_outb':
> cmd_ide.c:828: warning: implicit declaration of function 'outb'
> cmd_ide.c: In function '__ide_inb':
> cmd_ide.c:837: warning: implicit declaration of function 'inb'
> common/libcommon.a(cmd_ide.o): In function `__ide_outb':
> /home/wd/git/u-boot/master/common/cmd_ide.c:828: undefined reference to `outb'
> common/libcommon.a(cmd_ide.o): In function `__ide_inb':
> /home/wd/git/u-boot/master/common/cmd_ide.c:837: undefined reference to `inb'
> make: *** [u-boot] Error 1
> 
> 
> Configuring for actux4 board...
> actux4.c: In function 'board_init':
> actux4.c:83: warning: left shift count >= width of type
> actux4.c:83: warning: left shift count >= width of type
> 
> 
> Configuring for davinci_schmoogie board...
> env_nand.c: In function 'saveenv':
> env_nand.c:200: warning: passing argument 3 of 'nand_write' from incompatible pointer type
> env_nand.c: In function 'env_relocate_spec':
> env_nand.c:275: warning: passing argument 3 of 'nand_read' from incompatible pointer type
> 
> 
> 
> Board maintainers / custodians - please help!
> 
> 
> Best regards,
> 
> Wolfgang Denk
> 

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

* [U-Boot-Users] 1.3.3-rc2 - release status
  2008-04-30  7:36 ` Matthias Fuchs
@ 2008-04-30  7:49   ` Wolfgang Denk
  2008-05-01  7:46     ` Matthias Fuchs
  0 siblings, 1 reply; 12+ messages in thread
From: Wolfgang Denk @ 2008-04-30  7:49 UTC (permalink / raw)
  To: u-boot

In message <200804300936.47258.matthias.fuchs@esd-electronics.com> you wrote:
> 
> I noticed some warnings when compiling recent U-Boot source for some 4xx bo> ards 
> with gcc 4.0.0 or 4.2.2. Perhaps somebody has a clue on how to get rid of t> hese:

That's your host GCC. WHich distro is this?

I didn't see such warnigns yet...

> gcc -g -Wall -pedantic -idirafter /data/home/matthias/git/u-boot/u-boot/inc> lude -idirafter /data/home/matthias/git/u-boot/u-boot/include2 -idirafter /> data/home/matthias/git/u-boot/u-boot/include -DTEXT_BASE=0xFFFA0000 -DUSE> _HOSTCC -O -c -o md5.o md5.c
> md5.c: In function ???MD5Update???:
> md5.c:95: warning: implicit declaration of function ???memmove??> ?
> md5.c:95: warning: incompatible implicit declaration of built-in function > ???memmove???
> md5.c:98: warning: incompatible implicit declaration of built-in function > ???memmove???
> md5.c:107: warning: incompatible implicit declaration of built-in function > ???memmove???
> md5.c:116: warning: incompatible implicit declaration of built-in function > ???memmove???

I smell some header file incompatibitilies -  eventually  this  is  a
distro issue.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"A fractal is by definition a set for which the Hausdorff Besicovitch
dimension strictly exceeds the topological dimension."
- Mandelbrot, _The Fractal Geometry of Nature_

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

* [U-Boot-Users] [PATCH] env_nand: fix incompatible pointer type on nand_{read, write}
  2008-04-29 21:49 [U-Boot-Users] 1.3.3-rc2 - release status Wolfgang Denk
  2008-04-30  7:36 ` Matthias Fuchs
@ 2008-04-30 20:29 ` Jean-Christophe PLAGNIOL-VILLARD
  2008-04-30 20:38   ` [U-Boot-Users] [PATCH] cmd_nand: fix warning: str2long ncompatible pointer type Jean-Christophe PLAGNIOL-VILLARD
  2008-04-30 20:56   ` [U-Boot-Users] [PATCH] env_nand: fix incompatible pointer type on nand_{read, write} Wolfgang Denk
  1 sibling, 2 replies; 12+ messages in thread
From: Jean-Christophe PLAGNIOL-VILLARD @ 2008-04-30 20:29 UTC (permalink / raw)
  To: u-boot

env_nand.c: In function 'saveenv':
env_nand.c:200: warning: passing argument 3 of 'nand_write' from incompatible pointer type
env_nand.c: In function 'env_relocate_spec':
env_nand.c:275: warning: passing argument 3 of 'nand_read' from incompatible pointer type

Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>

diff --git a/common/env_nand.c b/common/env_nand.c
index 0dddddf..d8874a6 100644
--- a/common/env_nand.c
+++ b/common/env_nand.c
@@ -188,7 +188,7 @@ int saveenv(void)
 #else /* ! CFG_ENV_OFFSET_REDUND */
 int saveenv(void)
 {
-	ulong total;
+	size_t total;
 	int ret = 0;
 
 	puts ("Erasing Nand...");
@@ -268,12 +268,12 @@ void env_relocate_spec (void)
 void env_relocate_spec (void)
 {
 #if !defined(ENV_IS_EMBEDDED)
-	ulong total;
+	size_t total;
 	int ret;
 
 	total = CFG_ENV_SIZE;
 	ret = nand_read(&nand_info[0], CFG_ENV_OFFSET, &total, (u_char*)env_ptr);
-  	if (ret || total != CFG_ENV_SIZE)
+	if (ret || total != CFG_ENV_SIZE)
 		return use_default();
 
 	if (crc32(0, env_ptr->data, ENV_SIZE) != env_ptr->crc)
-- 
1.5.4.5

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

* [U-Boot-Users] [PATCH] cmd_nand: fix warning: str2long ncompatible pointer type
  2008-04-30 20:29 ` [U-Boot-Users] [PATCH] env_nand: fix incompatible pointer type on nand_{read, write} Jean-Christophe PLAGNIOL-VILLARD
@ 2008-04-30 20:38   ` Jean-Christophe PLAGNIOL-VILLARD
  2008-04-30 20:59     ` Wolfgang Denk
  2008-05-01  0:13     ` [U-Boot-Users] [PATCH 0/1] pxa: fix pcmcia operation on 'i' may be undefined Jean-Christophe PLAGNIOL-VILLARD
  2008-04-30 20:56   ` [U-Boot-Users] [PATCH] env_nand: fix incompatible pointer type on nand_{read, write} Wolfgang Denk
  1 sibling, 2 replies; 12+ messages in thread
From: Jean-Christophe PLAGNIOL-VILLARD @ 2008-04-30 20:38 UTC (permalink / raw)
  To: u-boot

Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>

diff --git a/common/cmd_nand.c b/common/cmd_nand.c
index 1a3c1df..37eb41b 100644
--- a/common/cmd_nand.c
+++ b/common/cmd_nand.c
@@ -110,7 +110,7 @@ arg_off_size(int argc, char *argv[], nand_info_t *nand, ulong *off, size_t *size
 			}
 			*off = part->offset;
 			if (argc >= 2) {
-				if (!(str2long(argv[1], size))) {
+				if (!(str2long(argv[1], (ulong *)size))) {
 					printf("'%s' is not a number\n", argv[1]);
 					return -1;
 				}
-- 
1.5.4.5

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

* [U-Boot-Users] [PATCH] env_nand: fix incompatible pointer type on nand_{read, write}
  2008-04-30 20:29 ` [U-Boot-Users] [PATCH] env_nand: fix incompatible pointer type on nand_{read, write} Jean-Christophe PLAGNIOL-VILLARD
  2008-04-30 20:38   ` [U-Boot-Users] [PATCH] cmd_nand: fix warning: str2long ncompatible pointer type Jean-Christophe PLAGNIOL-VILLARD
@ 2008-04-30 20:56   ` Wolfgang Denk
  1 sibling, 0 replies; 12+ messages in thread
From: Wolfgang Denk @ 2008-04-30 20:56 UTC (permalink / raw)
  To: u-boot

In message <1209587348-15402-1-git-send-email-plagnioj@jcrosoft.com> you wrote:
> env_nand.c: In function 'saveenv':
> env_nand.c:200: warning: passing argument 3 of 'nand_write' from incompatible pointer type
> env_nand.c: In function 'env_relocate_spec':
> env_nand.c:275: warning: passing argument 3 of 'nand_read' from incompatible pointer type
> 
> Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>

That was already fixed before.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
CAUTION:  The Mass of This Product Contains the Energy Equivalent  of
85 Million Tons of TNT per Net Ounce of Weight.

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

* [U-Boot-Users] [PATCH] cmd_nand: fix warning: str2long ncompatible pointer type
  2008-04-30 20:38   ` [U-Boot-Users] [PATCH] cmd_nand: fix warning: str2long ncompatible pointer type Jean-Christophe PLAGNIOL-VILLARD
@ 2008-04-30 20:59     ` Wolfgang Denk
  2008-05-01  0:13     ` [U-Boot-Users] [PATCH 0/1] pxa: fix pcmcia operation on 'i' may be undefined Jean-Christophe PLAGNIOL-VILLARD
  1 sibling, 0 replies; 12+ messages in thread
From: Wolfgang Denk @ 2008-04-30 20:59 UTC (permalink / raw)
  To: u-boot

In message <1209587897-25440-1-git-send-email-plagnioj@jcrosoft.com> you wrote:
> Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
> 
> diff --git a/common/cmd_nand.c b/common/cmd_nand.c

Applied, thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
There are bugs and then there are bugs.  And then there are bugs.
                                                    - Karl Lehenbauer

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

* [U-Boot-Users] [PATCH 0/1] pxa: fix pcmcia operation on 'i' may be undefined
  2008-04-30 20:38   ` [U-Boot-Users] [PATCH] cmd_nand: fix warning: str2long ncompatible pointer type Jean-Christophe PLAGNIOL-VILLARD
  2008-04-30 20:59     ` Wolfgang Denk
@ 2008-05-01  0:13     ` Jean-Christophe PLAGNIOL-VILLARD
  2008-05-01  0:13       ` [U-Boot-Users] [PATCH 1/1] pxa: fix previous definition on cpu init Jean-Christophe PLAGNIOL-VILLARD
  2008-05-03 18:55       ` [U-Boot-Users] [PATCH 0/1] pxa: fix pcmcia operation on 'i' may be undefined Wolfgang Denk
  1 sibling, 2 replies; 12+ messages in thread
From: Jean-Christophe PLAGNIOL-VILLARD @ 2008-05-01  0:13 UTC (permalink / raw)
  To: u-boot

Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>

diff --git a/drivers/pcmcia/pxa_pcmcia.c b/drivers/pcmcia/pxa_pcmcia.c
index 6020e46..7b02050 100644
--- a/drivers/pcmcia/pxa_pcmcia.c
+++ b/drivers/pcmcia/pxa_pcmcia.c
@@ -24,7 +24,8 @@ int pcmcia_on (void)
 
 		0, 0
 	};
-	int i, rc;
+	int rc;
+	int i = 0;
 
 #ifdef CONFIG_EXADRON1
 	int cardDetect;
@@ -34,9 +35,10 @@ int pcmcia_on (void)
 
 	debug ("%s\n", __FUNCTION__);
 
-	i = 0;
-	while (reg_arr[i])
-		*((volatile unsigned int *) reg_arr[i++]) |= reg_arr[i++];
+	while (reg_arr[i]) {
+		(*(volatile unsigned int *) reg_arr[i]) |= reg_arr[i + 1];
+		i += 2;
+	}
 	udelay (1000);
 
 	debug ("%s: programmed mem controller \n", __FUNCTION__);
@@ -44,7 +46,7 @@ int pcmcia_on (void)
 #ifdef CONFIG_EXADRON1
 
 /*define useful BCR masks */
-#define BCR_CF_INIT_VAL  		    0x00007230
+#define BCR_CF_INIT_VAL			    0x00007230
 #define BCR_CF_PWRON_BUSOFF_RESETOFF_VAL    0x00007231
 #define BCR_CF_PWRON_BUSOFF_RESETON_VAL     0x00007233
 #define BCR_CF_PWRON_BUSON_RESETON_VAL      0x00007213
-- 
1.5.4.5

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

* [U-Boot-Users] [PATCH 1/1] pxa: fix previous definition on cpu init
  2008-05-01  0:13     ` [U-Boot-Users] [PATCH 0/1] pxa: fix pcmcia operation on 'i' may be undefined Jean-Christophe PLAGNIOL-VILLARD
@ 2008-05-01  0:13       ` Jean-Christophe PLAGNIOL-VILLARD
  2008-05-03 18:56         ` Wolfgang Denk
  2008-05-03 18:55       ` [U-Boot-Users] [PATCH 0/1] pxa: fix pcmcia operation on 'i' may be undefined Wolfgang Denk
  1 sibling, 1 reply; 12+ messages in thread
From: Jean-Christophe PLAGNIOL-VILLARD @ 2008-05-01  0:13 UTC (permalink / raw)
  To: u-boot

start.S:183:1: warning: "ICMR" redefined
In file included from start.S:33:
/home/j/u-boot-at91/include/asm/arch/pxa-regs.h:935:1: warning: this is the location of the previous definition
start.S:187:1: warning: "RCSR" redefined
/home/j/u-boot-at91/include/asm/arch/pxa-regs.h:1602:1: warning: this is the location of the previous definition
start.S:191:1: warning: "OSMR3" redefined
/home/j/u-boot-at91/include/asm/arch/pxa-regs.h:869:1: warning: this is the location of the previous definition
start.S:192:1: warning: "OSCR" redefined
/home/j/u-boot-at91/include/asm/arch/pxa-regs.h:870:1: warning: this is the location of the previous definition
start.S:193:1: warning: "OWER" redefined
/home/j/u-boot-at91/include/asm/arch/pxa-regs.h:872:1: warning: this is the location of the previous definition
start.S:194:1: warning: "OIER" redefined
/home/j/u-boot-at91/include/asm/arch/pxa-regs.h:873:1: warning: this is the location of the previous definition
start.S:207:1: warning: "CCCR" redefined
/home/j/u-boot-at91/include/asm/arch/pxa-regs.h:1770:1: warning: this is the location of the previous definition

Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>

diff --git a/cpu/pxa/start.S b/cpu/pxa/start.S
index 31f408d..1cdb709 100644
--- a/cpu/pxa/start.S
+++ b/cpu/pxa/start.S
@@ -166,17 +166,13 @@ _start_armboot: .word start_armboot
 /*									    */
 /****************************************************************************/
 /* mk at tbd: Fix this! */
-#if defined(CONFIG_PXA250) || defined(CONFIG_CPU_MONAHANS)
+#undef RCSR
 #undef ICMR
 #undef OSMR3
 #undef OSCR
 #undef OWER
 #undef OIER
-#endif /* CONFIG_PXA250 || CONFIG_CPU_MONAHANS */
-#ifdef CONFIG_PXA250
-#undef RCSR
 #undef CCCR
-#endif /* CONFIG_PXA250 */
 
 /* Interrupt-Controller base address					    */
 IC_BASE:	   .word	   0x40d00000
-- 
1.5.4.5

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

* [U-Boot-Users] 1.3.3-rc2 - release status
  2008-04-30  7:49   ` Wolfgang Denk
@ 2008-05-01  7:46     ` Matthias Fuchs
  0 siblings, 0 replies; 12+ messages in thread
From: Matthias Fuchs @ 2008-05-01  7:46 UTC (permalink / raw)
  To: u-boot

On Wednesday 30 April 2008 09:49:20 Wolfgang Denk wrote:
> In message <200804300936.47258.matthias.fuchs@esd-electronics.com> you 
wrote:
> > I noticed some warnings when compiling recent U-Boot source for some 4xx
> > bo> ards with gcc 4.0.0 or 4.2.2. Perhaps somebody has a clue on how to
> > get rid of t> hese:
>
> That's your host GCC. WHich distro is this?
Yes you are right. 

It's Debian Etch.

matthias at debby:~$ gcc -v
Using built-in specs.
Target: i486-linux-gnu
Configured 
with: ../src/configure -v --enable-languages=c,c++,fortran,objc,obj-c++,treelang --prefix=/usr --enable-shared --with-system-zlib --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --enable-nls --program-suffix=-4.1 --enable-__cxa_atexit --enable-clocale=gnu --enable-libstdcxx-debug --enable-mpfr --with-tune=i686 --enable-checking=release 
i486-linux-gnu
Thread model: posix
gcc version 4.1.2 20061115 (prerelease) (Debian 4.1.1-21

Matthias

>
> I didn't see such warnigns yet...
>
> > gcc -g -Wall -pedantic -idirafter
> > /data/home/matthias/git/u-boot/u-boot/inc> lude -idirafter
> > /data/home/matthias/git/u-boot/u-boot/include2 -idirafter />
> > data/home/matthias/git/u-boot/u-boot/include -DTEXT_BASE=0xFFFA0000
> > -DUSE> _HOSTCC -O -c -o md5.o md5.c md5.c: In function ???MD5Update???:
> > md5.c:95: warning: implicit declaration of function ???memmove??> ?
> > md5.c:95: warning: incompatible implicit declaration of built-in function
> > > ???memmove??? md5.c:98: warning: incompatible implicit declaration of
> > built-in function > ???memmove??? md5.c:107: warning: incompatible
> > implicit declaration of built-in function > ???memmove??? md5.c:116:
> > warning: incompatible implicit declaration of built-in function >
> > ???memmove???
>
> I smell some header file incompatibitilies -  eventually  this  is  a
> distro issue.
>
> Best regards,
>
> Wolfgang Denk

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

* [U-Boot-Users] [PATCH 0/1] pxa: fix pcmcia operation on 'i' may be undefined
  2008-05-01  0:13     ` [U-Boot-Users] [PATCH 0/1] pxa: fix pcmcia operation on 'i' may be undefined Jean-Christophe PLAGNIOL-VILLARD
  2008-05-01  0:13       ` [U-Boot-Users] [PATCH 1/1] pxa: fix previous definition on cpu init Jean-Christophe PLAGNIOL-VILLARD
@ 2008-05-03 18:55       ` Wolfgang Denk
  1 sibling, 0 replies; 12+ messages in thread
From: Wolfgang Denk @ 2008-05-03 18:55 UTC (permalink / raw)
  To: u-boot

In message <1209600824-12950-1-git-send-email-plagnioj@jcrosoft.com> you wrote:
> Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
> 
> diff --git a/drivers/pcmcia/pxa_pcmcia.c b/drivers/pcmcia/pxa_pcmcia.c
> index 6020e46..7b02050 100644
> --- a/drivers/pcmcia/pxa_pcmcia.c
> +++ b/drivers/pcmcia/pxa_pcmcia.c

Applied, thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"God is a comedian playing to an audience too afraid to laugh."
- Voltaire

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

* [U-Boot-Users] [PATCH 1/1] pxa: fix previous definition on cpu init
  2008-05-01  0:13       ` [U-Boot-Users] [PATCH 1/1] pxa: fix previous definition on cpu init Jean-Christophe PLAGNIOL-VILLARD
@ 2008-05-03 18:56         ` Wolfgang Denk
  0 siblings, 0 replies; 12+ messages in thread
From: Wolfgang Denk @ 2008-05-03 18:56 UTC (permalink / raw)
  To: u-boot

In message <1209600824-12950-2-git-send-email-plagnioj@jcrosoft.com> you wrote:
> start.S:183:1: warning: "ICMR" redefined
> In file included from start.S:33:
> /home/j/u-boot-at91/include/asm/arch/pxa-regs.h:935:1: warning: this is the location of the previous definition
> start.S:187:1: warning: "RCSR" redefined
> /home/j/u-boot-at91/include/asm/arch/pxa-regs.h:1602:1: warning: this is the location of the previous definition
> start.S:191:1: warning: "OSMR3" redefined
> /home/j/u-boot-at91/include/asm/arch/pxa-regs.h:869:1: warning: this is the location of the previous definition
> start.S:192:1: warning: "OSCR" redefined
> /home/j/u-boot-at91/include/asm/arch/pxa-regs.h:870:1: warning: this is the location of the previous definition
> start.S:193:1: warning: "OWER" redefined
> /home/j/u-boot-at91/include/asm/arch/pxa-regs.h:872:1: warning: this is the location of the previous definition
> start.S:194:1: warning: "OIER" redefined
> /home/j/u-boot-at91/include/asm/arch/pxa-regs.h:873:1: warning: this is the location of the previous definition
> start.S:207:1: warning: "CCCR" redefined
> /home/j/u-boot-at91/include/asm/arch/pxa-regs.h:1770:1: warning: this is the location of the previous definition
> 
> Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
> 
> diff --git a/cpu/pxa/start.S b/cpu/pxa/start.S
> index 31f408d..1cdb709 100644
> --- a/cpu/pxa/start.S
> +++ b/cpu/pxa/start.S

Applied, thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Nearly everyone is in favor of going  to  heaven  but  too  many  are
hoping  they'll  live  long  enough  to see an easing of the entrance
requirements. Never appeal to a man's "better nature." he  might  not
have one.

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

end of thread, other threads:[~2008-05-03 18:56 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-04-29 21:49 [U-Boot-Users] 1.3.3-rc2 - release status Wolfgang Denk
2008-04-30  7:36 ` Matthias Fuchs
2008-04-30  7:49   ` Wolfgang Denk
2008-05-01  7:46     ` Matthias Fuchs
2008-04-30 20:29 ` [U-Boot-Users] [PATCH] env_nand: fix incompatible pointer type on nand_{read, write} Jean-Christophe PLAGNIOL-VILLARD
2008-04-30 20:38   ` [U-Boot-Users] [PATCH] cmd_nand: fix warning: str2long ncompatible pointer type Jean-Christophe PLAGNIOL-VILLARD
2008-04-30 20:59     ` Wolfgang Denk
2008-05-01  0:13     ` [U-Boot-Users] [PATCH 0/1] pxa: fix pcmcia operation on 'i' may be undefined Jean-Christophe PLAGNIOL-VILLARD
2008-05-01  0:13       ` [U-Boot-Users] [PATCH 1/1] pxa: fix previous definition on cpu init Jean-Christophe PLAGNIOL-VILLARD
2008-05-03 18:56         ` Wolfgang Denk
2008-05-03 18:55       ` [U-Boot-Users] [PATCH 0/1] pxa: fix pcmcia operation on 'i' may be undefined Wolfgang Denk
2008-04-30 20:56   ` [U-Boot-Users] [PATCH] env_nand: fix incompatible pointer type on nand_{read, write} Wolfgang Denk

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