summaryrefslogtreecommitdiff
path: root/drivers/block
diff options
context:
space:
mode:
authorColin Ian King <colin.i.king@gmail.com>2022-01-13 00:05:45 +0000
committerJens Axboe <axboe@kernel.dk>2022-01-13 13:00:45 -0700
commita6431e351c6ec5bb6800787d259b343088f369a3 (patch)
tree0ef65fe0836014b8dc14dd07445a38b18ac54d30 /drivers/block
parent413ec8057bc3d368574abd05dd27e747063b2f59 (diff)
aoe: remove redundant assignment on variable n
The variable n is being bit-wise or'd with a value and reassigned before being returned. The update of n is redundant, replace the |= operator with | instead. Cleans up clang scan warning: drivers/block/aoe/aoecmd.c:125:9: warning: Although the value stored to 'n' is used in the enclosing expression, the value is never actually read from 'n' [deadcode.DeadStores] Signed-off-by: Colin Ian King <colin.i.king@gmail.com> Link: https://lore.kernel.org/r/20220113000545.1307091-1-colin.i.king@gmail.com Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'drivers/block')
-rw-r--r--drivers/block/aoe/aoecmd.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/block/aoe/aoecmd.c b/drivers/block/aoe/aoecmd.c
index 588889bea7c3..6af111f568e4 100644
--- a/drivers/block/aoe/aoecmd.c
+++ b/drivers/block/aoe/aoecmd.c
@@ -122,7 +122,7 @@ newtag(struct aoedev *d)
register ulong n;
n = jiffies & 0xffff;
- return n |= (++d->lasttag & 0x7fff) << 16;
+ return n | (++d->lasttag & 0x7fff) << 16;
}
static u32