On Tue, Aug 4, 2020 at 5:26 AM Heinrich Schuchardt <xypron.g...@gmx.de> wrote: > > Building with CONFIG_SPL_SMP=n results in: > > arch/riscv/lib/spl.c: In function ‘jump_to_image_no_args’: > arch/riscv/lib/spl.c:33:6: > error: unused variable ‘ret’ [-Werror=unused-variable] > 33 | int ret; > | ^~~ > > Define the variable ret as __maybe_unused. > > Fixes: 191636e44898 ("riscv: Introduce SPL_SMP Kconfig option for U-Boot > SPL")
This should be on the same line > Fixes: 8c59f2023cc8 ("riscv: add SPL support") > Signed-off-by: Heinrich Schuchardt <xypron.g...@gmx.de> > --- > arch/riscv/lib/spl.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/riscv/lib/spl.c b/arch/riscv/lib/spl.c > index c47dcd46ce..ef00ec2bcc 100644 > --- a/arch/riscv/lib/spl.c > +++ b/arch/riscv/lib/spl.c > @@ -30,7 +30,7 @@ void __noreturn jump_to_image_no_args(struct spl_image_info > *spl_image) > { > typedef void __noreturn (*image_entry_riscv_t)(ulong hart, void *dtb); > void *fdt_blob; > - int ret; > + __maybe_unused int ret; > > #if CONFIG_IS_ENABLED(LOAD_FIT) || CONFIG_IS_ENABLED(LOAD_FIT_FULL) > fdt_blob = spl_image->fdt_addr; Reviewed-by: Bin Meng <bin.m...@windriver.com>