Re: [PATCH] staging: ccree: fix boolreturn.cocci warning

From: Tobin C. Harding
Date: Mon Oct 16 2017 - 17:19:14 EST


On Mon, Oct 16, 2017 at 03:39:57PM +0530, sunil.m@xxxxxxxxxxxx wrote:
> From: Suniel Mahesh <sunil.m@xxxxxxxxxxxx>
>
> This fixes the following coccinelle warning:
> WARNING: return of 0/1 in function 'ssi_is_hw_key' with return type bool.

Perhaps

Coccinelle emits WARNING: return of 0/1 in function 'ssi_is_hw_key' with return type bool.

Return 'false' instead of 0.

> Signed-off-by: Suniel Mahesh <sunil.m@xxxxxxxxxxxx>
> ---
> Note:
> - Patch was tested and built(ARCH=arm) on latest
> linux-next.
> - No build issues reported, however it was not
> tested on real hardware.
> - Please discard this changeset, if this is not
> helping the code look better.
> ---
> drivers/staging/ccree/ssi_cipher.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/ccree/ssi_cipher.h b/drivers/staging/ccree/ssi_cipher.h
> index c9a83df..f499962 100644
> --- a/drivers/staging/ccree/ssi_cipher.h
> +++ b/drivers/staging/ccree/ssi_cipher.h
> @@ -75,7 +75,7 @@ struct arm_hw_key_info {
>
> static inline bool ssi_is_hw_key(struct crypto_tfm *tfm)
> {
> - return 0;
> + return false;
> }

Hope this helps,
Tobin.