[2/7] arm64: dts: qcom: sdm845-cheza: commonize uart6 pinconf

Message ID 20221016143300.1738550-3-caleb@connolly.tech
State New
Headers
Series arm64: dts: qcom: sdm845: commonize bluetooth pinconf |

Commit Message

Caleb Connolly Oct. 16, 2022, 2:33 p.m. UTC
  Use the new qup_uart6_4pin configuration in sd845.dtsi and fix alias
name.

Signed-off-by: Caleb Connolly <caleb@connolly.tech>
---
 arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi | 39 ++--------------------
 1 file changed, 3 insertions(+), 36 deletions(-)

--
2.38.0
  

Comments

Bjorn Andersson Oct. 17, 2022, 11:04 p.m. UTC | #1
On Sun, Oct 16, 2022 at 02:33:53PM +0000, Caleb Connolly wrote:
> -----BEGIN PGP MESSAGE-----
> Version: ProtonMail
> 
> wcFMA4ktbss2xKXfAQ/7Bk2KbfGuwIpyiq5398IVuMDT4UNEOivVgsH/jG5M
> qk6QZmFNWAZ1AF4p6kMn6L6ivKbbzB2zCSDl+ZAcsL7xUn8dmRJ34bhJ5qTh
> fbysmQVA7MJAJL1bGuiQZJH1DcDtybNPg2bYo32qULz5eEKuU5EmOKi8/Vic
> 468ieMS2W67oD9ShOjOnMhTYogzXfJcyoYnGCG0EdDbpLkQz3a3pWQMDWLDg
> T+iZ8iAnYrgnLKlwKjK1QdcCtoihHyEsxx9EQJN1oMrfEmOwUmFGuTHxBjOW
> J3PLxxD7fB3/AzMTP0KmzboFw9mLzMR6QHToqW3QuI7M9FWDnR0YsKxUFuEQ
> yTX6Atr72T519NHUSZua7MYlO/+yWR1bjp/4EtePd0TaEUwKqGTtQRm32ele
> HP2msQXtTqQEv+kqxmWcqK5X58mhjKa8uybFha2ME2CmIPgkgJnYqhcDfHVB
> GIb9C8DV927CJYALRJ48tqVU9yxyXB/pXS4buuvV1tBLAg7Oa+8Pvib9OwdU
> G/faaD6KFpnuueAdemCcQHbqodTMjsJfnvPo82mT6fKaGaZdJa2Z9wua2i7o
> vohR7RlXUsewKbU1L/kfaQ0rV1o9QjLq+P76lgfOdbbGY3dMShZ0suDq4WRC
> ePeGvY1esLACy57qqApa2O2DV8GMlaf50yKOYUNuz6fS6gEh49eQSNaFb2Ht
> 49czhnTYqKq6nCXDCINgHz8w/vAY4r8zNpLmRk2oUv5DMoz9rWW0r/cX8VTu
> 0OykvN7wiJ2Bthd3u1CLXlQFJ7SpBSfMaI5O401vBMijo0gBKVCJ9uL5GHNQ
> PvC/oGaAb2O8Ph3zKWksHBgt5fhxtUQE/JbovsNI293Yg4lmsyFZQDFemZEL
> hEBun5Ib+jUI+jc7iZt6hP70EoBx/G3zoi4Ljo42ZoEutWp9SF3z25JRzpyw
> +96BLszbi8uo2nxPjyN6jlrfhhA1isKa8KLMamumic+N0IWsKzLvzg+/byDh
> tcqvUgam6BnwpingZE3PNCJKERsmDm1eEIil6e3vDctpq37z5cGZv1122JCo
> bFWL2sy/O2R9T77+52Q6cf7MwKVPnk0IXyElllELNF/C0vy0hq76FufGTeFT
> GAT3mn3GGuB64/kFvcP2l/N8M0M2W5V3jcsHfTByznJ+EUq/V/bYAdNJ1gP+
> hMLHyuprTLYtMEgPE6zA5nUTibV83rvQj63GP2+HBKSOi0rTvA29aAyp+Wlu
> JLo85mDUXa2f7AzPZdjiNIMnkP637kak9SRPwX9yNvdKT4UuFR7L+2LXFoTF
> cIH73B7NoSXL14z4QfnHVS6GWNjEFMpdvVj2eeqAOtybIQ5cMdub23qi4qFJ
> 80g/eb6dyjGwgyLBuotrB4/Nv1XDdSoZqIhyH2nPfFUwDEgYhmrSy4pRp+WK
> CVdjitG+gU5EIrw0eZOHYI8rh3D9CYy2RKFqozmQGU0FNkZ5Y8ZpLeyb5Ss/
> ZbETWwABPX0MP1DN1UB52YFp1eKLU9kRlJSLfWEMZo1x1NeDsB9IGfMMFmvG
> D+r+2DAI2LU45zYH/Ghiv7/nhnt6d5BZfyQ9RpNyGSlHUu/YG8jDdqplkwoD
> nfr8pfO08sDZgIj+nTpS6nG1Ifxa3ES2r2tknoHvNiqYjDnJ5p9IyMZ5apYL
> VFZbzD0XW1DPpPcLfW7IFKo8kuihcQUy7tSD9wVPyu6o+JHUw9uH7JzEP3mB
> hKiwHbCjYwYrQdd9icRro0J8aH7IU9CC8Dye73Wqne2q63cnsJREOz3zmHps
> 2PNbsZbvKFTkgZwEung/+6KcSbTCIhA2NZTO5KSKkopD8WFYrWj2ZKoZ2Kn7
> 7SLffDQs0pyolxzVhKSmjBgx3cKVfsHLVG7CylLvWUT3gdIuHEqgJ8BS8PBn
> eUUtqbHiUGN3RwvNKw9pSCZoo54WhdTWVaHWepisknphvoFYkfXODNjdE7cJ
> vdDAO3QtnmML1YCge7RgZ9g9b7H9dV1KE5NizSN7t3u44HXjtEIV6TPkwN5X
> unM7k0AbkGI511XT3q4xI3BUf0z155Xp1YuaXf5YekhsX/xRm+4a41fTFa+p
> chLV2yQC5QqEbXmeLiMQ6GPaCfmFLNkxxkgsXfHcfHCpg3CYrSdmI3i+ED2E
> YqqZa+w8kwDNGXkmTT7DED++EcRTbn86mZUntwMBA63LcW2J1L9OBwrS0XD/
> Ngcf59X5OcJl/TVi7RRtqxZfeUhyy0FDlJQpd8xGpNt0GaNSGgCNAgJbLY3x
> cXtkJzpG9M+FQlhImrcnpjlvgWju12cj8kUDoZ27ulmw0AfQurDKO+FfEOo/
> kb8l2wzPykSrnTImpiH9fyfbOoDLwEET9Vi1okz44FP1gLUDWnf1b0k276NG
> twV6QVjdhE1EuQLpnBbYEf84zxIm2MfBesw7LaiN2omtUzzV/oDhN6ptQ9WA
> GGPyNEe9f9n2NOq/3S8eg0yCSobvvdRmJTBDJGnMMvKerWD4+L3zl4zOJxbC
> i1RoUy7Xudd4vuqvvAT9sjAn/4MppVNxM3NVjlASnv5hOCe+uuHBlJn5Bz7i
> K286gPw1Qtoa6UTW410kCr5VuTCRcW8V03dqzpwAxIc63ua9fd5VJkUYEux9
> zpUn88Toyp2egnADJvC4h2jvs9nhqcy7v0WViEUy2wywvg+i/IWJj69zyWtG
> IDD5n7V9q8UNv6q+Q56MrH66X3z5rOUyh2yVJLKuG/AJHRVc2EvDWx/ppLIU
> JZWpNe0YsEbQYgY7I0WyGpDJbMw6qDO6GSV/L10VrJzEzDqgNzVqLp6gS0Pm
> 0x0n1VXF/VpTblajF357anD7PmuUcxuAay/Dl8DVWRkh8tjSFv8u2eda69Tr
> y1VWszpxmvhKpE1+mvDEXzkX1HL92iQtA7+57fDauINf5EfhJTusAnmd3z3M
> bLUUW5LxnWfEsFQLZjAExB4f7b2EILsk++TlrCsSOzi9vmMnXyYECClzuRWh
> 0vfo0Rr/+MySKQ+o0r9YWeL548db4hEGbMniz3tCLnewLgavXw/iW4E7mfrQ
> oPD2uZVxQsKddbUY/unkW7CDZ9CgjjEqoMgYww+akCfMZieZZ0TXnZL5WmYx
> R6NTqdjcDGoMPPDuhHkAe0qbJ9eTLAtBIno8ybAg+2t/yZsJ+G9poIH+pD+r
> 6kW+B7oGEm3XkqTGoRD+DcOr3mzQhXP+z+HVNjE5yAMVXal7aOMvIYfuKDeH
> fRnXLa7SU5MnQBi7dC/6N00AxihfX46KkFFblAx0q0V/Shk50ci9Nl1NrCZ5
> mBAJBSkN3ZSrZkHisvm1Ycg4cJK0B/bv7iDVRzENMmBZqqTjJzQ4ReXJJCtH
> ZAAVpDiPFT4NtAtpGO9pFirlaB2v9cmPamBg0fJBajCPDKplm0x811BmU1L1
> No0oEoeWwkzoy0f3KYnAvCdIaCgKjAfLpiYazW94YxG8DwmLbyJa97ZnWOvA
> ldj0uhJvKEb+CgMAni5uqPIBkQa3QFvA9wApULLc+IL0EU1fAZq1ouU9iHBn
> 7bjqpm7aHCodn9SPw9+6zvjS9WxBr0l7oao48CicDdbfXvC/YJU+YClLL6kr
> 0bosxFxUd9y+l7opdpERgUvOY5TWR2sJh3c2ae7mdjI/kBSZ+CVX7mnj1ayH
> COlEvAvJXurAk4DxBDfAnIvH4zoBLSJCbbME/kD2yWsvKPgOcfFHzkjNGScn
> pFh0kRfDZkufwGDjAHXp/x6UvFkHUCf7TqeBAhNrGk8fFRD8jnVFKUz2wyWx
> DKivP6qKhG/x6YS+llgVDanUXVjOV/41V8xLjt/xnL5WUTTica59mb6kdFyC
> CWbZo4aPrNZXkkp+RLAONn71uDHKXH0qqOr4h2CYz1eoOQFK5P95TSQRkwrc
> 5pxFFDUKLufmYjvOD/WcpUgMD4BoTj8oNVuvSlOnt3UDb1Y/
> =lHx9
> -----END PGP MESSAGE-----

Can you please resubmit this in a way that I can read it, with
cover-letter and proper threading?

(Or let me know what I'm doing wrong when trying to look at these
patches?)

Thanks,
Bjorn
  

Patch

diff --git a/arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi b/arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi
index b5eb8f7eca1d..a8ae6de51af4 100644
--- a/arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi
@@ -16,7 +16,7 @@ 
 / {
 	aliases {
 		bluetooth0 = &bluetooth;
-		hsuart0 = &uart6;
+		serial1 = &uart6;
 		serial0 = &uart9;
 		wifi0 = &wifi;
 	};
@@ -860,6 +860,8 @@  i2c_tunnel: i2c-tunnel {
 &uart6 {
 	status = "okay";

+	pinctrl-0 = <&qup_uart6_4pin>;
+
 	bluetooth: wcn3990-bt {
 		compatible = "qcom,wcn3990-bt";
 		vddio-supply = <&src_pp1800_s4a>;
@@ -1079,41 +1081,6 @@  pinconf {
 	};
 };

-&qup_uart6_default {
-	/* Change pinmux to all 4 pins since CTS and RTS are connected */
-	pinmux {
-		pins = "gpio45", "gpio46",
-		       "gpio47", "gpio48";
-	};
-
-	pinconf-cts {
-		/*
-		 * Configure a pull-down on 45 (CTS) to match the pull of
-		 * the Bluetooth module.
-		 */
-		pins = "gpio45";
-		bias-pull-down;
-	};
-
-	pinconf-rts-tx {
-		/* We'll drive 46 (RTS) and 47 (TX), so no pull */
-		pins = "gpio46", "gpio47";
-		drive-strength = <2>;
-		bias-disable;
-	};
-
-	pinconf-rx {
-		/*
-		 * Configure a pull-up on 48 (RX). This is needed to avoid
-		 * garbage data when the TX pin of the Bluetooth module is
-		 * in tri-state (module powered off or not driving the
-		 * signal yet).
-		 */
-		pins = "gpio48";
-		bias-pull-up;
-	};
-};
-
 &qup_uart9_default {
 	pinconf-tx {
 		pins = "gpio4";