From: Nick Xie <[email protected]>
Reduce the MCU FAN trigger temperature from
80 degree centigrade to 50 degree centigrade.
Signed-off-by: Nick Xie <[email protected]>
---
arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi b/arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi
index 66d67524b031..a9c34fee91f4 100644
--- a/arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi
+++ b/arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi
@@ -256,7 +256,7 @@ &cecb_AO {
&cpu_thermal {
trips {
cpu_active: cpu-active {
- temperature = <80000>; /* millicelsius */
+ temperature = <50000>; /* millicelsius */
hysteresis = <2000>; /* millicelsius */
type = "active";
};
--
2.25.1
Hi Nick,
On 09/06/2021 03:28, [email protected] wrote:
> From: Nick Xie <[email protected]>
>
> Reduce the MCU FAN trigger temperature from
> 80 degree centigrade to 50 degree centigrade.
Can you add some more rationale to this change ?
>
> Signed-off-by: Nick Xie <[email protected]>
> ---
> arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi b/arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi
> index 66d67524b031..a9c34fee91f4 100644
> --- a/arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi
> +++ b/arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi
> @@ -256,7 +256,7 @@ &cecb_AO {
> &cpu_thermal {
> trips {
> cpu_active: cpu-active {
> - temperature = <80000>; /* millicelsius */
> + temperature = <50000>; /* millicelsius */
> hysteresis = <2000>; /* millicelsius */
> type = "active";
> };
>
With that:
Reviewed-by: Neil Armstrong <[email protected]>
Hi,
On 11/06/2021 08:54, Nick Xie wrote:
> Hello Neil,
>
> The MCU FAN won't work until the temperature reach to 80 degree centigrade,
> Do you think the temperature is too high?
I think 80 is tool high, but I fear 50 is way too low, but if you think 50 is OK after you ran some tests, just add it to the commit log and it's ok for me.
Neil
>
> Thanks.
>
> Neil Armstrong <[email protected] <mailto:[email protected]>> 于2021年6月11日周五 下午2:44写道：
>
> Hi Nick,
>
> On 09/06/2021 03:28, [email protected] <mailto:[email protected]> wrote:
> > From: Nick Xie <[email protected] <mailto:[email protected]>>
> >
> > Reduce the MCU FAN trigger temperature from
> > 80 degree centigrade to 50 degree centigrade.
>
> Can you add some more rationale to this change ?
>
> >
> > Signed-off-by: Nick Xie <[email protected] <mailto:[email protected]>>
> > ---
> > arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi b/arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi
> > index 66d67524b031..a9c34fee91f4 100644
> > --- a/arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi
> > +++ b/arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi
> > @@ -256,7 +256,7 @@ &cecb_AO {
> > &cpu_thermal {
> > trips {
> > cpu_active: cpu-active {
> > - temperature = <80000>; /* millicelsius */
> > + temperature = <50000>; /* millicelsius */
> > hysteresis = <2000>; /* millicelsius */
> > type = "active";
> > };
> >
>
> With that:
> Reviewed-by: Neil Armstrong <[email protected] <mailto:[email protected]>>
>
Hello Neil,
Neil Armstrong <[email protected]> 于2021年6月11日周五 下午3:15写道：
>
> Hi,
>
> On 11/06/2021 08:54, Nick Xie wrote:
> > Hello Neil,
> >
> > The MCU FAN won't work until the temperature reach to 80 degree centigrade,
> > Do you think the temperature is too high?
>
> I think 80 is tool high, but I fear 50 is way too low, but if you think 50 is OK after you ran some tests, just add it to the commit log and it's ok for me.
I think it is OK, we use it for a long time.
Thanks.
>
> Neil
>
> >
> > Thanks.
> >
> > Neil Armstrong <[email protected] <mailto:[email protected]>> 于2021年6月11日周五 下午2:44写道：
> >
> > Hi Nick,
> >
> > On 09/06/2021 03:28, [email protected] <mailto:[email protected]> wrote:
> > > From: Nick Xie <[email protected] <mailto:[email protected]>>
> > >
> > > Reduce the MCU FAN trigger temperature from
> > > 80 degree centigrade to 50 degree centigrade.
> >
> > Can you add some more rationale to this change ?
> >
> > >
> > > Signed-off-by: Nick Xie <[email protected] <mailto:[email protected]>>
> > > ---
> > > arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi b/arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi
> > > index 66d67524b031..a9c34fee91f4 100644
> > > --- a/arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi
> > > +++ b/arch/arm64/boot/dts/amlogic/meson-khadas-vim3.dtsi
> > > @@ -256,7 +256,7 @@ &cecb_AO {
> > > &cpu_thermal {
> > > trips {
> > > cpu_active: cpu-active {
> > > - temperature = <80000>; /* millicelsius */
> > > + temperature = <50000>; /* millicelsius */
> > > hysteresis = <2000>; /* millicelsius */
> > > type = "active";
> > > };
> > >
> >
> > With that:
> > Reviewed-by: Neil Armstrong <[email protected] <mailto:[email protected]>>
> >
>
On 11/06/2021 09:30, Nick Xie wrote:
> Hello Neil,
>
>
> Neil Armstrong <[email protected]> 于2021年6月11日周五 下午3:15写道：
>>
>> Hi,
>>
>> On 11/06/2021 08:54, Nick Xie wrote:
>>> Hello Neil,
>>>
>>> The MCU FAN won't work until the temperature reach to 80 degree centigrade,
>>> Do you think the temperature is too high?
>>
>> I think 80 is tool high, but I fear 50 is way too low, but if you think 50 is OK after you ran some tests, just add it to the commit log and it's ok for me.
>
> I think it is OK, we use it for a long time.
50°C sounds like normal temperature with regular load. That means the
fan will be switched on most of the time when the board is used.
And this change may be put in regard with the other trip points in the
common DT. For instance, enable the fan when reaching 80°C but set the
passive trip point to 75°C instead of 85°C.
All that depends on the hardware characteristics and TDP which are in
the TRM normally.
--
<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
Hi,
On 11/06/2021 09:45, Daniel Lezcano wrote:
> On 11/06/2021 09:30, Nick Xie wrote:
>> Hello Neil,
>>
>>
>> Neil Armstrong <[email protected]> 于2021年6月11日周五 下午3:15写道：
>>>
>>> Hi,
>>>
>>> On 11/06/2021 08:54, Nick Xie wrote:
>>>> Hello Neil,
>>>>
>>>> The MCU FAN won't work until the temperature reach to 80 degree centigrade,
>>>> Do you think the temperature is too high?
>>>
>>> I think 80 is tool high, but I fear 50 is way too low, but if you think 50 is OK after you ran some tests, just add it to the commit log and it's ok for me.
>>
>> I think it is OK, we use it for a long time.
>
> 50°C sounds like normal temperature with regular load. That means the
> fan will be switched on most of the time when the board is used.
>
> And this change may be put in regard with the other trip points in the
> common DT. For instance, enable the fan when reaching 80°C but set the
> passive trip point to 75°C instead of 85°C.
>
> All that depends on the hardware characteristics and TDP which are in
> the TRM normally.
>
I don't think such data exist in the datasheet or other public document, but the vendor source tree
sets:
- passive: 75°C
- hot 85°C
- critical 110°C
we set upstream:
- passive: 85°C
- hot 95°C
- critical 110°C
so maybe we should lower the passive & hot trip points, then the FAN 80°C trip point would be ok now according to Daniel's explanation.
Neil