RE: + minmaxh-simplify-the-variants-of-clamp.patch added to mm-nonmm-unstable branch

[David Laight <David.Laight@ACULAB.COM>]

From: Lorenzo Stoakes
> Sent: 29 November 2024 12:06
> 
> On Fri, Nov 29, 2024 at 11:56:01AM +0000, David Laight wrote:
> > From: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
> > > Sent: 29 November 2024 11:46
> > >
> > > On Fri, Nov 29, 2024 at 03:11:43AM -0800, Andrew Morton wrote:
> > > >
> > > > The patch titled
> > > >      Subject: minmax.h: simplify the variants of clamp()
> > > > has been added to the -mm mm-nonmm-unstable branch.  Its filename is
> > > >      minmaxh-simplify-the-variants-of-clamp.patch
> > >
> > > Hmm, this was generating kernel test bot reports, should we be taking it without
> > > that being fixed?
> >
> > That is reported for W=1 builds and is correctly reporting some very
> > badly broken code - that has always been broken.
> >
> > Probably also only affects allmodconfig builds because the code in
> > question is unlikely to be included.
> 
> Ah fair play, you are poking around in some creaky, cobweb filled bits of
> the kernel so perhaps not a surprise...
> 
> We should probably try and sort out the W=1 stuff though! And by 'we' I
> mean you obviously ;)

Not me, Jakob is trying to fix the magnetometer/yamaha-yas530.c driver.
But I keep on pointing out just how broken the code was.
So there should be a patch to it (that needs backporting) soon.

	David

-
Registered Address Lakeside, Bramley Road, Mount Farm, Milton Keynes, MK1 1PT, UK
Registration No: 1397386 (Wales)