diff mbox series

[7/7] arm64: dts: ti: k3-*: Fix up node_name_chars_strict errors

Message ID 20200901223059.14801-8-nm@ti.com (mailing list archive)
State New, archived
Headers show
Series arm64: dts: ti: k3-*: Squash up node_name_chars_strict warnings | expand

Commit Message

Nishanth Menon Sept. 1, 2020, 10:30 p.m. UTC
Building with W=2 throws up a bunch of easy to fixup errors..
node_name_chars_strict is one of them.. Knock those out.

Signed-off-by: Nishanth Menon <nm@ti.com>
---
 arch/arm64/boot/dts/ti/k3-am65-main.dtsi       |  6 +++---
 arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi        |  4 ++--
 arch/arm64/boot/dts/ti/k3-am654-base-board.dts | 16 ++++++++--------
 .../dts/ti/k3-am654-industrial-thermal.dtsi    | 12 ++++++------
 .../boot/dts/ti/k3-j721e-common-proc-board.dts | 18 +++++++++---------
 arch/arm64/boot/dts/ti/k3-j721e-main.dtsi      |  6 +++---
 .../arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi |  2 +-
 arch/arm64/boot/dts/ti/k3-j721e-som-p0.dtsi    |  2 +-
 8 files changed, 33 insertions(+), 33 deletions(-)

Comments

Suman Anna Sept. 2, 2020, 4:54 p.m. UTC | #1
On 9/1/20 5:30 PM, Nishanth Menon wrote:
> Building with W=2 throws up a bunch of easy to fixup errors..
> node_name_chars_strict is one of them.. Knock those out.
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
>  arch/arm64/boot/dts/ti/k3-am65-main.dtsi       |  6 +++---
>  arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi        |  4 ++--
>  arch/arm64/boot/dts/ti/k3-am654-base-board.dts | 16 ++++++++--------
>  .../dts/ti/k3-am654-industrial-thermal.dtsi    | 12 ++++++------
>  .../boot/dts/ti/k3-j721e-common-proc-board.dts | 18 +++++++++---------
>  arch/arm64/boot/dts/ti/k3-j721e-main.dtsi      |  6 +++---
>  .../arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi |  2 +-
>  arch/arm64/boot/dts/ti/k3-j721e-som-p0.dtsi    |  2 +-
>  8 files changed, 33 insertions(+), 33 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
> index 9c96e3f58c86..ff3e38408dbc 100644
> --- a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
> @@ -305,7 +305,7 @@
>  		no-1-8-v;
>  	};
>  
> -	scm_conf: scm_conf@100000 {
> +	scm_conf: scm-conf@100000 {
>  		compatible = "syscon", "simple-mfd";
>  		reg = <0 0x00100000 0 0x1c000>;
>  		#address-cells = <1>;
> @@ -344,7 +344,7 @@
>  					<0x4090 0x3>; /* SERDES1 lane select */
>  		};
>  
> -		dss_oldi_io_ctrl: dss_oldi_io_ctrl@41E0 {
> +		dss_oldi_io_ctrl: dss-oldi-io-ctrl@41E0 {
>  			compatible = "syscon";
>  			reg = <0x0000041E0 0x14>;

We should %s/41E0/41e0/, but that change is not directly associated with the
patch subject line. Perhaps, an additional patch? Remember seeing similar
warning on some downstream crypto nodes, but strangely I am not seeing on these
upstream nodes.

Otherwise,
Reviewed-by: Suman Anna <s-anna@ti.com>

regards
Suman

>  		};
> @@ -445,7 +445,7 @@
>  		ti,interrupt-ranges = <0 392 32>;
>  	};
>  
> -	main_navss {
> +	main-navss {
>  		compatible = "simple-mfd";
>  		#address-cells = <2>;
>  		#size-cells = <2>;
> diff --git a/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi b/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi
> index 6dfec68ac865..435e51019287 100644
> --- a/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi
> @@ -6,7 +6,7 @@
>   */
>  
>  &cbass_mcu {
> -	mcu_conf: scm_conf@40f00000 {
> +	mcu_conf: scm-conf@40f00000 {
>  		compatible = "syscon", "simple-mfd";
>  		reg = <0x0 0x40f00000 0x0 0x20000>;
>  		#address-cells = <1>;
> @@ -116,7 +116,7 @@
>  		};
>  	};
>  
> -	mcu_navss {
> +	mcu-navss {
>  		compatible = "simple-mfd";
>  		#address-cells = <2>;
>  		#size-cells = <2>;
> diff --git a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
> index 86c9074cb070..d12dd89f3405 100644
> --- a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
> @@ -29,7 +29,7 @@
>  		#address-cells = <2>;
>  		#size-cells = <2>;
>  		ranges;
> -		secure_ddr: secure_ddr@9e800000 {
> +		secure_ddr: secure-ddr@9e800000 {
>  			reg = <0 0x9e800000 0 0x01800000>; /* for OP-TEE */
>  			alignment = <0x1000>;
>  			no-map;
> @@ -70,14 +70,14 @@
>  		>;
>  	};
>  
> -	push_button_pins_default: push_button__pins_default {
> +	push_button_pins_default: push-button-pins-default {
>  		pinctrl-single,pins = <
>  			AM65X_WKUP_IOPAD(0x0030, PIN_INPUT, 7) /* (R5) WKUP_GPIO0_24 */
>  			AM65X_WKUP_IOPAD(0x003c, PIN_INPUT, 7) /* (P2) WKUP_GPIO0_27 */
>  		>;
>  	};
>  
> -	mcu_fss0_ospi0_pins_default: mcu-fss0-ospi0-pins_default {
> +	mcu_fss0_ospi0_pins_default: mcu-fss0-ospi0-pins-default {
>  		pinctrl-single,pins = <
>  			AM65X_WKUP_IOPAD(0x0000, PIN_OUTPUT, 0) /* (V1) MCU_OSPI0_CLK */
>  			AM65X_WKUP_IOPAD(0x0008, PIN_INPUT, 0)	 /* (U2) MCU_OSPI0_DQS */
> @@ -93,13 +93,13 @@
>  		>;
>  	};
>  
> -	wkup_pca554_default: wkup_pca554_default {
> +	wkup_pca554_default: wkup-pca554-default {
>  		pinctrl-single,pins = <
>  			AM65X_WKUP_IOPAD(0x0034, PIN_INPUT, 7) /* (T1) MCU_OSPI1_CLK.WKUP_GPIO0_25 */
>  		>;
>  	};
>  
> -	mcu_cpsw_pins_default: mcu_cpsw_pins_default {
> +	mcu_cpsw_pins_default: mcu-cpsw-pins-default {
>  		pinctrl-single,pins = <
>  			AM65X_WKUP_IOPAD(0x0058, PIN_OUTPUT, 0) /* (N4) MCU_RGMII1_TX_CTL */
>  			AM65X_WKUP_IOPAD(0x005c, PIN_INPUT, 0) /* (N5) MCU_RGMII1_RX_CTL */
> @@ -116,7 +116,7 @@
>  		>;
>  	};
>  
> -	mcu_mdio_pins_default: mcu_mdio1_pins_default {
> +	mcu_mdio_pins_default: mcu-mdio1-pins-default {
>  		pinctrl-single,pins = <
>  			AM65X_WKUP_IOPAD(0x008c, PIN_OUTPUT, 0) /* (L1) MCU_MDIO0_MDC */
>  			AM65X_WKUP_IOPAD(0x0088, PIN_INPUT, 0) /* (L4) MCU_MDIO0_MDIO */
> @@ -167,7 +167,7 @@
>  		>;
>  	};
>  
> -	main_mmc1_pins_default: main_mmc1_pins_default {
> +	main_mmc1_pins_default: main-mmc1-pins-default {
>  		pinctrl-single,pins = <
>  			AM65X_IOPAD(0x02d4, PIN_INPUT_PULLDOWN, 0) /* (C27) MMC1_CLK */
>  			AM65X_IOPAD(0x02d8, PIN_INPUT_PULLUP, 0) /* (C28) MMC1_CMD */
> @@ -180,7 +180,7 @@
>  		>;
>  	};
>  
> -	usb1_pins_default: usb1_pins_default {
> +	usb1_pins_default: usb1-pins-default {
>  		pinctrl-single,pins = <
>  			AM65X_IOPAD(0x02c0, PIN_OUTPUT, 0) /* (AC8) USB1_DRVVBUS */
>  		>;
> diff --git a/arch/arm64/boot/dts/ti/k3-am654-industrial-thermal.dtsi b/arch/arm64/boot/dts/ti/k3-am654-industrial-thermal.dtsi
> index cdc3d40c3f60..9021c738056b 100644
> --- a/arch/arm64/boot/dts/ti/k3-am654-industrial-thermal.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am654-industrial-thermal.dtsi
> @@ -2,13 +2,13 @@
>  
>  #include <dt-bindings/thermal/thermal.h>
>  
> -mpu0_thermal: mpu0_thermal {
> +mpu0_thermal: mpu0-thermal {
>  	polling-delay-passive = <250>; /* milliseconds */
>  	polling-delay = <500>; /* milliseconds */
>  	thermal-sensors = <&wkup_vtm0 0>;
>  
>  	trips {
> -		mpu0_crit: mpu0_crit {
> +		mpu0_crit: mpu0-crit {
>  			temperature = <125000>; /* milliCelsius */
>  			hysteresis = <2000>; /* milliCelsius */
>  			type = "critical";
> @@ -16,13 +16,13 @@ mpu0_thermal: mpu0_thermal {
>  	};
>  };
>  
> -mpu1_thermal: mpu1_thermal {
> +mpu1_thermal: mpu1-thermal {
>  	polling-delay-passive = <250>; /* milliseconds */
>  	polling-delay = <500>; /* milliseconds */
>  	thermal-sensors = <&wkup_vtm0 1>;
>  
>  	trips {
> -		mpu1_crit: mpu1_crit {
> +		mpu1_crit: mpu1-crit {
>  			temperature = <125000>; /* milliCelsius */
>  			hysteresis = <2000>; /* milliCelsius */
>  			type = "critical";
> @@ -30,13 +30,13 @@ mpu1_thermal: mpu1_thermal {
>  	};
>  };
>  
> -mcu_thermal: mcu_thermal {
> +mcu_thermal: mcu-thermal {
>  	polling-delay-passive = <250>; /* milliseconds */
>  	polling-delay = <500>; /* milliseconds */
>  	thermal-sensors = <&wkup_vtm0 2>;
>  
>  	trips {
> -		mcu_crit: mcu_crit {
> +		mcu_crit: mcu-crit {
>  			temperature = <125000>; /* milliCelsius */
>  			hysteresis = <2000>; /* milliCelsius */
>  			type = "critical";
> diff --git a/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
> index c355692796a9..648267284582 100644
> --- a/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
> +++ b/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
> @@ -86,13 +86,13 @@
>  };
>  
>  &main_pmx0 {
> -	sw10_button_pins_default: sw10_button_pins_default {
> +	sw10_button_pins_default: sw10-button-pins-default {
>  		pinctrl-single,pins = <
>  			J721E_IOPAD(0x0, PIN_INPUT, 7) /* (AC18) EXTINTn.GPIO0_0 */
>  		>;
>  	};
>  
> -	main_mmc1_pins_default: main_mmc1_pins_default {
> +	main_mmc1_pins_default: main-mmc1-pins-default {
>  		pinctrl-single,pins = <
>  			J721E_IOPAD(0x254, PIN_INPUT, 0) /* (R29) MMC1_CMD */
>  			J721E_IOPAD(0x250, PIN_INPUT, 0) /* (P25) MMC1_CLK */
> @@ -106,14 +106,14 @@
>  		>;
>  	};
>  
> -	main_usbss0_pins_default: main_usbss0_pins_default {
> +	main_usbss0_pins_default: main-usbss0-pins-default {
>  		pinctrl-single,pins = <
>  			J721E_IOPAD(0x290, PIN_OUTPUT, 0) /* (U6) USB0_DRVVBUS */
>  			J721E_IOPAD(0x210, PIN_INPUT, 7) /* (W3) MCAN1_RX.GPIO1_3 */
>  		>;
>  	};
>  
> -	main_usbss1_pins_default: main_usbss1_pins_default {
> +	main_usbss1_pins_default: main-usbss1-pins-default {
>  		pinctrl-single,pins = <
>  			J721E_IOPAD(0x214, PIN_OUTPUT, 4) /* (V4) MCAN1_TX.USB1_DRVVBUS */
>  		>;
> @@ -153,7 +153,7 @@
>  		>;
>  	};
>  
> -	mcasp10_pins_default: mcasp10_pins_default {
> +	mcasp10_pins_default: mcasp10-pins-default {
>  		pinctrl-single,pins = <
>  			J721E_IOPAD(0x158, PIN_OUTPUT_PULLDOWN, 12) /* (U23) RGMII5_TX_CTL.MCASP10_ACLKX */
>  			J721E_IOPAD(0x15c, PIN_OUTPUT_PULLDOWN, 12) /* (U26) RGMII5_RX_CTL.MCASP10_AFSX */
> @@ -167,7 +167,7 @@
>  		>;
>  	};
>  
> -	audi_ext_refclk2_pins_default: audi_ext_refclk2_pins_default {
> +	audi_ext_refclk2_pins_default: audi-ext-refclk2-pins-default {
>  		pinctrl-single,pins = <
>  			J721E_IOPAD(0x1a4, PIN_OUTPUT, 3) /* (W26) RGMII6_RXC.AUDIO_EXT_REFCLK2 */
>  		>;
> @@ -175,7 +175,7 @@
>  };
>  
>  &wkup_pmx0 {
> -	sw11_button_pins_default: sw11_button_pins_default {
> +	sw11_button_pins_default: sw11-button-pins-default {
>  		pinctrl-single,pins = <
>  			J721E_WKUP_IOPAD(0xcc, PIN_INPUT, 7) /* (G28) WKUP_GPIO0_7 */
>  		>;
> @@ -194,7 +194,7 @@
>  		>;
>  	};
>  
> -	mcu_cpsw_pins_default: mcu_cpsw_pins_default {
> +	mcu_cpsw_pins_default: mcu-cpsw-pins-default {
>  		pinctrl-single,pins = <
>  			J721E_WKUP_IOPAD(0x0058, PIN_OUTPUT, 0) /* MCU_RGMII1_TX_CTL */
>  			J721E_WKUP_IOPAD(0x005c, PIN_INPUT, 0) /* MCU_RGMII1_RX_CTL */
> @@ -211,7 +211,7 @@
>  		>;
>  	};
>  
> -	mcu_mdio_pins_default: mcu_mdio1_pins_default {
> +	mcu_mdio_pins_default: mcu-mdio1-pins-default {
>  		pinctrl-single,pins = <
>  			J721E_WKUP_IOPAD(0x008c, PIN_OUTPUT, 0) /* MCU_MDIO0_MDC */
>  			J721E_WKUP_IOPAD(0x0088, PIN_INPUT, 0) /* MCU_MDIO0_MDIO */
> diff --git a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi
> index 1d2a7c05b6f3..ea57d07777ba 100644
> --- a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi
> @@ -86,7 +86,7 @@
>  		ti,interrupt-ranges = <8 392 56>;
>  	};
>  
> -	main_navss {
> +	main-navss {
>  		compatible = "simple-mfd";
>  		#address-cells = <2>;
>  		#size-cells = <2>;
> @@ -888,7 +888,7 @@
>  		no-1-8-v;
>  	};
>  
> -	usbss0: cdns_usb@4104000 {
> +	usbss0: cdns-usb@4104000 {
>  		compatible = "ti,j721e-usb";
>  		reg = <0x00 0x4104000 0x00 0x100>;
>  		dma-coherent;
> @@ -918,7 +918,7 @@
>  		};
>  	};
>  
> -	usbss1: cdns_usb@4114000 {
> +	usbss1: cdns-usb@4114000 {
>  		compatible = "ti,j721e-usb";
>  		reg = <0x00 0x4114000 0x00 0x100>;
>  		dma-coherent;
> diff --git a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
> index 81801f519a61..03fa69c0a038 100644
> --- a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
> @@ -249,7 +249,7 @@
>  		};
>  	};
>  
> -	mcu_navss {
> +	mcu-navss {
>  		compatible = "simple-mfd";
>  		#address-cells = <2>;
>  		#size-cells = <2>;
> diff --git a/arch/arm64/boot/dts/ti/k3-j721e-som-p0.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-som-p0.dtsi
> index d69d90c8b5e3..5dc3ba739131 100644
> --- a/arch/arm64/boot/dts/ti/k3-j721e-som-p0.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-j721e-som-p0.dtsi
> @@ -71,7 +71,7 @@
>  };
>  
>  &wkup_pmx0 {
> -	wkup_i2c0_pins_default: wkup_i2c0_pins_default {
> +	wkup_i2c0_pins_default: wkup-i2c0-pins-default {
>  		pinctrl-single,pins = <
>  			J721E_WKUP_IOPAD(0xf8, PIN_INPUT_PULLUP, 0) /* (J25) WKUP_I2C0_SCL */
>  			J721E_WKUP_IOPAD(0xfc, PIN_INPUT_PULLUP, 0) /* (H24) WKUP_I2C0_SDA */
>
Nishanth Menon Sept. 2, 2020, 6:16 p.m. UTC | #2
On 11:54-20200902, Suman Anna wrote:
> On 9/1/20 5:30 PM, Nishanth Menon wrote:
> > Building with W=2 throws up a bunch of easy to fixup errors..
> > node_name_chars_strict is one of them.. Knock those out.
> > 
> > Signed-off-by: Nishanth Menon <nm@ti.com>
> > ---
> >  arch/arm64/boot/dts/ti/k3-am65-main.dtsi       |  6 +++---
> >  arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi        |  4 ++--
> >  arch/arm64/boot/dts/ti/k3-am654-base-board.dts | 16 ++++++++--------
> >  .../dts/ti/k3-am654-industrial-thermal.dtsi    | 12 ++++++------
> >  .../boot/dts/ti/k3-j721e-common-proc-board.dts | 18 +++++++++---------
> >  arch/arm64/boot/dts/ti/k3-j721e-main.dtsi      |  6 +++---
> >  .../arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi |  2 +-
> >  arch/arm64/boot/dts/ti/k3-j721e-som-p0.dtsi    |  2 +-
> >  8 files changed, 33 insertions(+), 33 deletions(-)
> > 
> > diff --git a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
> > index 9c96e3f58c86..ff3e38408dbc 100644
> > --- a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
> > +++ b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
> > @@ -305,7 +305,7 @@
> >  		no-1-8-v;
> >  	};
> >  
> > -	scm_conf: scm_conf@100000 {
> > +	scm_conf: scm-conf@100000 {
> >  		compatible = "syscon", "simple-mfd";
> >  		reg = <0 0x00100000 0 0x1c000>;
> >  		#address-cells = <1>;
> > @@ -344,7 +344,7 @@
> >  					<0x4090 0x3>; /* SERDES1 lane select */
> >  		};
> >  
> > -		dss_oldi_io_ctrl: dss_oldi_io_ctrl@41E0 {
> > +		dss_oldi_io_ctrl: dss-oldi-io-ctrl@41E0 {
> >  			compatible = "syscon";
> >  			reg = <0x0000041E0 0x14>;
> 
> We should %s/41E0/41e0/, but that change is not directly associated with the
> patch subject line. Perhaps, an additional patch? Remember seeing similar
> warning on some downstream crypto nodes, but strangely I am not seeing on these
> upstream nodes.

yeah - probably a separate patch, but thanks, i can fix it as well(will
check if there are similar ones around as well).
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
index 9c96e3f58c86..ff3e38408dbc 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
@@ -305,7 +305,7 @@ 
 		no-1-8-v;
 	};
 
-	scm_conf: scm_conf@100000 {
+	scm_conf: scm-conf@100000 {
 		compatible = "syscon", "simple-mfd";
 		reg = <0 0x00100000 0 0x1c000>;
 		#address-cells = <1>;
@@ -344,7 +344,7 @@ 
 					<0x4090 0x3>; /* SERDES1 lane select */
 		};
 
-		dss_oldi_io_ctrl: dss_oldi_io_ctrl@41E0 {
+		dss_oldi_io_ctrl: dss-oldi-io-ctrl@41E0 {
 			compatible = "syscon";
 			reg = <0x0000041E0 0x14>;
 		};
@@ -445,7 +445,7 @@ 
 		ti,interrupt-ranges = <0 392 32>;
 	};
 
-	main_navss {
+	main-navss {
 		compatible = "simple-mfd";
 		#address-cells = <2>;
 		#size-cells = <2>;
diff --git a/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi b/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi
index 6dfec68ac865..435e51019287 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi
@@ -6,7 +6,7 @@ 
  */
 
 &cbass_mcu {
-	mcu_conf: scm_conf@40f00000 {
+	mcu_conf: scm-conf@40f00000 {
 		compatible = "syscon", "simple-mfd";
 		reg = <0x0 0x40f00000 0x0 0x20000>;
 		#address-cells = <1>;
@@ -116,7 +116,7 @@ 
 		};
 	};
 
-	mcu_navss {
+	mcu-navss {
 		compatible = "simple-mfd";
 		#address-cells = <2>;
 		#size-cells = <2>;
diff --git a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
index 86c9074cb070..d12dd89f3405 100644
--- a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
+++ b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
@@ -29,7 +29,7 @@ 
 		#address-cells = <2>;
 		#size-cells = <2>;
 		ranges;
-		secure_ddr: secure_ddr@9e800000 {
+		secure_ddr: secure-ddr@9e800000 {
 			reg = <0 0x9e800000 0 0x01800000>; /* for OP-TEE */
 			alignment = <0x1000>;
 			no-map;
@@ -70,14 +70,14 @@ 
 		>;
 	};
 
-	push_button_pins_default: push_button__pins_default {
+	push_button_pins_default: push-button-pins-default {
 		pinctrl-single,pins = <
 			AM65X_WKUP_IOPAD(0x0030, PIN_INPUT, 7) /* (R5) WKUP_GPIO0_24 */
 			AM65X_WKUP_IOPAD(0x003c, PIN_INPUT, 7) /* (P2) WKUP_GPIO0_27 */
 		>;
 	};
 
-	mcu_fss0_ospi0_pins_default: mcu-fss0-ospi0-pins_default {
+	mcu_fss0_ospi0_pins_default: mcu-fss0-ospi0-pins-default {
 		pinctrl-single,pins = <
 			AM65X_WKUP_IOPAD(0x0000, PIN_OUTPUT, 0) /* (V1) MCU_OSPI0_CLK */
 			AM65X_WKUP_IOPAD(0x0008, PIN_INPUT, 0)	 /* (U2) MCU_OSPI0_DQS */
@@ -93,13 +93,13 @@ 
 		>;
 	};
 
-	wkup_pca554_default: wkup_pca554_default {
+	wkup_pca554_default: wkup-pca554-default {
 		pinctrl-single,pins = <
 			AM65X_WKUP_IOPAD(0x0034, PIN_INPUT, 7) /* (T1) MCU_OSPI1_CLK.WKUP_GPIO0_25 */
 		>;
 	};
 
-	mcu_cpsw_pins_default: mcu_cpsw_pins_default {
+	mcu_cpsw_pins_default: mcu-cpsw-pins-default {
 		pinctrl-single,pins = <
 			AM65X_WKUP_IOPAD(0x0058, PIN_OUTPUT, 0) /* (N4) MCU_RGMII1_TX_CTL */
 			AM65X_WKUP_IOPAD(0x005c, PIN_INPUT, 0) /* (N5) MCU_RGMII1_RX_CTL */
@@ -116,7 +116,7 @@ 
 		>;
 	};
 
-	mcu_mdio_pins_default: mcu_mdio1_pins_default {
+	mcu_mdio_pins_default: mcu-mdio1-pins-default {
 		pinctrl-single,pins = <
 			AM65X_WKUP_IOPAD(0x008c, PIN_OUTPUT, 0) /* (L1) MCU_MDIO0_MDC */
 			AM65X_WKUP_IOPAD(0x0088, PIN_INPUT, 0) /* (L4) MCU_MDIO0_MDIO */
@@ -167,7 +167,7 @@ 
 		>;
 	};
 
-	main_mmc1_pins_default: main_mmc1_pins_default {
+	main_mmc1_pins_default: main-mmc1-pins-default {
 		pinctrl-single,pins = <
 			AM65X_IOPAD(0x02d4, PIN_INPUT_PULLDOWN, 0) /* (C27) MMC1_CLK */
 			AM65X_IOPAD(0x02d8, PIN_INPUT_PULLUP, 0) /* (C28) MMC1_CMD */
@@ -180,7 +180,7 @@ 
 		>;
 	};
 
-	usb1_pins_default: usb1_pins_default {
+	usb1_pins_default: usb1-pins-default {
 		pinctrl-single,pins = <
 			AM65X_IOPAD(0x02c0, PIN_OUTPUT, 0) /* (AC8) USB1_DRVVBUS */
 		>;
diff --git a/arch/arm64/boot/dts/ti/k3-am654-industrial-thermal.dtsi b/arch/arm64/boot/dts/ti/k3-am654-industrial-thermal.dtsi
index cdc3d40c3f60..9021c738056b 100644
--- a/arch/arm64/boot/dts/ti/k3-am654-industrial-thermal.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am654-industrial-thermal.dtsi
@@ -2,13 +2,13 @@ 
 
 #include <dt-bindings/thermal/thermal.h>
 
-mpu0_thermal: mpu0_thermal {
+mpu0_thermal: mpu0-thermal {
 	polling-delay-passive = <250>; /* milliseconds */
 	polling-delay = <500>; /* milliseconds */
 	thermal-sensors = <&wkup_vtm0 0>;
 
 	trips {
-		mpu0_crit: mpu0_crit {
+		mpu0_crit: mpu0-crit {
 			temperature = <125000>; /* milliCelsius */
 			hysteresis = <2000>; /* milliCelsius */
 			type = "critical";
@@ -16,13 +16,13 @@  mpu0_thermal: mpu0_thermal {
 	};
 };
 
-mpu1_thermal: mpu1_thermal {
+mpu1_thermal: mpu1-thermal {
 	polling-delay-passive = <250>; /* milliseconds */
 	polling-delay = <500>; /* milliseconds */
 	thermal-sensors = <&wkup_vtm0 1>;
 
 	trips {
-		mpu1_crit: mpu1_crit {
+		mpu1_crit: mpu1-crit {
 			temperature = <125000>; /* milliCelsius */
 			hysteresis = <2000>; /* milliCelsius */
 			type = "critical";
@@ -30,13 +30,13 @@  mpu1_thermal: mpu1_thermal {
 	};
 };
 
-mcu_thermal: mcu_thermal {
+mcu_thermal: mcu-thermal {
 	polling-delay-passive = <250>; /* milliseconds */
 	polling-delay = <500>; /* milliseconds */
 	thermal-sensors = <&wkup_vtm0 2>;
 
 	trips {
-		mcu_crit: mcu_crit {
+		mcu_crit: mcu-crit {
 			temperature = <125000>; /* milliCelsius */
 			hysteresis = <2000>; /* milliCelsius */
 			type = "critical";
diff --git a/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
index c355692796a9..648267284582 100644
--- a/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
+++ b/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
@@ -86,13 +86,13 @@ 
 };
 
 &main_pmx0 {
-	sw10_button_pins_default: sw10_button_pins_default {
+	sw10_button_pins_default: sw10-button-pins-default {
 		pinctrl-single,pins = <
 			J721E_IOPAD(0x0, PIN_INPUT, 7) /* (AC18) EXTINTn.GPIO0_0 */
 		>;
 	};
 
-	main_mmc1_pins_default: main_mmc1_pins_default {
+	main_mmc1_pins_default: main-mmc1-pins-default {
 		pinctrl-single,pins = <
 			J721E_IOPAD(0x254, PIN_INPUT, 0) /* (R29) MMC1_CMD */
 			J721E_IOPAD(0x250, PIN_INPUT, 0) /* (P25) MMC1_CLK */
@@ -106,14 +106,14 @@ 
 		>;
 	};
 
-	main_usbss0_pins_default: main_usbss0_pins_default {
+	main_usbss0_pins_default: main-usbss0-pins-default {
 		pinctrl-single,pins = <
 			J721E_IOPAD(0x290, PIN_OUTPUT, 0) /* (U6) USB0_DRVVBUS */
 			J721E_IOPAD(0x210, PIN_INPUT, 7) /* (W3) MCAN1_RX.GPIO1_3 */
 		>;
 	};
 
-	main_usbss1_pins_default: main_usbss1_pins_default {
+	main_usbss1_pins_default: main-usbss1-pins-default {
 		pinctrl-single,pins = <
 			J721E_IOPAD(0x214, PIN_OUTPUT, 4) /* (V4) MCAN1_TX.USB1_DRVVBUS */
 		>;
@@ -153,7 +153,7 @@ 
 		>;
 	};
 
-	mcasp10_pins_default: mcasp10_pins_default {
+	mcasp10_pins_default: mcasp10-pins-default {
 		pinctrl-single,pins = <
 			J721E_IOPAD(0x158, PIN_OUTPUT_PULLDOWN, 12) /* (U23) RGMII5_TX_CTL.MCASP10_ACLKX */
 			J721E_IOPAD(0x15c, PIN_OUTPUT_PULLDOWN, 12) /* (U26) RGMII5_RX_CTL.MCASP10_AFSX */
@@ -167,7 +167,7 @@ 
 		>;
 	};
 
-	audi_ext_refclk2_pins_default: audi_ext_refclk2_pins_default {
+	audi_ext_refclk2_pins_default: audi-ext-refclk2-pins-default {
 		pinctrl-single,pins = <
 			J721E_IOPAD(0x1a4, PIN_OUTPUT, 3) /* (W26) RGMII6_RXC.AUDIO_EXT_REFCLK2 */
 		>;
@@ -175,7 +175,7 @@ 
 };
 
 &wkup_pmx0 {
-	sw11_button_pins_default: sw11_button_pins_default {
+	sw11_button_pins_default: sw11-button-pins-default {
 		pinctrl-single,pins = <
 			J721E_WKUP_IOPAD(0xcc, PIN_INPUT, 7) /* (G28) WKUP_GPIO0_7 */
 		>;
@@ -194,7 +194,7 @@ 
 		>;
 	};
 
-	mcu_cpsw_pins_default: mcu_cpsw_pins_default {
+	mcu_cpsw_pins_default: mcu-cpsw-pins-default {
 		pinctrl-single,pins = <
 			J721E_WKUP_IOPAD(0x0058, PIN_OUTPUT, 0) /* MCU_RGMII1_TX_CTL */
 			J721E_WKUP_IOPAD(0x005c, PIN_INPUT, 0) /* MCU_RGMII1_RX_CTL */
@@ -211,7 +211,7 @@ 
 		>;
 	};
 
-	mcu_mdio_pins_default: mcu_mdio1_pins_default {
+	mcu_mdio_pins_default: mcu-mdio1-pins-default {
 		pinctrl-single,pins = <
 			J721E_WKUP_IOPAD(0x008c, PIN_OUTPUT, 0) /* MCU_MDIO0_MDC */
 			J721E_WKUP_IOPAD(0x0088, PIN_INPUT, 0) /* MCU_MDIO0_MDIO */
diff --git a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi
index 1d2a7c05b6f3..ea57d07777ba 100644
--- a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi
@@ -86,7 +86,7 @@ 
 		ti,interrupt-ranges = <8 392 56>;
 	};
 
-	main_navss {
+	main-navss {
 		compatible = "simple-mfd";
 		#address-cells = <2>;
 		#size-cells = <2>;
@@ -888,7 +888,7 @@ 
 		no-1-8-v;
 	};
 
-	usbss0: cdns_usb@4104000 {
+	usbss0: cdns-usb@4104000 {
 		compatible = "ti,j721e-usb";
 		reg = <0x00 0x4104000 0x00 0x100>;
 		dma-coherent;
@@ -918,7 +918,7 @@ 
 		};
 	};
 
-	usbss1: cdns_usb@4114000 {
+	usbss1: cdns-usb@4114000 {
 		compatible = "ti,j721e-usb";
 		reg = <0x00 0x4114000 0x00 0x100>;
 		dma-coherent;
diff --git a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
index 81801f519a61..03fa69c0a038 100644
--- a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
@@ -249,7 +249,7 @@ 
 		};
 	};
 
-	mcu_navss {
+	mcu-navss {
 		compatible = "simple-mfd";
 		#address-cells = <2>;
 		#size-cells = <2>;
diff --git a/arch/arm64/boot/dts/ti/k3-j721e-som-p0.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-som-p0.dtsi
index d69d90c8b5e3..5dc3ba739131 100644
--- a/arch/arm64/boot/dts/ti/k3-j721e-som-p0.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j721e-som-p0.dtsi
@@ -71,7 +71,7 @@ 
 };
 
 &wkup_pmx0 {
-	wkup_i2c0_pins_default: wkup_i2c0_pins_default {
+	wkup_i2c0_pins_default: wkup-i2c0-pins-default {
 		pinctrl-single,pins = <
 			J721E_WKUP_IOPAD(0xf8, PIN_INPUT_PULLUP, 0) /* (J25) WKUP_I2C0_SCL */
 			J721E_WKUP_IOPAD(0xfc, PIN_INPUT_PULLUP, 0) /* (H24) WKUP_I2C0_SDA */