diff mbox series

ARM: dts: exynos: Fix L2 cache-controller nodename

Message ID 20200624110435.1150-1-linux.amoon@gmail.com (mailing list archive)
State Mainlined
Commit a184ea9f41da3a65150713656cea58dc964af454
Headers show
Series ARM: dts: exynos: Fix L2 cache-controller nodename | expand

Commit Message

Anand Moon June 24, 2020, 11:04 a.m. UTC
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(-)

Comments

Krzysztof Kozlowski June 26, 2020, 7:13 a.m. UTC | #1
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
Anand Moon June 28, 2020, 8:26 a.m. UTC | #2
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
diff mbox series

Patch

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;