diff mbox

[2/2] cpufreq: dt: add support for rockchip rv1108

Message ID 1503306624-28937-1-git-send-email-andy.yan@rock-chips.com (mailing list archive)
State Superseded, archived
Headers show

Commit Message

Andy Yan Aug. 21, 2017, 9:10 a.m. UTC
Add the compatible string to support the generic device tree
cpufreq-dt driver on rockchip rv1108 soc.

Signed-off-by: Andy Yan <andy.yan@rock-chips.com>
---

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

Comments

Heiko Stuebner Aug. 22, 2017, 10:54 p.m. UTC | #1
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 <andy.yan@rock-chips.com>

Reviewed-by: Heiko Stuebner <heiko@sntech.de>
Viresh Kumar Sept. 13, 2017, 7:18 p.m. UTC | #2
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 <andy.yan@rock-chips.com>
> ---
> 
>  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.
Andy Yan Sept. 14, 2017, 12:52 a.m. UTC | #3
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 <andy.yan@rock-chips.com>
>> ---
>>
>>   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.
>
Viresh Kumar Sept. 19, 2017, 6:48 p.m. UTC | #4
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.
diff mbox

Patch

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", },