Commit 645c98c8 authored by Horst Hummel's avatar Horst Hummel Committed by Martin Schwidefsky

[S390] return 'count' for successful execution of dasd_eer_enable.

Currently the return value of 'dasd_eer_enable' is returned - even if the
function returned '0'. Now return 'count' for successful execution.
Signed-off-by: default avatarHorst Hummel <horst.hummel@de.ibm.com>
Signed-off-by: default avatarMartin Schwidefsky <schwidefsky@de.ibm.com>
parent 29145a6c
...@@ -877,12 +877,13 @@ dasd_eer_store(struct device *dev, struct device_attribute *attr, ...@@ -877,12 +877,13 @@ dasd_eer_store(struct device *dev, struct device_attribute *attr,
if (((endp + 1) < (buf + count)) || (val > 1)) if (((endp + 1) < (buf + count)) || (val > 1))
return -EINVAL; return -EINVAL;
rc = count; if (val) {
if (val)
rc = dasd_eer_enable(devmap->device); rc = dasd_eer_enable(devmap->device);
else if (rc)
return rc;
} else
dasd_eer_disable(devmap->device); dasd_eer_disable(devmap->device);
return rc; return count;
} }
static DEVICE_ATTR(eer_enabled, 0644, dasd_eer_show, dasd_eer_store); static DEVICE_ATTR(eer_enabled, 0644, dasd_eer_show, dasd_eer_store);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment