[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240410000730.628043-1-treapking@chromium.org>
Date: Wed, 10 Apr 2024 08:07:28 +0800
From: Pin-yen Lin <treapking@...omium.org>
To: Daniel Lezcano <daniel.lezcano@...aro.org>,
"Rafael J . Wysocki" <rafael@...nel.org>,
Zhang Rui <rui.zhang@...el.com>,
Lukasz Luba <lukasz.luba@....com>,
Matthias Brugger <matthias.bgg@...il.com>,
AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>
Cc: linux-arm-kernel@...ts.infradead.org,
Alexandre Mergnat <amergnat@...libre.com>,
linux-kernel@...r.kernel.org,
linux-mediatek@...ts.infradead.org,
NĂcolas F . R . A . Prado <nfraprado@...labora.com>,
Minjie Du <duminjie@...o.com>,
linux-pm@...r.kernel.org,
Frank Wunderlich <frank-w@...lic-files.de>,
Balsam CHIHI <bchihi@...libre.com>,
Pin-yen Lin <treapking@...omium.org>
Subject: [PATCH] thermal/drivers/mediatek/lvts_thermal: Remove redundant code in lvts_ctrl_configure
The removed variable assignment is never written to the register, so it
has no effect on the device behavior. Mediatek has confirmed that it
is not required to initialize this register for current platforms, so
remove this segment to avoid confusion.
Signed-off-by: Pin-yen Lin <treapking@...omium.org>
---
drivers/thermal/mediatek/lvts_thermal.c | 6 ------
1 file changed, 6 deletions(-)
diff --git a/drivers/thermal/mediatek/lvts_thermal.c b/drivers/thermal/mediatek/lvts_thermal.c
index fd4bd650c77a..48d2f8ba3f18 100644
--- a/drivers/thermal/mediatek/lvts_thermal.c
+++ b/drivers/thermal/mediatek/lvts_thermal.c
@@ -985,12 +985,6 @@ static int lvts_ctrl_configure(struct device *dev, struct lvts_ctrl *lvts_ctrl)
value = LVTS_TSSEL_CONF;
writel(value, LVTS_TSSEL(lvts_ctrl->base));
- /*
- * LVTS_CALSCALE : ADC voltage round
- */
- value = 0x300;
- value = LVTS_CALSCALE_CONF;
-
/*
* LVTS_MSRCTL0 : Sensor filtering strategy
*
--
2.44.0.478.gd926399ef9-goog
Powered by blists - more mailing lists