Linux-Samsung-soc Archive on lore.kernel.org
 help / color / Atom feed
* [PATCH] ARM: dts: exynos: Fix L2 cache-controller nodename
@ 2020-06-24 11:04 Anand Moon
  2020-06-26  7:13 ` Krzysztof Kozlowski
  0 siblings, 1 reply; 3+ messages in thread
From: Anand Moon @ 2020-06-24 11:04 UTC (permalink / raw)
  To: devicetree, linux-arm-kernel, linux-samsung-soc
  Cc: Rob Herring, Krzysztof Kozlowski, Marek Szyprowski,
	Bartlomiej Zolnierkiewicz

Fix warning message by changing nodename to 'cache-controller'.

make ARCH=arm dtbs_check
DT_SCHEMA_FILES=Documentation/devicetree/bindings/arm/l2c2x0.yaml
arch/arm/boot/dts/exynos4210-i9100.dt.yaml: l2-cache-controller@10502000:
	$nodename:0: 'l2-cache-controller@10502000' does not match '
	^(cache-controller|cpu)(@[0-9a-f,]+)*$'
arch/arm/boot/dts/exynos4412-i9300.dt.yaml: l2-cache-controller@10502000:
        $nodename:0: 'l2-cache-controller@10502000' does not match '
        ^(cache-controller|cpu)(@[0-9a-f,]+)*$'

Fixes: 56b60b8bce4a ("ARM: 8265/1: dts: exynos4: Add nodes for L2 cache controller")
Signed-off-by: Anand Moon <linux.amoon@gmail.com>
---
 arch/arm/boot/dts/exynos4210.dtsi | 2 +-
 arch/arm/boot/dts/exynos4412.dtsi | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/exynos4210.dtsi b/arch/arm/boot/dts/exynos4210.dtsi
index b4466232f0c1..33435ce79ce4 100644
--- a/arch/arm/boot/dts/exynos4210.dtsi
+++ b/arch/arm/boot/dts/exynos4210.dtsi
@@ -97,7 +97,7 @@ pd_lcd1: power-domain@10023ca0 {
 			label = "LCD1";
 		};
 
-		l2c: l2-cache-controller@10502000 {
+		l2c: cache-controller@10502000 {
 			compatible = "arm,pl310-cache";
 			reg = <0x10502000 0x1000>;
 			cache-unified;
diff --git a/arch/arm/boot/dts/exynos4412.dtsi b/arch/arm/boot/dts/exynos4412.dtsi
index 48868947373e..7002832eb4c0 100644
--- a/arch/arm/boot/dts/exynos4412.dtsi
+++ b/arch/arm/boot/dts/exynos4412.dtsi
@@ -213,7 +213,7 @@ pd_isp: power-domain@10023ca0 {
 			label = "ISP";
 		};
 
-		l2c: l2-cache-controller@10502000 {
+		l2c: cache-controller@10502000 {
 			compatible = "arm,pl310-cache";
 			reg = <0x10502000 0x1000>;
 			cache-unified;
-- 
2.27.0


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] ARM: dts: exynos: Fix L2 cache-controller nodename
  2020-06-24 11:04 [PATCH] ARM: dts: exynos: Fix L2 cache-controller nodename Anand Moon
@ 2020-06-26  7:13 ` Krzysztof Kozlowski
  2020-06-28  8:26   ` Anand Moon
  0 siblings, 1 reply; 3+ messages in thread
From: Krzysztof Kozlowski @ 2020-06-26  7:13 UTC (permalink / raw)
  To: Anand Moon
  Cc: devicetree, linux-arm-kernel, linux-samsung-soc, Rob Herring,
	Marek Szyprowski, Bartlomiej Zolnierkiewicz

On Wed, Jun 24, 2020 at 11:04:35AM +0000, Anand Moon wrote:
> Fix warning message by changing nodename to 'cache-controller'.
> 
> make ARCH=arm dtbs_check
> DT_SCHEMA_FILES=Documentation/devicetree/bindings/arm/l2c2x0.yaml
> arch/arm/boot/dts/exynos4210-i9100.dt.yaml: l2-cache-controller@10502000:
> 	$nodename:0: 'l2-cache-controller@10502000' does not match '
> 	^(cache-controller|cpu)(@[0-9a-f,]+)*$'
> arch/arm/boot/dts/exynos4412-i9300.dt.yaml: l2-cache-controller@10502000:
>         $nodename:0: 'l2-cache-controller@10502000' does not match '
>         ^(cache-controller|cpu)(@[0-9a-f,]+)*$'
> 
> Fixes: 56b60b8bce4a ("ARM: 8265/1: dts: exynos4: Add nodes for L2 cache controller")

It is just naming convention introduced with dtschema so there is no
error to fix.  Applied with modified commit msg.

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] ARM: dts: exynos: Fix L2 cache-controller nodename
  2020-06-26  7:13 ` Krzysztof Kozlowski
@ 2020-06-28  8:26   ` Anand Moon
  0 siblings, 0 replies; 3+ messages in thread
From: Anand Moon @ 2020-06-28  8:26 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: devicetree, linux-arm-kernel, linux-samsung-soc, Rob Herring,
	Marek Szyprowski, Bartlomiej Zolnierkiewicz

Hi Krzysztof,

On Fri, 26 Jun 2020 at 12:44, Krzysztof Kozlowski <krzk@kernel.org> wrote:
>
> On Wed, Jun 24, 2020 at 11:04:35AM +0000, Anand Moon wrote:
> > Fix warning message by changing nodename to 'cache-controller'.
> >
> > make ARCH=arm dtbs_check
> > DT_SCHEMA_FILES=Documentation/devicetree/bindings/arm/l2c2x0.yaml
> > arch/arm/boot/dts/exynos4210-i9100.dt.yaml: l2-cache-controller@10502000:
> >       $nodename:0: 'l2-cache-controller@10502000' does not match '
> >       ^(cache-controller|cpu)(@[0-9a-f,]+)*$'
> > arch/arm/boot/dts/exynos4412-i9300.dt.yaml: l2-cache-controller@10502000:
> >         $nodename:0: 'l2-cache-controller@10502000' does not match '
> >         ^(cache-controller|cpu)(@[0-9a-f,]+)*$'
> >
> > Fixes: 56b60b8bce4a ("ARM: 8265/1: dts: exynos4: Add nodes for L2 cache controller")
>
> It is just naming convention introduced with dtschema so there is no
> error to fix.  Applied with modified commit msg.
>
> Best regards,
> Krzysztof
>

Thanks & Regards
-Anand

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, back to index

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-24 11:04 [PATCH] ARM: dts: exynos: Fix L2 cache-controller nodename Anand Moon
2020-06-26  7:13 ` Krzysztof Kozlowski
2020-06-28  8:26   ` Anand Moon

Linux-Samsung-soc Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-samsung-soc/0 linux-samsung-soc/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-samsung-soc linux-samsung-soc/ https://lore.kernel.org/linux-samsung-soc \
		linux-samsung-soc@vger.kernel.org
	public-inbox-index linux-samsung-soc

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-samsung-soc


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git