Re: [PATCH] drivers/thermal/mediatek/lvts_thermal: Remove unneeded semicolon

[Daniel Lezcano]

On 14/06/2023 10:48, baomingtong001@xxxxxxxxxx wrote:
> ./drivers/thermal/mediatek/lvts_thermal.c:522:2-3: Unneeded semicolon

Please elaborate a bit the changelog when submitting patches.

I did not see this warning/error when compiling. So in which 
circumstances this message appears?

> Signed-off-by: Mingtong Bao <baomingtong001@xxxxxxxxxx>
> ---
>   drivers/thermal/mediatek/lvts_thermal.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/thermal/mediatek/lvts_thermal.c 
> b/drivers/thermal/mediatek/lvts_thermal.c
> index d0a3f95b7884..1a5ae51b2c51 100644
> --- a/drivers/thermal/mediatek/lvts_thermal.c
> +++ b/drivers/thermal/mediatek/lvts_thermal.c
> @@ -519,7 +519,7 @@ static int lvts_sensor_init(struct device *dev, 
> struct lvts_ctrl *lvts_ctrl,
>            */
>           lvts_sensor[i].msr = lvts_ctrl_data->mode == 
> LVTS_MSR_IMMEDIATE_MODE ?
>               imm_regs[i] : msr_regs[i];
> -    };
> +    }
>
>       lvts_ctrl->num_lvts_sensor = lvts_ctrl_data->num_lvts_sensor;

--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog