[U-Boot] [PATCH v3 01/14] MIPS: initialize board_init_f() argument to zero.

Daniel Schwierzeck
2016-01-12T12:29:24Z

2016-01-12 11:18 GMT+01:00 Purna Chandra Mandal <purna.mandal@microchip.com>:
> Signed-off-by: Purna Chandra Mandal <purna.mandal@microchip.com>
> ---
>
> Changes in v3: None
> Changes in v2: None
>
> arch/mips/cpu/start.S | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/arch/mips/cpu/start.S b/arch/mips/cpu/start.S
> index e95cdca..35d9650 100644
> --- a/arch/mips/cpu/start.S
> +++ b/arch/mips/cpu/start.S
> @@ -185,6 +185,8 @@ reset:
> PTR_ADDU t0, k0, GD_MALLOC_BASE # gd->malloc_base offset
> sw sp, 0(t0)
> #endif
> + /* Initialize args to zero */
> + move a0, zero

the comment should be on the same line and more precise. That is
useful in disassemblies or when debugging. E.g.

move a0, zero # a0 <-- boot_flags = 0

Also add a commit message please explaining why the change is
required. Something like that the boot_flags of board_init_f should be
set to 0 because $a0 may be utilized in lowlevel_init or
mips_cache_reset.

>
> PTR_LA t9, board_init_f
> jr t9
> --
> 1.8.3.1
>

--
- Daniel