From: Andrew Davis <afd@ti.com>
To: Nishanth Menon <nm@ti.com>, Vignesh Raghavendra <vigneshr@ti.com>,
Tero Kristo <kristo@kernel.org>, Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Conor Dooley <conor+dt@kernel.org>, Andrew Lunn <andrew@lunn.ch>,
Vaishnav M A <vaishnav@beagleboard.org>,
Derek Kiernan <derek.kiernan@amd.com>,
Dragan Cvetic <dragan.cvetic@amd.com>,
Arnd Bergmann <arnd@arndb.de>, Michael Walle <mwalle@kernel.org>,
Jason Kridner <jkridner@beagleboard.org>,
Robert Nelson <robertcnelson@beagleboard.org>,
Robert Nelson <robertcnelson@gmail.com>,
Ayush Singh <ayush@beagleboard.org>,
Ayush Singh <ayushdevel1325@gmail.com>
Cc: <linux-arm-kernel@lists.infradead.org>,
<devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
Andrew Davis <afd@ti.com>
Subject: [PATCH RFC 1/3] arm64: dts: ti: k3-am625-beagleplay: Add Grove connector pinmux options
Date: Tue, 2 Jul 2024 11:44:01 -0500 [thread overview]
Message-ID: <20240702164403.29067-2-afd@ti.com> (raw)
In-Reply-To: <20240702164403.29067-1-afd@ti.com>
The Grove connector pins on BeaglePlay supports 3 different functions.
Describe these here.
Also move the default to be selected by a new Grove connector node.
As I2C is not the only possible function, do not enable main_i2c1.
It should only be enabled when a add-on board is connected that
makes use of it. That enabling should be done by the add-on board's
overlay.
Signed-off-by: Andrew Davis <afd@ti.com>
---
.../arm64/boot/dts/ti/k3-am625-beagleplay.dts | 32 +++++++++++++++----
1 file changed, 25 insertions(+), 7 deletions(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts b/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts
index 70de288d728e4..004eb5c24a84c 100644
--- a/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts
+++ b/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts
@@ -227,6 +227,10 @@ simple-audio-card,codec {
};
};
+ grove_connector0: grove-connector0 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&grove_pins_default>;
+ };
};
&main_pmx0 {
@@ -270,6 +274,27 @@ AM62X_IOPAD(0x01ec, PIN_INPUT_PULLUP, 0) /* (A17) I2C1_SDA */
>;
};
+ grove_pins_digital: grove-digital-pins {
+ pinctrl-single,pins = <
+ AM62X_IOPAD(0x01e8, PIN_INPUT, 7) /* (B17) I2C1_SCL.GPIO1_28 */
+ AM62X_IOPAD(0x01ec, PIN_INPUT, 7) /* (A17) I2C1_SDA.GPIO1_29 */
+ >;
+ };
+
+ grove_pins_uart: grove-uart-pins {
+ pinctrl-single,pins = <
+ AM62X_IOPAD(0x01e8, PIN_INPUT, 1) /* (B17) I2C1_SCL.UART1_RXD */
+ AM62X_IOPAD(0x01ec, PIN_OUTPUT, 1) /* (A17) I2C1_SDA.UART1_TXD */
+ >;
+ };
+
+ grove_pins_i2c: grove-i2c-pins {
+ pinctrl-single,pins = <
+ AM62X_IOPAD(0x01e8, PIN_INPUT_PULLUP, 0) /* (B17) I2C1_SCL */
+ AM62X_IOPAD(0x01ec, PIN_INPUT_PULLUP, 0) /* (A17) I2C1_SDA */
+ >;
+ };
+
local_i2c_pins_default: local-i2c-default-pins {
bootph-all;
pinctrl-single,pins = <
@@ -753,13 +778,6 @@ ldo4_reg: ldo4 {
};
};
-&main_i2c1 {
- pinctrl-names = "default";
- pinctrl-0 = <&grove_pins_default>;
- clock-frequency = <100000>;
- status = "okay";
-};
-
&main_i2c2 {
pinctrl-names = "default";
pinctrl-0 = <&i2c2_1v8_pins_default>;
--
2.39.2
next prev parent reply other threads:[~2024-07-02 16:44 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-07-02 16:44 [PATCH RFC 0/3] Add generic Overlay for Grove Sunlight Sensor Andrew Davis
2024-07-02 16:44 ` Andrew Davis [this message]
2024-07-02 16:44 ` [PATCH RFC 2/3] arm64: dts: ti: k3-am625-beagleplay: Add Grove connector adapter overlays Andrew Davis
2024-07-02 16:44 ` [PATCH RFC 3/3] arm64: dts: ti: grove: Add Grove Sunlight Sensor overlay Andrew Davis
2024-07-03 14:15 ` Ayush Singh
2024-07-04 16:55 ` Andrew Davis
2024-07-04 17:29 ` Ayush Singh
2024-07-05 8:35 ` Geert Uytterhoeven
2024-07-03 0:52 ` [PATCH RFC 0/3] Add generic Overlay for Grove Sunlight Sensor Robert Nelson
2024-07-03 4:22 ` Ayush Singh
2024-07-05 8:28 ` Geert Uytterhoeven
2024-08-09 15:22 ` Ayush Singh
2024-08-26 18:44 ` Andrew Davis
2024-08-27 13:20 ` Ayush Singh
2024-08-27 15:22 ` Andrew Davis
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240702164403.29067-2-afd@ti.com \
--to=afd@ti.com \
--cc=andrew@lunn.ch \
--cc=arnd@arndb.de \
--cc=ayush@beagleboard.org \
--cc=ayushdevel1325@gmail.com \
--cc=conor+dt@kernel.org \
--cc=derek.kiernan@amd.com \
--cc=devicetree@vger.kernel.org \
--cc=dragan.cvetic@amd.com \
--cc=jkridner@beagleboard.org \
--cc=kristo@kernel.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mwalle@kernel.org \
--cc=nm@ti.com \
--cc=robertcnelson@beagleboard.org \
--cc=robertcnelson@gmail.com \
--cc=robh+dt@kernel.org \
--cc=vaishnav@beagleboard.org \
--cc=vigneshr@ti.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).