2023-10-26 07:02:12

by Nitin Yadav

Subject: [PATCH 0/2] Enable UHS card support on AM62a

This series enables class UHS mode on support on AM62a SK device.

Nitin Yadav (2):
arm64: dts: ti: k3-am62a: Enable UHS mode support for SD cards
arm64: dts: ti: k3-am62a-main: Remove no-1-8-v property

arch/arm64/boot/dts/ti/k3-am62a-main.dtsi | 1 -
arch/arm64/boot/dts/ti/k3-am62a7-sk.dts | 20 ++++++++++++++++++++
2 files changed, 20 insertions(+), 1 deletion(-)

--
2.25.1

2023-10-26 07:02:16

by Nitin Yadav

Subject: [PATCH 1/2] arm64: dts: ti: k3-am62a: Enable UHS mode support for SD cards

Hook up required IO voltage regulators to support UHS modes on
SD cards.

Signed-off-by: Nitin Yadav <[email protected]>
---
arch/arm64/boot/dts/ti/k3-am62a7-sk.dts | 20 ++++++++++++++++++++
1 file changed, 20 insertions(+)

diff --git a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
index 9f3dfbfbf369..3ae1ea1997a4 100644
--- a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
+++ b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
@@ -102,6 +102,19 @@ vdd_mmc1: regulator-3 {
gpio = <&exp1 3 GPIO_ACTIVE_HIGH>;
};

+ vddshv_sdio: regulator-4 {
+ compatible = "regulator-gpio";
+ regulator-name = "vddshv_sdio";
+ pinctrl-names = "default";
+ pinctrl-0 = <&vddshv_sdio_default_pins>;
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-boot-on;
+ gpios = <&main_gpio0 31 GPIO_ACTIVE_HIGH>;
+ states = <1800000 0x0>,
+ <3300000 0x1>;
+ };
+
leds {
compatible = "gpio-leds";
pinctrl-names = "default";
@@ -189,6 +202,12 @@ AM62AX_IOPAD(0x1f8, PIN_INPUT_PULLUP, 0) /* (AC2) MMC0_DAT7 */
>;
};

+ vddshv_sdio_default_pins: vddshv-sdio-default-pins {
+ pinctrl-single,pins = <
+ AM62AX_IOPAD(0x07c, PIN_OUTPUT, 7) /* (M19) GPMC0_CLK.GPIO0_31 */
+ >;
+ };
+
main_mmc1_pins_default: main-mmc1-default-pins {
pinctrl-single,pins = <
AM62AX_IOPAD(0x23c, PIN_INPUT, 0) /* (A21) MMC1_CMD */
@@ -294,6 +313,7 @@ &sdhci1 {
/* SD/MMC */
status = "okay";
vmmc-supply = <&vdd_mmc1>;
+ vqmmc-supply = <&vddshv_sdio>;
pinctrl-names = "default";
pinctrl-0 = <&main_mmc1_pins_default>;
ti,driver-strength-ohm = <50>;
--
2.25.1

2023-10-26 07:02:22

by Nitin Yadav

Subject: [PATCH 2/2] arm64: dts: ti: k3-am62a-main: Remove no-1-8-v property

Remove no-1-8-v property from sdhci1 node in k3-am62a-main to
support High Speed cards in AM62a devices.

Signed-off-by: Nitin Yadav <[email protected]>
---
arch/arm64/boot/dts/ti/k3-am62a-main.dtsi | 1 -
1 file changed, 1 deletion(-)

diff --git a/arch/arm64/boot/dts/ti/k3-am62a-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62a-main.dtsi
index aeb32240e5b3..75e447ceb224 100644
--- a/arch/arm64/boot/dts/ti/k3-am62a-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am62a-main.dtsi
@@ -533,7 +533,6 @@ sdhci1: mmc@fa00000 {
ti,itap-del-sel-sdr25 = <0x0>;
ti,clkbuf-sel = <0x7>;
bus-width = <4>;
- no-1-8-v;
status = "disabled";
};

--
2.25.1

2023-11-30 22:20:22

by Judith Mendez

Subject: Re: [PATCH 1/2] arm64: dts: ti: k3-am62a: Enable UHS mode support for SD cards

On 10/26/23 2:01 AM, Nitin Yadav wrote:
> Hook up required IO voltage regulators to support UHS modes on
> SD cards.
>
> Signed-off-by: Nitin Yadav <[email protected]>
> ---
> arch/arm64/boot/dts/ti/k3-am62a7-sk.dts | 20 ++++++++++++++++++++
> 1 file changed, 20 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
> index 9f3dfbfbf369..3ae1ea1997a4 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
> @@ -102,6 +102,19 @@ vdd_mmc1: regulator-3 {
> gpio = <&exp1 3 GPIO_ACTIVE_HIGH>;
> };
>
> + vddshv_sdio: regulator-4 {
> + compatible = "regulator-gpio";
> + regulator-name = "vddshv_sdio";
> + pinctrl-names = "default";
> + pinctrl-0 = <&vddshv_sdio_default_pins>;
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <3300000>;
> + regulator-boot-on;
> + gpios = <&main_gpio0 31 GPIO_ACTIVE_HIGH>;
> + states = <1800000 0x0>,
> + <3300000 0x1>;
> + };
> +
> leds {
> compatible = "gpio-leds";
> pinctrl-names = "default";
> @@ -189,6 +202,12 @@ AM62AX_IOPAD(0x1f8, PIN_INPUT_PULLUP, 0) /* (AC2) MMC0_DAT7 */
> >;
> };
>
> + vddshv_sdio_default_pins: vddshv-sdio-default-pins {
> + pinctrl-single,pins = <
> + AM62AX_IOPAD(0x07c, PIN_OUTPUT, 7) /* (M19) GPMC0_CLK.GPIO0_31 */

Isn't this ball name N22? I might be wrong, but I did check using
SYSCONFIG tool and datasheet.

> + >;
> + };
> +
> main_mmc1_pins_default: main-mmc1-default-pins {
> pinctrl-single,pins = <
> AM62AX_IOPAD(0x23c, PIN_INPUT, 0) /* (A21) MMC1_CMD */
> @@ -294,6 +313,7 @@ &sdhci1 {
> /* SD/MMC */
> status = "okay";
> vmmc-supply = <&vdd_mmc1>;
> + vqmmc-supply = <&vddshv_sdio>;
> pinctrl-names = "default";
> pinctrl-0 = <&main_mmc1_pins_default>;
> ti,driver-strength-ohm = <50>;