thermal/drivers/ti-soc-thermal/bandgap Remove unused variable 'val'
authorLin Ruizhe <linruizhe@huawei.com>
Wed, 21 Apr 2021 08:42:56 +0000 (16:42 +0800)
committerDaniel Lezcano <daniel.lezcano@linaro.org>
Wed, 21 Apr 2021 11:37:18 +0000 (13:37 +0200)
The function ti_bandgap_restore_ctxt() restores the context at resume
time. It checks if the sensor has a counter, reads the register but
does nothing with the value.

The block was probably omitted by the commit b87ea759a4cc.

Remove the unused variable as well as the block using it as we can
consider it as dead code.

Reported-by: Hulk Robot <hulkci@huawei.com>
Fixes: b87ea759a4cc ("staging: omap-thermal: fix context restore function")
Signed-off-by: Lin Ruizhe <linruizhe@huawei.com>
Reviewed-by: Tony Lindgren <tony@atomide.com>
Tested-by: Tony Lindgren <tony@atomide.com>
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Link: https://lore.kernel.org/r/20210421084256.57591-1-linruizhe@huawei.com
drivers/thermal/ti-soc-thermal/ti-bandgap.c

index 008fbed5a118a41ae20ba2b1e1d5b5c610c152d4..ebe7cb70bfb6334990f3306411949f04b5f5a783 100644 (file)
@@ -1142,14 +1142,10 @@ static int ti_bandgap_restore_ctxt(struct ti_bandgap *bgp)
        for (i = 0; i < bgp->conf->sensor_count; i++) {
                struct temp_sensor_registers *tsr;
                struct temp_sensor_regval *rval;
-               u32 val = 0;
 
                rval = &bgp->regval[i];
                tsr = bgp->conf->sensors[i].registers;
 
-               if (TI_BANDGAP_HAS(bgp, COUNTER))
-                       val = ti_bandgap_readl(bgp, tsr->bgap_counter);
-
                if (TI_BANDGAP_HAS(bgp, TSHUT_CONFIG))
                        ti_bandgap_writel(bgp, rval->tshut_threshold,
                                          tsr->tshut_threshold);