Re: [PATCH v4 1/2] platform/x86: Add Uniwill laptop driver

[Armin Wolf]

Am 06.10.25 um 21:08 schrieb Nathan Chancellor:

> On Sun, Oct 05, 2025 at 08:06:57PM +0200, Armin Wolf wrote:
> > Am 03.10.25 um 01:36 schrieb Nathan Chancellor:
> >
> > > Hi Armin,
> > >
> > > On Thu, Oct 02, 2025 at 08:41:19PM +0200, Armin Wolf wrote:
> > > > i think this is a problem inside the clang compiler. I did not encounter this warning when
> > > > build for x86-64 using gcc.
> > > Clang is actually saving you from yourself, it is a bug in GCC that it
> > > does not warn for this:
> > >
> > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91951
> > Oh my, i didn't expect that. Thank you for explaining this issue to to me,
> > it seems that i still have much to learn.
> In your defense, it is quite a subtle interaction. It is probably worth
> throwing something in the documentation that explicitly calls this out,
> though I am not immediately sure of where...
>
> Cheers,
> Nathan
Maybe Documentation/core-api/cleanup.rst would be a suitable place for warning
people about this.

Thanks,
Armin Wolf