linux-kernel - Re: [PATCH 3/6] MIPS: dec: Create reset.h

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <8dcb5c6d-be4f-4891-a999-137d53edfc05@linaro.org>
Date: Thu, 10 Apr 2025 14:34:17 +0200
From: Philippe Mathieu-Daudé <philmd@...aro.org>
To: WangYuli <wangyuli@...ontech.com>
Cc: guanwentao@...ontech.com, linux-kernel@...r.kernel.org,
 linux-mips@...r.kernel.org, macro@...am.me.uk, niecheng1@...ontech.com,
 tsbogend@...ha.franken.de, zhanjun@...ontech.com
Subject: Re: [PATCH 3/6] MIPS: dec: Create reset.h

On 7/4/25 09:36, WangYuli wrote:
> Declare externally used functions in reset.c to resolve compilation
> warnings.
> 
> Fix follow errors with gcc-14 when -Werror:
> 
> arch/mips/dec/reset.c:22:17: error: no previous prototype for ‘dec_machine_restart’ [-Werror=missing-prototypes]
>     22 | void __noreturn dec_machine_restart(char *command)
>        |                 ^~~~~~~~~~~~~~~~~~~
> arch/mips/dec/reset.c:27:17: error: no previous prototype for ‘dec_machine_halt’ [-Werror=missing-prototypes]
>     27 | void __noreturn dec_machine_halt(void)
>        |                 ^~~~~~~~~~~~~~~~
> arch/mips/dec/reset.c:32:17: error: no previous prototype for ‘dec_machine_power_off’ [-Werror=missing-prototypes]
>     32 | void __noreturn dec_machine_power_off(void)
>        |                 ^~~~~~~~~~~~~~~~~~~~~
> arch/mips/dec/reset.c:38:13: error: no previous prototype for ‘dec_intr_halt’ [-Werror=missing-prototypes]
>     38 | irqreturn_t dec_intr_halt(int irq, void *dev_id)
>        |             ^~~~~~~~~~~~~
> cc1: all warnings being treated as errors
> make[7]: *** [scripts/Makefile.build:207: arch/mips/dec/reset.o] Error 1
> make[7]: *** Waiting for unfinished jobs....
> 
> In passing, also correct the include file ordering in setup.c as it
> doesn't merit a separate commit.
> 
> Link: https://lore.kernel.org/all/Z8A0JeFYfBxXOFCD@alpha.franken.de/
> Signed-off-by: WangYuli <wangyuli@...ontech.com>
> ---
>   arch/mips/dec/prom/init.c         |  3 +--
>   arch/mips/dec/reset.c             |  2 ++
>   arch/mips/dec/setup.c             |  9 ++-------
>   arch/mips/include/asm/dec/reset.h | 20 ++++++++++++++++++++
>   4 files changed, 25 insertions(+), 9 deletions(-)
>   create mode 100644 arch/mips/include/asm/dec/reset.h

Reviewed-by: Philippe Mathieu-Daudé <philmd@...aro.org>