Re: [PATCH] lib/vsprintf: remove redundant header files

[Stephen Rothwell]

Hi all,

On Tue, 19 Aug 2025 15:37:20 +0200 Miguel Ojeda <miguel.ojeda.sandonis@xxxxxxxxx> wrote:
>
> On Tue, Aug 19, 2025 at 3:13 PM Liao Yuanhong <liaoyuanhong@xxxxxxxx> wrote:
> >
> > The header file <linux/stdarg.h> is already included on line 6. Remove the
> > redundant include.
> >
> > Fixes: 0dec7201788b9 ("sprintf.h requires stdarg.h")  
> 
> I don't think this fixes that commit -- `stdarg.h` was not there back then.
> 
> Instead, this commit
> 
>     be06b53d3af0 ("lib/vsprintf: include stdarg.h from sprintf.h to
> provide va_list")
> 
> also added the line, and I assume Stephen only needs to drop one of
> the lines in this merge resolution next time:
> 
>     3f413b9a3770 ("Merge branch 'next' of
> https://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git";)

I have done that today.

-- 
Cheers,
Stephen Rothwell

Attachment: pgpBjdLkI2Cw2.pgp
Description: OpenPGP digital signature