All of lore.kernel.org
 help / color / mirror / Atom feed
From: Chanwoo Choi <cw00.choi@samsung.com>
To: Krzysztof Kozlowski <krzk@kernel.org>,
	Kukjin Kim <kgene@kernel.org>,
	Sylwester Nawrocki <s.nawrocki@samsung.com>,
	Marek Szyprowski <m.szyprowski@samsung.com>,
	Marian Mihailescu <mihailescu2m@gmail.com>,
	devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org
Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Subject: Re: [PATCH 1/2] ARM: dts: exynos: Fix address of PPMU ACP on Exynos4210
Date: Fri, 23 Feb 2018 09:30:06 +0900	[thread overview]
Message-ID: <5A8F608E.4030509@samsung.com> (raw)
In-Reply-To: <20180222191111.23747-1-krzk@kernel.org>

On 2018년 02월 23일 04:11, Krzysztof Kozlowski wrote:
> Typo in unit address of PPMU ACP caused DTC warnings:
> 
>     arch/arm/boot/dts/exynos4412-odroidu3.dtb:
>         Warning (simple_bus_reg): Node /soc/ppmu_acp@10ae0000 simple-bus unit address format error, expected "106e0000"
> 
> PPMU ACP (under 0x10ae0000 address) is present only in Exynos4210 so
> move this node to exynos4210.dtsi with fixing the reg address to proper
> one.
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
>  arch/arm/boot/dts/exynos4.dtsi    | 6 ------
>  arch/arm/boot/dts/exynos4210.dtsi | 6 ++++++
>  2 files changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/exynos4.dtsi b/arch/arm/boot/dts/exynos4.dtsi
> index 967cd02401c5..909a9f2bf5be 100644
> --- a/arch/arm/boot/dts/exynos4.dtsi
> +++ b/arch/arm/boot/dts/exynos4.dtsi
> @@ -821,12 +821,6 @@
>  			status = "disabled";
>  		};
>  
> -		ppmu_acp: ppmu_acp@10ae0000 {
> -			compatible = "samsung,exynos-ppmu";
> -			reg = <0x106e0000 0x2000>;
> -			status = "disabled";
> -		};
> -
>  		ppmu_rightbus: ppmu_rightbus@112a0000 {
>  			compatible = "samsung,exynos-ppmu";
>  			reg = <0x112a0000 0x2000>;
> diff --git a/arch/arm/boot/dts/exynos4210.dtsi b/arch/arm/boot/dts/exynos4210.dtsi
> index 853604d91158..88fb47cef9a8 100644
> --- a/arch/arm/boot/dts/exynos4210.dtsi
> +++ b/arch/arm/boot/dts/exynos4210.dtsi
> @@ -162,6 +162,12 @@
>  			iommus = <&sysmmu_g2d>;
>  		};
>  
> +		ppmu_acp: ppmu_acp@10ae0000 {
> +			compatible = "samsung,exynos-ppmu";
> +			reg = <0x10ae0000 0x2000>;
> +			status = "disabled";
> +		};
> +
>  		ppmu_lcd1: ppmu_lcd1@12240000 {
>  			compatible = "samsung,exynos-ppmu";
>  			reg = <0x12240000 0x2000>;
> 

Reviewed-by: Chanwoo Choi <cw00.choi@samsung.com>

-- 
Best Regards,
Chanwoo Choi
Samsung Electronics

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

WARNING: multiple messages have this Message-ID (diff)
From: cw00.choi@samsung.com (Chanwoo Choi)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/2] ARM: dts: exynos: Fix address of PPMU ACP on Exynos4210
Date: Fri, 23 Feb 2018 09:30:06 +0900	[thread overview]
Message-ID: <5A8F608E.4030509@samsung.com> (raw)
In-Reply-To: <20180222191111.23747-1-krzk@kernel.org>

On 2018? 02? 23? 04:11, Krzysztof Kozlowski wrote:
> Typo in unit address of PPMU ACP caused DTC warnings:
> 
>     arch/arm/boot/dts/exynos4412-odroidu3.dtb:
>         Warning (simple_bus_reg): Node /soc/ppmu_acp at 10ae0000 simple-bus unit address format error, expected "106e0000"
> 
> PPMU ACP (under 0x10ae0000 address) is present only in Exynos4210 so
> move this node to exynos4210.dtsi with fixing the reg address to proper
> one.
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
>  arch/arm/boot/dts/exynos4.dtsi    | 6 ------
>  arch/arm/boot/dts/exynos4210.dtsi | 6 ++++++
>  2 files changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/exynos4.dtsi b/arch/arm/boot/dts/exynos4.dtsi
> index 967cd02401c5..909a9f2bf5be 100644
> --- a/arch/arm/boot/dts/exynos4.dtsi
> +++ b/arch/arm/boot/dts/exynos4.dtsi
> @@ -821,12 +821,6 @@
>  			status = "disabled";
>  		};
>  
> -		ppmu_acp: ppmu_acp at 10ae0000 {
> -			compatible = "samsung,exynos-ppmu";
> -			reg = <0x106e0000 0x2000>;
> -			status = "disabled";
> -		};
> -
>  		ppmu_rightbus: ppmu_rightbus at 112a0000 {
>  			compatible = "samsung,exynos-ppmu";
>  			reg = <0x112a0000 0x2000>;
> diff --git a/arch/arm/boot/dts/exynos4210.dtsi b/arch/arm/boot/dts/exynos4210.dtsi
> index 853604d91158..88fb47cef9a8 100644
> --- a/arch/arm/boot/dts/exynos4210.dtsi
> +++ b/arch/arm/boot/dts/exynos4210.dtsi
> @@ -162,6 +162,12 @@
>  			iommus = <&sysmmu_g2d>;
>  		};
>  
> +		ppmu_acp: ppmu_acp at 10ae0000 {
> +			compatible = "samsung,exynos-ppmu";
> +			reg = <0x10ae0000 0x2000>;
> +			status = "disabled";
> +		};
> +
>  		ppmu_lcd1: ppmu_lcd1 at 12240000 {
>  			compatible = "samsung,exynos-ppmu";
>  			reg = <0x12240000 0x2000>;
> 

Reviewed-by: Chanwoo Choi <cw00.choi@samsung.com>

-- 
Best Regards,
Chanwoo Choi
Samsung Electronics

WARNING: multiple messages have this Message-ID (diff)
From: Chanwoo Choi <cw00.choi@samsung.com>
To: Krzysztof Kozlowski <krzk@kernel.org>,
	Kukjin Kim <kgene@kernel.org>,
	Sylwester Nawrocki <s.nawrocki@samsung.com>,
	Marek Szyprowski <m.szyprowski@samsung.com>,
	Marian Mihailescu <mihailescu2m@gmail.com>,
	devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org
Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Subject: Re: [PATCH 1/2] ARM: dts: exynos: Fix address of PPMU ACP on Exynos4210
Date: Fri, 23 Feb 2018 09:30:06 +0900	[thread overview]
Message-ID: <5A8F608E.4030509@samsung.com> (raw)
In-Reply-To: <20180222191111.23747-1-krzk@kernel.org>

On 2018년 02월 23일 04:11, Krzysztof Kozlowski wrote:
> Typo in unit address of PPMU ACP caused DTC warnings:
> 
>     arch/arm/boot/dts/exynos4412-odroidu3.dtb:
>         Warning (simple_bus_reg): Node /soc/ppmu_acp@10ae0000 simple-bus unit address format error, expected "106e0000"
> 
> PPMU ACP (under 0x10ae0000 address) is present only in Exynos4210 so
> move this node to exynos4210.dtsi with fixing the reg address to proper
> one.
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
>  arch/arm/boot/dts/exynos4.dtsi    | 6 ------
>  arch/arm/boot/dts/exynos4210.dtsi | 6 ++++++
>  2 files changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/exynos4.dtsi b/arch/arm/boot/dts/exynos4.dtsi
> index 967cd02401c5..909a9f2bf5be 100644
> --- a/arch/arm/boot/dts/exynos4.dtsi
> +++ b/arch/arm/boot/dts/exynos4.dtsi
> @@ -821,12 +821,6 @@
>  			status = "disabled";
>  		};
>  
> -		ppmu_acp: ppmu_acp@10ae0000 {
> -			compatible = "samsung,exynos-ppmu";
> -			reg = <0x106e0000 0x2000>;
> -			status = "disabled";
> -		};
> -
>  		ppmu_rightbus: ppmu_rightbus@112a0000 {
>  			compatible = "samsung,exynos-ppmu";
>  			reg = <0x112a0000 0x2000>;
> diff --git a/arch/arm/boot/dts/exynos4210.dtsi b/arch/arm/boot/dts/exynos4210.dtsi
> index 853604d91158..88fb47cef9a8 100644
> --- a/arch/arm/boot/dts/exynos4210.dtsi
> +++ b/arch/arm/boot/dts/exynos4210.dtsi
> @@ -162,6 +162,12 @@
>  			iommus = <&sysmmu_g2d>;
>  		};
>  
> +		ppmu_acp: ppmu_acp@10ae0000 {
> +			compatible = "samsung,exynos-ppmu";
> +			reg = <0x10ae0000 0x2000>;
> +			status = "disabled";
> +		};
> +
>  		ppmu_lcd1: ppmu_lcd1@12240000 {
>  			compatible = "samsung,exynos-ppmu";
>  			reg = <0x12240000 0x2000>;
> 

Reviewed-by: Chanwoo Choi <cw00.choi@samsung.com>

-- 
Best Regards,
Chanwoo Choi
Samsung Electronics

  parent reply	other threads:[~2018-02-23  0:30 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CGME20180222191146epcas2p1e0fc10e3f8b278f0c3671d32e2d76350@epcas2p1.samsung.com>
2018-02-22 19:11 ` [PATCH 1/2] ARM: dts: exynos: Fix address of PPMU ACP on Exynos4210 Krzysztof Kozlowski
2018-02-22 19:11   ` Krzysztof Kozlowski
2018-02-22 19:11   ` [PATCH 2/2] ARM: dts: exynos: Fix unit addresses of PDMA nodes in Exynos5410 Krzysztof Kozlowski
2018-02-22 19:11     ` Krzysztof Kozlowski
2018-02-23  0:30     ` Chanwoo Choi
2018-02-23  0:30       ` Chanwoo Choi
2018-02-23  0:30       ` Chanwoo Choi
2018-02-23  0:30   ` Chanwoo Choi [this message]
2018-02-23  0:30     ` [PATCH 1/2] ARM: dts: exynos: Fix address of PPMU ACP on Exynos4210 Chanwoo Choi
2018-02-23  0:30     ` Chanwoo Choi
2018-02-26 17:04     ` Krzysztof Kozlowski
2018-02-26 17:04       ` Krzysztof Kozlowski

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5A8F608E.4030509@samsung.com \
    --to=cw00.choi@samsung.com \
    --cc=b.zolnierkie@samsung.com \
    --cc=devicetree@vger.kernel.org \
    --cc=kgene@kernel.org \
    --cc=krzk@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=m.szyprowski@samsung.com \
    --cc=mihailescu2m@gmail.com \
    --cc=s.nawrocki@samsung.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.