[PATCH] ARM: dts: exynos: Fix L2 cache-controller nodename

[PATCH] ARM: dts: exynos: Fix L2 cache-controller nodename

[Anand Moon]

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

Re: [PATCH] ARM: dts: exynos: Fix L2 cache-controller nodename

[Krzysztof Kozlowski]

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

Re: [PATCH] ARM: dts: exynos: Fix L2 cache-controller nodename

[Anand Moon]

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