Re: [PATCH v2] include/linux/byteorder/generic.h: fix signed/unsigned warnings

From: Anatol Belski
Date: Mon Oct 21 2019 - 06:08:05 EST


On Mon, 2019-10-21 at 12:38 +0300, Andy Shevchenko wrote:
> We have explicitly disabled this warnings in the kernel Makefile.
> How did you achieve this? (yes, I know the possible answer, perhaps
> this has to be mentioned as well)
Added W=12 to the make command, used GCC 9.1.0. There are quite some
warnings of this art yet, so this is just a first shot.

Additional issue i'd see in this particular case also, that iterating
through an array should use some unsigned to ensure there's no signed
overflow. But otherwise, it's better to fix similar cases to reduce the
noise.

> > Signed-off-by: Anatol Belski <anbelski@xxxxxxxxxxxxx>
> > ---
>
> Changelog is missing.
> (No need to resend just for this, it's for your future contributions)
>
Thanks for pointing to this. I also probably should have added a Cc:.
Will go through the submitting patches document more carefully.

Thanks!

anatol