diff options
| author | Srivatsa S. Bhat <srivatsa.bhat@linux.vnet.ibm.com> | 2013-04-30 15:17:16 +0530 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-05-11 07:18:37 -0700 |
| commit | f2e426a46a35a8cd30c7c7c39e099f7bd857ce0b (patch) | |
| tree | 3cc3841846964ea84405f69295c4a80af7bb6977 /include/linux/platform_data/asoc-mx27vis.h | |
| parent | d2775711193f1bad066bc5349969ec0bcc9dccb1 (diff) | |
| download | lwn-f2e426a46a35a8cd30c7c7c39e099f7bd857ce0b.tar.gz lwn-f2e426a46a35a8cd30c7c7c39e099f7bd857ce0b.zip | |
EDAC: Don't give write permission to read-only files
commit c8c64d165ccfd2274058ac84e0c680f9b48c4ec1 upstream.
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>
[ shorten commit message ]
Signed-off-by: Borislav Petkov <bp@suse.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'include/linux/platform_data/asoc-mx27vis.h')
0 files changed, 0 insertions, 0 deletions