diff options
author | Srivatsa S. Bhat <srivatsa.bhat@linux.vnet.ibm.com> | 2013-04-30 11:47:16 +0200 |
---|---|---|
committer | Borislav Petkov <bp@suse.de> | 2013-05-09 12:40:45 +0200 |
commit | c8c64d165ccfd2274058ac84e0c680f9b48c4ec1 (patch) | |
tree | 83d1831be5946a304b5d9d1551813832e2633eb3 /drivers/firewire/net.c | |
parent | EDAC, mc_sysfs.c: Fix string array pointer types (diff) | |
download | linux-c8c64d165ccfd2274058ac84e0c680f9b48c4ec1.tar.xz linux-c8c64d165ccfd2274058ac84e0c680f9b48c4ec1.zip |
EDAC: Don't give write permission to read-only files
I get the following warning on boot:
------------[ cut here ]------------
WARNING: at drivers/base/core.c:575 device_create_file+0x9a/0xa0()
Hardware name: -[8737R2A]-
Write permission without 'store'
...
</snip>
Drilling down, this is related to dynamic channel ce_count attribute
files sporting a S_IWUSR mode without a ->store() function. Looking
around, it appears that they aren't supposed to have a ->store()
function. So remove the bogus write permission to get rid of the
warning.
Signed-off-by: Srivatsa S. Bhat <srivatsa.bhat@linux.vnet.ibm.com>
Cc: Mauro Carvalho Chehab <mchehab@redhat.com>
Cc: <stable@vger.kernel.org> # 3.[89]
[ shorten commit message ]
Signed-off-by: Borislav Petkov <bp@suse.de>
Diffstat (limited to 'drivers/firewire/net.c')
0 files changed, 0 insertions, 0 deletions