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

baomingtong001@208suo.com posted 1 patch 2 years, 7 months ago
There is a newer version of this series
drivers/thermal/mediatek/lvts_thermal.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] drivers/thermal/mediatek/lvts_thermal: Remove unneeded semicolon
Posted by baomingtong001@208suo.com 2 years, 7 months ago
./drivers/thermal/mediatek/lvts_thermal.c:522:2-3: Unneeded semicolon

Signed-off-by: Mingtong Bao <baomingtong001@208suo.com>
---
  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;
Re: [PATCH] drivers/thermal/mediatek/lvts_thermal: Remove unneeded semicolon
Posted by Daniel Lezcano 2 years, 7 months ago
On 14/06/2023 10:48, baomingtong001@208suo.com 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@208suo.com>
> ---
>   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