From: "Clément Péron" <peron.clem@gmail.com> To: "Ondřej Jirman" <megous@megous.com>, "Clément Péron" <peron.clem@gmail.com>, "Maxime Ripard" <mripard@kernel.org>, "Chen-Yu Tsai" <wens@csie.org>, "Rob Herring" <robh+dt@kernel.org>, linux-arm-kernel <linux-arm-kernel@lists.infradead.org>, devicetree <devicetree@vger.kernel.org>, linux-kernel <linux-kernel@vger.kernel.org>, linux-sunxi <linux-sunxi@googlegroups.com> Subject: Re: [linux-sunxi] [PATCH 0/7] Add support for Allwinner H6 DVFS Date: Sun, 5 Apr 2020 17:01:51 +0200 [thread overview] Message-ID: <CAJiuCceROWv1ni3YZ6rU71O9R__t6ZOgnkZQDjBhJvhqHRAVng@mail.gmail.com> (raw) In-Reply-To: <20200405145443.uycsh73kxmurdmil@core.my.home> Hi, On Sun, 5 Apr 2020 at 16:54, Ondřej Jirman <megous@megous.com> wrote: > > Hi, > > On Sun, Apr 05, 2020 at 04:33:37PM +0200, Clément Péron wrote: > > Hi Ondřej, > > [ ... ] > > > Good point, this information should be added for both CPU and GPU regulator. > > This could be nice to confirm this point with a scope. > > > > Also I remark that Allwinner user higher temperature than what we set : > > alarm_low_temp = <105000>; > > alarm_high_temp = <110000>; > > alarm_temp_hysteresis = <15000>; > > shut_temp= <115000>; > > https://github.com/orangepi-xunlong/OrangePiH6_Linux4_9/blob/master/arch/arm64/boot/dts/sunxi/sun50iw6p1.dtsi#L1924 > > > > Don't you think that we can push a bit higher the temperature it's > > actually at 80°C ? > > We probably can. But just 5°C between high temp and criticla temp is probably > not enough. I'd probably set our value to something like 90-100°C so that quick > temperature spike before regulation kicks in would not shutdown the CPU. OK I will push it to 85°C and leave the critical to 100°C. Test it again and will send a v2 Regards, Clément > > regards, > o. > > > Thanks for the review, > > Clement > >
WARNING: multiple messages have this Message-ID (diff)
From: "Clément Péron" <peron.clem@gmail.com> To: "Ondřej Jirman" <megous@megous.com>, "Clément Péron" <peron.clem@gmail.com>, "Maxime Ripard" <mripard@kernel.org>, "Chen-Yu Tsai" <wens@csie.org>, "Rob Herring" <robh+dt@kernel.org>, linux-arm-kernel <linux-arm-kernel@lists.infradead.org>, devicetree <devicetree@vger.kernel.org>, linux-kernel <linux-kernel@vger.kernel.org>, linux-sunxi <linux-sunxi@googlegroups.com> Subject: Re: [linux-sunxi] [PATCH 0/7] Add support for Allwinner H6 DVFS Date: Sun, 5 Apr 2020 17:01:51 +0200 [thread overview] Message-ID: <CAJiuCceROWv1ni3YZ6rU71O9R__t6ZOgnkZQDjBhJvhqHRAVng@mail.gmail.com> (raw) In-Reply-To: <20200405145443.uycsh73kxmurdmil@core.my.home> Hi, On Sun, 5 Apr 2020 at 16:54, Ondřej Jirman <megous@megous.com> wrote: > > Hi, > > On Sun, Apr 05, 2020 at 04:33:37PM +0200, Clément Péron wrote: > > Hi Ondřej, > > [ ... ] > > > Good point, this information should be added for both CPU and GPU regulator. > > This could be nice to confirm this point with a scope. > > > > Also I remark that Allwinner user higher temperature than what we set : > > alarm_low_temp = <105000>; > > alarm_high_temp = <110000>; > > alarm_temp_hysteresis = <15000>; > > shut_temp= <115000>; > > https://github.com/orangepi-xunlong/OrangePiH6_Linux4_9/blob/master/arch/arm64/boot/dts/sunxi/sun50iw6p1.dtsi#L1924 > > > > Don't you think that we can push a bit higher the temperature it's > > actually at 80°C ? > > We probably can. But just 5°C between high temp and criticla temp is probably > not enough. I'd probably set our value to something like 90-100°C so that quick > temperature spike before regulation kicks in would not shutdown the CPU. OK I will push it to 85°C and leave the critical to 100°C. Test it again and will send a v2 Regards, Clément > > regards, > o. > > > Thanks for the review, > > Clement > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-04-05 15:02 UTC|newest] Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-04-05 10:49 [PATCH 0/7] Add support for Allwinner H6 DVFS Clément Péron 2020-04-05 10:49 ` Clément Péron 2020-04-05 10:49 ` [PATCH 1/7] arm64: dts: allwinner: h6: Add clock to CPU cores Clément Péron 2020-04-05 10:49 ` Clément Péron 2020-04-05 10:49 ` [PATCH 2/7] arm64: dts: allwinner: h6: Add thermal trip points/cooling map Clément Péron 2020-04-05 10:49 ` Clément Péron 2020-04-05 10:49 ` [PATCH 3/7] arm64: dts: allwinner: h6: set thermal polling time Clément Péron 2020-04-05 10:49 ` Clément Péron 2020-04-05 11:24 ` [linux-sunxi] " Ondřej Jirman 2020-04-05 11:24 ` Ondřej Jirman 2020-04-05 14:35 ` Clément Péron 2020-04-05 14:35 ` Clément Péron 2020-04-05 10:49 ` [PATCH 4/7] arm64: dts: allwinner: h6: Add CPU Operating Performance Points table Clément Péron 2020-04-05 10:49 ` Clément Péron 2020-04-05 10:49 ` [PATCH 5/7] arm64: dts: allwinner: h6: Add GPU " Clément Péron 2020-04-05 10:49 ` Clément Péron 2020-04-05 10:49 ` [PATCH 6/7] arm64: configs: Enable sun50i cpufreq nvmem Clément Péron 2020-04-05 10:49 ` Clément Péron 2020-04-05 10:49 ` [PATCH 7/7] arm64: dts: allwinner: h6: Enable CPU and GPU opp tables for Beelink GS1 Clément Péron 2020-04-05 10:49 ` Clément Péron 2020-04-05 11:31 ` [linux-sunxi] " Ondřej Jirman 2020-04-05 11:31 ` Ondřej Jirman 2020-04-05 11:51 ` [linux-sunxi] [PATCH 0/7] Add support for Allwinner H6 DVFS Ondřej Jirman 2020-04-05 11:51 ` Ondřej Jirman 2020-04-05 14:33 ` Clément Péron 2020-04-05 14:33 ` Clément Péron 2020-04-05 14:54 ` Ondřej Jirman 2020-04-05 14:54 ` Ondřej Jirman 2020-04-05 15:01 ` Clément Péron [this message] 2020-04-05 15:01 ` Clément Péron 2020-04-05 15:30 ` Clément Péron 2020-04-05 15:30 ` Clément Péron
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=CAJiuCceROWv1ni3YZ6rU71O9R__t6ZOgnkZQDjBhJvhqHRAVng@mail.gmail.com \ --to=peron.clem@gmail.com \ --cc=devicetree@vger.kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-sunxi@googlegroups.com \ --cc=megous@megous.com \ --cc=mripard@kernel.org \ --cc=robh+dt@kernel.org \ --cc=wens@csie.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.