Message ID | 20190401041418.5999-14-andrew.smirnov@gmail.com (mailing list archive) |
---|---|
State | Superseded, archived |
Delegated to: | Eduardo Valentin |
Headers | show |
Series | QorIQ TMU multi-sensor and HWMON support | expand |
On 01/04/2019 06:14, Andrey Smirnov wrote: > Expose thermal readings as a HWMON device, so that it could be > accessed using lm-sensors. > > Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com> > Cc: Chris Healy <cphealy@gmail.com> > Cc: Lucas Stach <l.stach@pengutronix.de> > Cc: Zhang Rui <rui.zhang@intel.com> > Cc: Eduardo Valentin <edubezval@gmail.com> > Cc: Daniel Lezcano <daniel.lezcano@linaro.org> > Cc: Angus Ainslie (Purism) <angus@akkea.ca> > Cc: linux-imx@nxp.com > Cc: linux-pm@vger.kernel.org > Cc: linux-kernel@vger.kernel.org > --- > drivers/thermal/qoriq_thermal.c | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/drivers/thermal/qoriq_thermal.c b/drivers/thermal/qoriq_thermal.c > index 9d227654f879..7fb9321a0d8c 100644 > --- a/drivers/thermal/qoriq_thermal.c > +++ b/drivers/thermal/qoriq_thermal.c > @@ -12,6 +12,7 @@ > #include <linux/thermal.h> > > #include "thermal_core.h" > +#include "thermal_hwmon.h" > > #define SITES_MAX 16 > > @@ -103,7 +104,10 @@ static int qoriq_tmu_register_tmu_zone(struct device *dev, > case -ENODEV: > continue; > case 0: > - break; > + ret = devm_thermal_add_hwmon_sysfs(tzd); > + if (!ret) > + break; > + /* fallthrough */ Do we really want to disable the thermal zone if the hwmon fails to register ? > default: > regmap_write(qdata->regmap, REGS_TMR, TMR_DISABLE); > return ret; >
On Thu, Apr 4, 2019 at 2:12 AM Daniel Lezcano <daniel.lezcano@linaro.org> wrote: > > On 01/04/2019 06:14, Andrey Smirnov wrote: > > Expose thermal readings as a HWMON device, so that it could be > > accessed using lm-sensors. > > > > Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com> > > Cc: Chris Healy <cphealy@gmail.com> > > Cc: Lucas Stach <l.stach@pengutronix.de> > > Cc: Zhang Rui <rui.zhang@intel.com> > > Cc: Eduardo Valentin <edubezval@gmail.com> > > Cc: Daniel Lezcano <daniel.lezcano@linaro.org> > > Cc: Angus Ainslie (Purism) <angus@akkea.ca> > > Cc: linux-imx@nxp.com > > Cc: linux-pm@vger.kernel.org > > Cc: linux-kernel@vger.kernel.org > > --- > > drivers/thermal/qoriq_thermal.c | 6 +++++- > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/thermal/qoriq_thermal.c b/drivers/thermal/qoriq_thermal.c > > index 9d227654f879..7fb9321a0d8c 100644 > > --- a/drivers/thermal/qoriq_thermal.c > > +++ b/drivers/thermal/qoriq_thermal.c > > @@ -12,6 +12,7 @@ > > #include <linux/thermal.h> > > > > #include "thermal_core.h" > > +#include "thermal_hwmon.h" > > > > #define SITES_MAX 16 > > > > @@ -103,7 +104,10 @@ static int qoriq_tmu_register_tmu_zone(struct device *dev, > > case -ENODEV: > > continue; > > case 0: > > - break; > > + ret = devm_thermal_add_hwmon_sysfs(tzd); > > + if (!ret) > > + break; > > + /* fallthrough */ > > Do we really want to disable the thermal zone if the hwmon fails to > register ? Original code already disables all thermal zones if any single one fails to register, so I though we may as well be as strict for hwmon registration as well. Doesn't have to be though. Will change in v4. Thanks, Andrey Smirnov
diff --git a/drivers/thermal/qoriq_thermal.c b/drivers/thermal/qoriq_thermal.c index 9d227654f879..7fb9321a0d8c 100644 --- a/drivers/thermal/qoriq_thermal.c +++ b/drivers/thermal/qoriq_thermal.c @@ -12,6 +12,7 @@ #include <linux/thermal.h> #include "thermal_core.h" +#include "thermal_hwmon.h" #define SITES_MAX 16 @@ -103,7 +104,10 @@ static int qoriq_tmu_register_tmu_zone(struct device *dev, case -ENODEV: continue; case 0: - break; + ret = devm_thermal_add_hwmon_sysfs(tzd); + if (!ret) + break; + /* fallthrough */ default: regmap_write(qdata->regmap, REGS_TMR, TMR_DISABLE); return ret;
Expose thermal readings as a HWMON device, so that it could be accessed using lm-sensors. Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com> Cc: Chris Healy <cphealy@gmail.com> Cc: Lucas Stach <l.stach@pengutronix.de> Cc: Zhang Rui <rui.zhang@intel.com> Cc: Eduardo Valentin <edubezval@gmail.com> Cc: Daniel Lezcano <daniel.lezcano@linaro.org> Cc: Angus Ainslie (Purism) <angus@akkea.ca> Cc: linux-imx@nxp.com Cc: linux-pm@vger.kernel.org Cc: linux-kernel@vger.kernel.org --- drivers/thermal/qoriq_thermal.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-)