cpufreq: ti: Use generic platdev driver

PATCH New Dave Gerlach 3 participants 6 messages Sep 14, 2016 – Sep 19, 2016 CPU frequency scaling framework

Share & tools

Dave Gerlach

author

2016-09-14 20:41 UTC #0 lore raw

PATCH 1/1 cpufreq: ti: Use generic platdev driver

Now that the cpufreq-dt-platdev is used to create the cpufreq-dt platform device for all OMAP platforms and the platform code that did it before has been removed, add ti,am33xx and ti,dra7xx to the machine list in cpufreq-dt-platdev which had relied on the removed platform code to do this previously.

Fixes 7694ca6e1d6f ("cpufreq: omap: Use generic platdev driver")

Signed-off-by Dave Gerlach <[email protected]>

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

drivers/cpufreq/cpufreq-dt-platdev.c +2 -0

index 0bb44d5b5df4..d5e8ba50f8a3 100644
@@ -68,6 +68,8 @@ static const struct of_device_id machines[] __initconst = {
 
 	{ .compatible = "sigma,tango4" },
 
+	{ .compatible = "ti,am33xx", },
+	{ .compatible = "ti,dra7", },
 	{ .compatible = "ti,omap2", },
 	{ .compatible = "ti,omap3", },
 	{ .compatible = "ti,omap4", },

Signature · 1 line

2.9.3

Dave Gerlach

author

2016-09-14 21:36 UTC #1 lore raw

PATCH 1/1 cpufreq: ti: Use generic platdev driver

Quoted 9 lines · Dave Gerlach

Now that the cpufreq-dt-platdev is used to create the cpufreq-dt platform device for all OMAP platforms and the platform code that did it before has been removed, add ti,am33xx and ti,dra7xx to the machine list in cpufreq-dt-platdev which had relied on the removed platform code to do this previously.
Fixes 7694ca6e1d6f ("cpufreq: omap: Use generic platdev driver")
Signed-off-by Dave Gerlach <[email protected]>
---

This got left out when OMAP was originally added to the cpufreq-dt-platdev, so cpufreq disappeared on am335x and dra7 platforms. This patch together with the series here [1] will fix this regression.

Regards,
Dave

[1] http://marc.info/?l=linux-pm&m=147388667613317&w=2

Quoted 17 lines

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

drivers/cpufreq/cpufreq-dt-platdev.c +2 -0

index 0bb44d5b5df4..d5e8ba50f8a3 100644
@@ -68,6 +68,8 @@ static const struct of_device_id machines[] __initconst = {
 
  	{ .compatible = "sigma,tango4" },
 
+	{ .compatible = "ti,am33xx", },
+	{ .compatible = "ti,dra7", },
  	{ .compatible = "ti,omap2", },
  	{ .compatible = "ti,omap3", },
  	{ .compatible = "ti,omap4", },

Viresh Kumar

maintainer

2016-09-15 06:25 UTC #2 lore raw

PATCH 1/1 cpufreq: ti: Use generic platdev driver

In reply to Dave Gerlach

Quoted 8 lines · Dave Gerlach

Now that the cpufreq-dt-platdev is used to create the cpufreq-dt platform device for all OMAP platforms and the platform code that did it before has been removed, add ti,am33xx and ti,dra7xx to the machine list in cpufreq-dt-platdev which had relied on the removed platform code to do this previously.
Fixes 7694ca6e1d6f ("cpufreq: omap: Use generic platdev driver")
Signed-off-by Dave Gerlach <[email protected]>

Any stable trees you want this to be added to ?

Quoted 9 lines

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

drivers/cpufreq/cpufreq-dt-platdev.c +0 -0

index 0bb44d5b5df4..d5e8ba50f8a3 100644

drivers/cpufreq/cpufreq-dt-platdev.c static const struct of_device_id machines[] __initconst = { · line 75

Quoted 8 lines

drivers/cpufreq/cpufreq-dt-platdev.c +2 -0

@@ -68,6 +68,8 @@ static const struct of_device_id machines[] __initconst = {
 
 	{ .compatible = "sigma,tango4" },
 
+	{ .compatible = "ti,am33xx", },
+	{ .compatible = "ti,dra7", },
 	{ .compatible = "ti,omap2", },
 	{ .compatible = "ti,omap3", },
 	{ .compatible = "ti,omap4", },

Acked-by Viresh Kumar <[email protected]>

Signature · 1 line

viresh

Dave Gerlach

author

2016-09-16 21:29 UTC #3 lore raw

PATCH 1/1 cpufreq: ti: Use generic platdev driver

Hi,

Quoted 11 lines · Viresh Kumar

Quoted 8 lines · Dave Gerlach
Now that the cpufreq-dt-platdev is used to create the cpufreq-dt platform device for all OMAP platforms and the platform code that did it before has been removed, add ti,am33xx and ti,dra7xx to the machine list in cpufreq-dt-platdev which had relied on the removed platform code to do this previously.
Fixes 7694ca6e1d6f ("cpufreq: omap: Use generic platdev driver")
Signed-off-by Dave Gerlach <[email protected]>
Any stable trees you want this to be added to ?

I suppose it'll go for 4.7-stable as the change happened in v4.6.

Quoted 13 lines

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

drivers/cpufreq/cpufreq-dt-platdev.c +0 -0

index 0bb44d5b5df4..d5e8ba50f8a3 100644
@@ -68,6 +68,8 @@ static const struct of_device_id machines[] __initconst = {
 
  	{ .compatible = "sigma,tango4" },

Quoted 8 lines

drivers/cpufreq/cpufreq-dt-platdev.c static const struct of_device_id machines[] __initconst = { · line 75
Quoted 5 lines
drivers/cpufreq/cpufreq-dt-platdev.c +2 -0
+	{ .compatible = "ti,am33xx", },
+	{ .compatible = "ti,dra7", },
  	{ .compatible = "ti,omap2", },
  	{ .compatible = "ti,omap3", },
  	{ .compatible = "ti,omap4", },
Acked-by Viresh Kumar <[email protected]>

Thanks,
Dave

Rafael J. Wysocki

2016-09-16 21:53 UTC #4 lore raw

PATCH 1/1 cpufreq: ti: Use generic platdev driver

Quoted 9 lines · Dave Gerlach

Hi,
Quoted 7 lines · Viresh Kumar
Quoted 6 lines · Dave Gerlach
Now that the cpufreq-dt-platdev is used to create the cpufreq-dt platform device for all OMAP platforms and the platform code that did it before has been removed, add ti,am33xx and ti,dra7xx to the machine list in cpufreq-dt-platdev which had relied on the removed platform code to do this previously.

Quoted 6 lines

Quoted 4 lines
Quoted 2 lines
Fixes 7694ca6e1d6f ("cpufreq: omap: Use generic platdev driver")
Signed-off-by Dave Gerlach <[email protected]>
Any stable trees you want this to be added to ?
I suppose it'll go for 4.7-stable as the change happened in v4.6.

The commit in the fixes tag went in during the 4.7 cycle, so the fix should go into 4.7.y, but I'm not going to push it as urgent material for 4.8.

Thanks,
Rafael

Dave Gerlach

author

2016-09-19 20:41 UTC #5 lore raw

PATCH 1/1 cpufreq: ti: Use generic platdev driver

Quoted 13 lines · Rafael J. Wysocki

Quoted 12 lines · Dave Gerlach
Hi,
Quoted 10 lines · Viresh Kumar
Quoted 8 lines · Dave Gerlach
Now that the cpufreq-dt-platdev is used to create the cpufreq-dt platform device for all OMAP platforms and the platform code that did it before has been removed, add ti,am33xx and ti,dra7xx to the machine list in cpufreq-dt-platdev which had relied on the removed platform code to do this previously.
Fixes 7694ca6e1d6f ("cpufreq: omap: Use generic platdev driver")
Signed-off-by Dave Gerlach <[email protected]>

Quoted 7 lines

Quoted 3 lines
Quoted 1 line
Any stable trees you want this to be added to ?
I suppose it'll go for 4.7-stable as the change happened in v4.6.
The commit in the fixes tag went in during the 4.7 cycle, so the fix should go into 4.7.y, but I'm not going to push it as urgent material for 4.8.

Yes, that is fine, thanks a lot!

Regards,
Dave

Quoted 3 lines

Thanks,
Rafael