Re: [PATCH 2/2] ARM: dts: exynos: Fix unit addresses of PDMA nodes in Exynos5410

[Chanwoo Choi]

On 2018년 02월 23일 04:11, Krzysztof Kozlowski wrote:
> Fix the unit addresses of PDMA nodes in Exynos5410 (the reg property is
> correct) to get rid of DTC warnings like:
> 
> arch/arm/boot/dts/exynos5410-odroidxu.dtb: Warning (simple_bus_reg):
> Node /soc/amba/pdma@1268 simple-bus unit address format error, 
> expected "121a"
> 
> Signed-off-by: Krzysztof Kozlowski 
> ---
>  arch/arm/boot/dts/exynos5410.dtsi | 8 
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/exynos5410.dtsi 
> b/arch/arm/boot/dts/exynos5410.dtsi
> index 1e6b57d013c4..55509c690328 100644
> --- a/arch/arm/boot/dts/exynos5410.dtsi
> +++ b/arch/arm/boot/dts/exynos5410.dtsi
> @@ -196,9 +196,9 @@
>   interrupt-parent = <&gic>;
>   ranges;
>  
> - pdma0: pdma@1268 {
> + pdma0: pdma@121a {
>   compatible = "arm,pl330", "arm,primecell";
> - reg = <0x121A 0x1000>;
> + reg = <0x121a 0x1000>;
>   interrupts = ;
>   clocks = <&clock CLK_PDMA0>;
>   clock-names = "apb_pclk";
> @@ -207,9 +207,9 @@
>   #dma-requests = <32>;
>   };
>  
> - pdma1: pdma@1269 {
> + pdma1: pdma@121b {
>   compatible = "arm,pl330", "arm,primecell";
> - reg = <0x121B 0x1000>;
> + reg = <0x121b 0x1000>;
>   interrupts = ;
>   clocks = <&clock CLK_PDMA1>;
>   clock-names = "apb_pclk";
> 

Reviewed-by: Chanwoo Choi 

-- 
Best Regards,
Chanwoo Choi
Samsung Electronics

[PATCH 2/2] ARM: dts: exynos: Fix unit addresses of PDMA nodes in Exynos5410

[Krzysztof Kozlowski]

Fix the unit addresses of PDMA nodes in Exynos5410 (the reg property is
correct) to get rid of DTC warnings like:

arch/arm/boot/dts/exynos5410-odroidxu.dtb: Warning (simple_bus_reg):
Node /soc/amba/pdma@1268 simple-bus unit address format error, 
expected "121a"

Signed-off-by: Krzysztof Kozlowski 
---
 arch/arm/boot/dts/exynos5410.dtsi | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm/boot/dts/exynos5410.dtsi 
b/arch/arm/boot/dts/exynos5410.dtsi
index 1e6b57d013c4..55509c690328 100644
--- a/arch/arm/boot/dts/exynos5410.dtsi
+++ b/arch/arm/boot/dts/exynos5410.dtsi
@@ -196,9 +196,9 @@
interrupt-parent = <&gic>;
ranges;
 
-   pdma0: pdma@1268 {
+   pdma0: pdma@121a {
compatible = "arm,pl330", "arm,primecell";
-   reg = <0x121A 0x1000>;
+   reg = <0x121a 0x1000>;
interrupts = ;
clocks = <&clock CLK_PDMA0>;
clock-names = "apb_pclk";
@@ -207,9 +207,9 @@
#dma-requests = <32>;
};
 
-   pdma1: pdma@1269 {
+   pdma1: pdma@121b {
compatible = "arm,pl330", "arm,primecell";
-   reg = <0x121B 0x1000>;
+   reg = <0x121b 0x1000>;
interrupts = ;
clocks = <&clock CLK_PDMA1>;
clock-names = "apb_pclk";
-- 
2.14.1