Re: [linux-sunxi] [PATCH v3 30/30] ARM: sun8i: a83t: full range OPP tables and CPUfreq

Ondřej Jirman
2018-08-30T16:38:38Z

Hello,

On Thu, Aug 30, 2018 at 05:45:18PM +0200, Philipp Rossak wrote:
> Since we have now thermal trotteling enabeled we can now add the full
> range of the OPP table.

I'm not sure we can. I have a tablet with A83T SoC and it gets unstable
at these frequencies even with thermal throttling on mainline kernel. (Though
I have my own THS driver, but I doubt a different driver will change much.)

There might be some other issue left in the cpufreq code. I'll let others
test this on a better cooled boards though.

Did you/someone test this?

regards,
o.

> The operating points were found in Allwinner BSP and fex files.
>
> Signed-off-by: Philipp Rossak <embed3d@gmail.com>
> ---
> arch/arm/boot/dts/sun8i-a83t.dtsi | 32 ++++++++++++++++++++++++++++++++
> 1 file changed, 32 insertions(+)
>
> diff --git a/arch/arm/boot/dts/sun8i-a83t.dtsi b/arch/arm/boot/dts/sun8i-a83t.dtsi
> index 78aa448e869f..ddcf404f9c80 100644
> --- a/arch/arm/boot/dts/sun8i-a83t.dtsi
> +++ b/arch/arm/boot/dts/sun8i-a83t.dtsi
> @@ -250,6 +250,22 @@
> opp-microvolt = <840000>;
> clock-latency-ns = <244144>; /* 8 32k periods */
> };
> +
> + opp-1608000000 {
> + opp-hz = /bits/ 64 <1608000000>;
> + opp-microvolt = <920000>;
> + clock-latency-ns = <244144>; /* 8 32k periods */
> + };
> + opp-1800000000 { /* BOOT FREQ */
> + opp-hz = /bits/ 64 <1800000000>;
> + opp-microvolt = <1000000>;
> + clock-latency-ns = <244144>; /* 8 32k periods */
> + };
> + opp-2016000000 {
> + opp-hz = /bits/ 64 <2016000000>;
> + opp-microvolt = <1080000>;
> + clock-latency-ns = <244144>; /* 8 32k periods */
> + };
> };
>
> cpu1_opp_table: opp_table1 {
> @@ -303,6 +319,22 @@
> opp-microvolt = <840000>;
> clock-latency-ns = <244144>; /* 8 32k periods */
> };
> +
> + opp-1608000000 {
> + opp-hz = /bits/ 64 <1608000000>;
> + opp-microvolt = <920000>;
> + clock-latency-ns = <244144>; /* 8 32k periods */
> + };
> + opp-1800000000 { /* BOOT FREQ */
> + opp-hz = /bits/ 64 <1800000000>;
> + opp-microvolt = <1000000>;
> + clock-latency-ns = <244144>; /* 8 32k periods */
> + };
> + opp-2016000000 {
> + opp-hz = /bits/ 64 <2016000000>;
> + opp-microvolt = <1080000>;
> + clock-latency-ns = <244144>; /* 8 32k periods */
> + };
> };
>
> soc {
> --
> 2.11.0
>
> --
> You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.