diff mbox series

[04/12] ARM: dts: exynos: adjust node names to DT spec in Exynos4412 boards

Message ID 20201026181528.163143-5-krzk@kernel.org (mailing list archive)
State Superseded
Headers show
Series ARM: dts: samsung: minor node name cleanups | expand

Commit Message

Krzysztof Kozlowski Oct. 26, 2020, 6:15 p.m. UTC
The Devicetree specification expects device node names to have a generic
name, representing the class of a device.  Also the convention for node
names is to use hyphens, not underscores.

No functional changes.

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 arch/arm/boot/dts/exynos4412-galaxy-s3.dtsi   |  8 ++++----
 arch/arm/boot/dts/exynos4412-itop-elite.dts   |  2 +-
 .../boot/dts/exynos4412-itop-scp-core.dtsi    |  2 +-
 arch/arm/boot/dts/exynos4412-midas.dtsi       | 12 +++++------
 arch/arm/boot/dts/exynos4412-n710x.dts        |  2 +-
 .../boot/dts/exynos4412-odroid-common.dtsi    |  8 ++++----
 arch/arm/boot/dts/exynos4412-odroidx.dts      |  4 ++--
 arch/arm/boot/dts/exynos4412-origen.dts       | 14 ++++++-------
 arch/arm/boot/dts/exynos4412-smdk4412.dts     | 20 +++++++++----------
 9 files changed, 36 insertions(+), 36 deletions(-)

Comments

Marek Szyprowski Oct. 26, 2020, 9:02 p.m. UTC | #1
On 26.10.2020 19:15, Krzysztof Kozlowski wrote:
> The Devicetree specification expects device node names to have a generic
> name, representing the class of a device.  Also the convention for node
> names is to use hyphens, not underscores.
>
> No functional changes.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
>   arch/arm/boot/dts/exynos4412-galaxy-s3.dtsi   |  8 ++++----
>   arch/arm/boot/dts/exynos4412-itop-elite.dts   |  2 +-
>   .../boot/dts/exynos4412-itop-scp-core.dtsi    |  2 +-
>   arch/arm/boot/dts/exynos4412-midas.dtsi       | 12 +++++------
>   arch/arm/boot/dts/exynos4412-n710x.dts        |  2 +-
>   .../boot/dts/exynos4412-odroid-common.dtsi    |  8 ++++----
>   arch/arm/boot/dts/exynos4412-odroidx.dts      |  4 ++--
>   arch/arm/boot/dts/exynos4412-origen.dts       | 14 ++++++-------
>   arch/arm/boot/dts/exynos4412-smdk4412.dts     | 20 +++++++++----------
>   9 files changed, 36 insertions(+), 36 deletions(-)
>
> diff --git a/arch/arm/boot/dts/exynos4412-galaxy-s3.dtsi b/arch/arm/boot/dts/exynos4412-galaxy-s3.dtsi
> index 89ed81fb348d..db91678f5648 100644
> --- a/arch/arm/boot/dts/exynos4412-galaxy-s3.dtsi
> +++ b/arch/arm/boot/dts/exynos4412-galaxy-s3.dtsi
> @@ -15,7 +15,7 @@ aliases {
>   		i2c10 = &i2c_cm36651;
>   	};
>   
> -	aat1290 {
> +	led-controller {
>   		compatible = "skyworks,aat1290";
>   		flen-gpios = <&gpj1 1 GPIO_ACTIVE_HIGH>;
>   		enset-gpios = <&gpj1 2 GPIO_ACTIVE_HIGH>;
> @@ -60,7 +60,7 @@ i2c_ak8975: i2c-gpio-0 {
>   		#size-cells = <0>;
>   		status = "okay";
>   
> -		ak8975@c {
> +		magnetometer@c {
>   			compatible = "asahi-kasei,ak8975";
>   			reg = <0x0c>;
>   			gpios = <&gpj0 7 GPIO_ACTIVE_HIGH>;
> @@ -75,7 +75,7 @@ i2c_cm36651: i2c-gpio-2 {
>   		#address-cells = <1>;
>   		#size-cells = <0>;
>   
> -		cm36651@18 {
> +		sensor@18 {
light-sensor?
>   			compatible = "capella,cm36651";
>   			reg = <0x18>;
>   			interrupt-parent = <&gpx0>;
> @@ -133,7 +133,7 @@ timing-0 {
>   };
>   
>   &i2c_3 {
> -	mms114-touchscreen@48 {
> +	touchscreen@48 {
>   		compatible = "melfas,mms114";
>   		reg = <0x48>;
>   		interrupt-parent = <&gpm2>;
> diff --git a/arch/arm/boot/dts/exynos4412-itop-elite.dts b/arch/arm/boot/dts/exynos4412-itop-elite.dts
> index f6d0a5f5d339..3063e44fcbcc 100644
> --- a/arch/arm/boot/dts/exynos4412-itop-elite.dts
> +++ b/arch/arm/boot/dts/exynos4412-itop-elite.dts
> @@ -175,7 +175,7 @@ &i2c_4 {
>   	pinctrl-names = "default";
>   	status = "okay";
>   
> -	codec: wm8960@1a {
> +	codec: codec@1a {
audio-codec?
>   		compatible = "wlf,wm8960";
>   		reg = <0x1a>;
>   		clocks = <&pmu_system_controller 0>;
> diff --git a/arch/arm/boot/dts/exynos4412-itop-scp-core.dtsi b/arch/arm/boot/dts/exynos4412-itop-scp-core.dtsi
> index dfceb155b3a7..4583d342af39 100644
> --- a/arch/arm/boot/dts/exynos4412-itop-scp-core.dtsi
> +++ b/arch/arm/boot/dts/exynos4412-itop-scp-core.dtsi
> @@ -134,7 +134,7 @@ &i2c_1 {
>   	pinctrl-names = "default";
>   	status = "okay";
>   
> -	s5m8767: s5m8767-pmic@66 {
> +	s5m8767: pmic@66 {
>   		compatible = "samsung,s5m8767-pmic";
>   		reg = <0x66>;
>   
> diff --git a/arch/arm/boot/dts/exynos4412-midas.dtsi b/arch/arm/boot/dts/exynos4412-midas.dtsi
> index 7e7c243ff196..52a4f6e6c7f2 100644
> --- a/arch/arm/boot/dts/exynos4412-midas.dtsi
> +++ b/arch/arm/boot/dts/exynos4412-midas.dtsi
> @@ -220,7 +220,7 @@ i2c_max77693_fuel: i2c-gpio-3 {
>   		#size-cells = <0>;
>   		status = "okay";
>   
> -		max77693-fuel-gauge@36 {
> +		fuel-gauge@36 {
>   			compatible = "maxim,max17047";
>   			interrupt-parent = <&gpx2>;
>   			interrupts = <3 IRQ_TYPE_EDGE_FALLING>;
> @@ -550,7 +550,7 @@ &i2c_0 {
>   	pinctrl-names = "default";
>   	status = "okay";
>   
> -	s5c73m3: s5c73m3@3c {
> +	s5c73m3: sensor@3c {
image-sensor?
>   		compatible = "samsung,s5c73m3";
>   		reg = <0x3c>;
>   		xshutdown-gpios = <&gpf1 3 GPIO_ACTIVE_LOW>; /* ISP_RESET */
> @@ -577,7 +577,7 @@ &i2c1_isp {
>   	pinctrl-0 = <&fimc_is_i2c1>;
>   	pinctrl-names = "default";
>   
> -	s5k6a3@10 {
> +	sensor@10 {
image-sensor?
>   		compatible = "samsung,s5k6a3";
>   		reg = <0x10>;
>   		svdda-supply = <&cam_io_reg>;
> @@ -616,7 +616,7 @@ &i2c_4 {
>   	pinctrl-names = "default";
>   	status = "okay";
>   
> -	wm1811: wm1811@1a {
> +	wm1811: codec@1a {
audio-codec?
>   		compatible = "wlf,wm1811";
>   		reg = <0x1a>;
>   		clocks = <&pmu_system_controller 0>,
> @@ -665,7 +665,7 @@ &i2c_7 {
>   	pinctrl-names = "default";
>   	status = "okay";
>   
> -	max77686: max77686_pmic@9 {
> +	max77686: pmic@9 {
>   		compatible = "maxim,max77686";
>   		interrupt-parent = <&gpx0>;
>   		interrupts = <7 IRQ_TYPE_NONE>;
> @@ -1407,7 +1407,7 @@ &spi_1 {
>   	cs-gpios = <&gpb 5 GPIO_ACTIVE_HIGH>;
>   	status = "okay";
>   
> -	s5c73m3_spi: s5c73m3@0 {
> +	s5c73m3_spi: sensor@0 {
image-sensor?
>   		compatible = "samsung,s5c73m3";
>   		spi-max-frequency = <50000000>;
>   		reg = <0>;
> diff --git a/arch/arm/boot/dts/exynos4412-n710x.dts b/arch/arm/boot/dts/exynos4412-n710x.dts
> index a47b7f35fc80..c49dbb7847b8 100644
> --- a/arch/arm/boot/dts/exynos4412-n710x.dts
> +++ b/arch/arm/boot/dts/exynos4412-n710x.dts
> @@ -45,7 +45,7 @@ &i2c_3 {
>   	pinctrl-names = "default";
>   	status = "okay";
>   
> -	mms152-touchscreen@48 {
> +	touchscreen@48 {
>   		compatible = "melfas,mms152";
>   		reg = <0x48>;
>   		interrupt-parent = <&gpm2>;
> diff --git a/arch/arm/boot/dts/exynos4412-odroid-common.dtsi b/arch/arm/boot/dts/exynos4412-odroid-common.dtsi
> index c3b6e6d367ab..3dcd7f7f10f3 100644
> --- a/arch/arm/boot/dts/exynos4412-odroid-common.dtsi
> +++ b/arch/arm/boot/dts/exynos4412-odroid-common.dtsi
> @@ -27,7 +27,7 @@ gpio_keys: gpio-keys {
>   		pinctrl-names = "default";
>   		pinctrl-0 = <&gpio_power_key>;
>   
> -		power_key {
> +		power-key {
>   			gpios = <&gpx1 3 GPIO_ACTIVE_LOW>;
>   			linux,code = <KEY_POWER>;
>   			label = "power key";
> @@ -171,7 +171,7 @@ cooling_map1: map1 {
>   };
>   
>   &pinctrl_1 {
> -	gpio_power_key: power_key {
> +	gpio_power_key: power-key {
>   		samsung,pins = "gpx1-3";
>   		samsung,pin-pud = <EXYNOS_PIN_PULL_NONE>;
>   	};
> @@ -266,7 +266,7 @@ &i2c_0 {
>   	samsung,i2c-max-bus-freq = <400000>;
>   	status = "okay";
>   
> -	usb3503: usb3503@8 {
> +	usb3503: usb-hub@8 {
>   		compatible = "smsc,usb3503";
>   		reg = <0x08>;
>   
> @@ -492,7 +492,7 @@ buck8_reg: BUCK8 {
>   
>   &i2c_1 {
>   	status = "okay";
> -	max98090: max98090@10 {
> +	max98090: codec@10 {
audio-codec?
>   		compatible = "maxim,max98090";
>   		reg = <0x10>;
>   		interrupt-parent = <&gpx0>;
> diff --git a/arch/arm/boot/dts/exynos4412-odroidx.dts b/arch/arm/boot/dts/exynos4412-odroidx.dts
> index 68fe88074d1d..46381e9097f4 100644
> --- a/arch/arm/boot/dts/exynos4412-odroidx.dts
> +++ b/arch/arm/boot/dts/exynos4412-odroidx.dts
> @@ -36,7 +36,7 @@ led2 {
>   		};
>   	};
>   
> -	regulator_p3v3 {
> +	regulator-1 {
>   		compatible = "regulator-fixed";
>   		regulator-name = "p3v3_en";
>   		regulator-min-microvolt = <3300000>;
> @@ -81,7 +81,7 @@ &mshc_0 {
>   };
>   
>   &pinctrl_1 {
> -	gpio_home_key: home_key {
> +	gpio_home_key: home-key {
>   		samsung,pins = "gpx2-2";
>   		samsung,pin-pud = <EXYNOS_PIN_PULL_NONE>;
>   	};
> diff --git a/arch/arm/boot/dts/exynos4412-origen.dts b/arch/arm/boot/dts/exynos4412-origen.dts
> index c2e793b69e7d..e1f6de53e20e 100644
> --- a/arch/arm/boot/dts/exynos4412-origen.dts
> +++ b/arch/arm/boot/dts/exynos4412-origen.dts
> @@ -116,7 +116,7 @@ &i2c_0 {
>   	pinctrl-names = "default";
>   	status = "okay";
>   
> -	s5m8767_pmic@66 {
> +	pmic@66 {
>   		compatible = "samsung,s5m8767-pmic";
>   		reg = <0x66>;
>   
> @@ -453,37 +453,37 @@ &keypad {
>   	pinctrl-names = "default";
>   	status = "okay";
>   
> -	key_home {
> +	key-home {
>   		keypad,row = <0>;
>   		keypad,column = <0>;
>   		linux,code = <KEY_HOME>;
>   	};
>   
> -	key_down {
> +	key-down {
>   		keypad,row = <0>;
>   		keypad,column = <1>;
>   		linux,code = <KEY_DOWN>;
>   	};
>   
> -	key_up {
> +	key-up {
>   		keypad,row = <1>;
>   		keypad,column = <0>;
>   		linux,code = <KEY_UP>;
>   	};
>   
> -	key_menu {
> +	key-menu {
>   		keypad,row = <1>;
>   		keypad,column = <1>;
>   		linux,code = <KEY_MENU>;
>   	};
>   
> -	key_back {
> +	key-back {
>   		keypad,row = <2>;
>   		keypad,column = <0>;
>   		linux,code = <KEY_BACK>;
>   	};
>   
> -	key_enter {
> +	key-enter {
>   		keypad,row = <2>;
>   		keypad,column = <1>;
>   		linux,code = <KEY_ENTER>;
> diff --git a/arch/arm/boot/dts/exynos4412-smdk4412.dts b/arch/arm/boot/dts/exynos4412-smdk4412.dts
> index 49971203a8aa..cc99b955af0c 100644
> --- a/arch/arm/boot/dts/exynos4412-smdk4412.dts
> +++ b/arch/arm/boot/dts/exynos4412-smdk4412.dts
> @@ -71,61 +71,61 @@ &keypad {
>   	pinctrl-names = "default";
>   	status = "okay";
>   
> -	key_1 {
> +	key-1 {
>   		keypad,row = <1>;
>   		keypad,column = <3>;
>   		linux,code = <2>;
>   	};
>   
> -	key_2 {
> +	key-2 {
>   		keypad,row = <1>;
>   		keypad,column = <4>;
>   		linux,code = <3>;
>   	};
>   
> -	key_3 {
> +	key-3 {
>   		keypad,row = <1>;
>   		keypad,column = <5>;
>   		linux,code = <4>;
>   	};
>   
> -	key_4 {
> +	key-4 {
>   		keypad,row = <1>;
>   		keypad,column = <6>;
>   		linux,code = <5>;
>   	};
>   
> -	key_5 {
> +	key-5 {
>   		keypad,row = <1>;
>   		keypad,column = <7>;
>   		linux,code = <6>;
>   	};
>   
> -	key_A {
> +	key-A {
>   		keypad,row = <2>;
>   		keypad,column = <6>;
>   		linux,code = <30>;
>   	};
>   
> -	key_B {
> +	key-B {
>   		keypad,row = <2>;
>   		keypad,column = <7>;
>   		linux,code = <48>;
>   	};
>   
> -	key_C {
> +	key-C {
>   		keypad,row = <0>;
>   		keypad,column = <5>;
>   		linux,code = <46>;
>   	};
>   
> -	key_D {
> +	key-D {
>   		keypad,row = <2>;
>   		keypad,column = <5>;
>   		linux,code = <32>;
>   	};
>   
> -	key_E {
> +	key-E {
>   		keypad,row = <0>;
>   		keypad,column = <7>;
>   		linux,code = <18>;

Best regards
Krzysztof Kozlowski Oct. 27, 2020, 10:38 a.m. UTC | #2
On Mon, Oct 26, 2020 at 10:02:10PM +0100, Marek Szyprowski wrote:
> 
> On 26.10.2020 19:15, Krzysztof Kozlowski wrote:
> > The Devicetree specification expects device node names to have a generic
> > name, representing the class of a device.  Also the convention for node
> > names is to use hyphens, not underscores.
> >
> > No functional changes.
> >
> > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> > ---
> >   arch/arm/boot/dts/exynos4412-galaxy-s3.dtsi   |  8 ++++----
> >   arch/arm/boot/dts/exynos4412-itop-elite.dts   |  2 +-
> >   .../boot/dts/exynos4412-itop-scp-core.dtsi    |  2 +-
> >   arch/arm/boot/dts/exynos4412-midas.dtsi       | 12 +++++------
> >   arch/arm/boot/dts/exynos4412-n710x.dts        |  2 +-
> >   .../boot/dts/exynos4412-odroid-common.dtsi    |  8 ++++----
> >   arch/arm/boot/dts/exynos4412-odroidx.dts      |  4 ++--
> >   arch/arm/boot/dts/exynos4412-origen.dts       | 14 ++++++-------
> >   arch/arm/boot/dts/exynos4412-smdk4412.dts     | 20 +++++++++----------
> >   9 files changed, 36 insertions(+), 36 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/exynos4412-galaxy-s3.dtsi b/arch/arm/boot/dts/exynos4412-galaxy-s3.dtsi
> > index 89ed81fb348d..db91678f5648 100644
> > --- a/arch/arm/boot/dts/exynos4412-galaxy-s3.dtsi
> > +++ b/arch/arm/boot/dts/exynos4412-galaxy-s3.dtsi
> > @@ -15,7 +15,7 @@ aliases {
> >   		i2c10 = &i2c_cm36651;
> >   	};
> >   
> > -	aat1290 {
> > +	led-controller {
> >   		compatible = "skyworks,aat1290";
> >   		flen-gpios = <&gpj1 1 GPIO_ACTIVE_HIGH>;
> >   		enset-gpios = <&gpj1 2 GPIO_ACTIVE_HIGH>;
> > @@ -60,7 +60,7 @@ i2c_ak8975: i2c-gpio-0 {
> >   		#size-cells = <0>;
> >   		status = "okay";
> >   
> > -		ak8975@c {
> > +		magnetometer@c {
> >   			compatible = "asahi-kasei,ak8975";
> >   			reg = <0x0c>;
> >   			gpios = <&gpj0 7 GPIO_ACTIVE_HIGH>;
> > @@ -75,7 +75,7 @@ i2c_cm36651: i2c-gpio-2 {
> >   		#address-cells = <1>;
> >   		#size-cells = <0>;
> >   
> > -		cm36651@18 {
> > +		sensor@18 {
> light-sensor?

Ack, to this and all following. Thanks.

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/exynos4412-galaxy-s3.dtsi b/arch/arm/boot/dts/exynos4412-galaxy-s3.dtsi
index 89ed81fb348d..db91678f5648 100644
--- a/arch/arm/boot/dts/exynos4412-galaxy-s3.dtsi
+++ b/arch/arm/boot/dts/exynos4412-galaxy-s3.dtsi
@@ -15,7 +15,7 @@  aliases {
 		i2c10 = &i2c_cm36651;
 	};
 
-	aat1290 {
+	led-controller {
 		compatible = "skyworks,aat1290";
 		flen-gpios = <&gpj1 1 GPIO_ACTIVE_HIGH>;
 		enset-gpios = <&gpj1 2 GPIO_ACTIVE_HIGH>;
@@ -60,7 +60,7 @@  i2c_ak8975: i2c-gpio-0 {
 		#size-cells = <0>;
 		status = "okay";
 
-		ak8975@c {
+		magnetometer@c {
 			compatible = "asahi-kasei,ak8975";
 			reg = <0x0c>;
 			gpios = <&gpj0 7 GPIO_ACTIVE_HIGH>;
@@ -75,7 +75,7 @@  i2c_cm36651: i2c-gpio-2 {
 		#address-cells = <1>;
 		#size-cells = <0>;
 
-		cm36651@18 {
+		sensor@18 {
 			compatible = "capella,cm36651";
 			reg = <0x18>;
 			interrupt-parent = <&gpx0>;
@@ -133,7 +133,7 @@  timing-0 {
 };
 
 &i2c_3 {
-	mms114-touchscreen@48 {
+	touchscreen@48 {
 		compatible = "melfas,mms114";
 		reg = <0x48>;
 		interrupt-parent = <&gpm2>;
diff --git a/arch/arm/boot/dts/exynos4412-itop-elite.dts b/arch/arm/boot/dts/exynos4412-itop-elite.dts
index f6d0a5f5d339..3063e44fcbcc 100644
--- a/arch/arm/boot/dts/exynos4412-itop-elite.dts
+++ b/arch/arm/boot/dts/exynos4412-itop-elite.dts
@@ -175,7 +175,7 @@  &i2c_4 {
 	pinctrl-names = "default";
 	status = "okay";
 
-	codec: wm8960@1a {
+	codec: codec@1a {
 		compatible = "wlf,wm8960";
 		reg = <0x1a>;
 		clocks = <&pmu_system_controller 0>;
diff --git a/arch/arm/boot/dts/exynos4412-itop-scp-core.dtsi b/arch/arm/boot/dts/exynos4412-itop-scp-core.dtsi
index dfceb155b3a7..4583d342af39 100644
--- a/arch/arm/boot/dts/exynos4412-itop-scp-core.dtsi
+++ b/arch/arm/boot/dts/exynos4412-itop-scp-core.dtsi
@@ -134,7 +134,7 @@  &i2c_1 {
 	pinctrl-names = "default";
 	status = "okay";
 
-	s5m8767: s5m8767-pmic@66 {
+	s5m8767: pmic@66 {
 		compatible = "samsung,s5m8767-pmic";
 		reg = <0x66>;
 
diff --git a/arch/arm/boot/dts/exynos4412-midas.dtsi b/arch/arm/boot/dts/exynos4412-midas.dtsi
index 7e7c243ff196..52a4f6e6c7f2 100644
--- a/arch/arm/boot/dts/exynos4412-midas.dtsi
+++ b/arch/arm/boot/dts/exynos4412-midas.dtsi
@@ -220,7 +220,7 @@  i2c_max77693_fuel: i2c-gpio-3 {
 		#size-cells = <0>;
 		status = "okay";
 
-		max77693-fuel-gauge@36 {
+		fuel-gauge@36 {
 			compatible = "maxim,max17047";
 			interrupt-parent = <&gpx2>;
 			interrupts = <3 IRQ_TYPE_EDGE_FALLING>;
@@ -550,7 +550,7 @@  &i2c_0 {
 	pinctrl-names = "default";
 	status = "okay";
 
-	s5c73m3: s5c73m3@3c {
+	s5c73m3: sensor@3c {
 		compatible = "samsung,s5c73m3";
 		reg = <0x3c>;
 		xshutdown-gpios = <&gpf1 3 GPIO_ACTIVE_LOW>; /* ISP_RESET */
@@ -577,7 +577,7 @@  &i2c1_isp {
 	pinctrl-0 = <&fimc_is_i2c1>;
 	pinctrl-names = "default";
 
-	s5k6a3@10 {
+	sensor@10 {
 		compatible = "samsung,s5k6a3";
 		reg = <0x10>;
 		svdda-supply = <&cam_io_reg>;
@@ -616,7 +616,7 @@  &i2c_4 {
 	pinctrl-names = "default";
 	status = "okay";
 
-	wm1811: wm1811@1a {
+	wm1811: codec@1a {
 		compatible = "wlf,wm1811";
 		reg = <0x1a>;
 		clocks = <&pmu_system_controller 0>,
@@ -665,7 +665,7 @@  &i2c_7 {
 	pinctrl-names = "default";
 	status = "okay";
 
-	max77686: max77686_pmic@9 {
+	max77686: pmic@9 {
 		compatible = "maxim,max77686";
 		interrupt-parent = <&gpx0>;
 		interrupts = <7 IRQ_TYPE_NONE>;
@@ -1407,7 +1407,7 @@  &spi_1 {
 	cs-gpios = <&gpb 5 GPIO_ACTIVE_HIGH>;
 	status = "okay";
 
-	s5c73m3_spi: s5c73m3@0 {
+	s5c73m3_spi: sensor@0 {
 		compatible = "samsung,s5c73m3";
 		spi-max-frequency = <50000000>;
 		reg = <0>;
diff --git a/arch/arm/boot/dts/exynos4412-n710x.dts b/arch/arm/boot/dts/exynos4412-n710x.dts
index a47b7f35fc80..c49dbb7847b8 100644
--- a/arch/arm/boot/dts/exynos4412-n710x.dts
+++ b/arch/arm/boot/dts/exynos4412-n710x.dts
@@ -45,7 +45,7 @@  &i2c_3 {
 	pinctrl-names = "default";
 	status = "okay";
 
-	mms152-touchscreen@48 {
+	touchscreen@48 {
 		compatible = "melfas,mms152";
 		reg = <0x48>;
 		interrupt-parent = <&gpm2>;
diff --git a/arch/arm/boot/dts/exynos4412-odroid-common.dtsi b/arch/arm/boot/dts/exynos4412-odroid-common.dtsi
index c3b6e6d367ab..3dcd7f7f10f3 100644
--- a/arch/arm/boot/dts/exynos4412-odroid-common.dtsi
+++ b/arch/arm/boot/dts/exynos4412-odroid-common.dtsi
@@ -27,7 +27,7 @@  gpio_keys: gpio-keys {
 		pinctrl-names = "default";
 		pinctrl-0 = <&gpio_power_key>;
 
-		power_key {
+		power-key {
 			gpios = <&gpx1 3 GPIO_ACTIVE_LOW>;
 			linux,code = <KEY_POWER>;
 			label = "power key";
@@ -171,7 +171,7 @@  cooling_map1: map1 {
 };
 
 &pinctrl_1 {
-	gpio_power_key: power_key {
+	gpio_power_key: power-key {
 		samsung,pins = "gpx1-3";
 		samsung,pin-pud = <EXYNOS_PIN_PULL_NONE>;
 	};
@@ -266,7 +266,7 @@  &i2c_0 {
 	samsung,i2c-max-bus-freq = <400000>;
 	status = "okay";
 
-	usb3503: usb3503@8 {
+	usb3503: usb-hub@8 {
 		compatible = "smsc,usb3503";
 		reg = <0x08>;
 
@@ -492,7 +492,7 @@  buck8_reg: BUCK8 {
 
 &i2c_1 {
 	status = "okay";
-	max98090: max98090@10 {
+	max98090: codec@10 {
 		compatible = "maxim,max98090";
 		reg = <0x10>;
 		interrupt-parent = <&gpx0>;
diff --git a/arch/arm/boot/dts/exynos4412-odroidx.dts b/arch/arm/boot/dts/exynos4412-odroidx.dts
index 68fe88074d1d..46381e9097f4 100644
--- a/arch/arm/boot/dts/exynos4412-odroidx.dts
+++ b/arch/arm/boot/dts/exynos4412-odroidx.dts
@@ -36,7 +36,7 @@  led2 {
 		};
 	};
 
-	regulator_p3v3 {
+	regulator-1 {
 		compatible = "regulator-fixed";
 		regulator-name = "p3v3_en";
 		regulator-min-microvolt = <3300000>;
@@ -81,7 +81,7 @@  &mshc_0 {
 };
 
 &pinctrl_1 {
-	gpio_home_key: home_key {
+	gpio_home_key: home-key {
 		samsung,pins = "gpx2-2";
 		samsung,pin-pud = <EXYNOS_PIN_PULL_NONE>;
 	};
diff --git a/arch/arm/boot/dts/exynos4412-origen.dts b/arch/arm/boot/dts/exynos4412-origen.dts
index c2e793b69e7d..e1f6de53e20e 100644
--- a/arch/arm/boot/dts/exynos4412-origen.dts
+++ b/arch/arm/boot/dts/exynos4412-origen.dts
@@ -116,7 +116,7 @@  &i2c_0 {
 	pinctrl-names = "default";
 	status = "okay";
 
-	s5m8767_pmic@66 {
+	pmic@66 {
 		compatible = "samsung,s5m8767-pmic";
 		reg = <0x66>;
 
@@ -453,37 +453,37 @@  &keypad {
 	pinctrl-names = "default";
 	status = "okay";
 
-	key_home {
+	key-home {
 		keypad,row = <0>;
 		keypad,column = <0>;
 		linux,code = <KEY_HOME>;
 	};
 
-	key_down {
+	key-down {
 		keypad,row = <0>;
 		keypad,column = <1>;
 		linux,code = <KEY_DOWN>;
 	};
 
-	key_up {
+	key-up {
 		keypad,row = <1>;
 		keypad,column = <0>;
 		linux,code = <KEY_UP>;
 	};
 
-	key_menu {
+	key-menu {
 		keypad,row = <1>;
 		keypad,column = <1>;
 		linux,code = <KEY_MENU>;
 	};
 
-	key_back {
+	key-back {
 		keypad,row = <2>;
 		keypad,column = <0>;
 		linux,code = <KEY_BACK>;
 	};
 
-	key_enter {
+	key-enter {
 		keypad,row = <2>;
 		keypad,column = <1>;
 		linux,code = <KEY_ENTER>;
diff --git a/arch/arm/boot/dts/exynos4412-smdk4412.dts b/arch/arm/boot/dts/exynos4412-smdk4412.dts
index 49971203a8aa..cc99b955af0c 100644
--- a/arch/arm/boot/dts/exynos4412-smdk4412.dts
+++ b/arch/arm/boot/dts/exynos4412-smdk4412.dts
@@ -71,61 +71,61 @@  &keypad {
 	pinctrl-names = "default";
 	status = "okay";
 
-	key_1 {
+	key-1 {
 		keypad,row = <1>;
 		keypad,column = <3>;
 		linux,code = <2>;
 	};
 
-	key_2 {
+	key-2 {
 		keypad,row = <1>;
 		keypad,column = <4>;
 		linux,code = <3>;
 	};
 
-	key_3 {
+	key-3 {
 		keypad,row = <1>;
 		keypad,column = <5>;
 		linux,code = <4>;
 	};
 
-	key_4 {
+	key-4 {
 		keypad,row = <1>;
 		keypad,column = <6>;
 		linux,code = <5>;
 	};
 
-	key_5 {
+	key-5 {
 		keypad,row = <1>;
 		keypad,column = <7>;
 		linux,code = <6>;
 	};
 
-	key_A {
+	key-A {
 		keypad,row = <2>;
 		keypad,column = <6>;
 		linux,code = <30>;
 	};
 
-	key_B {
+	key-B {
 		keypad,row = <2>;
 		keypad,column = <7>;
 		linux,code = <48>;
 	};
 
-	key_C {
+	key-C {
 		keypad,row = <0>;
 		keypad,column = <5>;
 		linux,code = <46>;
 	};
 
-	key_D {
+	key-D {
 		keypad,row = <2>;
 		keypad,column = <5>;
 		linux,code = <32>;
 	};
 
-	key_E {
+	key-E {
 		keypad,row = <0>;
 		keypad,column = <7>;
 		linux,code = <18>;