leandrof/linux
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

scsi: ips: Change the return type of ips_release() into 'void'

Browse Source 
ips_release() has one caller and that caller ignores the value returned by
ips_release(). Hence change the return type of that function into 'void'.

Link: https://lore.kernel.org/r/20220218195117.25689-3-bvanassche@acm.org
Cc: Johannes Thumshirn <johannes.thumshirn@wdc.com>
Cc: John Garry <john.garry@huawei.com>
Reviewed-by: Hannes Reinecke <hare@suse.com>
Reviewed-by: Himanshu Madhani <himanshu.madhani@oracle.com>
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
This commit is contained in:
Bart Van Assche
2022-02-18 11:50:30 -08:00
committed by Martin K. Petersen
parent 2cf0e0a9da
commit be33e2f8d8
1 changed files with 2 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
drivers/scsi/ips.c
View File

@@ -638,8 +638,7 @@ ips_setup_funclist(ips_ha_t * ha)
/* Remove a driver */ /* Remove a driver */
/* */ /* */
/****************************************************************************/ /****************************************************************************/
static int static void ips_release(struct Scsi_Host *sh)
ips_release(struct Scsi_Host *sh)
{ {
ips_scb_t *scb; ips_scb_t *scb;
ips_ha_t *ha; ips_ha_t *ha;
@@ -660,7 +659,7 @@ ips_release(struct Scsi_Host *sh)
ha = IPS_HA(sh); ha = IPS_HA(sh);
if (!ha) if (!ha)
return (FALSE); return;
/* flush the cache on the controller */ /* flush the cache on the controller */
scb = &ha->scbs[ha->max_cmds - 1]; scb = &ha->scbs[ha->max_cmds - 1];
@@ -698,8 +697,6 @@ ips_release(struct Scsi_Host *sh)
scsi_host_put(sh); scsi_host_put(sh);
ips_released_controllers++; ips_released_controllers++;
return (FALSE);
} }
/****************************************************************************/ /****************************************************************************/