Re: [PATCH V4 5/7] ARM: sun8i: Use - instead of @ for DT OPP entries

2017-04-20 Thread Viresh Kumar 
On 20-04-17, 10:02, Maxime Ripard wrote:
> Hi,
> 
> On Thu, Apr 20, 2017 at 11:14:16AM +0530, Viresh Kumar wrote:
> > 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: Maxime Ripard 
> > Acked-by: Rob Herring 
> 
> This patch doesn't apply, please make sure to base them on linux-next

I can do that tomorrow once linux-next includes the new PM tree as the
old PM tree had my patch from V3.

-- 
viresh

Re: [PATCH V4 5/7] ARM: sun8i: Use - instead of @ for DT OPP entries

2017-04-20 Thread Viresh Kumar 
On 20-04-17, 10:02, Maxime Ripard wrote:
> Hi,
> 
> On Thu, Apr 20, 2017 at 11:14:16AM +0530, Viresh Kumar wrote:
> > 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: Maxime Ripard 
> > Acked-by: Rob Herring 
> 
> This patch doesn't apply, please make sure to base them on linux-next

I can do that tomorrow once linux-next includes the new PM tree as the
old PM tree had my patch from V3.

-- 
viresh

Re: [PATCH V4 5/7] ARM: sun8i: Use - instead of @ for DT OPP entries

2017-04-20 Thread Maxime Ripard 
Hi,

On Thu, Apr 20, 2017 at 11:14:16AM +0530, Viresh Kumar wrote:
> 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: Maxime Ripard 
> Acked-by: Rob Herring 

This patch doesn't apply, please make sure to base them on linux-next

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com


signature.asc
Description: PGP signature

Re: [PATCH V4 5/7] ARM: sun8i: Use - instead of @ for DT OPP entries

2017-04-20 Thread Maxime Ripard 
Hi,

On Thu, Apr 20, 2017 at 11:14:16AM +0530, Viresh Kumar wrote:
> 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: Maxime Ripard 
> Acked-by: Rob Herring 

This patch doesn't apply, please make sure to base them on linux-next

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com


signature.asc
Description: PGP signature

[PATCH V4 5/7] ARM: sun8i: Use - instead of @ for DT OPP entries

2017-04-19 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: Maxime Ripard 
Acked-by: Rob Herring 
---
 arch/arm/boot/dts/sun8i-a33.dtsi | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/arm/boot/dts/sun8i-a33.dtsi b/arch/arm/boot/dts/sun8i-a33.dtsi
index 306af6cadf26..a2c555d6475c 100644
--- a/arch/arm/boot/dts/sun8i-a33.dtsi
+++ b/arch/arm/boot/dts/sun8i-a33.dtsi
@@ -49,19 +49,19 @@
compatible = "operating-points-v2";
opp-shared;
 
-   opp@64800 {
+   opp-64800 {
opp-hz = /bits/ 64 <64800>;
opp-microvolt = <104>;
clock-latency-ns = <244144>; /* 8 32k periods */
};
 
-   opp@81600 {
+   opp-81600 {
opp-hz = /bits/ 64 <81600>;
opp-microvolt = <110>;
clock-latency-ns = <244144>; /* 8 32k periods */
};
 
-   opp@100800 {
+   opp-100800 {
opp-hz = /bits/ 64 <100800>;
opp-microvolt = <120>;
clock-latency-ns = <244144>; /* 8 32k periods */
-- 
2.12.0.432.g71c3a4f4ba37

[PATCH V4 5/7] ARM: sun8i: Use - instead of @ for DT OPP entries

2017-04-19 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: Maxime Ripard 
Acked-by: Rob Herring 
---
 arch/arm/boot/dts/sun8i-a33.dtsi | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/arm/boot/dts/sun8i-a33.dtsi b/arch/arm/boot/dts/sun8i-a33.dtsi
index 306af6cadf26..a2c555d6475c 100644
--- a/arch/arm/boot/dts/sun8i-a33.dtsi
+++ b/arch/arm/boot/dts/sun8i-a33.dtsi
@@ -49,19 +49,19 @@
compatible = "operating-points-v2";
opp-shared;
 
-   opp@64800 {
+   opp-64800 {
opp-hz = /bits/ 64 <64800>;
opp-microvolt = <104>;
clock-latency-ns = <244144>; /* 8 32k periods */
};
 
-   opp@81600 {
+   opp-81600 {
opp-hz = /bits/ 64 <81600>;
opp-microvolt = <110>;
clock-latency-ns = <244144>; /* 8 32k periods */
};
 
-   opp@100800 {
+   opp-100800 {
opp-hz = /bits/ 64 <100800>;
opp-microvolt = <120>;
clock-latency-ns = <244144>; /* 8 32k periods */
-- 
2.12.0.432.g71c3a4f4ba37