diff options
author | Dmitry Torokhov <dtor_core@ameritech.net> | 2005-04-29 02:27:34 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2005-06-20 18:15:03 -0400 |
commit | 70f2817a43c89b784dc2ec3d06ba5bf3064f8235 (patch) | |
tree | 210bbd16599d4e402051e4ec30c82e70b8b427ef /drivers/cpufreq/cpufreq.c | |
parent | 6c1852a08e444a2e66367352a99c0e93c8bf3e97 (diff) |
[PATCH] sysfs: (rest) if show/store is missing return -EIO
sysfs: fix the rest of the kernel so if an attribute doesn't
implement show or store method read/write will return
-EIO instead of 0 or -EINVAL or -EPERM.
Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/cpufreq/cpufreq.c')
-rw-r--r-- | drivers/cpufreq/cpufreq.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c index 03b5fb2ddcf4..bf62dfe4976a 100644 --- a/drivers/cpufreq/cpufreq.c +++ b/drivers/cpufreq/cpufreq.c | |||
@@ -521,7 +521,7 @@ static ssize_t show(struct kobject * kobj, struct attribute * attr ,char * buf) | |||
521 | policy = cpufreq_cpu_get(policy->cpu); | 521 | policy = cpufreq_cpu_get(policy->cpu); |
522 | if (!policy) | 522 | if (!policy) |
523 | return -EINVAL; | 523 | return -EINVAL; |
524 | ret = fattr->show ? fattr->show(policy,buf) : 0; | 524 | ret = fattr->show ? fattr->show(policy,buf) : -EIO; |
525 | cpufreq_cpu_put(policy); | 525 | cpufreq_cpu_put(policy); |
526 | return ret; | 526 | return ret; |
527 | } | 527 | } |
@@ -535,7 +535,7 @@ static ssize_t store(struct kobject * kobj, struct attribute * attr, | |||
535 | policy = cpufreq_cpu_get(policy->cpu); | 535 | policy = cpufreq_cpu_get(policy->cpu); |
536 | if (!policy) | 536 | if (!policy) |
537 | return -EINVAL; | 537 | return -EINVAL; |
538 | ret = fattr->store ? fattr->store(policy,buf,count) : 0; | 538 | ret = fattr->store ? fattr->store(policy,buf,count) : -EIO; |
539 | cpufreq_cpu_put(policy); | 539 | cpufreq_cpu_put(policy); |
540 | return ret; | 540 | return ret; |
541 | } | 541 | } |