Skip to content

Commit

Permalink
thermal/drivers/mediatek: Fix bank number settings on mt8183
Browse files Browse the repository at this point in the history
MT8183_NUM_ZONES should be set to 1
because MT8183 doesn't have multiple banks.

Fixes: a4ffe6b ("thermal: mediatek: add support for MT8183")
Signed-off-by: Michael Kao <michael.kao@mediatek.com>
Signed-off-by: Hsin-Yi Wang <hsinyi@chromium.org>
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Link: https://lore.kernel.org/r/20200323121537.22697-6-michael.kao@mediatek.com
  • Loading branch information
Michael Kao authored and Daniel Lezcano committed Jun 29, 2020
1 parent b45fd13 commit 14533a5
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion drivers/thermal/mtk_thermal.c
Original file line number Diff line number Diff line change
Expand Up @@ -211,6 +211,9 @@ enum {
/* The total number of temperature sensors in the MT8183 */
#define MT8183_NUM_SENSORS 6

/* The number of banks in the MT8183 */
#define MT8183_NUM_ZONES 1

/* The number of sensing points per bank */
#define MT8183_NUM_SENSORS_PER_ZONE 6

Expand Down Expand Up @@ -497,7 +500,7 @@ static const struct mtk_thermal_data mt7622_thermal_data = {
*/
static const struct mtk_thermal_data mt8183_thermal_data = {
.auxadc_channel = MT8183_TEMP_AUXADC_CHANNEL,
.num_banks = MT8183_NUM_SENSORS_PER_ZONE,
.num_banks = MT8183_NUM_ZONES,
.num_sensors = MT8183_NUM_SENSORS,
.vts_index = mt8183_vts_index,
.cali_val = MT8183_CALIBRATION,
Expand Down

0 comments on commit 14533a5

Please sign in to comment.