Re: [PATCH 00/10] Fix a bunch SCSI related W=1 warnings

[Lee Jones]

On Wed, 08 Jul 2020, Martin K. Petersen wrote:

> On Tue, 7 Jul 2020 15:00:45 +0100, Lee Jones wrote:
> 
> > This set is part of a larger effort attempting to clean-up W=1
> > kernel builds, which are currently overwhelmingly riddled with
> > niggly little warnings.
> > 
> > There are a whole lot more of these.  More fixes to follow.
> > 
> > Lee Jones (10):
> >   scsi: megaraid: megaraid_mm: Strip excess function param description
> >   scsi: megaraid: megaraid_mbox: Fix some kerneldoc bitrot
> >   scsi: fdomain: Mark 'fdomain_pm_ops' as __maybe_unused
> >   scsi: megaraid: megaraid_sas_fusion: Fix-up a whole myriad of
> >     kerneldoc misdemeanours
> >   scsi: megaraid: megaraid_sas_base: Provide prototypes for non-static
> >     functions
> >   scsi: aha152x: Remove unused variable 'ret'
> >   scsi: pcmcia: nsp_cs: Use new __printf() format notation
> >   scsi: pcmcia: nsp_cs: Remove unused variable 'dummy'
> >   scsi: libfc: fc_disc: Fix-up some incorrectly referenced function
> >     parameters
> >   scsi: megaraid: megaraid_sas: Convert forward-declarations to
> >     prototypes
> > 
> > [...]
> 
> Applied to 5.9/scsi-queue, thanks!
> 
> [03/10] scsi: fdomain: Mark 'fdomain_pm_ops' as __maybe_unused
>         https://git.kernel.org/mkp/scsi/c/4be1fa2b55a8
> [06/10] scsi: aha152x: Remove unused variable 'ret'
>         https://git.kernel.org/mkp/scsi/c/3c011793aca7
> [07/10] scsi: pcmcia: nsp_cs: Use new __printf() format notation
>         https://git.kernel.org/mkp/scsi/c/af0b55d06004
> [08/10] scsi: pcmcia: nsp_cs: Remove unused variable 'dummy'
>         https://git.kernel.org/mkp/scsi/c/97a33483425d
> [09/10] scsi: libfc: fc_disc: Fix-up some incorrectly referenced function parameters
>         https://git.kernel.org/mkp/scsi/c/b1987c884585

Thanks Martin.

Out of interest, do you know of any other efforts to fix W=1 warnings
in SCSI?  I was going to clean the whole sub-system, but I really
would like to avoid duplicating efforts if at all possible.

-- 
Lee Jones [æçæ]
Senior Technical Lead - Developer Services
Linaro.org â Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog