Re: [PATCH v7 1/4] bitops: Introduce the the for_each_set_clump macro
From: Andy Shevchenko
Date: Wed Jun 03 2020 - 17:59:39 EST
On Thu, Jun 4, 2020 at 12:58 AM Andy Shevchenko
<andy.shevchenko@xxxxxxxxx> wrote:
>
> On Thu, Jun 4, 2020 at 12:53 AM Rikard Falkeborn
> <rikard.falkeborn@xxxxxxxxx> wrote:
> > On Wed, Jun 03, 2020 at 11:49:37AM +0300, Andy Shevchenko wrote:
> > > On Tue, Jun 2, 2020 at 10:01 PM Rikard Falkeborn
> > > <rikard.falkeborn@xxxxxxxxx> wrote:
>
> ...
>
> > I'd be very surprised if compilers warned for explicit casts but I'll
> > send a proper patch soon to let the build robot try it.
>
> I noticed that you should have received kbuild bot report about a
> driver where it appears.
>
> You patch broke all cases where (l) = 0 and (h) is type of unsigned
> (not a const from compiler point of view).
I will ask to revert for rc1 if there will be no fix.
--
With Best Regards,
Andy Shevchenko