The temperature sensor support on the MT7981 is exactly the same
as on the MT7986.
Signed-off-by: Aleksander Jan Bajkowski <olek2@wp.pl>
---
drivers/thermal/mediatek/auxadc_thermal.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/drivers/thermal/mediatek/auxadc_thermal.c b/drivers/thermal/mediatek/auxadc_thermal.c
index 9ee2e7283435..020324901505 100644
--- a/drivers/thermal/mediatek/auxadc_thermal.c
+++ b/drivers/thermal/mediatek/auxadc_thermal.c
@@ -1149,6 +1149,10 @@ static const struct of_device_id mtk_thermal_of_match[] = {
.compatible = "mediatek,mt7622-thermal",
.data = (void *)&mt7622_thermal_data,
},
+ {
+ .compatible = "mediatek,mt7981-thermal",
+ .data = (void *)&mt7986_thermal_data,
+ },
{
.compatible = "mediatek,mt7986-thermal",
.data = (void *)&mt7986_thermal_data,
--
2.39.5
Il 29/06/25 00:38, Aleksander Jan Bajkowski ha scritto:
> The temperature sensor support on the MT7981 is exactly the same
> as on the MT7986.
>
If it is exactly the same, great! ... but then this change is wrong :-)
You should, instead, change the bindings to allow you to specify
compatible = "mediatek,mt7981-thermal", "mediatek,mt7986-thermal";
...and that's about it, no driver changes to be done.
Cheers,
Angelo
> Signed-off-by: Aleksander Jan Bajkowski <olek2@wp.pl>
> ---
> drivers/thermal/mediatek/auxadc_thermal.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/drivers/thermal/mediatek/auxadc_thermal.c b/drivers/thermal/mediatek/auxadc_thermal.c
> index 9ee2e7283435..020324901505 100644
> --- a/drivers/thermal/mediatek/auxadc_thermal.c
> +++ b/drivers/thermal/mediatek/auxadc_thermal.c
> @@ -1149,6 +1149,10 @@ static const struct of_device_id mtk_thermal_of_match[] = {
> .compatible = "mediatek,mt7622-thermal",
> .data = (void *)&mt7622_thermal_data,
> },
> + {
> + .compatible = "mediatek,mt7981-thermal",
> + .data = (void *)&mt7986_thermal_data,
> + },
> {
> .compatible = "mediatek,mt7986-thermal",
> .data = (void *)&mt7986_thermal_data,
On Sun, 29 Jun 2025 00:38:36 +0200
Aleksander Jan Bajkowski <olek2@wp.pl> wrote:
> The temperature sensor support on the MT7981 is exactly the same
> as on the MT7986.
>
> Signed-off-by: Aleksander Jan Bajkowski <olek2@wp.pl>
> ---
> drivers/thermal/mediatek/auxadc_thermal.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/drivers/thermal/mediatek/auxadc_thermal.c b/drivers/thermal/mediatek/auxadc_thermal.c
> index 9ee2e7283435..020324901505 100644
> --- a/drivers/thermal/mediatek/auxadc_thermal.c
> +++ b/drivers/thermal/mediatek/auxadc_thermal.c
> @@ -1149,6 +1149,10 @@ static const struct of_device_id mtk_thermal_of_match[] = {
> .compatible = "mediatek,mt7622-thermal",
> .data = (void *)&mt7622_thermal_data,
> },
> + {
> + .compatible = "mediatek,mt7981-thermal",
> + .data = (void *)&mt7986_thermal_data,
Random drive by question. Why the (void *) casts?
static const struct mtk_thermal_data mt7986_thermal_data;
is the definition and data is a const void * so
you should be able to assign directly without a cast and not
have the casting away of const only to assign it to a
const void *
> + },
> {
> .compatible = "mediatek,mt7986-thermal",
> .data = (void *)&mt7986_thermal_data,
© 2016 - 2026 Red Hat, Inc.