Re: [PATCH] m68k: amiga: Mark amiga_reset() as __noreturn

[Thorsten Blum <thorsten.blum@linux.dev>]

Hi Daniel,

On Fri, Jun 26, 2026 at 10:49:59PM +0900, Daniel Palmer wrote:
> Hi Thorsten,
> 
> On Fri, 26 Jun 2026 at 19:43, Thorsten Blum <thorsten.blum@linux.dev> wrote:
> >
> > Mark both the forward declaration and the function definition as
> > __noreturn, and remove the redundant redeclaration.
> >
> <snip>
> 
> I tried to build and boot test this on my Amiga 4000 but the build is borken:
> 
>   UPD     include/config/kernel.release
>  UPD     include/generated/utsrelease.h
>  CC      init/version.o
>  AR      init/built-in.a
>  CC      arch/m68k/amiga/config.o
> arch/m68k/amiga/config.c:546:1: error: attributes should be specified
> before the declarator in a function definition
>  546 | static void amiga_reset(void) __noreturn
>      | ^~~~~~
> arch/m68k/amiga/config.c:99:13: warning: ‘amiga_reset’ used but never defined
>   99 | static void amiga_reset(void) __noreturn;
>      |             ^~~~~~~~~~~
> 
> GCC version is:  m68k-linux-gnu-gcc (Debian 15.2.0-17) 15.2.0

I also compiled it, but then moved the __noreturn to the end which is
apparently not allowed. Feel free to try:

	static void __noreturn amiga_reset(void);

That should work. I'll send a v2 later.

Thanks,
Thorsten