* [PATCH] ARM: dts: stm32: Enable thermal sensor support on stm32mp15xx-dhcor
@ 2020-09-23 23:25 Marek Vasut
2020-09-24 5:16 ` Manivannan Sadhasivam
0 siblings, 1 reply; 8+ messages in thread
From: Marek Vasut @ 2020-09-23 23:25 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Marek Vasut, linux-stm32, Alexandre Torgue, Manivannan Sadhasivam
Enable STM32 Digital Thermal Sensor driver for stm32mp15xx-dhcor SoMs.
Fixes: 94cafe1b6482 ("ARM: dts: stm32: Add Avenger96 devicetree support based on STM32MP157A")
Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Alexandre Torgue <alexandre.torgue@st.com>
Cc: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Cc: linux-stm32@st-md-mailman.stormreply.com
To: linux-arm-kernel@lists.infradead.org
---
arch/arm/boot/dts/stm32mp15xx-dhcor-som.dtsi | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/arch/arm/boot/dts/stm32mp15xx-dhcor-som.dtsi b/arch/arm/boot/dts/stm32mp15xx-dhcor-som.dtsi
index 04fbb324a541..803eb8bc9c85 100644
--- a/arch/arm/boot/dts/stm32mp15xx-dhcor-som.dtsi
+++ b/arch/arm/boot/dts/stm32mp15xx-dhcor-som.dtsi
@@ -21,6 +21,10 @@ memory@c0000000 {
};
};
+&dts {
+ status = "okay";
+};
+
&i2c4 {
pinctrl-names = "default";
pinctrl-0 = <&i2c4_pins_a>;
--
2.28.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: dts: stm32: Enable thermal sensor support on stm32mp15xx-dhcor
2020-09-23 23:25 [PATCH] ARM: dts: stm32: Enable thermal sensor support on stm32mp15xx-dhcor Marek Vasut
@ 2020-09-24 5:16 ` Manivannan Sadhasivam
2020-09-24 10:41 ` Marek Vasut
0 siblings, 1 reply; 8+ messages in thread
From: Manivannan Sadhasivam @ 2020-09-24 5:16 UTC (permalink / raw)
To: Marek Vasut, linux-arm-kernel; +Cc: Alexandre Torgue, linux-stm32
On 24 September 2020 4:55:35 AM IST, Marek Vasut <marex@denx.de> wrote:
>Enable STM32 Digital Thermal Sensor driver for stm32mp15xx-dhcor SoMs.
>
>Fixes: 94cafe1b6482 ("ARM: dts: stm32: Add Avenger96 devicetree support
>based on STM32MP157A")
The change looks good but what does this patch fixes?
Thanks,
Mani
>Signed-off-by: Marek Vasut <marex@denx.de>
>Cc: Alexandre Torgue <alexandre.torgue@st.com>
>Cc: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
>Cc: linux-stm32@st-md-mailman.stormreply.com
>To: linux-arm-kernel@lists.infradead.org
>---
> arch/arm/boot/dts/stm32mp15xx-dhcor-som.dtsi | 4 ++++
> 1 file changed, 4 insertions(+)
>
>diff --git a/arch/arm/boot/dts/stm32mp15xx-dhcor-som.dtsi
>b/arch/arm/boot/dts/stm32mp15xx-dhcor-som.dtsi
>index 04fbb324a541..803eb8bc9c85 100644
>--- a/arch/arm/boot/dts/stm32mp15xx-dhcor-som.dtsi
>+++ b/arch/arm/boot/dts/stm32mp15xx-dhcor-som.dtsi
>@@ -21,6 +21,10 @@ memory@c0000000 {
> };
> };
>
>+&dts {
>+ status = "okay";
>+};
>+
> &i2c4 {
> pinctrl-names = "default";
> pinctrl-0 = <&i2c4_pins_a>;
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: dts: stm32: Enable thermal sensor support on stm32mp15xx-dhcor
2020-09-24 5:16 ` Manivannan Sadhasivam
@ 2020-09-24 10:41 ` Marek Vasut
2020-09-25 2:21 ` Manivannan Sadhasivam
0 siblings, 1 reply; 8+ messages in thread
From: Marek Vasut @ 2020-09-24 10:41 UTC (permalink / raw)
To: Manivannan Sadhasivam, linux-arm-kernel; +Cc: Alexandre Torgue, linux-stm32
On 9/24/20 7:16 AM, Manivannan Sadhasivam wrote:
>
>
> On 24 September 2020 4:55:35 AM IST, Marek Vasut <marex@denx.de> wrote:
>> Enable STM32 Digital Thermal Sensor driver for stm32mp15xx-dhcor SoMs.
>>
>> Fixes: 94cafe1b6482 ("ARM: dts: stm32: Add Avenger96 devicetree support
>> based on STM32MP157A")
>
> The change looks good but what does this patch fixes?
The missing temp sensor, which helps you detect overheat of the SoC.
That is esp. important on the 800 MHz AV96.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: dts: stm32: Enable thermal sensor support on stm32mp15xx-dhcor
2020-09-24 10:41 ` Marek Vasut
@ 2020-09-25 2:21 ` Manivannan Sadhasivam
2020-09-25 11:12 ` Marek Vasut
0 siblings, 1 reply; 8+ messages in thread
From: Manivannan Sadhasivam @ 2020-09-25 2:21 UTC (permalink / raw)
To: Marek Vasut, linux-arm-kernel; +Cc: Alexandre Torgue, linux-stm32
On 24 September 2020 4:11:11 PM IST, Marek Vasut <marex@denx.de> wrote:
>On 9/24/20 7:16 AM, Manivannan Sadhasivam wrote:
>>
>>
>> On 24 September 2020 4:55:35 AM IST, Marek Vasut <marex@denx.de>
>wrote:
>>> Enable STM32 Digital Thermal Sensor driver for stm32mp15xx-dhcor
>SoMs.
>>>
>>> Fixes: 94cafe1b6482 ("ARM: dts: stm32: Add Avenger96 devicetree
>support
>>> based on STM32MP157A")
>>
>> The change looks good but what does this patch fixes?
>
>The missing temp sensor, which helps you detect overheat of the SoC.
>That is esp. important on the 800 MHz AV96.
This doesn't quality as a "fix". Essentially you're just adding a missing feature and not fixing any issues. So please remove the fixes tag and resubmit.
Thanks,
Mani
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: dts: stm32: Enable thermal sensor support on stm32mp15xx-dhcor
2020-09-25 2:21 ` Manivannan Sadhasivam
@ 2020-09-25 11:12 ` Marek Vasut
2020-09-25 16:20 ` Manivannan Sadhasivam
0 siblings, 1 reply; 8+ messages in thread
From: Marek Vasut @ 2020-09-25 11:12 UTC (permalink / raw)
To: Manivannan Sadhasivam, linux-arm-kernel; +Cc: Alexandre Torgue, linux-stm32
On 9/25/20 4:21 AM, Manivannan Sadhasivam wrote:
>
>
> On 24 September 2020 4:11:11 PM IST, Marek Vasut <marex@denx.de> wrote:
>> On 9/24/20 7:16 AM, Manivannan Sadhasivam wrote:
>>>
>>>
>>> On 24 September 2020 4:55:35 AM IST, Marek Vasut <marex@denx.de>
>> wrote:
>>>> Enable STM32 Digital Thermal Sensor driver for stm32mp15xx-dhcor
>> SoMs.
>>>>
>>>> Fixes: 94cafe1b6482 ("ARM: dts: stm32: Add Avenger96 devicetree
>> support
>>>> based on STM32MP157A")
>>>
>>> The change looks good but what does this patch fixes?
>>
>> The missing temp sensor, which helps you detect overheat of the SoC.
>> That is esp. important on the 800 MHz AV96.
>
> This doesn't quality as a "fix". Essentially you're just adding a missing feature and not fixing any issues. So please remove the fixes tag and resubmit.
I would argue that if the system overheats and crashes, we want to know
about that, possibly in advance so thermal throttling can be applied.
Currently this is not possible and I think that is a bug.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: dts: stm32: Enable thermal sensor support on stm32mp15xx-dhcor
2020-09-25 11:12 ` Marek Vasut
@ 2020-09-25 16:20 ` Manivannan Sadhasivam
2020-09-25 18:06 ` Marek Vasut
2020-11-09 13:55 ` Alexandre Torgue
0 siblings, 2 replies; 8+ messages in thread
From: Manivannan Sadhasivam @ 2020-09-25 16:20 UTC (permalink / raw)
To: Marek Vasut; +Cc: Alexandre Torgue, linux-arm-kernel, linux-stm32
On Fri, Sep 25, 2020 at 01:12:12PM +0200, Marek Vasut wrote:
> On 9/25/20 4:21 AM, Manivannan Sadhasivam wrote:
> >
> >
> > On 24 September 2020 4:11:11 PM IST, Marek Vasut <marex@denx.de> wrote:
> >> On 9/24/20 7:16 AM, Manivannan Sadhasivam wrote:
> >>>
> >>>
> >>> On 24 September 2020 4:55:35 AM IST, Marek Vasut <marex@denx.de>
> >> wrote:
> >>>> Enable STM32 Digital Thermal Sensor driver for stm32mp15xx-dhcor
> >> SoMs.
> >>>>
> >>>> Fixes: 94cafe1b6482 ("ARM: dts: stm32: Add Avenger96 devicetree
> >> support
> >>>> based on STM32MP157A")
> >>>
> >>> The change looks good but what does this patch fixes?
> >>
> >> The missing temp sensor, which helps you detect overheat of the SoC.
> >> That is esp. important on the 800 MHz AV96.
> >
> > This doesn't quality as a "fix". Essentially you're just adding a missing feature and not fixing any issues. So please remove the fixes tag and resubmit.
>
> I would argue that if the system overheats and crashes, we want to know
> about that, possibly in advance so thermal throttling can be applied.
> Currently this is not possible and I think that is a bug.
No, this is not a _bug_. This is a missing feature that the current kernel
doesn't support and you know about that! The fact that you can trigger a crash
due to hw limitation doesn't qualify as a bug IMO. And you can do that by other
means also (CPU throttling without CPUFreq support etc...)
Anyway, I'll stop here and let Alex to make a call.
Either case, feel free to add:
Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Thanks,
Mani
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: dts: stm32: Enable thermal sensor support on stm32mp15xx-dhcor
2020-09-25 16:20 ` Manivannan Sadhasivam
@ 2020-09-25 18:06 ` Marek Vasut
2020-11-09 13:55 ` Alexandre Torgue
1 sibling, 0 replies; 8+ messages in thread
From: Marek Vasut @ 2020-09-25 18:06 UTC (permalink / raw)
To: Manivannan Sadhasivam; +Cc: Alexandre Torgue, linux-arm-kernel, linux-stm32
On 9/25/20 6:20 PM, Manivannan Sadhasivam wrote:
> On Fri, Sep 25, 2020 at 01:12:12PM +0200, Marek Vasut wrote:
>> On 9/25/20 4:21 AM, Manivannan Sadhasivam wrote:
>>>
>>>
>>> On 24 September 2020 4:11:11 PM IST, Marek Vasut <marex@denx.de> wrote:
>>>> On 9/24/20 7:16 AM, Manivannan Sadhasivam wrote:
>>>>>
>>>>>
>>>>> On 24 September 2020 4:55:35 AM IST, Marek Vasut <marex@denx.de>
>>>> wrote:
>>>>>> Enable STM32 Digital Thermal Sensor driver for stm32mp15xx-dhcor
>>>> SoMs.
>>>>>>
>>>>>> Fixes: 94cafe1b6482 ("ARM: dts: stm32: Add Avenger96 devicetree
>>>> support
>>>>>> based on STM32MP157A")
>>>>>
>>>>> The change looks good but what does this patch fixes?
>>>>
>>>> The missing temp sensor, which helps you detect overheat of the SoC.
>>>> That is esp. important on the 800 MHz AV96.
>>>
>>> This doesn't quality as a "fix". Essentially you're just adding a missing feature and not fixing any issues. So please remove the fixes tag and resubmit.
>>
>> I would argue that if the system overheats and crashes, we want to know
>> about that, possibly in advance so thermal throttling can be applied.
>> Currently this is not possible and I think that is a bug.
>
> No, this is not a _bug_. This is a missing feature that the current kernel
> doesn't support and you know about that! The fact that you can trigger a crash
> due to hw limitation doesn't qualify as a bug IMO. And you can do that by other
> means also (CPU throttling without CPUFreq support etc...)
I would say that makes the hardware pretty useless, which is not the
expectation I would have of a system I would like to use in production.
Just assume your laptop would randomly overheat and crash, would you
consider that a missing feature or a bug that should be fixed ?
> Anyway, I'll stop here and let Alex to make a call.
I agree.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: dts: stm32: Enable thermal sensor support on stm32mp15xx-dhcor
2020-09-25 16:20 ` Manivannan Sadhasivam
2020-09-25 18:06 ` Marek Vasut
@ 2020-11-09 13:55 ` Alexandre Torgue
1 sibling, 0 replies; 8+ messages in thread
From: Alexandre Torgue @ 2020-11-09 13:55 UTC (permalink / raw)
To: Manivannan Sadhasivam, Marek Vasut; +Cc: linux-stm32, linux-arm-kernel
Hi Marek, Mani
On 9/25/20 6:20 PM, Manivannan Sadhasivam wrote:
> On Fri, Sep 25, 2020 at 01:12:12PM +0200, Marek Vasut wrote:
>> On 9/25/20 4:21 AM, Manivannan Sadhasivam wrote:
>>>
>>>
>>> On 24 September 2020 4:11:11 PM IST, Marek Vasut <marex@denx.de> wrote:
>>>> On 9/24/20 7:16 AM, Manivannan Sadhasivam wrote:
>>>>>
>>>>>
>>>>> On 24 September 2020 4:55:35 AM IST, Marek Vasut <marex@denx.de>
>>>> wrote:
>>>>>> Enable STM32 Digital Thermal Sensor driver for stm32mp15xx-dhcor
>>>> SoMs.
>>>>>>
>>>>>> Fixes: 94cafe1b6482 ("ARM: dts: stm32: Add Avenger96 devicetree
>>>> support
>>>>>> based on STM32MP157A")
>>>>>
>>>>> The change looks good but what does this patch fixes?
>>>>
>>>> The missing temp sensor, which helps you detect overheat of the SoC.
>>>> That is esp. important on the 800 MHz AV96.
>>>
>>> This doesn't quality as a "fix". Essentially you're just adding a missing feature and not fixing any issues. So please remove the fixes tag and resubmit.
>>
>> I would argue that if the system overheats and crashes, we want to know
>> about that, possibly in advance so thermal throttling can be applied.
>> Currently this is not possible and I think that is a bug.
>
> No, this is not a _bug_. This is a missing feature that the current kernel
> doesn't support and you know about that! The fact that you can trigger a crash
> due to hw limitation doesn't qualify as a bug IMO. And you can do that by other
> means also (CPU throttling without CPUFreq support etc...)
>
> Anyway, I'll stop here and let Alex to make a call.
I take this patch as fix this time. I understand your argument Mani, but
to avoid unexplained crashes at boot it's better to get also on older
version.
So applied on stm32-dt-for-v5.10-fixes.
regards
alex
> Either case, feel free to add:
>
> Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
>
> Thanks,
> Mani
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2020-11-09 13:57 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-23 23:25 [PATCH] ARM: dts: stm32: Enable thermal sensor support on stm32mp15xx-dhcor Marek Vasut
2020-09-24 5:16 ` Manivannan Sadhasivam
2020-09-24 10:41 ` Marek Vasut
2020-09-25 2:21 ` Manivannan Sadhasivam
2020-09-25 11:12 ` Marek Vasut
2020-09-25 16:20 ` Manivannan Sadhasivam
2020-09-25 18:06 ` Marek Vasut
2020-11-09 13:55 ` Alexandre Torgue
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.