Re: [PATCH 10/14] suspend: add a arch_resume_nosmt() prototype
From: Rafael J. Wysocki
Date: Wed May 24 2023 - 13:33:23 EST
On Wed, May 17, 2023 at 4:52 PM Arnd Bergmann <arnd@xxxxxxxx> wrote:
>
> On Wed, May 17, 2023, at 15:48, Rafael J. Wysocki wrote:
> > On Wed, May 17, 2023 at 3:12 PM Arnd Bergmann <arnd@xxxxxxxxxx> wrote:
> >>
> >> From: Arnd Bergmann <arnd@xxxxxxxx>
> >>
> >> The arch_resume_nosmt() has a __weak definition, plus an x86
> >> specific override, but no prototype that ensures the two have
> >> the same arguments. This causes a W=1 warning:
> >>
> >> arch/x86/power/hibernate.c:189:5: error: no previous prototype for 'arch_resume_nosmt' [-Werror=missing-prototypes]
> >>
> >> Add the prototype in linux/suspend.h, which is included in
> >> both places.
> >>
> >> Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx>
> >
> > Do you want me to pick this up?
>
> Yes, please do. Thanks,
Done, thanks!