Re: [PATCH V5 4/7] ARM: pxa: Use - instead of @ for DT OPP entries

2017-04-22 Thread Robert Jarzmik 
Robert Jarzmik  writes:

> Viresh Kumar  writes:
>
>> On 20-04-17, 22:14, Robert Jarzmik wrote:
>>> Viresh Kumar  writes:
>>> 
>>> > Compiling the DT file with W=1, DTC warns like follows:
>>> >
>>> > Warning (unit_address_vs_reg): Node /opp_table0/opp@10 has a
>>> > unit name, but no reg property
>>> >
>>> > Fix this by replacing '@' with '-' as the OPP nodes will never have a
>>> > "reg" property.
>>> >
>>> > Reported-by: Krzysztof Kozlowski 
>>> > Reported-by: Masahiro Yamada 
>>> > Suggested-by: Mark Rutland 
>>> > Signed-off-by: Viresh Kumar 
>>> > Acked-by: Rob Herring 
>>> Acked-by: Robert Jarzmik 
>>
>> Thanks. But I need you to pick it up for your pull request for arm-soc.
Applied to pxa/dt tree, thanks.

Cheers.

-- 
Robert

Re: [PATCH V5 4/7] ARM: pxa: Use - instead of @ for DT OPP entries

2017-04-22 Thread Robert Jarzmik 
Robert Jarzmik  writes:

> Viresh Kumar  writes:
>
>> On 20-04-17, 22:14, Robert Jarzmik wrote:
>>> Viresh Kumar  writes:
>>> 
>>> > Compiling the DT file with W=1, DTC warns like follows:
>>> >
>>> > Warning (unit_address_vs_reg): Node /opp_table0/opp@10 has a
>>> > unit name, but no reg property
>>> >
>>> > Fix this by replacing '@' with '-' as the OPP nodes will never have a
>>> > "reg" property.
>>> >
>>> > Reported-by: Krzysztof Kozlowski 
>>> > Reported-by: Masahiro Yamada 
>>> > Suggested-by: Mark Rutland 
>>> > Signed-off-by: Viresh Kumar 
>>> > Acked-by: Rob Herring 
>>> Acked-by: Robert Jarzmik 
>>
>> Thanks. But I need you to pick it up for your pull request for arm-soc.
Applied to pxa/dt tree, thanks.

Cheers.

-- 
Robert

Re: [PATCH V5 4/7] ARM: pxa: Use - instead of @ for DT OPP entries

2017-04-21 Thread Robert Jarzmik 
Viresh Kumar  writes:

> On 20-04-17, 22:14, Robert Jarzmik wrote:
>> Viresh Kumar  writes:
>> 
>> > Compiling the DT file with W=1, DTC warns like follows:
>> >
>> > Warning (unit_address_vs_reg): Node /opp_table0/opp@10 has a
>> > unit name, but no reg property
>> >
>> > Fix this by replacing '@' with '-' as the OPP nodes will never have a
>> > "reg" property.
>> >
>> > Reported-by: Krzysztof Kozlowski 
>> > Reported-by: Masahiro Yamada 
>> > Suggested-by: Mark Rutland 
>> > Signed-off-by: Viresh Kumar 
>> > Acked-by: Rob Herring 
>> Acked-by: Robert Jarzmik 
>
> Thanks. But I need you to pick it up for your pull request for arm-soc.
Sure, let me take it through my tree, no problem.

Cheers.

-- 
Robert

Re: [PATCH V5 4/7] ARM: pxa: Use - instead of @ for DT OPP entries

2017-04-21 Thread Robert Jarzmik 
Viresh Kumar  writes:

> On 20-04-17, 22:14, Robert Jarzmik wrote:
>> Viresh Kumar  writes:
>> 
>> > Compiling the DT file with W=1, DTC warns like follows:
>> >
>> > Warning (unit_address_vs_reg): Node /opp_table0/opp@10 has a
>> > unit name, but no reg property
>> >
>> > Fix this by replacing '@' with '-' as the OPP nodes will never have a
>> > "reg" property.
>> >
>> > Reported-by: Krzysztof Kozlowski 
>> > Reported-by: Masahiro Yamada 
>> > Suggested-by: Mark Rutland 
>> > Signed-off-by: Viresh Kumar 
>> > Acked-by: Rob Herring 
>> Acked-by: Robert Jarzmik 
>
> Thanks. But I need you to pick it up for your pull request for arm-soc.
Sure, let me take it through my tree, no problem.

Cheers.

-- 
Robert

Re: [PATCH V5 4/7] ARM: pxa: Use - instead of @ for DT OPP entries

2017-04-20 Thread Viresh Kumar 
On 20-04-17, 22:14, Robert Jarzmik wrote:
> Viresh Kumar  writes:
> 
> > Compiling the DT file with W=1, DTC warns like follows:
> >
> > Warning (unit_address_vs_reg): Node /opp_table0/opp@10 has a
> > unit name, but no reg property
> >
> > Fix this by replacing '@' with '-' as the OPP nodes will never have a
> > "reg" property.
> >
> > Reported-by: Krzysztof Kozlowski 
> > Reported-by: Masahiro Yamada 
> > Suggested-by: Mark Rutland 
> > Signed-off-by: Viresh Kumar 
> > Acked-by: Rob Herring 
> Acked-by: Robert Jarzmik 

Thanks. But I need you to pick it up for your pull request for arm-soc.

-- 
viresh

Re: [PATCH V5 4/7] ARM: pxa: Use - instead of @ for DT OPP entries

2017-04-20 Thread Viresh Kumar 
On 20-04-17, 22:14, Robert Jarzmik wrote:
> Viresh Kumar  writes:
> 
> > Compiling the DT file with W=1, DTC warns like follows:
> >
> > Warning (unit_address_vs_reg): Node /opp_table0/opp@10 has a
> > unit name, but no reg property
> >
> > Fix this by replacing '@' with '-' as the OPP nodes will never have a
> > "reg" property.
> >
> > Reported-by: Krzysztof Kozlowski 
> > Reported-by: Masahiro Yamada 
> > Suggested-by: Mark Rutland 
> > Signed-off-by: Viresh Kumar 
> > Acked-by: Rob Herring 
> Acked-by: Robert Jarzmik 

Thanks. But I need you to pick it up for your pull request for arm-soc.

-- 
viresh

Re: [PATCH V5 4/7] ARM: pxa: Use - instead of @ for DT OPP entries

2017-04-20 Thread Robert Jarzmik 
Viresh Kumar  writes:

> Compiling the DT file with W=1, DTC warns like follows:
>
> Warning (unit_address_vs_reg): Node /opp_table0/opp@10 has a
> unit name, but no reg property
>
> Fix this by replacing '@' with '-' as the OPP nodes will never have a
> "reg" property.
>
> Reported-by: Krzysztof Kozlowski 
> Reported-by: Masahiro Yamada 
> Suggested-by: Mark Rutland 
> Signed-off-by: Viresh Kumar 
> Acked-by: Rob Herring 
Acked-by: Robert Jarzmik 

Cheers.

-- 
Robert

Re: [PATCH V5 4/7] ARM: pxa: Use - instead of @ for DT OPP entries

2017-04-20 Thread Robert Jarzmik 
Viresh Kumar  writes:

> Compiling the DT file with W=1, DTC warns like follows:
>
> Warning (unit_address_vs_reg): Node /opp_table0/opp@10 has a
> unit name, but no reg property
>
> Fix this by replacing '@' with '-' as the OPP nodes will never have a
> "reg" property.
>
> Reported-by: Krzysztof Kozlowski 
> Reported-by: Masahiro Yamada 
> Suggested-by: Mark Rutland 
> Signed-off-by: Viresh Kumar 
> Acked-by: Rob Herring 
Acked-by: Robert Jarzmik 

Cheers.

-- 
Robert

[PATCH V5 4/7] ARM: pxa: Use - instead of @ for DT OPP entries

2017-04-20 Thread Viresh Kumar 
Compiling the DT file with W=1, DTC warns like follows:

Warning (unit_address_vs_reg): Node /opp_table0/opp@10 has a
unit name, but no reg property

Fix this by replacing '@' with '-' as the OPP nodes will never have a
"reg" property.

Reported-by: Krzysztof Kozlowski 
Reported-by: Masahiro Yamada 
Suggested-by: Mark Rutland 
Signed-off-by: Viresh Kumar 
Acked-by: Rob Herring 
---
 arch/arm/boot/dts/pxa25x.dtsi |  8 
 arch/arm/boot/dts/pxa27x.dtsi | 14 +++---
 2 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/arch/arm/boot/dts/pxa25x.dtsi b/arch/arm/boot/dts/pxa25x.dtsi
index f9f4726396a0..95d59be97213 100644
--- a/arch/arm/boot/dts/pxa25x.dtsi
+++ b/arch/arm/boot/dts/pxa25x.dtsi
@@ -93,22 +93,22 @@
pxa250_opp_table: opp_table0 {
compatible = "operating-points-v2";
 
-   opp@99532800 {
+   opp-99532800 {
opp-hz = /bits/ 64 <99532800>;
opp-microvolt = <100 95 165>;
clock-latency-ns = <20>;
};
-   opp@199065600 {
+   opp-199065600 {
opp-hz = /bits/ 64 <199065600>;
opp-microvolt = <100 95 165>;
clock-latency-ns = <20>;
};
-   opp@298598400 {
+   opp-298598400 {
opp-hz = /bits/ 64 <298598400>;
opp-microvolt = <110 1045000 165>;
clock-latency-ns = <20>;
};
-   opp@398131200 {
+   opp-398131200 {
opp-hz = /bits/ 64 <398131200>;
opp-microvolt = <130 1235000 165>;
clock-latency-ns = <20>;
diff --git a/arch/arm/boot/dts/pxa27x.dtsi b/arch/arm/boot/dts/pxa27x.dtsi
index e0fab48ba6fa..5f1d6da02a4c 100644
--- a/arch/arm/boot/dts/pxa27x.dtsi
+++ b/arch/arm/boot/dts/pxa27x.dtsi
@@ -141,37 +141,37 @@
pxa270_opp_table: opp_table0 {
compatible = "operating-points-v2";
 
-   opp@10400 {
+   opp-10400 {
opp-hz = /bits/ 64 <10400>;
opp-microvolt = <90 90 1705000>;
clock-latency-ns = <20>;
};
-   opp@15600 {
+   opp-15600 {
opp-hz = /bits/ 64 <15600>;
opp-microvolt = <100 100 1705000>;
clock-latency-ns = <20>;
};
-   opp@20800 {
+   opp-20800 {
opp-hz = /bits/ 64 <20800>;
opp-microvolt = <118 118 1705000>;
clock-latency-ns = <20>;
};
-   opp@31200 {
+   opp-31200 {
opp-hz = /bits/ 64 <31200>;
opp-microvolt = <125 125 1705000>;
clock-latency-ns = <20>;
};
-   opp@41600 {
+   opp-41600 {
opp-hz = /bits/ 64 <41600>;
opp-microvolt = <135 135 1705000>;
clock-latency-ns = <20>;
};
-   opp@52000 {
+   opp-52000 {
opp-hz = /bits/ 64 <52000>;
opp-microvolt = <145 145 1705000>;
clock-latency-ns = <20>;
};
-   opp@62400 {
+   opp-62400 {
opp-hz = /bits/ 64 <62400>;
opp-microvolt = <155 155 1705000>;
clock-latency-ns = <20>;
-- 
2.12.0.432.g71c3a4f4ba37

[PATCH V5 4/7] ARM: pxa: Use - instead of @ for DT OPP entries

2017-04-20 Thread Viresh Kumar 
Compiling the DT file with W=1, DTC warns like follows:

Warning (unit_address_vs_reg): Node /opp_table0/opp@10 has a
unit name, but no reg property

Fix this by replacing '@' with '-' as the OPP nodes will never have a
"reg" property.

Reported-by: Krzysztof Kozlowski 
Reported-by: Masahiro Yamada 
Suggested-by: Mark Rutland 
Signed-off-by: Viresh Kumar 
Acked-by: Rob Herring 
---
 arch/arm/boot/dts/pxa25x.dtsi |  8 
 arch/arm/boot/dts/pxa27x.dtsi | 14 +++---
 2 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/arch/arm/boot/dts/pxa25x.dtsi b/arch/arm/boot/dts/pxa25x.dtsi
index f9f4726396a0..95d59be97213 100644
--- a/arch/arm/boot/dts/pxa25x.dtsi
+++ b/arch/arm/boot/dts/pxa25x.dtsi
@@ -93,22 +93,22 @@
pxa250_opp_table: opp_table0 {
compatible = "operating-points-v2";
 
-   opp@99532800 {
+   opp-99532800 {
opp-hz = /bits/ 64 <99532800>;
opp-microvolt = <100 95 165>;
clock-latency-ns = <20>;
};
-   opp@199065600 {
+   opp-199065600 {
opp-hz = /bits/ 64 <199065600>;
opp-microvolt = <100 95 165>;
clock-latency-ns = <20>;
};
-   opp@298598400 {
+   opp-298598400 {
opp-hz = /bits/ 64 <298598400>;
opp-microvolt = <110 1045000 165>;
clock-latency-ns = <20>;
};
-   opp@398131200 {
+   opp-398131200 {
opp-hz = /bits/ 64 <398131200>;
opp-microvolt = <130 1235000 165>;
clock-latency-ns = <20>;
diff --git a/arch/arm/boot/dts/pxa27x.dtsi b/arch/arm/boot/dts/pxa27x.dtsi
index e0fab48ba6fa..5f1d6da02a4c 100644
--- a/arch/arm/boot/dts/pxa27x.dtsi
+++ b/arch/arm/boot/dts/pxa27x.dtsi
@@ -141,37 +141,37 @@
pxa270_opp_table: opp_table0 {
compatible = "operating-points-v2";
 
-   opp@10400 {
+   opp-10400 {
opp-hz = /bits/ 64 <10400>;
opp-microvolt = <90 90 1705000>;
clock-latency-ns = <20>;
};
-   opp@15600 {
+   opp-15600 {
opp-hz = /bits/ 64 <15600>;
opp-microvolt = <100 100 1705000>;
clock-latency-ns = <20>;
};
-   opp@20800 {
+   opp-20800 {
opp-hz = /bits/ 64 <20800>;
opp-microvolt = <118 118 1705000>;
clock-latency-ns = <20>;
};
-   opp@31200 {
+   opp-31200 {
opp-hz = /bits/ 64 <31200>;
opp-microvolt = <125 125 1705000>;
clock-latency-ns = <20>;
};
-   opp@41600 {
+   opp-41600 {
opp-hz = /bits/ 64 <41600>;
opp-microvolt = <135 135 1705000>;
clock-latency-ns = <20>;
};
-   opp@52000 {
+   opp-52000 {
opp-hz = /bits/ 64 <52000>;
opp-microvolt = <145 145 1705000>;
clock-latency-ns = <20>;
};
-   opp@62400 {
+   opp-62400 {
opp-hz = /bits/ 64 <62400>;
opp-microvolt = <155 155 1705000>;
clock-latency-ns = <20>;
-- 
2.12.0.432.g71c3a4f4ba37