Commit 12541f68 authored by Konrad Dybcio's avatar Konrad Dybcio Committed by Bjorn Andersson
Browse files

arm64: dts: qcom: msm8998*: Fix TLMM and pin nodes



Remove the unnecessary level of indentation, commonize SDC2 pins and notice
that SDCC2_CD_ON and _OFF is identical, deduplicate it!

Also, remove some unnecessary overrides and use decimal values in #-cells

Signed-off-by: default avatarKonrad Dybcio <konrad.dybcio@somainline.org>
Signed-off-by: default avatarBjorn Andersson <bjorn.andersson@linaro.org>
Link: https://lore.kernel.org/r/20220430162353.607709-1-konrad.dybcio@somainline.org
parent f424d754
Loading
Loading
Loading
Loading
+0 −9
Original line number Diff line number Diff line
@@ -37,12 +37,3 @@ keyboard@3a {
&sdhc2 {
	cd-gpios = <&tlmm 95 GPIO_ACTIVE_HIGH>;
};

&tlmm {
	touchpad: touchpad {
		config {
			pins = "gpio123";
			bias-pull-up;
		};
	};
};
+5 −7
Original line number Diff line number Diff line
@@ -289,11 +289,9 @@ &remoteproc_mss {
&tlmm {
	gpio-reserved-ranges = <0 4>, <81 4>;

	touchpad: touchpad {
		config {
	touchpad: touchpad-pin {
		pins = "gpio123";
			bias-pull-up;           /* pull up */
		};
		bias-pull-up;
	};
};

@@ -304,8 +302,8 @@ &sdhc2 {
	vqmmc-supply = <&vreg_l13a_2p95>;

	pinctrl-names = "default", "sleep";
	pinctrl-0 = <&sdc2_clk_on  &sdc2_cmd_on  &sdc2_data_on  &sdc2_cd_on>;
	pinctrl-1 = <&sdc2_clk_off &sdc2_cmd_off &sdc2_data_off &sdc2_cd_off>;
	pinctrl-0 = <&sdc2_on &sdc2_cd>;
	pinctrl-1 = <&sdc2_off &sdc2_cd>;
};

&ufshc {
+2 −2
Original line number Diff line number Diff line
@@ -348,8 +348,8 @@ &sdhc2 {
	vqmmc-supply = <&vreg_l13a_2p95>;

	pinctrl-names = "default", "sleep";
	pinctrl-0 = <&sdc2_clk_on  &sdc2_cmd_on  &sdc2_data_on  &sdc2_cd_on>;
	pinctrl-1 = <&sdc2_clk_off &sdc2_cmd_off &sdc2_data_off &sdc2_cd_off>;
	pinctrl-0 = <&sdc2_on &sdc2_cd>;
	pinctrl-1 = <&sdc2_off &sdc2_cd>;
};

&stm {
+2 −2
Original line number Diff line number Diff line
@@ -549,8 +549,8 @@ &sdhc2 {
	vqmmc-supply = <&vreg_l13a_2p95>;

	pinctrl-names = "default", "sleep";
	pinctrl-0 = <&sdc2_clk_on  &sdc2_cmd_on  &sdc2_data_on  &sdc2_cd_on>;
	pinctrl-1 = <&sdc2_clk_off &sdc2_cmd_off &sdc2_data_off &sdc2_cd_off>;
	pinctrl-0 = <&sdc2_on &sdc2_cd>;
	pinctrl-1 = <&sdc2_off &sdc2_cd>;
};

&tlmm {
+24 −51
Original line number Diff line number Diff line
@@ -1057,85 +1057,58 @@ tlmm: pinctrl@3400000 {
			reg = <0x03400000 0xc00000>;
			interrupts = <GIC_SPI 208 IRQ_TYPE_LEVEL_HIGH>;
			gpio-controller;
			#gpio-cells = <0x2>;
			#gpio-cells = <2>;
			interrupt-controller;
			#interrupt-cells = <0x2>;
			#interrupt-cells = <2>;

			sdc2_clk_on: sdc2_clk_on {
				config {
			sdc2_on: sdc2-on {
				clk {
					pins = "sdc2_clk";
					bias-disable;
					drive-strength = <16>;
				};
			};

			sdc2_clk_off: sdc2_clk_off {
				config {
					pins = "sdc2_clk";
					bias-disable;
					drive-strength = <2>;
				};
				};

			sdc2_cmd_on: sdc2_cmd_on {
				config {
				cmd {
					pins = "sdc2_cmd";
					bias-pull-up;
					drive-strength = <10>;
				};
			};

			sdc2_cmd_off: sdc2_cmd_off {
				config {
					pins = "sdc2_cmd";
					bias-pull-up;
					drive-strength = <2>;
				};
				};

			sdc2_data_on: sdc2_data_on {
				config {
				data {
					pins = "sdc2_data";
					bias-pull-up;
					drive-strength = <10>;
					bias-pull-up;
				};
			};

			sdc2_data_off: sdc2_data_off {
				config {
					pins = "sdc2_data";
					bias-pull-up;
			sdc2_off: sdc2-off {
				clk {
					pins = "sdc2_clk";
					drive-strength = <2>;
				};
					bias-disable;
				};

			sdc2_cd_on: sdc2_cd_on {
				mux {
					pins = "gpio95";
					function = "gpio";
				cmd {
					pins = "sdc2_cmd";
					drive-strength = <2>;
					bias-pull-up;
				};

				config {
					pins = "gpio95";
					bias-pull-up;
				data {
					pins = "sdc2_data";
					drive-strength = <2>;
					bias-pull-up;
				};
			};

			sdc2_cd_off: sdc2_cd_off {
				mux {
			sdc2_cd: sdc2-cd {
				pins = "gpio95";
				function = "gpio";
				};

				config {
					pins = "gpio95";
				bias-pull-up;
				drive-strength = <2>;
			};
			};

			blsp1_uart3_on: blsp1_uart3_on {
			blsp1_uart3_on: blsp1-uart3-on {
				tx {
					pins = "gpio45";
					function = "blsp_uart3_a";