Re: [PATCH][next] scsi: elx: efct: remove redundant initialization of variable lun

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Colin King <colin.king@xxxxxxxxxxxxx>
Subject: Re: [PATCH][next] scsi: elx: efct: remove redundant initialization of variable lun
From: "Martin K. Petersen" <martin.petersen@xxxxxxxxxx>
Date: Fri, 18 Jun 2021 22:24:27 -0400
Cc: James Smart <james.smart@xxxxxxxxxxxx>, Ram Vegesna <ram.vegesna@xxxxxxxxxxxx>, "James E . J . Bottomley" <jejb@xxxxxxxxxxxxx>, "Martin K . Petersen" <martin.petersen@xxxxxxxxxx>, Hannes Reinecke <hare@xxxxxxx>, linux-scsi@xxxxxxxxxxxxxxx, target-devel@xxxxxxxxxxxxxxx, kernel-janitors@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
In-reply-to: <20210616171621.16176-1-colin.king@canonical.com> (Colin King's message of "Wed, 16 Jun 2021 18:16:21 +0100")
Organization: Oracle Corporation
References: <20210616171621.16176-1-colin.king@canonical.com>

Colin,

> The variable lun is being initialized with a value that is never read,
> it is being updated later on. The assignment is redundant and can be
> removed.

Applied to 5.14/scsi-staging, thanks!

-- 
Martin K. Petersen	Oracle Linux Engineering

References:
- [PATCH][next] scsi: elx: efct: remove redundant initialization of variable lun
  - From: Colin King

Prev by Date: Re: [PATCH][next] scsi: elx: efct: Fix spelling mistake "Unexected" -> "Unexpected"
Next by Date: Re: [PATCH][next] scsi: elx: libefc: Fix less than zero comparison of a unsigned int
Previous by thread: Re: [PATCH][next] scsi: elx: efct: remove redundant initialization of variable lun
Next by thread: [PATCH] be2net: Fix an error handling path in 'be_probe()'
Index(es):
- Date
- Thread

[Index of Archives] [Kernel Development] [Kernel Announce] [Kernel Newbies] [Linux Networking Development] [Share Photos] [IDE] [Security] [Git] [Netfilter] [Yosemite News] [MIPS Linux] [ARM Linux] [Device Mapper]

Powered by Linux