ARM: dts: exynos: Fix L2 cache-controller nodename
diff mbox series

Message ID 20200624110435.1150-1-linux.amoon@gmail.com
State Accepted
Commit 34f7afe2538ef01f3ff53f913d848e69084b6875
Headers show
Series
  • ARM: dts: exynos: Fix L2 cache-controller nodename
Related show

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

Patch
diff mbox series

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;