diff options
author | dyoung@redhat.com <dyoung@redhat.com> | 2013-05-16 14:31:30 +0800 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-05-21 09:05:52 -0700 |
commit | 97521978c5ea80857d4f4f74d4e1fc93721482cf (patch) | |
tree | 7382ef10ce4cae297584ce0c08859da487d0994e /drivers/base | |
parent | 1c04fc3536b9e6d143991a8c5c16b04866baeed6 (diff) | |
download | op-kernel-dev-97521978c5ea80857d4f4f74d4e1fc93721482cf.zip op-kernel-dev-97521978c5ea80857d4f4f74d4e1fc93721482cf.tar.gz |
driver core: print sysfs attribute name when warning about bogus permissions
Make it obvious to see what attribute is using bogus permissions.
Signed-off-by: Dave Young <dyoung@redhat.com>
Acked-by: Felipe Balbi <balbi@ti.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/base')
-rw-r--r-- | drivers/base/core.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/base/core.c b/drivers/base/core.c index 016312437..2499cef 100644 --- a/drivers/base/core.c +++ b/drivers/base/core.c @@ -572,9 +572,11 @@ int device_create_file(struct device *dev, if (dev) { WARN(((attr->attr.mode & S_IWUGO) && !attr->store), - "Write permission without 'store'\n"); + "Attribute %s: write permission without 'store'\n", + attr->attr.name); WARN(((attr->attr.mode & S_IRUGO) && !attr->show), - "Read permission without 'show'\n"); + "Attribute %s: read permission without 'show'\n", + attr->attr.name); error = sysfs_create_file(&dev->kobj, &attr->attr); } |