diff options
author | Qiao Zhou <zhouqiao@marvell.com> | 2013-10-31 09:36:50 +0800 |
---|---|---|
committer | Qiao Zhou <zhouqiao@marvell.com> | 2013-10-31 09:36:50 +0800 |
commit | 4155e96bc418753a0c7a25d9ffb6d959d3933dc1 (patch) | |
tree | bc139fac15bfafe222e93e18b4f7f628f11380b4 | |
parent | cacc36f18cda9e2894704ace9dbfb15e6045b134 (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>
-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 e05fc259114..e373b625806 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) { |