diff options
| author | Anton Vorontsov <avorontsov@ru.mvista.com> | 2009-12-02 21:24:51 (GMT) |
|---|---|---|
| committer | Anton Vorontsov <avorontsov@ru.mvista.com> | 2009-12-02 21:38:06 (GMT) |
| commit | 9d233e8bb92e355fd20b14745c1d9ff402e0e685 (patch) | |
| tree | 8046983fd1d9a89a34e61619f35be6b3ec78789c | |
| parent | 0e19dbb73eab1f5de328e297b8b6d9887c3e73c2 (diff) | |
| download | linux-fsl-qoriq-9d233e8bb92e355fd20b14745c1d9ff402e0e685.tar.xz | |
power_supply_sysfs: Handle -ENODATA in a special way
There are cases when some device can not report any meaningful value,
e.g. TWL4030 charger can report voltage only when charging is
active.
In these cases drivers will return -ENODATA, and we shouldn't flood
kernel log with error messages.
Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com>
| -rw-r--r-- | drivers/power/power_supply_sysfs.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/power/power_supply_sysfs.c b/drivers/power/power_supply_sysfs.c index 0814439..c790e0c 100644 --- a/drivers/power/power_supply_sysfs.c +++ b/drivers/power/power_supply_sysfs.c @@ -65,7 +65,10 @@ static ssize_t power_supply_show_property(struct device *dev, ret = psy->get_property(psy, off, &value); if (ret < 0) { - if (ret != -ENODEV) + if (ret == -ENODATA) + dev_dbg(dev, "driver has no data for `%s' property\n", + attr->attr.name); + else if (ret != -ENODEV) dev_err(dev, "driver failed to report `%s' property\n", attr->attr.name); return ret; |