mbox series

[00/18,Set,3] Rid W=1 warnings in SCSI

Message ID 20210317085701.2891231-1-lee.jones@linaro.org
Headers show
Series Rid W=1 warnings in SCSI | expand

Message

Lee Jones March 17, 2021, 8:56 a.m. UTC
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This set contains functional changes.

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.

Lee Jones (18):
  scsi: aic94xx: aic94xx_dump: Remove code that has been unused for at
    least 13 years
  scsi: mpt3sas: mpt3sas_scs: Move a little data from the stack onto the
    heap
  scsi: bfa: bfa_fcs_lport: Move a large struct from the stack onto the
    heap
  scsi: esas2r: esas2r_log: Supply __printf(x, y) formatting for
    esas2r_log_master()
  scsi: BusLogic: Supply __printf(x, y) formatting for blogic_msg()
  scsi: nsp32: Supply __printf(x, y) formatting for nsp32_message()
  scsi: initio: Remove unused variable 'prev'
  scsi: a100u2w: Remove unused variable 'bios_phys'
  scsi: myrs: Remove a couple of unused 'status' variables
  scsi: 3w-xxxx: Remove 2 unused variables 'response_que_value' and
    'tw_dev'
  scsi: 3w-9xxx: Remove a few set but unused variables
  scsi: 3w-sas: Remove unused variables 'sglist' and 'tw_dev'
  scsi: nsp32: Remove or exclude unused variables
  scsi: FlashPoint: Remove unused variable 'TID' from
    'FlashPoint_AbortCCB()'
  scsi: sim710: Remove unused variable 'err' from sim710_init()
  scsi: isci: port: Make local function 'port_state_name()' static
  scsi: isci: remote_device: Make local function
    isci_remote_device_wait_for_resume_from_abort() static
  scsi: nsp32: Correct expected types in debug print formatting

 drivers/scsi/3w-9xxx.c               |  14 +-
 drivers/scsi/3w-sas.c                |  10 +-
 drivers/scsi/3w-xxxx.c               |   6 +-
 drivers/scsi/BusLogic.c              |   2 +-
 drivers/scsi/FlashPoint.c            |   4 -
 drivers/scsi/a100u2w.c               |   2 -
 drivers/scsi/aic94xx/aic94xx_dump.c  | 184 ---------------------------
 drivers/scsi/bfa/bfa_fcs_lport.c     |  20 ++-
 drivers/scsi/esas2r/esas2r_log.c     |   7 +
 drivers/scsi/initio.c                |   5 +-
 drivers/scsi/isci/port.c             |   2 +-
 drivers/scsi/isci/remote_device.c    |   2 +-
 drivers/scsi/mpt3sas/mpt3sas_scsih.c |  38 ++++--
 drivers/scsi/myrs.c                  |   6 +-
 drivers/scsi/nsp32.c                 |  31 ++---
 drivers/scsi/sim710.c                |  14 +-
 16 files changed, 77 insertions(+), 270 deletions(-)

Cc: Adam Radford <aradford@gmail.com>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: Andre Hedrick <andre@suse.com>
Cc: Anil Gurumurthy <anil.gurumurthy@qlogic.com>
Cc: Artur Paszkiewicz <artur.paszkiewicz@intel.com>
Cc: Bas Vermeulen <bvermeul@blackstar.xs4all.nl>
Cc: Bradley Grove <linuxdrivers@attotech.com>
Cc: Brian Macy <bmacy@sunshinecomputing.com>
Cc: c by <James.Bottomley@SteelEye.com>
Cc: Christoph Hellwig <hch@lst.de>
Cc: David Chaw <david_chaw@adaptec.com>
Cc: de Melo <acme@conectiva.com.br>
Cc: Doug Ledford <dledford@redhat.com>
Cc: GOTO Masanori <gotom@debian.or.jp>
Cc: gotom@debian.org
Cc: Hannes Reinecke <hare@kernel.org>
Cc: "James E.J. Bottomley" <jejb@linux.ibm.com>
Cc: Joel Jacobson <linux@3ware.com>
Cc: Khalid Aziz <khalid@gonehiking.org>
Cc: "Leonard N. Zubkoff" <lnz@dandelion.com>
Cc: Linux GmbH <hare@suse.com>
Cc: linux-scsi@vger.kernel.org
Cc: Luben Tuikov <luben_tuikov@adaptec.com>
Cc: "Martin K. Petersen" <martin.petersen@oracle.com>
Cc: MPT-FusionLinux.pdl@avagotech.com
Cc: MPT-FusionLinux.pdl@broadcom.com
Cc: Richard Hirst <richard@sleepie.demon.co.uk>
Cc: Sathya Prakash <sathya.prakash@broadcom.com>
Cc: Sreekanth Reddy <sreekanth.reddy@broadcom.com>
Cc: Sudarsana Kalluru <sudarsana.kalluru@qlogic.com>
Cc: Suganath Prabu Subramani <suganath-prabu.subramani@broadcom.com>
Cc: YOKOTA Hiroshi <yokota@netlab.is.tsukuba.ac.jp>
-- 
2.27.0

Comments

Lee Jones March 17, 2021, 9:05 a.m. UTC | #1
On Wed, 17 Mar 2021, Lee Jones wrote:

> MIME-Version: 1.0

> Content-Type: text/plain; charset=UTF-8

> Content-Transfer-Encoding: 8bit

> 

> This set contains functional changes.

> 

> 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.

> 

> Lee Jones (18):

>   scsi: aic94xx: aic94xx_dump: Remove code that has been unused for at

>     least 13 years

>   scsi: mpt3sas: mpt3sas_scs: Move a little data from the stack onto the

>     heap

>   scsi: bfa: bfa_fcs_lport: Move a large struct from the stack onto the

>     heap

>   scsi: esas2r: esas2r_log: Supply __printf(x, y) formatting for

>     esas2r_log_master()

>   scsi: BusLogic: Supply __printf(x, y) formatting for blogic_msg()

>   scsi: nsp32: Supply __printf(x, y) formatting for nsp32_message()

>   scsi: initio: Remove unused variable 'prev'

>   scsi: a100u2w: Remove unused variable 'bios_phys'

>   scsi: myrs: Remove a couple of unused 'status' variables

>   scsi: 3w-xxxx: Remove 2 unused variables 'response_que_value' and

>     'tw_dev'

>   scsi: 3w-9xxx: Remove a few set but unused variables

>   scsi: 3w-sas: Remove unused variables 'sglist' and 'tw_dev'

>   scsi: nsp32: Remove or exclude unused variables

>   scsi: FlashPoint: Remove unused variable 'TID' from

>     'FlashPoint_AbortCCB()'

>   scsi: sim710: Remove unused variable 'err' from sim710_init()

>   scsi: isci: port: Make local function 'port_state_name()' static

>   scsi: isci: remote_device: Make local function

>     isci_remote_device_wait_for_resume_from_abort() static

>   scsi: nsp32: Correct expected types in debug print formatting


Oh dear! Looks like I also took functional patches from sets that
have already been sent out.

Please silently disregard this set. Sorry for the noise.

-- 
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog