Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
Message-ID: <trinity-de0268f6-e2f2-4ce9-abdd-8016c593894c-1624874759475@3c-app-gmx-bs72>
From:   Frank Wunderlich <frank-w@public-files.de>
To:     Daniel Lezcano <daniel.lezcano@linaro.org>
Cc:     Matthias Brugger <matthias.bgg@gmail.com>,
        Frank Wunderlich <linux@fw-web.de>,
        linux-mediatek@lists.infradead.org,
        Amit Kucheria <amitk@kernel.org>, linux-pm@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: Aw: Re: [PATCH v4] thermal: mediatek: add sensors-support
Content-Type: text/plain; charset=UTF-8
Date:   Mon, 28 Jun 2021 12:05:59 +0200
Importance: normal
Sensitivity: Normal
In-Reply-To: <9eb45f06-1303-1438-7ba0-b9ccaa898b34@gmail.com>
References: <20210608154530.70074-1-linux@fw-web.de>
 <9eb45f06-1303-1438-7ba0-b9ccaa898b34@gmail.com>
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Hi,

A gentle ping as it is reviewed but not yet visible in thermal tree [1] i =
guess it should do,right?

regards Frank

[1] https://git.kernel.org/pub/scm/linux/kernel/git/thermal/linux.git/log/=
?h=3Dthermal/next

> Gesendet: Mittwoch, 09. Juni 2021 um 10:09 Uhr
> Von: "Matthias Brugger" <matthias.bgg@gmail.com>
> An: "Frank Wunderlich" <linux@fw-web.de>, linux-mediatek@lists.infradead=
.org
> Cc: "Frank Wunderlich" <frank-w@public-files.de>, "Daniel Lezcano" <dani=
el.lezcano@linaro.org>, "Amit Kucheria" <amitk@kernel.org>, linux-pm@vger.=
kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel=
.org
> Betreff: Re: [PATCH v4] thermal: mediatek: add sensors-support
>
>
>
> On 08/06/2021 17:45, Frank Wunderlich wrote:
> > From: Frank Wunderlich <frank-w@public-files.de>
> >
> > add HWMON-support to mediateks thermal driver to allow lm-sensors
> > userspace tools read soc temperature
> >
> > Signed-off-by: Frank Wunderlich <frank-w@public-files.de>
>
> Reviewed-by: Matthias Brugger <matthias.bgg@gmail.com>
>
> > ---
> > v4: change message to dev_warn as suggested by matthias
> > v3: drop no_hwmon
> > v2: drop ifdef and used devm_thermal_add_hwmon_sysfs
> > ---
> >  drivers/thermal/mtk_thermal.c | 6 ++++++
> >  1 file changed, 6 insertions(+)
> >
> > diff --git a/drivers/thermal/mtk_thermal.c b/drivers/thermal/mtk_therm=
al.c
> > index 97e8678ccf0e..ede94eadddda 100644
> > --- a/drivers/thermal/mtk_thermal.c
> > +++ b/drivers/thermal/mtk_thermal.c
> > @@ -23,6 +23,8 @@
> >  #include <linux/reset.h>
> >  #include <linux/types.h>
> >
> > +#include "thermal_hwmon.h"
> > +
> >  /* AUXADC Registers */
> >  #define AUXADC_CON1_SET_V	0x008
> >  #define AUXADC_CON1_CLR_V	0x00c
> > @@ -1087,6 +1089,10 @@ static int mtk_thermal_probe(struct platform_de=
vice *pdev)
> >  		goto err_disable_clk_peri_therm;
> >  	}
> >
> > +	ret =3D devm_thermal_add_hwmon_sysfs(tzdev);
> > +	if (ret)
> > +		dev_warn(&pdev->dev, "error in thermal_add_hwmon_sysfs");
> > +
> >  	return 0;
> >
> >  err_disable_clk_peri_therm:
> >
>