Re: [PATCH RFC 8/8] ARM: dts: exynos: Add ASV tables for exynos5422/5800
Hi Anand,
I'm using this frequencies on the board since 3 years, without any stability
issues. The kind of worklaod used is intensive CPU-bound benchmarks running
for 20 mins at least in order to stress the system, I'm studying schedulability
under high thermal constraints.
So, I would say these frequencies are safe by experience.
Best Regards,
Willy
On Thu, Apr 11, 2019 at 01:09:04PM +0530, Anand Moon wrote:
> Hi Sylwester,
>
> As per my knowledge HK soc introduce this table to support
> overclocking of cpufreq for Odroid XU3 XU4 family for boards.
>
> ARM Cortex-A15 Quad CPU (Eagle) 1800 to 2000
> ARM Cortex-A7 Quard CPU (Kingfisher) 1400 to 1500
>
> For Exynos5422 below table
> asv-table-[0-3] AVS range from 200 to 2100
> asv-table-[100-103] AVS range from 200 to 1500.
>
> Are these setting stable for this board to cover clock the cpu now.
>
> Can you also enable support for this clk-exynos5420.c
> For example exynos5800_eglclk_d and exynos5420_kfcclk_d table.
>
> Also can you update exynos5420.dtsi following opp table
> For example cluster_a15_opp_table and cluster_a7_opp_table opt binding.
>
> To make this work synchronously with AVS.
>
> Best Regards
> -Anand
>
> On Thu, 4 Apr 2019 at 22:55, Sylwester Nawrocki
> wrote:
> >
> > This patch ASV (Adaptive Supply Voltage) table entries for
> > Exynos5422/5800 SoC.
> >
> > Signed-off-by: Sylwester Nawrocki
> > ---
> > arch/arm/boot/dts/exynos5.dtsi| 2 +-
> > arch/arm/boot/dts/exynos5800.dtsi | 207 ++
> > 2 files changed, 208 insertions(+), 1 deletion(-)
> >
> > diff --git a/arch/arm/boot/dts/exynos5.dtsi b/arch/arm/boot/dts/exynos5.dtsi
> > index 67f9b4504a42..22eb951c614c 100644
> > --- a/arch/arm/boot/dts/exynos5.dtsi
> > +++ b/arch/arm/boot/dts/exynos5.dtsi
> > @@ -35,7 +35,7 @@
> > #size-cells = <1>;
> > ranges;
> >
> > - chipid@1000 {
> > + chipid: chipid@1000 {
> > compatible = "samsung,exynos4210-chipid";
> > reg = <0x1000 0x100>;
> > };
> > diff --git a/arch/arm/boot/dts/exynos5800.dtsi
> > b/arch/arm/boot/dts/exynos5800.dtsi
> > index 57d3b319fd65..5358865f5c0b 100644
> > --- a/arch/arm/boot/dts/exynos5800.dtsi
> > +++ b/arch/arm/boot/dts/exynos5800.dtsi
> > @@ -16,6 +16,213 @@
> > compatible = "samsung,exynos5800", "samsung,exynos5";
> > };
> >
> > + {
> > + asv {
> > + compatible = "samsung,exynos-asv-v1";
> > + asv-table-0 {
> > + /* ARM 0, 1 */
> > + samsung,asv-table-size = <20 15>;
> > + samsung,asv-table-id = <0x0>;
> > + samsung,asv-data =
> > + /* ASV0 ASV1ASV2ASV3ASV4
> >ASV5ASV6ASV7ASV8ASV9ASV10 ASV11 ASV12 ASV13 */
> > + <2100 1362500 1362500 135 1337500
> > 1325000 1312500 130 1275000 1262500 125 1237500 1225000 1212500
> > 120>,
> > + <2000 1312500 1312500 130 1287500
> > 1275000 1262500 125 1237500 1225000 1237500 1225000 1212500 120
> > 1187500>,
> > + <1900 125 1237500 1225000 1212500
> > 120 1187500 1175000 1162500 115 1162500 115 1137500 1125000
> > 1112500>,
> > + <1800 120 1187500 1175000 1162500
> > 115 1137500 1125000 1112500 110 1112500 110 1087500 1075000
> > 1062500>,
> > + <1700 1162500 115 1137500 1125000
> > 1112500 110 1087500 1075000 1062500 1075000 1062500 105 1037500
> > 1025000>,
> > + <1600 1125000 1112500 110 1087500
> > 1075000 1062500 105 1037500 1025000 1037500 1025000 1012500 100
> > 987500>,
> > + <1500 1087500 1075000 1062500 105
> > 1037500 1025000 1012500 100 987500 100 987500 975000 962500
> > 95>,
> > + <1400 1062500 105 1037500 1025000
> > 1012500 100 987500 975000 962500 975000 962500 95 937500
> > 925000>,
> > + <1300 105 1037500 1025000 1012500
> > 100 987500 975000 962500 95 962500 95 937500 925000
> > 912500>,
> > + <1200 1025000 1012500 100 987500
> > 975000 962500 95 937500 925000 937500 925000 912500 90
> > 90>,
> > + <1100 100 987500 975000 962500
> > 95 937500 925000 912500 90 90 90 90 90
> > 90>,
> > + <1000 975000 962500 95 937500
> > 925000 912500 90 90 90 90 90 90 90
> > 90>,
> > + < 900 95 937500 925000
Re: [PATCH RFC 8/8] ARM: dts: exynos: Add ASV tables for exynos5422/5800
Hi Sylwester,
As per my knowledge HK soc introduce this table to support
overclocking of cpufreq for Odroid XU3 XU4 family for boards.
ARM Cortex-A15 Quad CPU (Eagle) 1800 to 2000
ARM Cortex-A7 Quard CPU (Kingfisher) 1400 to 1500
For Exynos5422 below table
asv-table-[0-3] AVS range from 200 to 2100
asv-table-[100-103] AVS range from 200 to 1500.
Are these setting stable for this board to cover clock the cpu now.
Can you also enable support for this clk-exynos5420.c
For example exynos5800_eglclk_d and exynos5420_kfcclk_d table.
Also can you update exynos5420.dtsi following opp table
For example cluster_a15_opp_table and cluster_a7_opp_table opt binding.
To make this work synchronously with AVS.
Best Regards
-Anand
On Thu, 4 Apr 2019 at 22:55, Sylwester Nawrocki wrote:
>
> This patch ASV (Adaptive Supply Voltage) table entries for
> Exynos5422/5800 SoC.
>
> Signed-off-by: Sylwester Nawrocki
> ---
> arch/arm/boot/dts/exynos5.dtsi| 2 +-
> arch/arm/boot/dts/exynos5800.dtsi | 207 ++
> 2 files changed, 208 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/exynos5.dtsi b/arch/arm/boot/dts/exynos5.dtsi
> index 67f9b4504a42..22eb951c614c 100644
> --- a/arch/arm/boot/dts/exynos5.dtsi
> +++ b/arch/arm/boot/dts/exynos5.dtsi
> @@ -35,7 +35,7 @@
> #size-cells = <1>;
> ranges;
>
> - chipid@1000 {
> + chipid: chipid@1000 {
> compatible = "samsung,exynos4210-chipid";
> reg = <0x1000 0x100>;
> };
> diff --git a/arch/arm/boot/dts/exynos5800.dtsi
> b/arch/arm/boot/dts/exynos5800.dtsi
> index 57d3b319fd65..5358865f5c0b 100644
> --- a/arch/arm/boot/dts/exynos5800.dtsi
> +++ b/arch/arm/boot/dts/exynos5800.dtsi
> @@ -16,6 +16,213 @@
> compatible = "samsung,exynos5800", "samsung,exynos5";
> };
>
> + {
> + asv {
> + compatible = "samsung,exynos-asv-v1";
> + asv-table-0 {
> + /* ARM 0, 1 */
> + samsung,asv-table-size = <20 15>;
> + samsung,asv-table-id = <0x0>;
> + samsung,asv-data =
> + /* ASV0 ASV1ASV2ASV3ASV4
> ASV5ASV6ASV7ASV8ASV9ASV10 ASV11 ASV12 ASV13 */
> + <2100 1362500 1362500 135 1337500 1325000
> 1312500 130 1275000 1262500 125 1237500 1225000 1212500 120>,
> + <2000 1312500 1312500 130 1287500 1275000
> 1262500 125 1237500 1225000 1237500 1225000 1212500 120 1187500>,
> + <1900 125 1237500 1225000 1212500 120
> 1187500 1175000 1162500 115 1162500 115 1137500 1125000 1112500>,
> + <1800 120 1187500 1175000 1162500 115
> 1137500 1125000 1112500 110 1112500 110 1087500 1075000 1062500>,
> + <1700 1162500 115 1137500 1125000 1112500
> 110 1087500 1075000 1062500 1075000 1062500 105 1037500 1025000>,
> + <1600 1125000 1112500 110 1087500 1075000
> 1062500 105 1037500 1025000 1037500 1025000 1012500 100 987500>,
> + <1500 1087500 1075000 1062500 105 1037500
> 1025000 1012500 100 987500 100 987500 975000 962500 95>,
> + <1400 1062500 105 1037500 1025000 1012500
> 100 987500 975000 962500 975000 962500 95 937500 925000>,
> + <1300 105 1037500 1025000 1012500 100
> 987500 975000 962500 95 962500 95 937500 925000 912500>,
> + <1200 1025000 1012500 100 987500 975000
> 962500 95 937500 925000 937500 925000 912500 90 90>,
> + <1100 100 987500 975000 962500 95
> 937500 925000 912500 90 90 90 90 90 90>,
> + <1000 975000 962500 95 937500 925000
> 912500 90 90 90 90 90 90 90 90>,
> + < 900 95 937500 925000 912500 90
> 90 90 90 90 90 90 90 90 90>,
> + < 800 925000 912500 90 90 90
> 90 90 90 90 90 90 90 90 90>;
> +/* ASV0...13 */
> + samsung,asv-common-data = <700 90>,
> + <600 90>,
> + <500 90>,
> + <400 90>,
> + <300 90>,
> +
[PATCH RFC 8/8] ARM: dts: exynos: Add ASV tables for exynos5422/5800
This patch ASV (Adaptive Supply Voltage) table entries for
Exynos5422/5800 SoC.
Signed-off-by: Sylwester Nawrocki
---
arch/arm/boot/dts/exynos5.dtsi| 2 +-
arch/arm/boot/dts/exynos5800.dtsi | 207 ++
2 files changed, 208 insertions(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/exynos5.dtsi b/arch/arm/boot/dts/exynos5.dtsi
index 67f9b4504a42..22eb951c614c 100644
--- a/arch/arm/boot/dts/exynos5.dtsi
+++ b/arch/arm/boot/dts/exynos5.dtsi
@@ -35,7 +35,7 @@
#size-cells = <1>;
ranges;
- chipid@1000 {
+ chipid: chipid@1000 {
compatible = "samsung,exynos4210-chipid";
reg = <0x1000 0x100>;
};
diff --git a/arch/arm/boot/dts/exynos5800.dtsi
b/arch/arm/boot/dts/exynos5800.dtsi
index 57d3b319fd65..5358865f5c0b 100644
--- a/arch/arm/boot/dts/exynos5800.dtsi
+++ b/arch/arm/boot/dts/exynos5800.dtsi
@@ -16,6 +16,213 @@
compatible = "samsung,exynos5800", "samsung,exynos5";
};
+ {
+ asv {
+ compatible = "samsung,exynos-asv-v1";
+ asv-table-0 {
+ /* ARM 0, 1 */
+ samsung,asv-table-size = <20 15>;
+ samsung,asv-table-id = <0x0>;
+ samsung,asv-data =
+ /* ASV0 ASV1ASV2ASV3ASV4
ASV5ASV6ASV7ASV8ASV9ASV10 ASV11 ASV12 ASV13 */
+ <2100 1362500 1362500 135 1337500 1325000
1312500 130 1275000 1262500 125 1237500 1225000 1212500 120>,
+ <2000 1312500 1312500 130 1287500 1275000
1262500 125 1237500 1225000 1237500 1225000 1212500 120 1187500>,
+ <1900 125 1237500 1225000 1212500 120
1187500 1175000 1162500 115 1162500 115 1137500 1125000 1112500>,
+ <1800 120 1187500 1175000 1162500 115
1137500 1125000 1112500 110 1112500 110 1087500 1075000 1062500>,
+ <1700 1162500 115 1137500 1125000 1112500
110 1087500 1075000 1062500 1075000 1062500 105 1037500 1025000>,
+ <1600 1125000 1112500 110 1087500 1075000
1062500 105 1037500 1025000 1037500 1025000 1012500 100 987500>,
+ <1500 1087500 1075000 1062500 105 1037500
1025000 1012500 100 987500 100 987500 975000 962500 95>,
+ <1400 1062500 105 1037500 1025000 1012500
100 987500 975000 962500 975000 962500 95 937500 925000>,
+ <1300 105 1037500 1025000 1012500 100
987500 975000 962500 95 962500 95 937500 925000 912500>,
+ <1200 1025000 1012500 100 987500 975000
962500 95 937500 925000 937500 925000 912500 90 90>,
+ <1100 100 987500 975000 962500 95
937500 925000 912500 90 90 90 90 90 90>,
+ <1000 975000 962500 95 937500 925000
912500 90 90 90 90 90 90 90 90>,
+ < 900 95 937500 925000 912500 90
90 90 90 90 90 90 90 90 90>,
+ < 800 925000 912500 90 90 90
90 90 90 90 90 90 90 90 90>;
+/* ASV0...13 */
+ samsung,asv-common-data = <700 90>,
+ <600 90>,
+ <500 90>,
+ <400 90>,
+ <300 90>,
+ <200 90>;
+ };
+ asv-table-2 {
+ /* ARM 2 */
+ samsung,asv-table-size = <20 15>;
+ samsung,asv-table-id = <0x2>;
+ samsung,asv-data =
+ /* ASV0 ASV1ASV2ASV3ASV4
ASV5ASV6ASV7ASV8ASV9ASV10 ASV11 ASV12 ASV13 */
+ <2100 1362500 1362500 135 1337500 1325000
1312500 130 1275000 1262500 125 1237500 1225000 1212500 120>,
+ <2000 1312500 1312500 1312500 130 1275000
1262500 125 1237500 1225000 1237500 1225000 1212500 120 1187500>,
+ <1900 1262500 125 125 1237500 1212500
120 1187500 1175000 1162500 1175000 1162500 115 1137500 1125000>,
+ <1800
