[PATCH 1/1] thermal: imx8mm: enable ADC when enabling monitor

[PATCH 1/1] thermal: imx8mm: enable ADC when enabling monitor

[Alexander Stein]

From: Paul Gerber <Paul.Gerber@tq-group.com>

The i.MX 8MP has a ADC_PD bit in the TMU_TER register that controls the
operating mode of the ADC:
* 0 means normal operating mode
* 1 means power down mode

When enabling/disabling the TMU, the ADC operating mode must be set
accordingly.

i.MX 8M Mini & Nano are lacking this bit.

Signed-off-by: Paul Gerber <Paul.Gerber@tq-group.com>
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
Often this bit is set already by the bootloader, but in case this is not
done, the file /sys/class/thermal/thermal_zone0/temp will always return
-EAGAIN.

 drivers/thermal/imx8mm_thermal.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/thermal/imx8mm_thermal.c b/drivers/thermal/imx8mm_thermal.c
index 7442e013738f..af666bd9e8d4 100644
--- a/drivers/thermal/imx8mm_thermal.c
+++ b/drivers/thermal/imx8mm_thermal.c
@@ -21,6 +21,7 @@
 #define TPS			0x4
 #define TRITSR			0x20	/* TMU immediate temp */
 
+#define TER_ADC_PD		BIT(30)
 #define TER_EN			BIT(31)
 #define TRITSR_TEMP0_VAL_MASK	0xff
 #define TRITSR_TEMP1_VAL_MASK	0xff0000
@@ -113,6 +114,8 @@ static void imx8mm_tmu_enable(struct imx8mm_tmu *tmu, bool enable)
 
 	val = readl_relaxed(tmu->base + TER);
 	val = enable ? (val | TER_EN) : (val & ~TER_EN);
+	if (tmu->socdata->version == TMU_VER2)
+		val = enable ? (val & ~TER_ADC_PD) : (val | TER_ADC_PD);
 	writel_relaxed(val, tmu->base + TER);
 }
 
-- 
2.25.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: [PATCH 1/1] thermal: imx8mm: enable ADC when enabling monitor

[Daniel Lezcano]

On 22/11/2021 12:42, Alexander Stein wrote:
> From: Paul Gerber <Paul.Gerber@tq-group.com>
> 
> The i.MX 8MP has a ADC_PD bit in the TMU_TER register that controls the
> operating mode of the ADC:
> * 0 means normal operating mode
> * 1 means power down mode
> 
> When enabling/disabling the TMU, the ADC operating mode must be set
> accordingly.
> 
> i.MX 8M Mini & Nano are lacking this bit.
> 
> Signed-off-by: Paul Gerber <Paul.Gerber@tq-group.com>
> Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
> ---
> Often this bit is set already by the bootloader, but in case this is not
> done, the file /sys/class/thermal/thermal_zone0/temp will always return
> -EAGAIN.

Does it deserve a Fixes tag?

>  drivers/thermal/imx8mm_thermal.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/thermal/imx8mm_thermal.c b/drivers/thermal/imx8mm_thermal.c
> index 7442e013738f..af666bd9e8d4 100644
> --- a/drivers/thermal/imx8mm_thermal.c
> +++ b/drivers/thermal/imx8mm_thermal.c
> @@ -21,6 +21,7 @@
>  #define TPS			0x4
>  #define TRITSR			0x20	/* TMU immediate temp */
>  
> +#define TER_ADC_PD		BIT(30)
>  #define TER_EN			BIT(31)
>  #define TRITSR_TEMP0_VAL_MASK	0xff
>  #define TRITSR_TEMP1_VAL_MASK	0xff0000
> @@ -113,6 +114,8 @@ static void imx8mm_tmu_enable(struct imx8mm_tmu *tmu, bool enable)
>  
>  	val = readl_relaxed(tmu->base + TER);
>  	val = enable ? (val | TER_EN) : (val & ~TER_EN);
> +	if (tmu->socdata->version == TMU_VER2)
> +		val = enable ? (val & ~TER_ADC_PD) : (val | TER_ADC_PD);
>  	writel_relaxed(val, tmu->base + TER);
>  }
>  
> 


-- 
<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

Re: (EXT) Re: [PATCH 1/1] thermal: imx8mm: enable ADC when enabling monitor

[Alexander Stein]

Am Dienstag, dem 30.11.2021 um 15:49 +0100 schrieb Daniel Lezcano:
> On 22/11/2021 12:42, Alexander Stein wrote:
> > From: Paul Gerber <
> > Paul.Gerber@tq-group.com
> > >
> > 
> > The i.MX 8MP has a ADC_PD bit in the TMU_TER register that controls
> > the
> > operating mode of the ADC:
> > * 0 means normal operating mode
> > * 1 means power down mode
> > 
> > When enabling/disabling the TMU, the ADC operating mode must be set
> > accordingly.
> > 
> > i.MX 8M Mini & Nano are lacking this bit.
> > 
> > Signed-off-by: Paul Gerber <
> > Paul.Gerber@tq-group.com
> > >
> > Signed-off-by: Alexander Stein <
> > alexander.stein@ew.tq-group.com
> > >
> > ---
> > Often this bit is set already by the bootloader, but in case this
> > is not
> > done, the file /sys/class/thermal/thermal_zone0/temp will always
> > return
> > -EAGAIN.
> 
> Does it deserve a Fixes tag?

The more I think about it, the more I'm inclined to add one.
Please tell me if I shall iterate v2 adding a Fixes tag (or other
changes)

Best regards,
Alexander



_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: (EXT) Re: [PATCH 1/1] thermal: imx8mm: enable ADC when enabling monitor

[Daniel Lezcano]

On 01/12/2021 15:45, Alexander Stein wrote:
> Am Dienstag, dem 30.11.2021 um 15:49 +0100 schrieb Daniel Lezcano:
>> On 22/11/2021 12:42, Alexander Stein wrote:
>>> From: Paul Gerber <
>>> Paul.Gerber@tq-group.com
>>>>
>>>
>>> The i.MX 8MP has a ADC_PD bit in the TMU_TER register that controls
>>> the
>>> operating mode of the ADC:
>>> * 0 means normal operating mode
>>> * 1 means power down mode
>>>
>>> When enabling/disabling the TMU, the ADC operating mode must be set
>>> accordingly.
>>>
>>> i.MX 8M Mini & Nano are lacking this bit.
>>>
>>> Signed-off-by: Paul Gerber <
>>> Paul.Gerber@tq-group.com
>>>>
>>> Signed-off-by: Alexander Stein <
>>> alexander.stein@ew.tq-group.com
>>>>
>>> ---
>>> Often this bit is set already by the bootloader, but in case this
>>> is not
>>> done, the file /sys/class/thermal/thermal_zone0/temp will always
>>> return
>>> -EAGAIN.
>>
>> Does it deserve a Fixes tag?
> 
> The more I think about it, the more I'm inclined to add one.
> Please tell me if I shall iterate v2 adding a Fixes tag (or other
> changes)

It is not necessary to send a V2 as the commit log will have a link to
this thread. Just answer with the Fixes tag, I'll add it myself to the
change log.



-- 
<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

Re: (EXT) Re: (EXT) Re: [PATCH 1/1] thermal: imx8mm: enable ADC when enabling monitor

[Alexander Stein]

Am Mittwoch, dem 01.12.2021 um 15:52 +0100 schrieb Daniel Lezcano:
> On 01/12/2021 15:45, Alexander Stein wrote:
> > Am Dienstag, dem 30.11.2021 um 15:49 +0100 schrieb Daniel Lezcano:
> > > On 22/11/2021 12:42, Alexander Stein wrote:
> > > > From: Paul Gerber <
> > > > Paul.Gerber@tq-group.com
> > > > 
> > > > 
> > > > The i.MX 8MP has a ADC_PD bit in the TMU_TER register that
> > > > controls
> > > > the
> > > > operating mode of the ADC:
> > > > * 0 means normal operating mode
> > > > * 1 means power down mode
> > > > 
> > > > When enabling/disabling the TMU, the ADC operating mode must be
> > > > set
> > > > accordingly.
> > > > 
> > > > i.MX 8M Mini & Nano are lacking this bit.
> > > > 
> > > > Signed-off-by: Paul Gerber <
> > > > Paul.Gerber@tq-group.com
> > > > 
> > > > 
> > > > Signed-off-by: Alexander Stein <
> > > > alexander.stein@ew.tq-group.com
> > > > 
> > > > 
> > > > ---
> > > > Often this bit is set already by the bootloader, but in case
> > > > this
> > > > is not
> > > > done, the file /sys/class/thermal/thermal_zone0/temp will
> > > > always
> > > > return
> > > > -EAGAIN.
> > > 
> > > Does it deserve a Fixes tag?
> > 
> > The more I think about it, the more I'm inclined to add one.
> > Please tell me if I shall iterate v2 adding a Fixes tag (or other
> > changes)
> 
> It is not necessary to send a V2 as the commit log will have a link
> to
> this thread. Just answer with the Fixes tag, I'll add it myself to
> the
> change log.

Thanks.

Fixes: 2b8f1f0337c5 ("thermal: imx8mm: Add i.MX8MP support")

imx8mm_tmu_enable() was introduced in this commit and the fix is about
i.MX8MP as well, so this seems even more reasonable to me.

Best regards,
Alexander



_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel