diff options
| author | Russell King <rmk+kernel@armlinux.org.uk> | 2018-11-09 16:44:14 +0000 |
|---|---|---|
| committer | Eduardo Valentin <edubezval@gmail.com> | 2018-12-04 20:45:14 -0800 |
| commit | 70bb27b79adf63ea39e37371d09c823c7a8f93ce (patch) | |
| tree | 0cadd59b9944332dc6e157f0f74c358c3468eea0 /drivers/hwmon/w83l786ng.c | |
| parent | 0072a0c14d5b7cb72c611d396f143f5dcd73ebe2 (diff) | |
| download | linux-70bb27b79adf63ea39e37371d09c823c7a8f93ce.tar.gz linux-70bb27b79adf63ea39e37371d09c823c7a8f93ce.tar.bz2 linux-70bb27b79adf63ea39e37371d09c823c7a8f93ce.zip | |
thermal: armada: fix legacy validity test sense
Commit 8c0e64ac4075 ("thermal: armada: get rid of the ->is_valid()
pointer") removed the unnecessary indirection through a function
pointer, but in doing so, also removed the negation operator too:
- if (priv->data->is_valid && !priv->data->is_valid(priv)) {
+ if (armada_is_valid(priv)) {
which results in:
armada_thermal f06f808c.thermal: Temperature sensor reading not valid
armada_thermal f2400078.thermal: Temperature sensor reading not valid
armada_thermal f4400078.thermal: Temperature sensor reading not valid
at boot, or whenever the "temp" sysfs file is read. Replace the
negation operator.
Fixes: 8c0e64ac4075 ("thermal: armada: get rid of the ->is_valid() pointer")
Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
Diffstat (limited to 'drivers/hwmon/w83l786ng.c')
0 files changed, 0 insertions, 0 deletions