Re: [PATCH] ipmi_si: use bool type for initialized variable

From: Corey Minyard
Date: Fri May 17 2019 - 12:21:06 EST


On Fri, May 17, 2019 at 06:12:45PM +0800, Kefeng Wang wrote:
> Cover 'int' to 'bool' type for initialized variable.

A reasonable cleanup, queued for 5.3.

-corey

>
> Signed-off-by: Kefeng Wang <wangkefeng.wang@xxxxxxxxxx>
> ---
> drivers/char/ipmi/ipmi_si_intf.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/char/ipmi/ipmi_si_intf.c b/drivers/char/ipmi/ipmi_si_intf.c
> index f124a2d2bb9f..da5b6723329a 100644
> --- a/drivers/char/ipmi/ipmi_si_intf.c
> +++ b/drivers/char/ipmi/ipmi_si_intf.c
> @@ -71,7 +71,7 @@ enum si_intf_state {
>
> static const char * const si_to_str[] = { "invalid", "kcs", "smic", "bt" };
>
> -static int initialized;
> +static bool initialized;
>
> /*
> * Indexes into stats[] in smi_info below.
> @@ -2124,7 +2124,7 @@ static int __init init_ipmi_si(void)
> }
>
> skip_fallback_noirq:
> - initialized = 1;
> + initialized = true;
> mutex_unlock(&smi_infos_lock);
>
> if (type)
> --
> 2.20.1
>