diff options
author | Michael Trimarchi <michael@amarulasolutions.com> | 2013-02-16 02:42:51 +0400 |
---|---|---|
committer | Dmitry Torokhov <dmitry.torokhov@gmail.com> | 2013-02-16 06:52:39 +0400 |
commit | 79f34d19d7d1ece9918afe079529e61f978eeebe (patch) | |
tree | d764c4defc634e23f458364c7f6d14c8ae34d29b /include/linux/bma150.h | |
parent | cd401204873101245287afc07271b39c79194d9c (diff) | |
download | linux-79f34d19d7d1ece9918afe079529e61f978eeebe.tar.xz |
Input: bma150 - fix checking pm_runtime_get_sync() return value
When PM_RUNTIME is not defined, pm_runtime_get_sync() returns 1, see
include/linux/pm_runtime.c::__pm_runtime_resume(), and the check of the
return value was overlooking this, in this case bma150_open() would
return 1 which is not expected by upper layers.
Maybe the check for != -ENOSYS (Function not implemented) was meant to
cover this, but pm_runtime_get_sync() does not return this value.
For now fix the issue locally by checking explicitly for negative return
values.
Signed-off-by: Michael Trimarchi <michael@amarulasolutions.com>
Signed-off-by: Antonio Ospite <ao2@amarulasolutions.com>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Diffstat (limited to 'include/linux/bma150.h')
0 files changed, 0 insertions, 0 deletions