[U-Boot-Users] Final call: last minute fixes for 1.3.0 release ?

[Shinya Kuribayashi]

Jean-Christophe PLAGNIOL-VILLARD wrote:
>> * MIPS looks mostly OK, too, except here:
>>
>> 	gth2 board:
>>
>> 	lowlevel_init.S: Assembler messages:
>> 	lowlevel_init.S:413: Warning: Pretending global symbol used as branch target is local.
>>
> If I'm not wrong Shinya send a patch a bout it.

Hmm. I haven't ever seen such an warning with gcc 4.2.1 / bintuils 2.18.
Anyway patch is attached. Hope this helps.

---

[MIPS] board/gth2/lowlevel_init.S: Fix a build warning

From: Shinya Kuribayashi <shinya.kuribayashi at necel.com>

lowlevel_init.S: Assembler messages:
lowlevel_init.S:413: Warning: Pretending global symbol used as branch target is local.

Looking at codes, the `memtest' and `clearmem' are intentional mixed use of
`global symbols' and `label' for debugging purpose. To make it builds, just
disable global-symbols-use for now.  As a result `memtest' still remains as
unused, but leave it be...

Signed-off-by: Shinya Kuribayashi <shinya.kuribayashi at necel.com>
---

 board/gth2/lowlevel_init.S |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/board/gth2/lowlevel_init.S b/board/gth2/lowlevel_init.S
index 983ff70..eea378a 100644
--- a/board/gth2/lowlevel_init.S
+++ b/board/gth2/lowlevel_init.S
@@ -413,7 +413,9 @@ noCacheJump:
 	j clearmem
 	nop
 
+#if 0
 	.globl	memtest
+#endif
 memtest:
 	/* Fill memory with address */
 	li	t0, 0x80000000
@@ -434,7 +436,9 @@ mt1:	lw	t2, 0(t0)
 	bne	t1, zero, mt1
 	nop
 	nop
+#if 0
 	.globl	clearmem
+#endif
 clearmem:
 		/* Clear memory */
 	li	t0, 0x80000000