From: Daniel Lezcano <daniel.lezcano@linaro.org> To: Chen-Yu Tsai <wens@kernel.org>, Zhang Rui <rui.zhang@intel.com>, Amit Kucheria <amit.kucheria@verdurent.com>, Heiko Stuebner <heiko@sntech.de> Cc: linux-pm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] thermal: rockchip: enable hwmon Date: Tue, 17 Dec 2019 09:01:44 +0100 [thread overview] Message-ID: <7f2be28d-9bf5-79a8-8720-4615a4e9f463@linaro.org> (raw) In-Reply-To: <20191217052328.25633-1-wens@kernel.org> Hi Chen-Yu, On 17/12/2019 06:23, Chen-Yu Tsai wrote: > From: Chen-Yu Tsai <wens@csie.org> > > By default of-based thermal driver do not have hwmon entries registered. > > Do this explicitly so users can use standard hwmon interfaces and tools > to read the temperature. > > This is based on similar changes for bcm2835_thermal in commit > d56c19d07e0b ("thermal: bcm2835: enable hwmon explicitly"). Thanks for submitting this patch, but it is duplicate with: https://lore.kernel.org/linux-arm-kernel/20191212061702.BFE2D6E85603@corona.crabdance.com/ which I picked up. -- Daniel > Signed-off-by: Chen-Yu Tsai <wens@csie.org> > --- > drivers/thermal/rockchip_thermal.c | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/drivers/thermal/rockchip_thermal.c b/drivers/thermal/rockchip_thermal.c > index 9ed8085bb792..d53ba7dabf16 100644 > --- a/drivers/thermal/rockchip_thermal.c > +++ b/drivers/thermal/rockchip_thermal.c > @@ -19,6 +19,8 @@ > #include <linux/mfd/syscon.h> > #include <linux/pinctrl/consumer.h> > > +#include "thermal_hwmon.h" > + > /* > * If the temperature over a period of time High, > * the resulting TSHUT gave CRU module,let it reset the entire chip, > @@ -1210,7 +1212,11 @@ rockchip_thermal_register_sensor(struct platform_device *pdev, > return error; > } > > - return 0; > + /* thermal_zone doesn't enable hwmon as default, enable it here */ > + sensor->tzd->tzp->no_hwmon = false; > + error = thermal_add_hwmon_sysfs(sensor->tzd); > + > + return error; > } > > /** > -- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog
WARNING: multiple messages have this Message-ID (diff)
From: Daniel Lezcano <daniel.lezcano@linaro.org> To: Chen-Yu Tsai <wens@kernel.org>, Zhang Rui <rui.zhang@intel.com>, Amit Kucheria <amit.kucheria@verdurent.com>, Heiko Stuebner <heiko@sntech.de> Cc: linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-pm@vger.kernel.org Subject: Re: [PATCH] thermal: rockchip: enable hwmon Date: Tue, 17 Dec 2019 09:01:44 +0100 [thread overview] Message-ID: <7f2be28d-9bf5-79a8-8720-4615a4e9f463@linaro.org> (raw) In-Reply-To: <20191217052328.25633-1-wens@kernel.org> Hi Chen-Yu, On 17/12/2019 06:23, Chen-Yu Tsai wrote: > From: Chen-Yu Tsai <wens@csie.org> > > By default of-based thermal driver do not have hwmon entries registered. > > Do this explicitly so users can use standard hwmon interfaces and tools > to read the temperature. > > This is based on similar changes for bcm2835_thermal in commit > d56c19d07e0b ("thermal: bcm2835: enable hwmon explicitly"). Thanks for submitting this patch, but it is duplicate with: https://lore.kernel.org/linux-arm-kernel/20191212061702.BFE2D6E85603@corona.crabdance.com/ which I picked up. -- Daniel > Signed-off-by: Chen-Yu Tsai <wens@csie.org> > --- > drivers/thermal/rockchip_thermal.c | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/drivers/thermal/rockchip_thermal.c b/drivers/thermal/rockchip_thermal.c > index 9ed8085bb792..d53ba7dabf16 100644 > --- a/drivers/thermal/rockchip_thermal.c > +++ b/drivers/thermal/rockchip_thermal.c > @@ -19,6 +19,8 @@ > #include <linux/mfd/syscon.h> > #include <linux/pinctrl/consumer.h> > > +#include "thermal_hwmon.h" > + > /* > * If the temperature over a period of time High, > * the resulting TSHUT gave CRU module,let it reset the entire chip, > @@ -1210,7 +1212,11 @@ rockchip_thermal_register_sensor(struct platform_device *pdev, > return error; > } > > - return 0; > + /* thermal_zone doesn't enable hwmon as default, enable it here */ > + sensor->tzd->tzp->no_hwmon = false; > + error = thermal_add_hwmon_sysfs(sensor->tzd); > + > + return error; > } > > /** > -- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2019-12-17 8:01 UTC|newest] Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-12-17 5:23 [PATCH] thermal: rockchip: enable hwmon Chen-Yu Tsai 2019-12-17 5:23 ` Chen-Yu Tsai 2019-12-17 8:01 ` Daniel Lezcano [this message] 2019-12-17 8:01 ` Daniel Lezcano 2019-12-17 8:02 ` Chen-Yu Tsai 2019-12-17 8:02 ` Chen-Yu Tsai -- strict thread matches above, loose matches on Subject: below -- 2019-11-18 5:24 schaecsn 2019-11-18 5:24 ` schaecsn 2019-12-02 14:52 ` Stefan Schaeckeler 2019-12-02 14:52 ` Stefan Schaeckeler 2019-12-09 17:08 ` Daniel Lezcano 2019-12-09 17:08 ` Daniel Lezcano
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=7f2be28d-9bf5-79a8-8720-4615a4e9f463@linaro.org \ --to=daniel.lezcano@linaro.org \ --cc=amit.kucheria@verdurent.com \ --cc=heiko@sntech.de \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-pm@vger.kernel.org \ --cc=linux-rockchip@lists.infradead.org \ --cc=rui.zhang@intel.com \ --cc=wens@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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.