[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Y/I7dZiHqT0xH7tC@oden.dyn.berto.se>
Date: Sun, 19 Feb 2023 16:08:37 +0100
From: Niklas Söderlund <niklas.soderlund@...natech.se>
To: Daniel Lezcano <daniel.lezcano@...aro.org>
Cc: rafael@...nel.org, linux-pm@...r.kernel.org,
linux-kernel@...r.kernel.org, Amit Kucheria <amitk@...nel.org>,
Zhang Rui <rui.zhang@...el.com>,
Florian Fainelli <f.fainelli@...il.com>,
Broadcom internal kernel review list
<bcm-kernel-feedback-list@...adcom.com>,
Ray Jui <rjui@...adcom.com>,
Scott Branden <sbranden@...adcom.com>,
Heiko Stuebner <heiko@...ech.de>,
Stefan Wahren <stefan.wahren@...e.com>,
Kunihiko Hayashi <hayashi.kunihiko@...ionext.com>,
Miquel Raynal <miquel.raynal@...tlin.com>,
Neil Armstrong <neil.armstrong@...aro.org>,
ye xingchen <ye.xingchen@....com.cn>,
"moderated list:BROADCOM BCM2711/BCM2835 ARM ARCHITECTURE"
<linux-rpi-kernel@...ts.infradead.org>,
"moderated list:BROADCOM BCM2711/BCM2835 ARM ARCHITECTURE"
<linux-arm-kernel@...ts.infradead.org>,
"open list:RENESAS R-CAR THERMAL DRIVERS"
<linux-renesas-soc@...r.kernel.org>,
"open list:ARM/Rockchip SoC support"
<linux-rockchip@...ts.infradead.org>
Subject: Re: [PATCH v1 04/17] thermal/hwmon: Do not set no_hwmon before
calling thermal_add_hwmon_sysfs()
Hi Daniel,
Thanks for your work.
On 2023-02-19 15:36:44 +0100, Daniel Lezcano wrote:
> The thermal->tzp->no_hwmon parameter is only used when calling
> thermal_zone_device_register().
>
> Setting it to 'false' before calling thermal_add_hwmon_sysfs() has no
> effect.
>
> Remove the call and again prevent the drivers to access the thermal
> internals.
>
> Signed-off-by: Daniel Lezcano <daniel.lezcano@...aro.org>
> ---
> drivers/thermal/broadcom/bcm2711_thermal.c | 1 -
> drivers/thermal/broadcom/bcm2835_thermal.c | 1 -
> drivers/thermal/rcar_gen3_thermal.c | 1 -
> drivers/thermal/rcar_thermal.c | 5 -----
For R-Car,
Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@...natech.se>
> drivers/thermal/rockchip_thermal.c | 1 -
> drivers/thermal/rzg2l_thermal.c | 1 -
> 6 files changed, 10 deletions(-)
>
> diff --git a/drivers/thermal/broadcom/bcm2711_thermal.c b/drivers/thermal/broadcom/bcm2711_thermal.c
> index 18b1a4d9ecc7..0a499435431c 100644
> --- a/drivers/thermal/broadcom/bcm2711_thermal.c
> +++ b/drivers/thermal/broadcom/bcm2711_thermal.c
> @@ -98,7 +98,6 @@ static int bcm2711_thermal_probe(struct platform_device *pdev)
>
> priv->thermal = thermal;
>
> - thermal->tzp->no_hwmon = false;
> return thermal_add_hwmon_sysfs(thermal);
> }
>
> diff --git a/drivers/thermal/broadcom/bcm2835_thermal.c b/drivers/thermal/broadcom/bcm2835_thermal.c
> index de2f573863da..8ed7a80122a0 100644
> --- a/drivers/thermal/broadcom/bcm2835_thermal.c
> +++ b/drivers/thermal/broadcom/bcm2835_thermal.c
> @@ -267,7 +267,6 @@ static int bcm2835_thermal_probe(struct platform_device *pdev)
> * Thermal_zone doesn't enable hwmon as default,
> * enable it here
> */
> - tz->tzp->no_hwmon = false;
> err = thermal_add_hwmon_sysfs(tz);
> if (err)
> goto err_tz;
> diff --git a/drivers/thermal/rcar_gen3_thermal.c b/drivers/thermal/rcar_gen3_thermal.c
> index 8ad713cb4bf7..4ea9d4bd8e6f 100644
> --- a/drivers/thermal/rcar_gen3_thermal.c
> +++ b/drivers/thermal/rcar_gen3_thermal.c
> @@ -527,7 +527,6 @@ static int rcar_gen3_thermal_probe(struct platform_device *pdev)
> }
> tsc->zone = zone;
>
> - tsc->zone->tzp->no_hwmon = false;
> ret = thermal_add_hwmon_sysfs(tsc->zone);
> if (ret)
> goto error_unregister;
> diff --git a/drivers/thermal/rcar_thermal.c b/drivers/thermal/rcar_thermal.c
> index 538ed6731589..919f0e32d3c5 100644
> --- a/drivers/thermal/rcar_thermal.c
> +++ b/drivers/thermal/rcar_thermal.c
> @@ -509,11 +509,6 @@ static int rcar_thermal_probe(struct platform_device *pdev)
> }
>
> if (chip->use_of_thermal) {
> - /*
> - * thermal_zone doesn't enable hwmon as default,
> - * but, enable it here to keep compatible
> - */
> - priv->zone->tzp->no_hwmon = false;
> ret = thermal_add_hwmon_sysfs(priv->zone);
> if (ret)
> goto error_unregister;
> diff --git a/drivers/thermal/rockchip_thermal.c b/drivers/thermal/rockchip_thermal.c
> index bd593ec712ee..3211b70c85b9 100644
> --- a/drivers/thermal/rockchip_thermal.c
> +++ b/drivers/thermal/rockchip_thermal.c
> @@ -1451,7 +1451,6 @@ static int rockchip_thermal_probe(struct platform_device *pdev)
>
> for (i = 0; i < thermal->chip->chn_num; i++) {
> rockchip_thermal_toggle_sensor(&thermal->sensors[i], true);
> - thermal->sensors[i].tzd->tzp->no_hwmon = false;
> error = thermal_add_hwmon_sysfs(thermal->sensors[i].tzd);
> if (error)
> dev_warn(&pdev->dev,
> diff --git a/drivers/thermal/rzg2l_thermal.c b/drivers/thermal/rzg2l_thermal.c
> index d3ba2a74e42d..a2f21c941cfe 100644
> --- a/drivers/thermal/rzg2l_thermal.c
> +++ b/drivers/thermal/rzg2l_thermal.c
> @@ -216,7 +216,6 @@ static int rzg2l_thermal_probe(struct platform_device *pdev)
> }
>
> priv->zone = zone;
> - priv->zone->tzp->no_hwmon = false;
> ret = thermal_add_hwmon_sysfs(priv->zone);
> if (ret)
> goto err;
> --
> 2.34.1
>
--
Kind Regards,
Niklas Söderlund
Powered by blists - more mailing lists