2017-08-21 09:08:16

by Andy Yan

[permalink] [raw]
Subject: [PATCH 1/2] ARM: dts: rockchip: add cpu opp table for rv1108

Add cpu opp table for rv1108 to support frequency
from 408MHZ to 1008MHZ.

Signed-off-by: Andy Yan <[email protected]>
---

arch/arm/boot/dts/rv1108-evb.dts | 4 ++++
arch/arm/boot/dts/rv1108.dtsi | 9 +++++++++
2 files changed, 13 insertions(+)

diff --git a/arch/arm/boot/dts/rv1108-evb.dts b/arch/arm/boot/dts/rv1108-evb.dts
index 32fb09a..e517aa3 100644
--- a/arch/arm/boot/dts/rv1108-evb.dts
+++ b/arch/arm/boot/dts/rv1108-evb.dts
@@ -104,6 +104,10 @@
};
};

+&cpu0 {
+ cpu-supply = <&vdd_core>;
+};
+
&i2c0 {
status = "okay";
i2c-scl-rising-time-ns = <275>;
diff --git a/arch/arm/boot/dts/rv1108.dtsi b/arch/arm/boot/dts/rv1108.dtsi
index 064cec3..6ebf07c 100644
--- a/arch/arm/boot/dts/rv1108.dtsi
+++ b/arch/arm/boot/dts/rv1108.dtsi
@@ -69,6 +69,15 @@
device_type = "cpu";
compatible = "arm,cortex-a7";
reg = <0xf00>;
+ operating-points = <
+ /* KHz uV */
+ 408000 975000
+ 600000 975000
+ 816000 1025000
+ 1008000 1150000
+ >;
+ clock-latency = <40000>;
+ clocks = <&cru ARMCLK>;
};
};

--
2.7.4



2017-08-21 09:10:40

by Andy Yan

[permalink] [raw]
Subject: [PATCH 2/2] cpufreq: dt: add support for rockchip rv1108

Add the compatible string to support the generic device tree
cpufreq-dt driver on rockchip rv1108 soc.

Signed-off-by: Andy Yan <[email protected]>
---

drivers/cpufreq/cpufreq-dt-platdev.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c b/drivers/cpufreq/cpufreq-dt-platdev.c
index 1c26292..2d71c09 100644
--- a/drivers/cpufreq/cpufreq-dt-platdev.c
+++ b/drivers/cpufreq/cpufreq-dt-platdev.c
@@ -69,6 +69,7 @@ static const struct of_device_id machines[] __initconst = {
{ .compatible = "renesas,r8a7794", },
{ .compatible = "renesas,sh73a0", },

+ { .compatible = "rockchip,rv1108", },
{ .compatible = "rockchip,rk2928", },
{ .compatible = "rockchip,rk3036", },
{ .compatible = "rockchip,rk3066a", },
--
2.7.4


2017-08-21 15:30:26

by Heiko Stübner

[permalink] [raw]
Subject: Re: [PATCH 1/2] ARM: dts: rockchip: add cpu opp table for rv1108

Hi Andy,

Am Montag, 21. August 2017, 17:06:44 CEST schrieb Andy Yan:
> Add cpu opp table for rv1108 to support frequency
> from 408MHZ to 1008MHZ.
>
> Signed-off-by: Andy Yan <[email protected]>
> ---
>
> arch/arm/boot/dts/rv1108-evb.dts | 4 ++++
> arch/arm/boot/dts/rv1108.dtsi | 9 +++++++++
> 2 files changed, 13 insertions(+)
>
> diff --git a/arch/arm/boot/dts/rv1108-evb.dts b/arch/arm/boot/dts/rv1108-evb.dts
> index 32fb09a..e517aa3 100644
> --- a/arch/arm/boot/dts/rv1108-evb.dts
> +++ b/arch/arm/boot/dts/rv1108-evb.dts
> @@ -104,6 +104,10 @@
> };
> };
>
> +&cpu0 {
> + cpu-supply = <&vdd_core>;
> +};
> +

please split into two patches (operating points + cpu-supply)
and use the operating-points-v2 binding.


Thanks
Heiko

2017-08-22 22:54:24

by Heiko Stübner

[permalink] [raw]
Subject: Re: [PATCH 2/2] cpufreq: dt: add support for rockchip rv1108

Am Montag, 21. August 2017, 17:10:24 CEST schrieb Andy Yan:
> Add the compatible string to support the generic device tree
> cpufreq-dt driver on rockchip rv1108 soc.
>
> Signed-off-by: Andy Yan <[email protected]>

Reviewed-by: Heiko Stuebner <[email protected]>

2017-09-13 19:18:43

by Viresh Kumar

[permalink] [raw]
Subject: Re: [PATCH 2/2] cpufreq: dt: add support for rockchip rv1108

On 21-08-17, 17:10, Andy Yan wrote:
> Add the compatible string to support the generic device tree
> cpufreq-dt driver on rockchip rv1108 soc.
>
> Signed-off-by: Andy Yan <[email protected]>
> ---
>
> drivers/cpufreq/cpufreq-dt-platdev.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c b/drivers/cpufreq/cpufreq-dt-platdev.c
> index 1c26292..2d71c09 100644
> --- a/drivers/cpufreq/cpufreq-dt-platdev.c
> +++ b/drivers/cpufreq/cpufreq-dt-platdev.c
> @@ -69,6 +69,7 @@ static const struct of_device_id machines[] __initconst = {
> { .compatible = "renesas,r8a7794", },
> { .compatible = "renesas,sh73a0", },
>
> + { .compatible = "rockchip,rv1108", },
> { .compatible = "rockchip,rk2928", },
> { .compatible = "rockchip,rk3036", },
> { .compatible = "rockchip,rk3066a", },

You use operating-points-v2 or the V1 bindings for this SoC? The SoCs using
operating-points-v2 don't need such changes anymore.

--
viresh

2017-09-14 00:53:02

by Andy Yan

[permalink] [raw]
Subject: Re: [PATCH 2/2] cpufreq: dt: add support for rockchip rv1108

Hi Viresh:


On 2017年09月14日 03:18, Viresh Kumar wrote:
> On 21-08-17, 17:10, Andy Yan wrote:
>> Add the compatible string to support the generic device tree
>> cpufreq-dt driver on rockchip rv1108 soc.
>>
>> Signed-off-by: Andy Yan <[email protected]>
>> ---
>>
>> drivers/cpufreq/cpufreq-dt-platdev.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c b/drivers/cpufreq/cpufreq-dt-platdev.c
>> index 1c26292..2d71c09 100644
>> --- a/drivers/cpufreq/cpufreq-dt-platdev.c
>> +++ b/drivers/cpufreq/cpufreq-dt-platdev.c
>> @@ -69,6 +69,7 @@ static const struct of_device_id machines[] __initconst = {
>> { .compatible = "renesas,r8a7794", },
>> { .compatible = "renesas,sh73a0", },
>>
>> + { .compatible = "rockchip,rv1108", },
>> { .compatible = "rockchip,rk2928", },
>> { .compatible = "rockchip,rk3036", },
>> { .compatible = "rockchip,rk3066a", },
> You use operating-points-v2 or the V1 bindings for this SoC? The SoCs using
> operating-points-v2 don't need such changes anymore.

Yes, I use operating-points-v2.
>


2017-09-19 18:49:03

by Viresh Kumar

[permalink] [raw]
Subject: Re: [PATCH 2/2] cpufreq: dt: add support for rockchip rv1108

On 14-09-17, 08:52, Andy Yan wrote:
> Yes, I use operating-points-v2.

Did you check with current mainline? It should work without this patches.

--
viresh