diff options
author | Colin Ian King <colin.i.king@gmail.com> | 2024-04-15 13:19:28 +0300 |
---|---|---|
committer | Ira Weiny <ira.weiny@intel.com> | 2024-04-26 00:11:11 +0300 |
commit | 41147b006be2174b825a54b0620ecf4cc7ec5c84 (patch) | |
tree | 948351dbdc8a295de2ebc1d7be0b428870c6bd49 | |
parent | 57456adef68db15ab81578d42848bae5b542999a (diff) | |
download | linux-41147b006be2174b825a54b0620ecf4cc7ec5c84.tar.xz |
dax: remove redundant assignment to variable rc
The variable rc is being assigned an value and then is being re-assigned
a new value in the next statement. The assignment is redundant and can
be removed.
Cleans up clang scan build warning:
drivers/dax/bus.c:1207:2: warning: Value stored to 'rc' is never
read [deadcode.DeadStores]
Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Reviewed-by: Alison Schofield <alison.schofield@intel.com>
Reviewed-by: Dave Jiang <dave.jiang@intel.com>
Reviewed-by: Vishal Verma <vishal.l.verma@intel.com>
Link: https://lore.kernel.org/r/20240415101928.484143-1-colin.i.king@gmail.com
Signed-off-by: Ira Weiny <ira.weiny@intel.com>
-rw-r--r-- | drivers/dax/bus.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/dax/bus.c b/drivers/dax/bus.c index 018f81ec82ac..6ec5db8013e8 100644 --- a/drivers/dax/bus.c +++ b/drivers/dax/bus.c @@ -1204,7 +1204,6 @@ static ssize_t mapping_store(struct device *dev, struct device_attribute *attr, if (rc) return rc; - rc = -ENXIO; rc = down_write_killable(&dax_region_rwsem); if (rc) return rc; |