From: Jacky Bai <ping.bai@nxp.com>
To: krzk@kernel.org, rafael@kernel.org, daniel.lezcano@linaro.org,
rui.zhang@intel.com, robh@kernel.org, krzk+dt@kernel.org,
conor+dt@kernel.org, shawnguo@kernel.org
Cc: linux-pm@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
imx@lists.linux.dev, kernel@pengutronix.de, festevam@gmail.com
Subject: [PATCH v2 3/4] thermal: qoriq: workaround the tmu temp jump on imx93
Date: Fri, 1 Aug 2025 16:11:18 +0800 [thread overview]
Message-ID: <20250801081119.1999298-3-ping.bai@nxp.com> (raw)
In-Reply-To: <20250801081119.1999298-1-ping.bai@nxp.com>
On i.MX93, the temp read from tmu may jump wrongly(ERR052243), and invalid
temp will be read out. To workaround such issue, we need to use the
raising/falling edge threshold to filter out the wrong temp. When reading
the temp, need to check the TIDR register to make sure no jump happens.
Please refer to NXP errata ERR052243 for more details.
Signed-off-by: Jacky Bai <ping.bai@nxp.com>
---
drivers/thermal/qoriq_thermal.c | 31 +++++++++++++++++++++++++++++--
1 file changed, 29 insertions(+), 2 deletions(-)
diff --git a/drivers/thermal/qoriq_thermal.c b/drivers/thermal/qoriq_thermal.c
index fd4d162fc188..664cd5f7d8af 100644
--- a/drivers/thermal/qoriq_thermal.c
+++ b/drivers/thermal/qoriq_thermal.c
@@ -46,6 +46,13 @@
#define REGS_TIER 0x020 /* Interrupt Enable Register */
#define TIER_DISABLE 0x0
+#define REGS_TIDR 0x24
+#define TMRTRCTR 0x70
+#define TMRTRCTR_EN BIT(31)
+#define TMRTRCTR_TEMP(x) ((x) & 0xFF)
+#define TMFTRCTR 0x74
+#define TMFTRCTR_EN BIT(31)
+#define TMFTRCTR_TEMP(x) ((x) & 0xFF)
#define REGS_TTCFGR 0x080 /* Temperature Configuration Register */
#define REGS_TSCFGR 0x084 /* Sensor Configuration Register */
@@ -93,7 +100,7 @@ static int tmu_get_temp(struct thermal_zone_device *tz, int *temp)
{
struct qoriq_sensor *qsensor = thermal_zone_device_priv(tz);
struct qoriq_tmu_data *qdata = qoriq_sensor_to_data(qsensor);
- u32 val;
+ u32 val, tidr;
/*
* REGS_TRITSR(id) has the following layout:
*
@@ -118,6 +125,15 @@ static int tmu_get_temp(struct thermal_zone_device *tz, int *temp)
if (!(val & TMR_ME))
return -EAGAIN;
+ /* ERR052243: If there raising or falling edge happens, try later */
+ if (qdata->ver == TMU_VER93) {
+ regmap_read(qdata->regmap, REGS_TIDR, &tidr);
+ if (tidr & GENMASK(25, 24)) {
+ regmap_write(qdata->regmap, REGS_TIDR, GENMASK(25, 24));
+ return -EAGAIN;
+ }
+ }
+
if (regmap_read_poll_timeout(qdata->regmap,
REGS_TRITSR(qsensor->id),
val,
@@ -126,6 +142,15 @@ static int tmu_get_temp(struct thermal_zone_device *tz, int *temp)
10 * USEC_PER_MSEC))
return -ENODATA;
+ /*ERR052243: If there raising or falling edge happens, try later */
+ if (qdata->ver == TMU_VER93) {
+ regmap_read(qdata->regmap, REGS_TIDR, &tidr);
+ if (tidr & GENMASK(25, 24)) {
+ regmap_write(qdata->regmap, REGS_TIDR, GENMASK(25, 24));
+ return -EAGAIN;
+ }
+ }
+
if (qdata->ver == TMU_VER1) {
*temp = (val & GENMASK(7, 0)) * MILLIDEGREE_PER_DEGREE;
} else {
@@ -232,12 +257,14 @@ static void qoriq_tmu_init_device(struct qoriq_tmu_data *data)
regmap_write(data->regmap, REGS_TIER, TIER_DISABLE);
/* Set update_interval */
-
if (data->ver == TMU_VER1) {
regmap_write(data->regmap, REGS_TMTMIR, TMTMIR_DEFAULT);
} else if (data->ver == TMU_VER93) {
regmap_write(data->regmap, REGS_V2_TMTMIR, TMTMIR_DEFAULT);
regmap_write(data->regmap, REGS_V2_TEUMR(0), TEUMR0_V21);
+ /* ERR052243: Set the raising & falling edge monitor */
+ regmap_write(data->regmap, TMRTRCTR, TMRTRCTR_EN | TMRTRCTR_TEMP(0x7));
+ regmap_write(data->regmap, TMFTRCTR, TMFTRCTR_EN | TMFTRCTR_TEMP(0x7));
} else {
regmap_write(data->regmap, REGS_V2_TMTMIR, TMTMIR_DEFAULT);
regmap_write(data->regmap, REGS_V2_TEUMR(0), TEUMR0_V2);
--
2.34.1
next prev parent reply other threads:[~2025-08-01 8:09 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-08-01 8:11 [PATCH v2 1/4] dt-bindings: thermal: qoriq: Update compatible string for imx93 Jacky Bai
2025-08-01 8:11 ` [PATCH v2 2/4] thermal: qoriq: add i.MX93 TMU support Jacky Bai
2025-08-01 14:09 ` Frank Li
2025-08-01 14:54 ` Jacky Bai
2025-08-01 8:11 ` Jacky Bai [this message]
2025-08-01 14:11 ` [PATCH v2 3/4] thermal: qoriq: workaround the tmu temp jump on imx93 Frank Li
2025-08-01 15:10 ` Jacky Bai
2025-08-01 8:11 ` [PATCH v2 4/4] arm64: dts: imx93: update the tmu compatible string Jacky Bai
2025-08-01 8:55 ` [PATCH v2 1/4] dt-bindings: thermal: qoriq: Update compatible string for imx93 Krzysztof Kozlowski
2025-08-01 14:48 ` Jacky Bai
2025-08-01 14:14 ` Frank Li
2025-08-01 14:50 ` Jacky Bai
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250801081119.1999298-3-ping.bai@nxp.com \
--to=ping.bai@nxp.com \
--cc=conor+dt@kernel.org \
--cc=daniel.lezcano@linaro.org \
--cc=festevam@gmail.com \
--cc=imx@lists.linux.dev \
--cc=kernel@pengutronix.de \
--cc=krzk+dt@kernel.org \
--cc=krzk@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-pm@vger.kernel.org \
--cc=rafael@kernel.org \
--cc=robh@kernel.org \
--cc=rui.zhang@intel.com \
--cc=shawnguo@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox