On Fri, Jan 14, 2022 at 3:25 PM Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> wrote: > > Register thermal zones as hwmon sensors to let userspace read > temperatures using standard hwmon interface. > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> > --- > drivers/thermal/qcom/tsens.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/drivers/thermal/qcom/tsens.c b/drivers/thermal/qcom/tsens.c > index 99a8d9f3e03c..c13093e8a642 100644 > --- a/drivers/thermal/qcom/tsens.c > +++ b/drivers/thermal/qcom/tsens.c > @@ -19,6 +19,7 @@ > #include <linux/slab.h> > #include <linux/thermal.h> > #include "tsens.h" > +#include "../thermal_hwmon.h" Just a nit, but can you move this above the tsen.h along with other system headers? With that, feel free to add Acked-by: Amit Kucheria <amitk@xxxxxxxxxx> > > /** > * struct tsens_irq_data - IRQ status and temperature violations > @@ -1060,6 +1061,10 @@ static int tsens_register(struct tsens_priv *priv) > priv->sensor[i].tzd = tzd; > if (priv->ops->enable) > priv->ops->enable(priv, i); > + > + if (devm_thermal_add_hwmon_sysfs(tzd)) > + dev_warn(priv->dev, > + "Failed to add hwmon sysfs attributes\n"); > } > > /* VER_0 require to set MIN and MAX THRESH > -- > 2.34.1 >
