diff options
| author | Colin Ian King <colin.i.king@gmail.com> | 2024-04-15 11:19:28 +0100 |
|---|---|---|
| committer | Ira Weiny <ira.weiny@intel.com> | 2024-04-25 14:11:11 -0700 |
| commit | 41147b006be2174b825a54b0620ecf4cc7ec5c84 (patch) | |
| tree | 948351dbdc8a295de2ebc1d7be0b428870c6bd49 /drivers/dax | |
| parent | 57456adef68db15ab81578d42848bae5b542999a (diff) | |
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>
Diffstat (limited to 'drivers/dax')
| -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; |