diff mbox series

[U-Boot] boston: Drop unused return value

Message ID 20170915183353.11321-1-paul.burton@imgtec.com
State Accepted
Commit 555a6529532c0ac82d92f26d56e9becad72216db
Delegated to: Daniel Schwierzeck
Headers show
Series [U-Boot] boston: Drop unused return value | expand

Commit Message

Paul Burton Sept. 15, 2017, 6:33 p.m. UTC
The boston lowlevel_init() function zeroes the return register v0,
despite the function not being expected to return a value & that value
never being used.

Remove the redundant assignment to v0.

Signed-off-by: Paul Burton <paul.burton@imgtec.com>
Cc: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
---

 board/imgtec/boston/lowlevel_init.S | 1 -
 1 file changed, 1 deletion(-)

Comments

Daniel Schwierzeck Sept. 15, 2017, 8:24 p.m. UTC | #1
Am 15.09.2017 um 20:33 schrieb Paul Burton:
> The boston lowlevel_init() function zeroes the return register v0,
> despite the function not being expected to return a value & that value
> never being used.
> 
> Remove the redundant assignment to v0.
> 
> Signed-off-by: Paul Burton <paul.burton@imgtec.com>
> Cc: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
> ---
> 
>  board/imgtec/boston/lowlevel_init.S | 1 -
>  1 file changed, 1 deletion(-)

applied to u-boot-mips, thanks
diff mbox series

Patch

diff --git a/board/imgtec/boston/lowlevel_init.S b/board/imgtec/boston/lowlevel_init.S
index 0c01aa981d..46c9c1d269 100644
--- a/board/imgtec/boston/lowlevel_init.S
+++ b/board/imgtec/boston/lowlevel_init.S
@@ -34,7 +34,6 @@  LEAF(lowlevel_init)
 	PTR_LA	a0, msg_ddr_ok
 	bal	lowlevel_display
 
-	move	v0, zero
 	jr	s0
 	END(lowlevel_init)