Re: [RFT][PATCH] arm64: dts: exynos: Remove unneeded address space mapping for soc node

2018-04-29 Thread Krzysztof Kozlowski
On Wed, Apr 25, 2018 at 07:52:45AM +0200, Marek Szyprowski wrote:
> Hi Krzysztof,
> 
> On 2018-04-24 19:36, Krzysztof Kozlowski wrote:
> > Remove the address space mapping between root and soc nodes to fix
> > DTC warnings in Exynos5433 and Exynos7 like:
> >
> >  arch/arm64/boot/dts/exynos/exynos5433-tm2.dtb:
> >  Warning (unit_address_vs_reg): /soc: node has a reg or ranges 
> > property, but no unit name
> >
> > Signed-off-by: Krzysztof Kozlowski 
> 
> Works fine on Samsung Exynos5433-based TM2e board.
> 
> Tested-by: Marek Szyprowski 

Thanks for testing! Applied.


Best regards,
Krzysztof



Re: [RFT][PATCH] arm64: dts: exynos: Remove unneeded address space mapping for soc node

2018-04-29 Thread Krzysztof Kozlowski
On Wed, Apr 25, 2018 at 07:52:45AM +0200, Marek Szyprowski wrote:
> Hi Krzysztof,
> 
> On 2018-04-24 19:36, Krzysztof Kozlowski wrote:
> > Remove the address space mapping between root and soc nodes to fix
> > DTC warnings in Exynos5433 and Exynos7 like:
> >
> >  arch/arm64/boot/dts/exynos/exynos5433-tm2.dtb:
> >  Warning (unit_address_vs_reg): /soc: node has a reg or ranges 
> > property, but no unit name
> >
> > Signed-off-by: Krzysztof Kozlowski 
> 
> Works fine on Samsung Exynos5433-based TM2e board.
> 
> Tested-by: Marek Szyprowski 

Thanks for testing! Applied.


Best regards,
Krzysztof



Re: [RFT][PATCH] arm64: dts: exynos: Remove unneeded address space mapping for soc node

2018-04-24 Thread Marek Szyprowski
Hi Krzysztof,

On 2018-04-24 19:36, Krzysztof Kozlowski wrote:
> Remove the address space mapping between root and soc nodes to fix
> DTC warnings in Exynos5433 and Exynos7 like:
>
>  arch/arm64/boot/dts/exynos/exynos5433-tm2.dtb:
>  Warning (unit_address_vs_reg): /soc: node has a reg or ranges 
> property, but no unit name
>
> Signed-off-by: Krzysztof Kozlowski 

Works fine on Samsung Exynos5433-based TM2e board.

Tested-by: Marek Szyprowski 

> ---
>
> Not tested.
> ---
>   arch/arm64/boot/dts/exynos/exynos5433.dtsi | 6 +++---
>   arch/arm64/boot/dts/exynos/exynos7.dtsi| 6 +++---
>   2 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/exynos/exynos5433.dtsi 
> b/arch/arm64/boot/dts/exynos/exynos5433.dtsi
> index 828996a06610..ba8157ceaa56 100644
> --- a/arch/arm64/boot/dts/exynos/exynos5433.dtsi
> +++ b/arch/arm64/boot/dts/exynos/exynos5433.dtsi
> @@ -18,8 +18,8 @@
>   
>   / {
>   compatible = "samsung,exynos5433";
> - #address-cells = <2>;
> - #size-cells = <2>;
> + #address-cells = <1>;
> + #size-cells = <1>;
>   
>   interrupt-parent = <>;
>   
> @@ -235,7 +235,7 @@
>   compatible = "simple-bus";
>   #address-cells = <1>;
>   #size-cells = <1>;
> - ranges = <0x0 0x0 0x0 0x1800>;
> + ranges;
>   
>   arm_a53_pmu {
>   compatible = "arm,cortex-a53-pmu", "arm,armv8-pmuv3";
> diff --git a/arch/arm64/boot/dts/exynos/exynos7.dtsi 
> b/arch/arm64/boot/dts/exynos/exynos7.dtsi
> index 0b98d2334cad..93a84338938a 100644
> --- a/arch/arm64/boot/dts/exynos/exynos7.dtsi
> +++ b/arch/arm64/boot/dts/exynos/exynos7.dtsi
> @@ -12,8 +12,8 @@
>   / {
>   compatible = "samsung,exynos7";
>   interrupt-parent = <>;
> - #address-cells = <2>;
> - #size-cells = <2>;
> + #address-cells = <1>;
> + #size-cells = <1>;
>   
>   aliases {
>   pinctrl0 = _alive;
> @@ -70,7 +70,7 @@
>   compatible = "simple-bus";
>   #address-cells = <1>;
>   #size-cells = <1>;
> - ranges = <0 0 0 0x1800>;
> + ranges;
>   
>   chipid@1000 {
>   compatible = "samsung,exynos4210-chipid";

Best regards
-- 
Marek Szyprowski, PhD
Samsung R Institute Poland



Re: [RFT][PATCH] arm64: dts: exynos: Remove unneeded address space mapping for soc node

2018-04-24 Thread Marek Szyprowski
Hi Krzysztof,

On 2018-04-24 19:36, Krzysztof Kozlowski wrote:
> Remove the address space mapping between root and soc nodes to fix
> DTC warnings in Exynos5433 and Exynos7 like:
>
>  arch/arm64/boot/dts/exynos/exynos5433-tm2.dtb:
>  Warning (unit_address_vs_reg): /soc: node has a reg or ranges 
> property, but no unit name
>
> Signed-off-by: Krzysztof Kozlowski 

Works fine on Samsung Exynos5433-based TM2e board.

Tested-by: Marek Szyprowski 

> ---
>
> Not tested.
> ---
>   arch/arm64/boot/dts/exynos/exynos5433.dtsi | 6 +++---
>   arch/arm64/boot/dts/exynos/exynos7.dtsi| 6 +++---
>   2 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/exynos/exynos5433.dtsi 
> b/arch/arm64/boot/dts/exynos/exynos5433.dtsi
> index 828996a06610..ba8157ceaa56 100644
> --- a/arch/arm64/boot/dts/exynos/exynos5433.dtsi
> +++ b/arch/arm64/boot/dts/exynos/exynos5433.dtsi
> @@ -18,8 +18,8 @@
>   
>   / {
>   compatible = "samsung,exynos5433";
> - #address-cells = <2>;
> - #size-cells = <2>;
> + #address-cells = <1>;
> + #size-cells = <1>;
>   
>   interrupt-parent = <>;
>   
> @@ -235,7 +235,7 @@
>   compatible = "simple-bus";
>   #address-cells = <1>;
>   #size-cells = <1>;
> - ranges = <0x0 0x0 0x0 0x1800>;
> + ranges;
>   
>   arm_a53_pmu {
>   compatible = "arm,cortex-a53-pmu", "arm,armv8-pmuv3";
> diff --git a/arch/arm64/boot/dts/exynos/exynos7.dtsi 
> b/arch/arm64/boot/dts/exynos/exynos7.dtsi
> index 0b98d2334cad..93a84338938a 100644
> --- a/arch/arm64/boot/dts/exynos/exynos7.dtsi
> +++ b/arch/arm64/boot/dts/exynos/exynos7.dtsi
> @@ -12,8 +12,8 @@
>   / {
>   compatible = "samsung,exynos7";
>   interrupt-parent = <>;
> - #address-cells = <2>;
> - #size-cells = <2>;
> + #address-cells = <1>;
> + #size-cells = <1>;
>   
>   aliases {
>   pinctrl0 = _alive;
> @@ -70,7 +70,7 @@
>   compatible = "simple-bus";
>   #address-cells = <1>;
>   #size-cells = <1>;
> - ranges = <0 0 0 0x1800>;
> + ranges;
>   
>   chipid@1000 {
>   compatible = "samsung,exynos4210-chipid";

Best regards
-- 
Marek Szyprowski, PhD
Samsung R Institute Poland



[RFT][PATCH] arm64: dts: exynos: Remove unneeded address space mapping for soc node

2018-04-24 Thread Krzysztof Kozlowski
Remove the address space mapping between root and soc nodes to fix
DTC warnings in Exynos5433 and Exynos7 like:

arch/arm64/boot/dts/exynos/exynos5433-tm2.dtb:
Warning (unit_address_vs_reg): /soc: node has a reg or ranges property, 
but no unit name

Signed-off-by: Krzysztof Kozlowski 

---

Not tested.
---
 arch/arm64/boot/dts/exynos/exynos5433.dtsi | 6 +++---
 arch/arm64/boot/dts/exynos/exynos7.dtsi| 6 +++---
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/arch/arm64/boot/dts/exynos/exynos5433.dtsi 
b/arch/arm64/boot/dts/exynos/exynos5433.dtsi
index 828996a06610..ba8157ceaa56 100644
--- a/arch/arm64/boot/dts/exynos/exynos5433.dtsi
+++ b/arch/arm64/boot/dts/exynos/exynos5433.dtsi
@@ -18,8 +18,8 @@
 
 / {
compatible = "samsung,exynos5433";
-   #address-cells = <2>;
-   #size-cells = <2>;
+   #address-cells = <1>;
+   #size-cells = <1>;
 
interrupt-parent = <>;
 
@@ -235,7 +235,7 @@
compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
-   ranges = <0x0 0x0 0x0 0x1800>;
+   ranges;
 
arm_a53_pmu {
compatible = "arm,cortex-a53-pmu", "arm,armv8-pmuv3";
diff --git a/arch/arm64/boot/dts/exynos/exynos7.dtsi 
b/arch/arm64/boot/dts/exynos/exynos7.dtsi
index 0b98d2334cad..93a84338938a 100644
--- a/arch/arm64/boot/dts/exynos/exynos7.dtsi
+++ b/arch/arm64/boot/dts/exynos/exynos7.dtsi
@@ -12,8 +12,8 @@
 / {
compatible = "samsung,exynos7";
interrupt-parent = <>;
-   #address-cells = <2>;
-   #size-cells = <2>;
+   #address-cells = <1>;
+   #size-cells = <1>;
 
aliases {
pinctrl0 = _alive;
@@ -70,7 +70,7 @@
compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
-   ranges = <0 0 0 0x1800>;
+   ranges;
 
chipid@1000 {
compatible = "samsung,exynos4210-chipid";
-- 
2.14.1



[RFT][PATCH] arm64: dts: exynos: Remove unneeded address space mapping for soc node

2018-04-24 Thread Krzysztof Kozlowski
Remove the address space mapping between root and soc nodes to fix
DTC warnings in Exynos5433 and Exynos7 like:

arch/arm64/boot/dts/exynos/exynos5433-tm2.dtb:
Warning (unit_address_vs_reg): /soc: node has a reg or ranges property, 
but no unit name

Signed-off-by: Krzysztof Kozlowski 

---

Not tested.
---
 arch/arm64/boot/dts/exynos/exynos5433.dtsi | 6 +++---
 arch/arm64/boot/dts/exynos/exynos7.dtsi| 6 +++---
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/arch/arm64/boot/dts/exynos/exynos5433.dtsi 
b/arch/arm64/boot/dts/exynos/exynos5433.dtsi
index 828996a06610..ba8157ceaa56 100644
--- a/arch/arm64/boot/dts/exynos/exynos5433.dtsi
+++ b/arch/arm64/boot/dts/exynos/exynos5433.dtsi
@@ -18,8 +18,8 @@
 
 / {
compatible = "samsung,exynos5433";
-   #address-cells = <2>;
-   #size-cells = <2>;
+   #address-cells = <1>;
+   #size-cells = <1>;
 
interrupt-parent = <>;
 
@@ -235,7 +235,7 @@
compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
-   ranges = <0x0 0x0 0x0 0x1800>;
+   ranges;
 
arm_a53_pmu {
compatible = "arm,cortex-a53-pmu", "arm,armv8-pmuv3";
diff --git a/arch/arm64/boot/dts/exynos/exynos7.dtsi 
b/arch/arm64/boot/dts/exynos/exynos7.dtsi
index 0b98d2334cad..93a84338938a 100644
--- a/arch/arm64/boot/dts/exynos/exynos7.dtsi
+++ b/arch/arm64/boot/dts/exynos/exynos7.dtsi
@@ -12,8 +12,8 @@
 / {
compatible = "samsung,exynos7";
interrupt-parent = <>;
-   #address-cells = <2>;
-   #size-cells = <2>;
+   #address-cells = <1>;
+   #size-cells = <1>;
 
aliases {
pinctrl0 = _alive;
@@ -70,7 +70,7 @@
compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
-   ranges = <0 0 0 0x1800>;
+   ranges;
 
chipid@1000 {
compatible = "samsung,exynos4210-chipid";
-- 
2.14.1