Re: [PATCH] platform/x86: intel_pmc_core: Assign boolean values to a bool variable

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Nov 7, 2020 at 2:57 PM <xiakaixu1987@xxxxxxxxx> wrote:
>
> From: Kaixu Xia <kaixuxia@xxxxxxxxxxx>
>
> Fix the following coccinelle warnings:
>
> ./drivers/platform/x86/intel_pmc_core.c:932:1-16: WARNING: Assignment of 0/1 to bool variable

Thanks for the patch! Looks good to me.
Reviewed-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx>

P.S. Don't forget to include subsystem maintainers.

>
> Signed-off-by: Kaixu Xia <kaixuxia@xxxxxxxxxxx>
> ---
>  drivers/platform/x86/intel_pmc_core.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/platform/x86/intel_pmc_core.c b/drivers/platform/x86/intel_pmc_core.c
> index 3e5fe66333f1..ee2f757515b0 100644
> --- a/drivers/platform/x86/intel_pmc_core.c
> +++ b/drivers/platform/x86/intel_pmc_core.c
> @@ -929,7 +929,7 @@ static void pmc_core_slps0_dbg_latch(struct pmc_dev *pmcdev, bool reset)
>                 fd |= CNP_PMC_LATCH_SLPS0_EVENTS;
>         pmc_core_reg_write(pmcdev, map->slps0_dbg_offset, fd);
>
> -       slps0_dbg_latch = 0;
> +       slps0_dbg_latch = false;
>
>  out_unlock:
>         mutex_unlock(&pmcdev->lock);
> --
> 2.20.0
>


-- 
With Best Regards,
Andy Shevchenko



[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux