diff options
author | Gustavo A. R. Silva <gustavo@embeddedor.com> | 2018-08-06 15:38:32 -0500 |
---|---|---|
committer | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2018-08-15 13:17:48 +0300 |
commit | aa7d16bfa3bd7a753c59b9bae1ac1ccda33bc27a (patch) | |
tree | c809f9299951342021bc9cca36dd17b34678934c /drivers/platform | |
parent | d22296d9c38fd29a96bb5079fb8d17cee278f40e (diff) | |
download | talos-obmc-linux-aa7d16bfa3bd7a753c59b9bae1ac1ccda33bc27a.tar.gz talos-obmc-linux-aa7d16bfa3bd7a753c59b9bae1ac1ccda33bc27a.zip |
platform/x86: acer-wmi: refactor function has_cap
Refactor function has_cap in order to avoid returning integer
values, when instead it should return booleans.
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Reviewed-by: "Lee, Chun-Yi" <jlee@suse.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Diffstat (limited to 'drivers/platform')
-rw-r--r-- | drivers/platform/x86/acer-wmi.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/platform/x86/acer-wmi.c b/drivers/platform/x86/acer-wmi.c index 8952173dd380..3294bb2b37fc 100644 --- a/drivers/platform/x86/acer-wmi.c +++ b/drivers/platform/x86/acer-wmi.c @@ -672,10 +672,7 @@ static void __init find_quirks(void) static bool has_cap(u32 cap) { - if ((interface->capability & cap) != 0) - return 1; - - return 0; + return interface->capability & cap; } /* |