mfd: dbx500-prcmu: Use true and false for bool variable
diff mbox series

Message ID 1612940856-72537-1-git-send-email-jiapeng.chong@linux.alibaba.com
State Accepted
Commit 1c7c1488898e1b3fd15d38f373a8c947a2d9b1e3
Headers show
Series
  • mfd: dbx500-prcmu: Use true and false for bool variable
Related show

Commit Message

Jiapeng Chong Feb. 10, 2021, 7:07 a.m. UTC
Fix the following coccicheck warning:

./include/linux/mfd/db8500-prcmu.h:723:8-9: WARNING: return of 0/1 in
function 'db8500_prcmu_is_ac_wake_requested' with return type bool.

Reported-by: Abaci Robot<abaci@linux.alibaba.com>
Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>
---
 include/linux/mfd/db8500-prcmu.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Lee Jones March 8, 2021, 1:27 p.m. UTC | #1
On Wed, 10 Feb 2021, Jiapeng Chong wrote:

> Fix the following coccicheck warning:
> 
> ./include/linux/mfd/db8500-prcmu.h:723:8-9: WARNING: return of 0/1 in
> function 'db8500_prcmu_is_ac_wake_requested' with return type bool.
> 
> Reported-by: Abaci Robot<abaci@linux.alibaba.com>

This needs a ' ' before the '<'.

> Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>
> ---
>  include/linux/mfd/db8500-prcmu.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

I fixed the above and applied the patch, thanks.

Patch
diff mbox series

diff --git a/include/linux/mfd/db8500-prcmu.h b/include/linux/mfd/db8500-prcmu.h
index 4b63d3e..a62de3d 100644
--- a/include/linux/mfd/db8500-prcmu.h
+++ b/include/linux/mfd/db8500-prcmu.h
@@ -720,7 +720,7 @@  static inline int db8500_prcmu_load_a9wdog(u8 id, u32 val)
 
 static inline bool db8500_prcmu_is_ac_wake_requested(void)
 {
-	return 0;
+	return false;
 }
 
 static inline int db8500_prcmu_set_arm_opp(u8 opp)