• Tom Rix's avatar
    scsi: ipr: Remove several unused variables · 392e4daa
    Tom Rix authored
    gcc with W=1 reports
    drivers/scsi/ipr.c: In function ‘ipr_init_res_entry’:
    drivers/scsi/ipr.c:1104:22: error: variable ‘proto’
      set but not used [-Werror=unused-but-set-variable]
     1104 |         unsigned int proto;
          |                      ^~~~~
    drivers/scsi/ipr.c: In function ‘ipr_update_res_entry’:
    drivers/scsi/ipr.c:1261:22: error: variable ‘proto’
      set but not used [-Werror=unused-but-set-variable]
     1261 |         unsigned int proto;
          |                      ^~~~~
    drivers/scsi/ipr.c: In function ‘ipr_change_queue_depth’:
    drivers/scsi/ipr.c:4417:36: error: variable ‘res’
      set but not used [-Werror=unused-but-set-variable]
     4417 |         struct ipr_resource_entry *res;
          |                                    ^~~
    
    These variables are not used, so remove them. The lock around res is not
    needed so remove that. This makes ioa_cfg and lock_flags unneeded so remove
    them as well.
    
    Fixes: 65a15d65 ("scsi: ipr: Remove SATA support")
    Signed-off-by: default avatarTom Rix <trix@redhat.com>
    Link: https://lore.kernel.org/r/20230420125035.3888188-1-trix@redhat.comAcked-by: default avatarBrian King <brking@linux.vnet.ibm.com>
    Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
    392e4daa
ipr.c 282 KB