2021-03-10 03:56:33

by Jiapeng Chong

Subject: [PATCH v2] mfd: dbx500-prcmu: Use true and false for bool variable

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 <[email protected]>
Signed-off-by: Jiapeng Chong <[email protected]>
---
Changes in v2:
- Make the commit message more clearer.

include/linux/mfd/db8500-prcmu.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

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

2021-03-10 08:32:34

by Lee Jones

Subject: Re: [PATCH v2] mfd: dbx500-prcmu: Use true and false for bool variable

On Wed, 10 Mar 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 <[email protected]>
> Signed-off-by: Jiapeng Chong <[email protected]>
> ---
> Changes in v2:
> - Make the commit message more clearer.

I already fixed and applied this.

Please take a look at my reply to v1.

> include/linux/mfd/db8500-prcmu.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)

--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog