From: Gustavo A. R. Silva <gustavo@embeddedor.com>
Date: Mon, 6 Aug 2018 20:38:32 +0000 (-0500)
Subject: platform/x86: acer-wmi: refactor function has_cap
X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=aa7d16bfa3bd7a753c59b9bae1ac1ccda33bc27a;p=linux.git

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>
---

diff --git a/drivers/platform/x86/acer-wmi.c b/drivers/platform/x86/acer-wmi.c
index 8952173dd380b..3294bb2b37fce 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;
 }
 
 /*