diff options
author | Qiao Zhou <zhouqiao@marvell.com> | 2013-10-31 09:36:50 +0800 |
---|---|---|
committer | John Stultz <john.stultz@linaro.org> | 2016-02-16 13:51:00 -0800 |
commit | dc66dee02dcd6ea774e3ed4ae32e88b0f3b4bee7 (patch) | |
tree | 53c30479bb24859f00008348c7373150b024f019 /drivers | |
parent | 8e180508c265026cc4653b68ca216db5e9b709fc (diff) |
drivers: switch: remove S_IWUSR from dev_attr
it doesn't need S_IWUSR attribute since xxx_store API is un-needed.
otherwise the WARN check in device_create_file will alert.
Change-Id: I6360bf022dcd659bfb3f41c84624f954d5d15ea5
Signed-off-by: Qiao Zhou <zhouqiao@marvell.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/switch/switch_class.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/switch/switch_class.c b/drivers/switch/switch_class.c index e05fc2591147..e373b625806e 100644 --- a/drivers/switch/switch_class.c +++ b/drivers/switch/switch_class.c @@ -54,8 +54,8 @@ static ssize_t name_show(struct device *dev, struct device_attribute *attr, return sprintf(buf, "%s\n", sdev->name); } -static DEVICE_ATTR(state, S_IRUGO | S_IWUSR, state_show, NULL); -static DEVICE_ATTR(name, S_IRUGO | S_IWUSR, name_show, NULL); +static DEVICE_ATTR(state, S_IRUGO, state_show, NULL); +static DEVICE_ATTR(name, S_IRUGO, name_show, NULL); void switch_set_state(struct switch_dev *sdev, int state) { |