All of lore.kernel.org
 help / color / mirror / Atom feed
From: Nicolas Schier <nsc@kernel.org>
To: Nathan Chancellor <nathan@kernel.org>
Cc: "Thomas Weißschuh" <thomas.weissschuh@linutronix.de>,
	linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/6] kbuild: uapi: various fixes
Date: Wed, 13 Aug 2025 07:54:56 +0200	[thread overview]
Message-ID: <aJwosJCbyZoeD5DT@levanger> (raw)
In-Reply-To: <20250812234458.GA52733@ax162>

[-- Attachment #1: Type: text/plain, Size: 1349 bytes --]

On Tue, Aug 12, 2025 at 04:44:58PM -0700, Nathan Chancellor wrote:
> Hi Thomas,
> 
> On Tue, Aug 12, 2025 at 07:33:03AM +0200, Thomas Weißschuh wrote:
> > Various fixes and promotion of warnings to real errors.
> > 
> > I didn't add Fixes: tags as digging up the original changes would be
> > cumbersome as the code moved a lot over the years.
> > Backporting this doesn't make sense anyways.
> > 
> > Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
> > ---
> > Thomas Weißschuh (6):
> >       kbuild: uapi: rerun header tests when headers_check.pl changes
> >       kbuild: uapi: only update hdrtest output on success
> >       kbuild: uapi: fail header test on compiler warnings
> >       kbuild: uapi: upgrade warning on asm/types.h inclusion to error
> >       kbuild: uapi: upgrade check_sizetypes() warning to error
> >       kbuild: uapi: upgrade check_declarations() warning to error
> 
> This series seems reasonable. I did a build of usr/ on arm64 and x86_64
> allmodconfig, which showed no issues. I will give Nicolas a few days to
> comment, after which I will apply these to the Kbuild tree so we can
> start soak testing in -next.

thanks.  With the exception of the already canceled patch 2, this looks
good to me.

Reviewed-by: Nicolas Schier <nsc@kernel.org>

Kind regards,
Nicolas

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

      reply	other threads:[~2025-08-13  5:55 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-08-12  5:33 [PATCH 0/6] kbuild: uapi: various fixes Thomas Weißschuh
2025-08-12  5:33 ` [PATCH 1/6] kbuild: uapi: rerun header tests when headers_check.pl changes Thomas Weißschuh
2025-08-12  5:33 ` [PATCH 2/6] kbuild: uapi: only update hdrtest output on success Thomas Weißschuh
2025-08-13  0:29   ` Masahiro Yamada
2025-08-13  5:29     ` Thomas Weißschuh
2025-08-13  5:46     ` Nicolas Schier
2025-08-12  5:33 ` [PATCH 3/6] kbuild: uapi: fail header test on compiler warnings Thomas Weißschuh
2025-08-12  5:33 ` [PATCH 4/6] kbuild: uapi: upgrade warning on asm/types.h inclusion to error Thomas Weißschuh
2025-08-12  5:33 ` [PATCH 5/6] kbuild: uapi: upgrade check_sizetypes() warning " Thomas Weißschuh
2025-08-12  5:33 ` [PATCH 6/6] kbuild: uapi: upgrade check_declarations() " Thomas Weißschuh
2025-08-12 23:44 ` [PATCH 0/6] kbuild: uapi: various fixes Nathan Chancellor
2025-08-13  5:54   ` Nicolas Schier [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=aJwosJCbyZoeD5DT@levanger \
    --to=nsc@kernel.org \
    --cc=linux-kbuild@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nathan@kernel.org \
    --cc=thomas.weissschuh@linutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.