]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
scsi: message: fusion: Remove redundant pointer 'hd'
authorColin Ian King <colin.i.king@gmail.com>
Thu, 18 Jan 2024 12:20:39 +0000 (12:20 +0000)
committerMartin K. Petersen <martin.petersen@oracle.com>
Wed, 24 Jan 2024 02:39:21 +0000 (21:39 -0500)
The pointer 'hd' is being assigned a value that is not being read
later. The variable is redundant and can be removed.

Cleans up clang scan build warning:

warning: Although the value stored to 'hd' is used in the enclosing
expression, the value is never actually read from 'hd'
[deadcode.DeadStores]

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Link: https://lore.kernel.org/r/20240118122039.2541425-1-colin.i.king@gmail.com
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/message/fusion/mptfc.c

index 0581f855c72e89154188412a35dc823a4d706d08..c459f709107b7cf65782d900dab7a50c7db3b346 100644 (file)
@@ -1401,7 +1401,6 @@ static struct pci_driver mptfc_driver = {
 static int
 mptfc_event_process(MPT_ADAPTER *ioc, EventNotificationReply_t *pEvReply)
 {
-       MPT_SCSI_HOST *hd;
        u8 event = le32_to_cpu(pEvReply->Event) & 0xFF;
        unsigned long flags;
        int rc=1;
@@ -1412,8 +1411,7 @@ mptfc_event_process(MPT_ADAPTER *ioc, EventNotificationReply_t *pEvReply)
        devtverboseprintk(ioc, printk(MYIOC_s_DEBUG_FMT "MPT event (=%02Xh) routed to SCSI host driver!\n",
                        ioc->name, event));
 
-       if (ioc->sh == NULL ||
-               ((hd = shost_priv(ioc->sh)) == NULL))
+       if (ioc->sh == NULL || shost_priv(ioc->sh) == NULL)
                return 1;
 
        switch (event) {