Re: [PATCH] NIOS2: setup.c: drop unused variable 'dram_start'

From: Mike Rapoport
Date: Sun Sep 26 2021 - 01:36:37 EST


On Fri, Sep 24, 2021 at 02:05:25PM -0700, Randy Dunlap wrote:
> This is a nuisance when CONFIG_WERROR is set, so drop the variable
> declaration since the code that used it was removed.
>
> ../arch/nios2/kernel/setup.c: In function 'setup_arch':
> ../arch/nios2/kernel/setup.c:152:13: warning: unused variable 'dram_start' [-Wunused-variable]
> 152 | int dram_start;
>
> Fixes: 7f7bc20bc41a ("nios2: Don't use _end for calculating min_low_pfn")
> Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> Reported-by: kernel test robot <lkp@xxxxxxxxx>
> Cc: Dinh Nguyen <dinguyen@xxxxxxxxxx>
> Cc: Andreas Oetken <andreas.oetken@xxxxxxxxxxx>
> Cc: Mike Rapoport <rppt@xxxxxxxxxxxxxxxxxx>

Reviewed-by: Mike Rapoport <rppt@xxxxxxxxxxxxx>

> ---
> arch/nios2/kernel/setup.c | 2 --
> 1 file changed, 2 deletions(-)
>
> --- linux-next-20210917.orig/arch/nios2/kernel/setup.c
> +++ linux-next-20210917/arch/nios2/kernel/setup.c
> @@ -149,8 +149,6 @@ static void __init find_limits(unsigned
>
> void __init setup_arch(char **cmdline_p)
> {
> - int dram_start;
> -
> console_verbose();
>
> memory_start = memblock_start_of_DRAM();

--
Sincerely yours,
Mike.