arm64: dts: qcom: msm8916: avoid using _ in node names

Many nodes in the MSM8916 device trees use '_' in node names
(especially pinctrl), even though (seemingly) '-' is preferred now.
Make this more consistent by replacing '_' with '-' where possible.

Similar naming is used for pinctrl in newer device trees (e.g. sdm845.dtsi).
Signed-off-by: Stephan Gerhold <stephan@gerhold.net>
Link: https://lore.kernel.org/r/20200514112754.148919-1-stephan@gerhold.netSigned-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>

arch/arm64/boot/dts/qcom/apq8016-sbc-pmic-pins.dtsi

@@ -4,7 +4,7 @@
 
 &pm8916_gpios {
 
-	usb_hub_reset_pm: usb_hub_reset_pm {
+	usb_hub_reset_pm: usb-hub-reset-pm {
 		pinconf {
 			pins = "gpio3";
 			function = PMIC_GPIO_FUNC_NORMAL;
@@ -13,7 +13,7 @@ pinconf {
 		};
 	};
 
-	usb_hub_reset_pm_device: usb_hub_reset_pm_device {
+	usb_hub_reset_pm_device: usb-hub-reset-pm-device {
 		pinconf {
 			pins = "gpio3";
 			function = PMIC_GPIO_FUNC_NORMAL;
@@ -21,7 +21,7 @@ pinconf {
 		};
 	};
 
-	usb_sw_sel_pm: usb_sw_sel_pm {
+	usb_sw_sel_pm: usb-sw-sel-pm {
 		pinconf {
 			pins = "gpio4";
 			function = PMIC_GPIO_FUNC_NORMAL;
@@ -31,7 +31,7 @@ pinconf {
 		};
 	};
 
-	usb_sw_sel_pm_device: usb_sw_sel_pm_device {
+	usb_sw_sel_pm_device: usb-sw-sel-pm-device {
 		pinconf {
 			pins = "gpio4";
 			function = PMIC_GPIO_FUNC_NORMAL;
@@ -41,7 +41,7 @@ pinconf {
 		};
 	};
 
-	pm8916_gpios_leds: pm8916_gpios_leds {
+	pm8916_gpios_leds: pm8916-gpios-leds {
 		pinconf {
 			pins = "gpio1", "gpio2";
 			function = PMIC_GPIO_FUNC_NORMAL;
@@ -55,7 +55,7 @@ &pm8916_mpps {
 	pinctrl-names = "default";
 	pinctrl-0 = <&ls_exp_gpio_f>;
 
-	ls_exp_gpio_f: pm8916_mpp4 {
+	ls_exp_gpio_f: pm8916-mpp4 {
 		pinconf {
 			pins = "mpp4";
 			function = "digital";
@@ -64,7 +64,7 @@ pinconf {
 		};
 	};
 
-	pm8916_mpps_leds: pm8916_mpps_leds {
+	pm8916_mpps_leds: pm8916-mpps-leds {
 		pinconf {
 			pins = "mpp2", "mpp3";
 			function = "digital";

arch/arm64/boot/dts/qcom/apq8016-sbc-soc-pins.dtsi

@@ -4,7 +4,7 @@
 
 &msmgpio {
 
-	msmgpio_leds: msmgpio_leds {
+	msmgpio_leds: msmgpio-leds {
 		pinconf {
 			pins = "gpio21", "gpio120";
 			function = "gpio";
@@ -26,7 +26,7 @@ pinconf {
 		};
 	};
 
-	adv7533_int_active: adv533_int_active {
+	adv7533_int_active: adv533-int-active {
 		pinmux {
 			function = "gpio";
 			pins = "gpio31";
@@ -38,7 +38,7 @@ pinconf {
 		};
 	};
 
-	adv7533_int_suspend: adv7533_int_suspend {
+	adv7533_int_suspend: adv7533-int-suspend {
 		pinmux {
 			function = "gpio";
 			pins = "gpio31";
@@ -50,7 +50,7 @@ pinconf {
 		};
 	};
 
-	adv7533_switch_active: adv7533_switch_active {
+	adv7533_switch_active: adv7533-switch-active {
 		pinmux {
 			function = "gpio";
 			pins = "gpio32";
@@ -62,7 +62,7 @@ pinconf {
 		};
 	};
 
-	adv7533_switch_suspend: adv7533_switch_suspend {
+	adv7533_switch_suspend: adv7533-switch-suspend {
 		pinmux {
 			function = "gpio";
 			pins = "gpio32";
@@ -74,7 +74,7 @@ pinconf {
 		};
 	};
 
-	msm_key_volp_n_default: msm_key_volp_n_default {
+	msm_key_volp_n_default: msm-key-volp-n-default {
 		pinmux {
 			function = "gpio";
 			pins = "gpio107";

arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi

@@ -51,7 +51,7 @@ chosen {
 		stdout-path = "serial0";
 	};
 
-	camera_vdddo_1v8: camera_vdddo_1v8 {
+	camera_vdddo_1v8: camera-vdddo-1v8 {
 		compatible = "regulator-fixed";
 		regulator-name = "camera_vdddo";
 		regulator-min-microvolt = <1800000>;
@@ -59,7 +59,7 @@ camera_vdddo_1v8: camera_vdddo_1v8 {
 		regulator-always-on;
 	};
 
-	camera_vdda_2v8: camera_vdda_2v8 {
+	camera_vdda_2v8: camera-vdda-2v8 {
 		compatible = "regulator-fixed";
 		regulator-name = "camera_vdda";
 		regulator-min-microvolt = <2800000>;
@@ -67,7 +67,7 @@ camera_vdda_2v8: camera_vdda_2v8 {
 		regulator-always-on;
 	};
 
-	camera_vddd_1v5: camera_vddd_1v5 {
+	camera_vddd_1v5: camera-vddd-1v5 {
 		compatible = "regulator-fixed";
 		regulator-name = "camera_vddd";
 		regulator-min-microvolt = <1500000>;
@@ -566,7 +566,7 @@ hdmi_con: endpoint {
 		};
 	};
 
-	gpio_keys {
+	gpio-keys {
 		compatible = "gpio-keys";
 		#address-cells = <1>;
 		#size-cells = <0>;

arch/arm64/boot/dts/qcom/msm8916-longcheer-l8150.dts

@@ -109,7 +109,7 @@ volume-up {
 };
 
 &msmgpio {
-	gpio_keys_default: gpio_keys_default {
+	gpio_keys_default: gpio-keys-default {
 		pinmux {
 			function = "gpio";
 			pins = "gpio107";

arch/arm64/boot/dts/qcom/msm8916-samsung-a2015-common.dtsi

@@ -168,7 +168,7 @@ muic: sm5502@25 {
 };
 
 &msmgpio {
-	gpio_keys_default: gpio_keys_default {
+	gpio_keys_default: gpio-keys-default {
 		pinmux {
 			function = "gpio";
 			pins = "gpio107", "gpio109";
@@ -180,7 +180,7 @@ pinconf {
 		};
 	};
 
-	gpio_hall_sensor_default: gpio_hall_sensor_default {
+	gpio_hall_sensor_default: gpio-hall-sensor-default {
 		pinmux {
 			function = "gpio";
 			pins = "gpio52";
@@ -192,7 +192,7 @@ pinconf {
 		};
 	};
 
-	muic_int_default: muic_int_default {
+	muic_int_default: muic-int-default {
 		pinmux {
 			function = "gpio";
 			pins = "gpio12";

arch/arm64/boot/dts/qcom/msm8916.dtsi

@@ -340,7 +340,7 @@ modem_alert0: trip-point0 {
 
 	};
 
-	cpu_opp_table: cpu_opp_table {
+	cpu_opp_table: cpu-opp-table {
 		compatible = "operating-points-v2";
 		opp-shared;
 
@@ -358,7 +358,7 @@ opp-998400000 {
 		};
 	};
 
-	gpu_opp_table: opp_table {
+	gpu_opp_table: gpu-opp-table {
 		compatible = "operating-points-v2";
 
 		opp-400000000 {
@@ -378,13 +378,13 @@ timer {
 	};
 
 	clocks {
-		xo_board: xo_board {
+		xo_board: xo-board {
 			compatible = "fixed-clock";
 			#clock-cells = <0>;
 			clock-frequency = <19200000>;
 		};
 
-		sleep_clk: sleep_clk {
+		sleep_clk: sleep-clk {
 			compatible = "fixed-clock";
 			#clock-cells = <0>;
 			clock-frequency = <32768>;
@@ -1783,7 +1783,7 @@ rpm {
 			qcom,ipc = <&apcs 8 0>;
 			qcom,smd-edge = <15>;
 
-			rpm_requests {
+			rpm-requests {
 				compatible = "qcom,rpm-msm8916";
 				qcom,smd-channels = "rpm_requests";