[PATCH v2 0/3] ARM: dts: Add operating-points-v2 tables for TI platforms

[PATCH v2 0/3] ARM: dts: Add operating-points-v2 tables for TI platforms

[Dave Gerlach]

Hi,
This is v2 of the series to add operating-points-v2 tables for am335x,
am437x, dra7, and am57x platforms, but really just updates the already
merged v1 found here [1].

It seems v1 was merged without the binding being finalized so this series
just updates the nodes that are already present in order to work with the
new bindings coming in v2 of the ti-cpufreq driver series which will follow
shortly.

Regards,
Dave

[1] http://lists.infradead.org/pipermail/linux-arm-kernel/2016-May/430205.html

Dave Gerlach (3):
  ARM: dts: am33xx: Change operating-points-v2 table to am3352 version
  ARM: dts: am4372: Change operating-points-v2 table to am4372 version
  ARM: dts: dra7: Change operating-points-v2 table to dra7 version

 arch/arm/boot/dts/am33xx.dtsi | 2 +-
 arch/arm/boot/dts/am4372.dtsi | 2 +-
 arch/arm/boot/dts/dra7.dtsi   | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

-- 
2.9.0

[PATCH v2 1/3] ARM: dts: am33xx: Change operating-points-v2 table to am3352 version

[Dave Gerlach]

The operating-points-v2 table got merged before the binding had
stabilized so update the compatible string to am3352 version.

Signed-off-by: Dave Gerlach <d-gerlach@ti.com>
---
 arch/arm/boot/dts/am33xx.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
index 98748c61ed99..a07212914130 100644
--- a/arch/arm/boot/dts/am33xx.dtsi
+++ b/arch/arm/boot/dts/am33xx.dtsi
@@ -57,7 +57,7 @@
 	};
 
 	cpu0_opp_table: opp_table0 {
-		compatible = "operating-points-v2";
+		compatible = "operating-points-v2-ti-am3352-cpu";
 
 		/*
 		 * The three following nodes are marked with opp-suspend
-- 
2.9.0

[PATCH v2 2/3] ARM: dts: am4372: Change operating-points-v2 table to am4372 version

[Dave Gerlach]

The operating-points-v2 table got merged before the binding had
stabilized so update the compatible string to am4372 version.

Signed-off-by: Dave Gerlach <d-gerlach@ti.com>
---
 arch/arm/boot/dts/am4372.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.dtsi
index 0fadae5396e1..cdadcd793229 100644
--- a/arch/arm/boot/dts/am4372.dtsi
+++ b/arch/arm/boot/dts/am4372.dtsi
@@ -53,7 +53,7 @@
 	};
 
 	cpu0_opp_table: opp_table0 {
-		compatible = "operating-points-v2";
+		compatible = "operating-points-v2-ti-am4372-cpu";
 
 		opp50 at 300000000 {
 			opp-hz = /bits/ 64 <300000000>;
-- 
2.9.0

[PATCH v2 3/3] ARM: dts: dra7: Change operating-points-v2 table to dra7 version

[Dave Gerlach]

The operating-points-v2 table got merged before the binding had
stabilized so update the compatible string to dra7 version.

Signed-off-by: Dave Gerlach <d-gerlach@ti.com>
---
 arch/arm/boot/dts/dra7.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi
index d9bfb94a2992..c138eca85efd 100644
--- a/arch/arm/boot/dts/dra7.dtsi
+++ b/arch/arm/boot/dts/dra7.dtsi
@@ -99,7 +99,7 @@
 	};
 
 	cpu0_opp_table: opp_table0 {
-		compatible = "operating-points-v2";
+		compatible = "operating-points-v2-ti-dra7-cpu";
 		opp-shared;
 
 		opp_nom at 1000000000 {
-- 
2.9.0

[PATCH v2 0/3] ARM: dts: Add operating-points-v2 tables for TI platforms

[Tony Lindgren]

Hi,

* Dave Gerlach <d-gerlach@ti.com> [160831 19:49]:
> Hi,
> This is v2 of the series to add operating-points-v2 tables for am335x,
> am437x, dra7, and am57x platforms, but really just updates the already
> merged v1 found here [1].
> 
> It seems v1 was merged without the binding being finalized so this series
> just updates the nodes that are already present in order to work with the
> new bindings coming in v2 of the ti-cpufreq driver series which will follow
> shortly.

Assuming this will have to wait based on what you mentioned about
am335x cpufreq regression. I guess you'll instead send few patches
to revert?

Regards,

Tony

> [1] http://lists.infradead.org/pipermail/linux-arm-kernel/2016-May/430205.html
> 
> Dave Gerlach (3):
>   ARM: dts: am33xx: Change operating-points-v2 table to am3352 version
>   ARM: dts: am4372: Change operating-points-v2 table to am4372 version
>   ARM: dts: dra7: Change operating-points-v2 table to dra7 version
> 
>  arch/arm/boot/dts/am33xx.dtsi | 2 +-
>  arch/arm/boot/dts/am4372.dtsi | 2 +-
>  arch/arm/boot/dts/dra7.dtsi   | 2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)
> 
> -- 
> 2.9.0
> 

[PATCH v2 0/3] ARM: dts: Add operating-points-v2 tables for TI platforms

[Dave Gerlach]

Hi Tony,
On 09/13/2016 05:02 PM, Tony Lindgren wrote:
> Hi,
>
> * Dave Gerlach <d-gerlach@ti.com> [160831 19:49]:
>> Hi,
>> This is v2 of the series to add operating-points-v2 tables for am335x,
>> am437x, dra7, and am57x platforms, but really just updates the already
>> merged v1 found here [1].
>>
>> It seems v1 was merged without the binding being finalized so this series
>> just updates the nodes that are already present in order to work with the
>> new bindings coming in v2 of the ti-cpufreq driver series which will follow
>> shortly.
>
> Assuming this will have to wait based on what you mentioned about
> am335x cpufreq regression. I guess you'll instead send few patches
> to revert?
>

Correct, these should not be merged, I will be sending a fixup series 
until this series is ready.

I needed to resend the ti-cpufreq driver at [2] to get a response about 
the dt binding and other updates Viresh asked for and sent this to 
update the merged nodes so they would work with the driver. But the 
driver has not been merged yet, so in order to re-enable the previously 
existing cpufreq support on those platforms I must revert three patches 
from v1 [1] of this series that got merged, which I will send.

I also found that 7694ca6e1d6f ("cpufreq: omap: Use generic platdev 
driver") which went in v4.6-rc1 had already broken the existing cpufreq 
support on am335x and dra7xx, so it's been missing for some time. I can 
send a patch fixing that as well but I will need to remove it once the 
ti-cpufreq driver in [2] below gets merged as that driver must create 
the cpufreq-dt platform device instead of cpufreq-dt-platdev. So I will 
send a series that can fix cpufreq until ti-cpufreq is ready which will 
give us better support for more OPPs on am335x, am437x (which has no 
implementation today), and dra7xx.

Regards,
Dave

[1] 
http://lists.infradead.org/pipermail/linux-arm-kernel/2016-May/430205.html
[2] http://www.spinics.net/lists/devicetree/msg141152.html

> Regards,
>
> Tony
>
>> [1] http://lists.infradead.org/pipermail/linux-arm-kernel/2016-May/430205.html
>>
>> Dave Gerlach (3):
>>    ARM: dts: am33xx: Change operating-points-v2 table to am3352 version
>>    ARM: dts: am4372: Change operating-points-v2 table to am4372 version
>>    ARM: dts: dra7: Change operating-points-v2 table to dra7 version
>>
>>   arch/arm/boot/dts/am33xx.dtsi | 2 +-
>>   arch/arm/boot/dts/am4372.dtsi | 2 +-
>>   arch/arm/boot/dts/dra7.dtsi   | 2 +-
>>   3 files changed, 3 insertions(+), 3 deletions(-)
>>
>> --
>> 2.9.0
>>

[PATCH v2 0/3] ARM: dts: Add operating-points-v2 tables for TI platforms

[Tony Lindgren]

* Dave Gerlach <d-gerlach@ti.com> [160914 09:45]:
> On 09/13/2016 05:02 PM, Tony Lindgren wrote:
> > Assuming this will have to wait based on what you mentioned about
> > am335x cpufreq regression. I guess you'll instead send few patches
> > to revert?
> > 
> 
> Correct, these should not be merged, I will be sending a fixup series until
> this series is ready.

OK

> I needed to resend the ti-cpufreq driver at [2] to get a response about the
> dt binding and other updates Viresh asked for and sent this to update the
> merged nodes so they would work with the driver. But the driver has not been
> merged yet, so in order to re-enable the previously existing cpufreq support
> on those platforms I must revert three patches from v1 [1] of this series
> that got merged, which I will send.
> 
> I also found that 7694ca6e1d6f ("cpufreq: omap: Use generic platdev driver")
> which went in v4.6-rc1 had already broken the existing cpufreq support on
> am335x and dra7xx, so it's been missing for some time. I can send a patch
> fixing that as well but I will need to remove it once the ti-cpufreq driver
> in [2] below gets merged as that driver must create the cpufreq-dt platform
> device instead of cpufreq-dt-platdev. So I will send a series that can fix
> cpufreq until ti-cpufreq is ready which will give us better support for more
> OPPs on am335x, am437x (which has no implementation today), and dra7xx.

OK yeah let's fix the regression & thanks for the update.

Regards,

Tony