* [PATCH v4 01/15] arm64: dts: ti: k3-am62a: Fix MMC0_CLK register address
2026-09-09 21:47 [PATCH v4 00/15] MMC pinmux configuration fixes Judith Mendez
@ 2026-09-09 21:47 ` Judith Mendez
2026-09-11 7:57 ` Kumar, Udit
2026-09-09 21:47 ` [PATCH v4 02/15] arm64: dts: ti: k3-am62l-evm: Remove unnecessary internal pullups for MMC0 Judith Mendez
` (15 subsequent siblings)
16 siblings, 1 reply; 26+ messages in thread
From: Judith Mendez @ 2026-09-09 21:47 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Wadim Egorov, Robert Nelson, Tony Lindgren,
Nora Schiffer, Stefano Radaelli, linux-arm-kernel, devicetree,
linux-kernel, Moteen Shah
MMC0_CLKLB and MMC0_CLK have incorrectly swapped register addresses
as per datasheet [0]. Set the correct MMC0_CLK address and drop the
unneeded MMC0_CLKLB pin configuration.
[0] https://www.ti.com/lit/gpn/am62a7 Rev. E: Table 5-1. Pin Attributes
Fixes: e041ec6e86e7 ("arm64: dts: ti: k3-am62a7-sk: Enable eMMC support")
Cc: <stable@vger.kernel.org>
Signed-off-by: Judith Mendez <jm@ti.com>
Reviewed-by: Moteen Shah <m-shah@ti.com>
---
arch/arm64/boot/dts/ti/k3-am62a7-sk.dts | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
index d3b3675e7a8fc..b04636cab71c7 100644
--- a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
+++ b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
@@ -328,8 +328,7 @@ AM62AX_IOPAD(0x0b4, PIN_INPUT_PULLUP, 1) /* (K24) GPMC0_CSn3.I2C2_SDA */
main_mmc0_pins_default: main-mmc0-default-pins {
pinctrl-single,pins = <
AM62AX_IOPAD(0x220, PIN_INPUT, 0) /* (Y3) MMC0_CMD */
- AM62AX_IOPAD(0x218, PIN_INPUT, 0) /* (AB1) MMC0_CLKLB */
- AM62AX_IOPAD(0x21c, PIN_INPUT, 0) /* (AB1) MMC0_CLK */
+ AM62AX_IOPAD(0x218, PIN_INPUT, 0) /* (AB1) MMC0_CLK */
AM62AX_IOPAD(0x214, PIN_INPUT, 0) /* (AA2) MMC0_DAT0 */
AM62AX_IOPAD(0x210, PIN_INPUT_PULLUP, 0) /* (AA1) MMC0_DAT1 */
AM62AX_IOPAD(0x20c, PIN_INPUT_PULLUP, 0) /* (AA3) MMC0_DAT2 */
--
2.34.1
^ permalink raw reply related [flat|nested] 26+ messages in thread* Re: [PATCH v4 01/15] arm64: dts: ti: k3-am62a: Fix MMC0_CLK register address
2026-09-09 21:47 ` [PATCH v4 01/15] arm64: dts: ti: k3-am62a: Fix MMC0_CLK register address Judith Mendez
@ 2026-09-11 7:57 ` Kumar, Udit
2026-09-11 16:54 ` Judith Mendez
0 siblings, 1 reply; 26+ messages in thread
From: Kumar, Udit @ 2026-09-11 7:57 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Wadim Egorov, Robert Nelson, Tony Lindgren,
Nora Schiffer, Stefano Radaelli, linux-arm-kernel, devicetree,
linux-kernel, Moteen Shah, u-kumar1
On 9/10/2026 3:17 AM, Judith Mendez wrote:
> MMC0_CLKLB and MMC0_CLK have incorrectly swapped register addresses
> as per datasheet [0]. Set the correct MMC0_CLK address and drop the
> unneeded MMC0_CLKLB pin configuration.
>
> [0] https://www.ti.com/lit/gpn/am62a7 Rev. E: Table 5-1. Pin Attributes
>
> Fixes: e041ec6e86e7 ("arm64: dts: ti: k3-am62a7-sk: Enable eMMC support")
> Cc: <stable@vger.kernel.org>
> Signed-off-by: Judith Mendez <jm@ti.com>
> Reviewed-by: Moteen Shah <m-shah@ti.com>
> ---
> arch/arm64/boot/dts/ti/k3-am62a7-sk.dts | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
> index d3b3675e7a8fc..b04636cab71c7 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
> @@ -328,8 +328,7 @@ AM62AX_IOPAD(0x0b4, PIN_INPUT_PULLUP, 1) /* (K24) GPMC0_CSn3.I2C2_SDA */
> main_mmc0_pins_default: main-mmc0-default-pins {
> pinctrl-single,pins = <
> AM62AX_IOPAD(0x220, PIN_INPUT, 0) /* (Y3) MMC0_CMD */
> - AM62AX_IOPAD(0x218, PIN_INPUT, 0) /* (AB1) MMC0_CLKLB */
> - AM62AX_IOPAD(0x21c, PIN_INPUT, 0) /* (AB1) MMC0_CLK */
> + AM62AX_IOPAD(0x218, PIN_INPUT, 0) /* (AB1) MMC0_CLK */
Shouldn't be MMC0_CLK be o/p, instead of PIN_INPUT
Similar to patch [0]
[0] https://lore.kernel.org/all/20260909214743.899477-4-jm@ti.com/
> AM62AX_IOPAD(0x214, PIN_INPUT, 0) /* (AA2) MMC0_DAT0 */
> AM62AX_IOPAD(0x210, PIN_INPUT_PULLUP, 0) /* (AA1) MMC0_DAT1 */
> AM62AX_IOPAD(0x20c, PIN_INPUT_PULLUP, 0) /* (AA3) MMC0_DAT2 */
^ permalink raw reply [flat|nested] 26+ messages in thread* Re: [PATCH v4 01/15] arm64: dts: ti: k3-am62a: Fix MMC0_CLK register address
2026-09-11 7:57 ` Kumar, Udit
@ 2026-09-11 16:54 ` Judith Mendez
2026-09-14 16:23 ` Kumar, Udit
0 siblings, 1 reply; 26+ messages in thread
From: Judith Mendez @ 2026-09-11 16:54 UTC (permalink / raw)
To: Kumar, Udit, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Wadim Egorov, Robert Nelson, Tony Lindgren,
Nora Schiffer, Stefano Radaelli, linux-arm-kernel, devicetree,
linux-kernel, Moteen Shah
Hi Udit,
On 9/11/26 2:57 AM, Kumar, Udit wrote:
>
>
> On 9/10/2026 3:17 AM, Judith Mendez wrote:
>> MMC0_CLKLB and MMC0_CLK have incorrectly swapped register addresses
>> as per datasheet [0]. Set the correct MMC0_CLK address and drop the
>> unneeded MMC0_CLKLB pin configuration.
>>
>> [0] https://www.ti.com/lit/gpn/am62a7 Rev. E: Table 5-1. Pin Attributes
>>
>> Fixes: e041ec6e86e7 ("arm64: dts: ti: k3-am62a7-sk: Enable eMMC support")
>> Cc: <stable@vger.kernel.org>
>> Signed-off-by: Judith Mendez <jm@ti.com>
>> Reviewed-by: Moteen Shah <m-shah@ti.com>
>> ---
>> arch/arm64/boot/dts/ti/k3-am62a7-sk.dts | 3 +--
>> 1 file changed, 1 insertion(+), 2 deletions(-)
>>
>> diff --git a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
>> index d3b3675e7a8fc..b04636cab71c7 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
>> +++ b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
>> @@ -328,8 +328,7 @@ AM62AX_IOPAD(0x0b4, PIN_INPUT_PULLUP, 1) /* (K24) GPMC0_CSn3.I2C2_SDA */
>> main_mmc0_pins_default: main-mmc0-default-pins {
>> pinctrl-single,pins = <
>> AM62AX_IOPAD(0x220, PIN_INPUT, 0) /* (Y3) MMC0_CMD */
>> - AM62AX_IOPAD(0x218, PIN_INPUT, 0) /* (AB1) MMC0_CLKLB */
>> - AM62AX_IOPAD(0x21c, PIN_INPUT, 0) /* (AB1) MMC0_CLK */
>> + AM62AX_IOPAD(0x218, PIN_INPUT, 0) /* (AB1) MMC0_CLK */
>
> Shouldn't be MMC0_CLK be o/p, instead of PIN_INPUT
> Similar to patch [0]
> [0] https://lore.kernel.org/all/20260909214743.899477-4-jm@ti.com/
Yes it should, and it is in the same series as you pointed out. This
current patch only fixes register address as per commit description. (:
~ Judith
>
>
>
>> AM62AX_IOPAD(0x214, PIN_INPUT, 0) /* (AA2) MMC0_DAT0 */
>> AM62AX_IOPAD(0x210, PIN_INPUT_PULLUP, 0) /* (AA1) MMC0_DAT1 */
>> AM62AX_IOPAD(0x20c, PIN_INPUT_PULLUP, 0) /* (AA3) MMC0_DAT2 */
>
^ permalink raw reply [flat|nested] 26+ messages in thread* Re: [PATCH v4 01/15] arm64: dts: ti: k3-am62a: Fix MMC0_CLK register address
2026-09-11 16:54 ` Judith Mendez
@ 2026-09-14 16:23 ` Kumar, Udit
0 siblings, 0 replies; 26+ messages in thread
From: Kumar, Udit @ 2026-09-14 16:23 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Wadim Egorov, Robert Nelson, Tony Lindgren,
Nora Schiffer, Stefano Radaelli, linux-arm-kernel, devicetree,
linux-kernel, Moteen Shah, u-kumar1
Thanks Judith
On 9/11/2026 10:24 PM, Judith Mendez wrote:
> Hi Udit,
>
> On 9/11/26 2:57 AM, Kumar, Udit wrote:
>>
>>
>> On 9/10/2026 3:17 AM, Judith Mendez wrote:
>>> MMC0_CLKLB and MMC0_CLK have incorrectly swapped register addresses
>>> as per datasheet [0]. Set the correct MMC0_CLK address and drop the
>>> unneeded MMC0_CLKLB pin configuration.
>>>
>>> [0] https://www.ti.com/lit/gpn/am62a7 Rev. E: Table 5-1. Pin Attributes
>>>
>>> Fixes: e041ec6e86e7 ("arm64: dts: ti: k3-am62a7-sk: Enable eMMC
>>> support")
>>> Cc: <stable@vger.kernel.org>
>>> Signed-off-by: Judith Mendez <jm@ti.com>
>>> Reviewed-by: Moteen Shah <m-shah@ti.com>
>>> ---
>>> arch/arm64/boot/dts/ti/k3-am62a7-sk.dts | 3 +--
>>> 1 file changed, 1 insertion(+), 2 deletions(-)
>>>
>>> diff --git a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts b/arch/arm64/
>>> boot/dts/ti/k3-am62a7-sk.dts
>>> index d3b3675e7a8fc..b04636cab71c7 100644
>>> --- a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
>>> +++ b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
>>> @@ -328,8 +328,7 @@ AM62AX_IOPAD(0x0b4, PIN_INPUT_PULLUP, 1) /* (K24)
>>> GPMC0_CSn3.I2C2_SDA */
>>> main_mmc0_pins_default: main-mmc0-default-pins {
>>> pinctrl-single,pins = <
>>> AM62AX_IOPAD(0x220, PIN_INPUT, 0) /* (Y3) MMC0_CMD */
>>> - AM62AX_IOPAD(0x218, PIN_INPUT, 0) /* (AB1) MMC0_CLKLB */
>>> - AM62AX_IOPAD(0x21c, PIN_INPUT, 0) /* (AB1) MMC0_CLK */
>>> + AM62AX_IOPAD(0x218, PIN_INPUT, 0) /* (AB1) MMC0_CLK */
>>
>> Shouldn't be MMC0_CLK be o/p, instead of PIN_INPUT
>> Similar to patch [0]
>> [0] https://lore.kernel.org/all/20260909214743.899477-4-jm@ti.com/
>
> Yes it should, and it is in the same series as you pointed out. This
> current patch only fixes register address as per commit description. (:
>
I see. These can be merged into one patch,
Anyway,
Acked-by: Udit Kumar <u-kumar1@ti.com> (for fixing offset)
> ~ Judith
>
>>
>>
>>
>>> AM62AX_IOPAD(0x214, PIN_INPUT, 0) /* (AA2) MMC0_DAT0 */
>>> AM62AX_IOPAD(0x210, PIN_INPUT_PULLUP, 0) /* (AA1)
>>> MMC0_DAT1 */
>>> AM62AX_IOPAD(0x20c, PIN_INPUT_PULLUP, 0) /* (AA3)
>>> MMC0_DAT2 */
>>
>
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH v4 02/15] arm64: dts: ti: k3-am62l-evm: Remove unnecessary internal pullups for MMC0
2026-09-09 21:47 [PATCH v4 00/15] MMC pinmux configuration fixes Judith Mendez
2026-09-09 21:47 ` [PATCH v4 01/15] arm64: dts: ti: k3-am62a: Fix MMC0_CLK register address Judith Mendez
@ 2026-09-09 21:47 ` Judith Mendez
2026-09-09 21:47 ` [PATCH v4 03/15] arm64: dts: ti: k3-am62a7-sk: Set eMMC clock pin as OUTPUT Judith Mendez
` (14 subsequent siblings)
16 siblings, 0 replies; 26+ messages in thread
From: Judith Mendez @ 2026-09-09 21:47 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Wadim Egorov, Robert Nelson, Tony Lindgren,
Nora Schiffer, Stefano Radaelli, linux-arm-kernel, devicetree,
linux-kernel, Moteen Shah
AM62L EVM has external pullups on MMC0 CMD and D0 pins [0]. Remove
internal pullups considering:
- Internal pullups are not required, without internal pullups, RDAT and
RCMD values of 47K are within spec
- having both pullups (stronger pull) results in unnecessary power
consumption
[0] https://www.ti.com/lit/zip/SPRCAL6
Fixes: 00fb4c73b67d ("arm64: dts: ti: k3-am62l: add initial reference board file")
Signed-off-by: Judith Mendez <jm@ti.com>
---
arch/arm64/boot/dts/ti/k3-am62l3-evm.dts | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
index 84a0cfaaf196a..964387eb51788 100644
--- a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
+++ b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts
@@ -253,9 +253,9 @@ AM62LX_IOPAD(0x01e0, PIN_INPUT_PULLUP, 0) /* (D8) I2C2_SDA */
mmc0_pins_default: mmc0-default-pins {
pinctrl-single,pins = <
- AM62LX_IOPAD(0x0214, PIN_INPUT_PULLUP, 0) /* (D2) MMC0_CMD */
+ AM62LX_IOPAD(0x0214, PIN_INPUT, 0) /* (D2) MMC0_CMD */
AM62LX_IOPAD(0x020c, PIN_OUTPUT, 0) /* (B2) MMC0_CLK */
- AM62LX_IOPAD(0x0208, PIN_INPUT_PULLUP, 0) /* (D3) MMC0_DAT0 */
+ AM62LX_IOPAD(0x0208, PIN_INPUT, 0) /* (D3) MMC0_DAT0 */
AM62LX_IOPAD(0x0204, PIN_INPUT_PULLUP, 0) /* (D4) MMC0_DAT1 */
AM62LX_IOPAD(0x0200, PIN_INPUT_PULLUP, 0) /* (C1) MMC0_DAT2 */
AM62LX_IOPAD(0x01fc, PIN_INPUT_PULLUP, 0) /* (C2) MMC0_DAT3 */
--
2.34.1
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH v4 03/15] arm64: dts: ti: k3-am62a7-sk: Set eMMC clock pin as OUTPUT
2026-09-09 21:47 [PATCH v4 00/15] MMC pinmux configuration fixes Judith Mendez
2026-09-09 21:47 ` [PATCH v4 01/15] arm64: dts: ti: k3-am62a: Fix MMC0_CLK register address Judith Mendez
2026-09-09 21:47 ` [PATCH v4 02/15] arm64: dts: ti: k3-am62l-evm: Remove unnecessary internal pullups for MMC0 Judith Mendez
@ 2026-09-09 21:47 ` Judith Mendez
2026-09-09 21:47 ` [PATCH v4 04/15] arm64: dts: ti: k3-am62a7-sk: Set SD " Judith Mendez
` (13 subsequent siblings)
16 siblings, 0 replies; 26+ messages in thread
From: Judith Mendez @ 2026-09-09 21:47 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Wadim Egorov, Robert Nelson, Tony Lindgren,
Nora Schiffer, Stefano Radaelli, linux-arm-kernel, devicetree,
linux-kernel, Moteen Shah
MMC clock pins must be configured as OUTPUT-only per TRM [0] with
receiver disabled (RXACTIVE bit = 0). Change MMC CLK pins from
PIN_INPUT to PIN_OUTPUT to match documentation.
[0] https://www.ti.com/lit/pdf/spruj16 Rev. C: Table 12-257. MMCSDi I/O Signals
Fixes: e041ec6e86e7 ("arm64: dts: ti: k3-am62a7-sk: Enable eMMC support")
Cc: <stable@vger.kernel.org>
Signed-off-by: Judith Mendez <jm@ti.com>
Reviewed-by: Moteen Shah <m-shah@ti.com>
---
arch/arm64/boot/dts/ti/k3-am62a7-sk.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
index b04636cab71c7..f6778f9798d40 100644
--- a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
+++ b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
@@ -328,7 +328,7 @@ AM62AX_IOPAD(0x0b4, PIN_INPUT_PULLUP, 1) /* (K24) GPMC0_CSn3.I2C2_SDA */
main_mmc0_pins_default: main-mmc0-default-pins {
pinctrl-single,pins = <
AM62AX_IOPAD(0x220, PIN_INPUT, 0) /* (Y3) MMC0_CMD */
- AM62AX_IOPAD(0x218, PIN_INPUT, 0) /* (AB1) MMC0_CLK */
+ AM62AX_IOPAD(0x218, PIN_OUTPUT, 0) /* (AB1) MMC0_CLK */
AM62AX_IOPAD(0x214, PIN_INPUT, 0) /* (AA2) MMC0_DAT0 */
AM62AX_IOPAD(0x210, PIN_INPUT_PULLUP, 0) /* (AA1) MMC0_DAT1 */
AM62AX_IOPAD(0x20c, PIN_INPUT_PULLUP, 0) /* (AA3) MMC0_DAT2 */
--
2.34.1
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH v4 04/15] arm64: dts: ti: k3-am62a7-sk: Set SD clock pin as OUTPUT
2026-09-09 21:47 [PATCH v4 00/15] MMC pinmux configuration fixes Judith Mendez
` (2 preceding siblings ...)
2026-09-09 21:47 ` [PATCH v4 03/15] arm64: dts: ti: k3-am62a7-sk: Set eMMC clock pin as OUTPUT Judith Mendez
@ 2026-09-09 21:47 ` Judith Mendez
2026-09-09 21:47 ` [PATCH v4 05/15] arm64: dts: ti: k3-am62a-phycore-som: Set eMMC " Judith Mendez
` (12 subsequent siblings)
16 siblings, 0 replies; 26+ messages in thread
From: Judith Mendez @ 2026-09-09 21:47 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Wadim Egorov, Robert Nelson, Tony Lindgren,
Nora Schiffer, Stefano Radaelli, linux-arm-kernel, devicetree,
linux-kernel, Moteen Shah
MMC clock pins must be configured as OUTPUT-only per TRM [0] with
receiver disabled (RXACTIVE bit = 0). Change MMC CLK pins from
PIN_INPUT to PIN_OUTPUT to match documentation.
[0] https://www.ti.com/lit/pdf/spruj16 Rev. C: Table 12-257. MMCSDi I/O Signals
Fixes: 38c4a08c820c ("arm64: dts: ti: Add support for AM62A7-SK")
Cc: <stable@vger.kernel.org>
Signed-off-by: Judith Mendez <jm@ti.com>
Reviewed-by: Moteen Shah <m-shah@ti.com>
---
arch/arm64/boot/dts/ti/k3-am62a7-sk.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
index f6778f9798d40..e63c5200a2370 100644
--- a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
+++ b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
@@ -344,7 +344,7 @@ AM62AX_IOPAD(0x1f8, PIN_INPUT_PULLUP, 0) /* (AC2) MMC0_DAT7 */
main_mmc1_pins_default: main-mmc1-default-pins {
pinctrl-single,pins = <
AM62AX_IOPAD(0x23c, PIN_INPUT, 0) /* (A21) MMC1_CMD */
- AM62AX_IOPAD(0x234, PIN_INPUT, 0) /* (B22) MMC1_CLK */
+ AM62AX_IOPAD(0x234, PIN_OUTPUT, 0) /* (B22) MMC1_CLK */
AM62AX_IOPAD(0x230, PIN_INPUT, 0) /* (A22) MMC1_DAT0 */
AM62AX_IOPAD(0x22c, PIN_INPUT, 0) /* (B21) MMC1_DAT1 */
AM62AX_IOPAD(0x228, PIN_INPUT, 0) /* (C21) MMC1_DAT2 */
--
2.34.1
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH v4 05/15] arm64: dts: ti: k3-am62a-phycore-som: Set eMMC clock pin as OUTPUT
2026-09-09 21:47 [PATCH v4 00/15] MMC pinmux configuration fixes Judith Mendez
` (3 preceding siblings ...)
2026-09-09 21:47 ` [PATCH v4 04/15] arm64: dts: ti: k3-am62a7-sk: Set SD " Judith Mendez
@ 2026-09-09 21:47 ` Judith Mendez
2026-09-15 16:54 ` Wadim Egorov
2026-09-09 21:47 ` [PATCH v4 06/15] arm64: dts: ti: k3-am62-lp-sk: Set eMMC clock pins " Judith Mendez
` (11 subsequent siblings)
16 siblings, 1 reply; 26+ messages in thread
From: Judith Mendez @ 2026-09-09 21:47 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Wadim Egorov, Robert Nelson, Tony Lindgren,
Nora Schiffer, Stefano Radaelli, linux-arm-kernel, devicetree,
linux-kernel, Moteen Shah
MMC clock pins must be configured as OUTPUT-only per TRM [0] with
receiver disabled (RXACTIVE bit = 0). Change MMC CLK pins from
PIN_INPUT to PIN_OUTPUT to match documentation.
[0] https://www.ti.com/lit/pdf/spruj16 Rev. C: Table 12-257. MMCSDi I/O Signals
Fixes: d69383885576 ("arm64: dts: ti: Add basic support for phyBOARD-Lyra-AM62Ax")
Cc: <stable@vger.kernel.org>
Signed-off-by: Judith Mendez <jm@ti.com>
---
arch/arm64/boot/dts/ti/k3-am62a-phycore-som.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am62a-phycore-som.dtsi b/arch/arm64/boot/dts/ti/k3-am62a-phycore-som.dtsi
index de4048a3564bc..5ad93be3d3363 100644
--- a/arch/arm64/boot/dts/ti/k3-am62a-phycore-som.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am62a-phycore-som.dtsi
@@ -120,7 +120,7 @@ AM62AX_IOPAD(0x15c, PIN_INPUT, 0) /* (V13) MDIO0_MDIO */
main_mmc0_pins_default: main-mmc0-default-pins {
pinctrl-single,pins = <
AM62AX_IOPAD(0x220, PIN_INPUT_PULLUP, 0) /* (Y6) MMC0_CMD */
- AM62AX_IOPAD(0x218, PIN_INPUT_PULLDOWN, 0) /* (AB7) MMC0_CLK */
+ AM62AX_IOPAD(0x218, PIN_OUTPUT_PULLDOWN, 0) /* (AB7) MMC0_CLK */
AM62AX_IOPAD(0x214, PIN_INPUT_PULLUP, 0) /* (AA6) MMC0_DAT0 */
AM62AX_IOPAD(0x210, PIN_INPUT_PULLUP, 0) /* (AB6) MMC0_DAT1 */
AM62AX_IOPAD(0x20c, PIN_INPUT_PULLUP, 0) /* (Y7) MMC0_DAT2 */
--
2.34.1
^ permalink raw reply related [flat|nested] 26+ messages in thread* Re: [PATCH v4 05/15] arm64: dts: ti: k3-am62a-phycore-som: Set eMMC clock pin as OUTPUT
2026-09-09 21:47 ` [PATCH v4 05/15] arm64: dts: ti: k3-am62a-phycore-som: Set eMMC " Judith Mendez
@ 2026-09-15 16:54 ` Wadim Egorov
0 siblings, 0 replies; 26+ messages in thread
From: Wadim Egorov @ 2026-09-15 16:54 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Robert Nelson, Tony Lindgren, Nora Schiffer,
Stefano Radaelli, linux-arm-kernel, devicetree, linux-kernel,
Moteen Shah
On 9/10/26 12:47 AM, Judith Mendez wrote:
> MMC clock pins must be configured as OUTPUT-only per TRM [0] with
> receiver disabled (RXACTIVE bit = 0). Change MMC CLK pins from
> PIN_INPUT to PIN_OUTPUT to match documentation.
>
> [0] https://www.ti.com/lit/pdf/spruj16 Rev. C: Table 12-257. MMCSDi I/O Signals
>
> Fixes: d69383885576 ("arm64: dts: ti: Add basic support for phyBOARD-Lyra-AM62Ax")
> Cc: <stable@vger.kernel.org>
> Signed-off-by: Judith Mendez <jm@ti.com>
Acked-by: Wadim Egorov <w.egorov@phytec.de>
> ---
> arch/arm64/boot/dts/ti/k3-am62a-phycore-som.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am62a-phycore-som.dtsi b/arch/arm64/boot/dts/ti/k3-am62a-phycore-som.dtsi
> index de4048a3564bc..5ad93be3d3363 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62a-phycore-som.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62a-phycore-som.dtsi
> @@ -120,7 +120,7 @@ AM62AX_IOPAD(0x15c, PIN_INPUT, 0) /* (V13) MDIO0_MDIO */
> main_mmc0_pins_default: main-mmc0-default-pins {
> pinctrl-single,pins = <
> AM62AX_IOPAD(0x220, PIN_INPUT_PULLUP, 0) /* (Y6) MMC0_CMD */
> - AM62AX_IOPAD(0x218, PIN_INPUT_PULLDOWN, 0) /* (AB7) MMC0_CLK */
> + AM62AX_IOPAD(0x218, PIN_OUTPUT_PULLDOWN, 0) /* (AB7) MMC0_CLK */
> AM62AX_IOPAD(0x214, PIN_INPUT_PULLUP, 0) /* (AA6) MMC0_DAT0 */
> AM62AX_IOPAD(0x210, PIN_INPUT_PULLUP, 0) /* (AB6) MMC0_DAT1 */
> AM62AX_IOPAD(0x20c, PIN_INPUT_PULLUP, 0) /* (Y7) MMC0_DAT2 */
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH v4 06/15] arm64: dts: ti: k3-am62-lp-sk: Set eMMC clock pins as OUTPUT
2026-09-09 21:47 [PATCH v4 00/15] MMC pinmux configuration fixes Judith Mendez
` (4 preceding siblings ...)
2026-09-09 21:47 ` [PATCH v4 05/15] arm64: dts: ti: k3-am62a-phycore-som: Set eMMC " Judith Mendez
@ 2026-09-09 21:47 ` Judith Mendez
2026-09-09 21:47 ` [PATCH v4 07/15] arm64: dts: ti: k3-am62-pocketbeagle2: Set SD clock pin " Judith Mendez
` (10 subsequent siblings)
16 siblings, 0 replies; 26+ messages in thread
From: Judith Mendez @ 2026-09-09 21:47 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Wadim Egorov, Robert Nelson, Tony Lindgren,
Nora Schiffer, Stefano Radaelli, linux-arm-kernel, devicetree,
linux-kernel, Moteen Shah
MMC clock pins must be configured as OUTPUT-only per TRM [0] with
receiver disabled (RXACTIVE bit = 0). Change MMC CLK pins from
PIN_INPUT to PIN_OUTPUT to match documentation.
[0] https://www.ti.com/lit/pdf/spruiv7 Rev. C: Table 12-246. MMCSDi I/O Signals
Fixes: e6a51ffabfc1 ("arm64: ti: dts: Add support for AM62x LP SK")
Cc: <stable@vger.kernel.org>
Signed-off-by: Judith Mendez <jm@ti.com>
Reviewed-by: Moteen Shah <m-shah@ti.com>
---
arch/arm64/boot/dts/ti/k3-am62-lp-sk.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am62-lp-sk.dts b/arch/arm64/boot/dts/ti/k3-am62-lp-sk.dts
index dfd65c00f5b95..7fd508f4eddf1 100644
--- a/arch/arm64/boot/dts/ti/k3-am62-lp-sk.dts
+++ b/arch/arm64/boot/dts/ti/k3-am62-lp-sk.dts
@@ -86,7 +86,7 @@ main_mmc0_pins_default: main-mmc0-default-pins {
bootph-all;
pinctrl-single,pins = <
AM62X_IOPAD(0x220, PIN_INPUT, 0) /* (V3) MMC0_CMD */
- AM62X_IOPAD(0x218, PIN_INPUT, 0) /* (Y1) MMC0_CLK */
+ AM62X_IOPAD(0x218, PIN_OUTPUT, 0) /* (Y1) MMC0_CLK */
AM62X_IOPAD(0x214, PIN_INPUT, 0) /* (V2) MMC0_DAT0 */
AM62X_IOPAD(0x210, PIN_INPUT_PULLUP, 0) /* (V1) MMC0_DAT1 */
AM62X_IOPAD(0x20c, PIN_INPUT_PULLUP, 0) /* (W2) MMC0_DAT2 */
--
2.34.1
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH v4 07/15] arm64: dts: ti: k3-am62-pocketbeagle2: Set SD clock pin as OUTPUT
2026-09-09 21:47 [PATCH v4 00/15] MMC pinmux configuration fixes Judith Mendez
` (5 preceding siblings ...)
2026-09-09 21:47 ` [PATCH v4 06/15] arm64: dts: ti: k3-am62-lp-sk: Set eMMC clock pins " Judith Mendez
@ 2026-09-09 21:47 ` Judith Mendez
2026-09-09 21:47 ` [PATCH v4 08/15] arm64: dts: ti: k3-am625-beagleplay: Set MMC clock pins " Judith Mendez
` (9 subsequent siblings)
16 siblings, 0 replies; 26+ messages in thread
From: Judith Mendez @ 2026-09-09 21:47 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Wadim Egorov, Robert Nelson, Tony Lindgren,
Nora Schiffer, Stefano Radaelli, linux-arm-kernel, devicetree,
linux-kernel, Moteen Shah
MMC clock pins must be configured as OUTPUT-only per TRM [0] with
receiver disabled (RXACTIVE bit = 0). Change MMC CLK pins from
PIN_INPUT to PIN_OUTPUT to match documentation.
[0] https://www.ti.com/lit/pdf/spruiv7 Rev. C: Table 12-246. MMCSDi I/O Signals
Fixes: 92d8c028aa92 ("arm64: dts: ti: Add k3-am62-pocketbeagle2")
Cc: <stable@vger.kernel.org>
Signed-off-by: Judith Mendez <jm@ti.com>
Reviewed-by: Moteen Shah <m-shah@ti.com>
---
arch/arm64/boot/dts/ti/k3-am62-pocketbeagle2.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am62-pocketbeagle2.dts b/arch/arm64/boot/dts/ti/k3-am62-pocketbeagle2.dts
index 7a4cffc27bda3..1c691629d4ad0 100644
--- a/arch/arm64/boot/dts/ti/k3-am62-pocketbeagle2.dts
+++ b/arch/arm64/boot/dts/ti/k3-am62-pocketbeagle2.dts
@@ -235,7 +235,7 @@ AM62X_IOPAD(0x0020, PIN_OUTPUT, 3) /* (J25) OSPI0_D5.UART6_TXD */
main_mmc1_pins_default: main-mmc1-default-pins {
pinctrl-single,pins = <
AM62X_IOPAD(0x23c, PIN_INPUT, 0) /* (A21/C18) MMC1_CMD */
- AM62X_IOPAD(0x234, PIN_INPUT, 0) /* (B22/A20) MMC1_CLK */
+ AM62X_IOPAD(0x234, PIN_OUTPUT, 0) /* (B22/A20) MMC1_CLK */
AM62X_IOPAD(0x230, PIN_INPUT, 0) /* (A22/A19) MMC1_DAT0 */
AM62X_IOPAD(0x22c, PIN_INPUT, 0) /* (B21/B19) MMC1_DAT1 */
AM62X_IOPAD(0x228, PIN_INPUT, 0) /* (C21/B20) MMC1_DAT2 */
--
2.34.1
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH v4 08/15] arm64: dts: ti: k3-am625-beagleplay: Set MMC clock pins as OUTPUT
2026-09-09 21:47 [PATCH v4 00/15] MMC pinmux configuration fixes Judith Mendez
` (6 preceding siblings ...)
2026-09-09 21:47 ` [PATCH v4 07/15] arm64: dts: ti: k3-am62-pocketbeagle2: Set SD clock pin " Judith Mendez
@ 2026-09-09 21:47 ` Judith Mendez
2026-09-09 21:47 ` [PATCH v4 09/15] arm64: dts: ti: k3-am62x-sk-common: Set SD clock pin " Judith Mendez
` (8 subsequent siblings)
16 siblings, 0 replies; 26+ messages in thread
From: Judith Mendez @ 2026-09-09 21:47 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Wadim Egorov, Robert Nelson, Tony Lindgren,
Nora Schiffer, Stefano Radaelli, linux-arm-kernel, devicetree,
linux-kernel, Moteen Shah
MMC clock pins must be configured as OUTPUT-only per TRM [0] with
receiver disabled (RXACTIVE bit = 0). Change MMC CLK pins from
PIN_INPUT to PIN_OUTPUT to match documentation.
[0] https://www.ti.com/lit/pdf/spruiv7 Rev. C: Table 12-246. MMCSDi I/O Signals
Fixes: f5a731f0787f ("arm64: dts: ti: Add k3-am625-beagleplay")
Cc: <stable@vger.kernel.org>
Signed-off-by: Judith Mendez <jm@ti.com>
Reviewed-by: Moteen Shah <m-shah@ti.com>
---
arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts b/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts
index c468b9c5fc09a..898e4ca3fd87e 100644
--- a/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts
+++ b/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts
@@ -323,7 +323,7 @@ emmc_pins_default: emmc-default-pins {
bootph-all;
pinctrl-single,pins = <
AM62X_IOPAD(0x0220, PIN_INPUT, 0) /* (Y3) MMC0_CMD */
- AM62X_IOPAD(0x0218, PIN_INPUT, 0) /* (AB1) MMC0_CLK */
+ AM62X_IOPAD(0x0218, PIN_OUTPUT, 0) /* (AB1) MMC0_CLK */
AM62X_IOPAD(0x0214, PIN_INPUT, 0) /* (AA2) MMC0_DAT0 */
AM62X_IOPAD(0x0210, PIN_INPUT, 0) /* (AA1) MMC0_DAT1 */
AM62X_IOPAD(0x020c, PIN_INPUT, 0) /* (AA3) MMC0_DAT2 */
@@ -346,7 +346,7 @@ sd_pins_default: sd-default-pins {
bootph-all;
pinctrl-single,pins = <
AM62X_IOPAD(0x023c, PIN_INPUT, 0) /* (A21) MMC1_CMD */
- AM62X_IOPAD(0x0234, PIN_INPUT, 0) /* (B22) MMC1_CLK */
+ AM62X_IOPAD(0x0234, PIN_OUTPUT, 0) /* (B22) MMC1_CLK */
AM62X_IOPAD(0x0230, PIN_INPUT, 0) /* (A22) MMC1_DAT0 */
AM62X_IOPAD(0x022c, PIN_INPUT, 0) /* (B21) MMC1_DAT1 */
AM62X_IOPAD(0x0228, PIN_INPUT, 0) /* (C21) MMC1_DAT2 */
@@ -358,7 +358,7 @@ AM62X_IOPAD(0x0240, PIN_INPUT, 7) /* (D17) MMC1_SDCD.GPIO1_48 */
wifi_pins_default: wifi-default-pins {
pinctrl-single,pins = <
AM62X_IOPAD(0x0120, PIN_INPUT, 0) /* (C24) MMC2_CMD */
- AM62X_IOPAD(0x0118, PIN_INPUT, 0) /* (D25) MMC2_CLK */
+ AM62X_IOPAD(0x0118, PIN_OUTPUT, 0) /* (D25) MMC2_CLK */
AM62X_IOPAD(0x0114, PIN_INPUT, 0) /* (B24) MMC2_DAT0 */
AM62X_IOPAD(0x0110, PIN_INPUT, 0) /* (C25) MMC2_DAT1 */
AM62X_IOPAD(0x010c, PIN_INPUT, 0) /* (E23) MMC2_DAT2 */
--
2.34.1
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH v4 09/15] arm64: dts: ti: k3-am62x-sk-common: Set SD clock pin as OUTPUT
2026-09-09 21:47 [PATCH v4 00/15] MMC pinmux configuration fixes Judith Mendez
` (7 preceding siblings ...)
2026-09-09 21:47 ` [PATCH v4 08/15] arm64: dts: ti: k3-am625-beagleplay: Set MMC clock pins " Judith Mendez
@ 2026-09-09 21:47 ` Judith Mendez
2026-09-09 21:47 ` [PATCH v4 10/15] arm64: dts: ti: k3-am625-sk-common: Set eMMC " Judith Mendez
` (7 subsequent siblings)
16 siblings, 0 replies; 26+ messages in thread
From: Judith Mendez @ 2026-09-09 21:47 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Wadim Egorov, Robert Nelson, Tony Lindgren,
Nora Schiffer, Stefano Radaelli, linux-arm-kernel, devicetree,
linux-kernel, Moteen Shah
MMC clock pins must be configured as OUTPUT-only per TRM [0] with
receiver disabled (RXACTIVE bit = 0). Change MMC CLK pins from
PIN_INPUT to PIN_OUTPUT to match documentation.
[0] https://www.ti.com/lit/pdf/spruiv7 Rev. C: Table 12-246. MMCSDi I/O Signals
Fixes: d19a66ae488a ("arm64: dts: ti: k3-am625-sk: Enable on board peripherals")
Cc: <stable@vger.kernel.org>
Signed-off-by: Judith Mendez <jm@ti.com>
Reviewed-by: Moteen Shah <m-shah@ti.com>
---
arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi b/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi
index ab9e58c2d225e..d2f41e2ed6239 100644
--- a/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi
@@ -194,7 +194,7 @@ main_mmc1_pins_default: main-mmc1-default-pins {
bootph-all;
pinctrl-single,pins = <
AM62X_IOPAD(0x23c, PIN_INPUT, 0) /* (A21/C18) MMC1_CMD */
- AM62X_IOPAD(0x234, PIN_INPUT, 0) /* (B22/A20) MMC1_CLK */
+ AM62X_IOPAD(0x234, PIN_OUTPUT, 0) /* (B22/A20) MMC1_CLK */
AM62X_IOPAD(0x230, PIN_INPUT, 0) /* (A22/A19) MMC1_DAT0 */
AM62X_IOPAD(0x22c, PIN_INPUT, 0) /* (B21/B19) MMC1_DAT1 */
AM62X_IOPAD(0x228, PIN_INPUT, 0) /* (C21/B20) MMC1_DAT2 */
--
2.34.1
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH v4 10/15] arm64: dts: ti: k3-am625-sk-common: Set eMMC clock pin as OUTPUT
2026-09-09 21:47 [PATCH v4 00/15] MMC pinmux configuration fixes Judith Mendez
` (8 preceding siblings ...)
2026-09-09 21:47 ` [PATCH v4 09/15] arm64: dts: ti: k3-am62x-sk-common: Set SD clock pin " Judith Mendez
@ 2026-09-09 21:47 ` Judith Mendez
2026-09-09 21:47 ` [PATCH v4 11/15] arm64: dts: ti: k3-am62-phycore-som: " Judith Mendez
` (6 subsequent siblings)
16 siblings, 0 replies; 26+ messages in thread
From: Judith Mendez @ 2026-09-09 21:47 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Wadim Egorov, Robert Nelson, Tony Lindgren,
Nora Schiffer, Stefano Radaelli, linux-arm-kernel, devicetree,
linux-kernel, Moteen Shah
MMC clock pins must be configured as OUTPUT-only per TRM [0] with
receiver disabled (RXACTIVE bit = 0). Change MMC CLK pins from
PIN_INPUT to PIN_OUTPUT to match documentation.
[0] https://www.ti.com/lit/pdf/spruiv7 Rev. C: Table 12-246. MMCSDi I/O Signals
Fixes: d19a66ae488a ("arm64: dts: ti: k3-am625-sk: Enable on board peripherals")
Cc: <stable@vger.kernel.org>
Signed-off-by: Judith Mendez <jm@ti.com>
Reviewed-by: Moteen Shah <m-shah@ti.com>
---
arch/arm64/boot/dts/ti/k3-am625-sk-common.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am625-sk-common.dtsi b/arch/arm64/boot/dts/ti/k3-am625-sk-common.dtsi
index 9c83626826453..085b067d96fbb 100644
--- a/arch/arm64/boot/dts/ti/k3-am625-sk-common.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am625-sk-common.dtsi
@@ -98,7 +98,7 @@ main_mmc0_pins_default: main-mmc0-default-pins {
bootph-all;
pinctrl-single,pins = <
AM62X_IOPAD(0x220, PIN_INPUT, 0) /* (Y3) MMC0_CMD */
- AM62X_IOPAD(0x218, PIN_INPUT, 0) /* (AB1) MMC0_CLK */
+ AM62X_IOPAD(0x218, PIN_OUTPUT, 0) /* (AB1) MMC0_CLK */
AM62X_IOPAD(0x214, PIN_INPUT, 0) /* (AA2) MMC0_DAT0 */
AM62X_IOPAD(0x210, PIN_INPUT_PULLUP, 0) /* (AA1) MMC0_DAT1 */
AM62X_IOPAD(0x20c, PIN_INPUT_PULLUP, 0) /* (AA3) MMC0_DAT2 */
--
2.34.1
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH v4 11/15] arm64: dts: ti: k3-am62-phycore-som: Set eMMC clock pin as OUTPUT
2026-09-09 21:47 [PATCH v4 00/15] MMC pinmux configuration fixes Judith Mendez
` (9 preceding siblings ...)
2026-09-09 21:47 ` [PATCH v4 10/15] arm64: dts: ti: k3-am625-sk-common: Set eMMC " Judith Mendez
@ 2026-09-09 21:47 ` Judith Mendez
2026-09-15 16:54 ` Wadim Egorov
2026-09-09 21:47 ` [PATCH v4 12/15] arm64: dts: ti: k3-am625-tqma62xx: Set MMC clock pins " Judith Mendez
` (5 subsequent siblings)
16 siblings, 1 reply; 26+ messages in thread
From: Judith Mendez @ 2026-09-09 21:47 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Wadim Egorov, Robert Nelson, Tony Lindgren,
Nora Schiffer, Stefano Radaelli, linux-arm-kernel, devicetree,
linux-kernel, Moteen Shah
MMC clock pins must be configured as OUTPUT-only per TRM [0] with
receiver disabled (RXACTIVE bit = 0). Change MMC CLK pins from
PIN_INPUT to PIN_OUTPUT to match documentation.
[0] https://www.ti.com/lit/pdf/spruiv7 Rev. C: Table 12-246. MMCSDi I/O Signals
Fixes: 3443c1c4ed61 ("arm64: dts: ti: Add basic support for phyBOARD-Lyra-AM625")
Cc: <stable@vger.kernel.org>
Signed-off-by: Judith Mendez <jm@ti.com>
---
arch/arm64/boot/dts/ti/k3-am62-phycore-som.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am62-phycore-som.dtsi b/arch/arm64/boot/dts/ti/k3-am62-phycore-som.dtsi
index 3baa653257bb6..ce3e5165bd310 100644
--- a/arch/arm64/boot/dts/ti/k3-am62-phycore-som.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am62-phycore-som.dtsi
@@ -153,7 +153,7 @@ AM62X_IOPAD(0x15c, PIN_INPUT, 0) /* (AB22) MDIO0_MDIO */
main_mmc0_pins_default: main-mmc0-default-pins {
pinctrl-single,pins = <
AM62X_IOPAD(0x220, PIN_INPUT_PULLUP, 0) /* (Y3) MMC0_CMD */
- AM62X_IOPAD(0x218, PIN_INPUT_PULLDOWN, 0) /* (AB1) MMC0_CLK */
+ AM62X_IOPAD(0x218, PIN_OUTPUT_PULLDOWN, 0) /* (AB1) MMC0_CLK */
AM62X_IOPAD(0x214, PIN_INPUT_PULLUP, 0) /* (AA2) MMC0_DAT0 */
AM62X_IOPAD(0x210, PIN_INPUT_PULLUP, 0) /* (AA1) MMC0_DAT1 */
AM62X_IOPAD(0x20c, PIN_INPUT_PULLUP, 0) /* (AA3) MMC0_DAT2 */
--
2.34.1
^ permalink raw reply related [flat|nested] 26+ messages in thread* Re: [PATCH v4 11/15] arm64: dts: ti: k3-am62-phycore-som: Set eMMC clock pin as OUTPUT
2026-09-09 21:47 ` [PATCH v4 11/15] arm64: dts: ti: k3-am62-phycore-som: " Judith Mendez
@ 2026-09-15 16:54 ` Wadim Egorov
0 siblings, 0 replies; 26+ messages in thread
From: Wadim Egorov @ 2026-09-15 16:54 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Robert Nelson, Tony Lindgren, Nora Schiffer,
Stefano Radaelli, linux-arm-kernel, devicetree, linux-kernel,
Moteen Shah
On 9/10/26 12:47 AM, Judith Mendez wrote:
> MMC clock pins must be configured as OUTPUT-only per TRM [0] with
> receiver disabled (RXACTIVE bit = 0). Change MMC CLK pins from
> PIN_INPUT to PIN_OUTPUT to match documentation.
>
> [0] https://www.ti.com/lit/pdf/spruiv7 Rev. C: Table 12-246. MMCSDi I/O Signals
>
> Fixes: 3443c1c4ed61 ("arm64: dts: ti: Add basic support for phyBOARD-Lyra-AM625")
> Cc: <stable@vger.kernel.org>
> Signed-off-by: Judith Mendez <jm@ti.com>
Acked-by: Wadim Egorov <w.egorov@phytec.de>
> ---
> arch/arm64/boot/dts/ti/k3-am62-phycore-som.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am62-phycore-som.dtsi b/arch/arm64/boot/dts/ti/k3-am62-phycore-som.dtsi
> index 3baa653257bb6..ce3e5165bd310 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62-phycore-som.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62-phycore-som.dtsi
> @@ -153,7 +153,7 @@ AM62X_IOPAD(0x15c, PIN_INPUT, 0) /* (AB22) MDIO0_MDIO */
> main_mmc0_pins_default: main-mmc0-default-pins {
> pinctrl-single,pins = <
> AM62X_IOPAD(0x220, PIN_INPUT_PULLUP, 0) /* (Y3) MMC0_CMD */
> - AM62X_IOPAD(0x218, PIN_INPUT_PULLDOWN, 0) /* (AB1) MMC0_CLK */
> + AM62X_IOPAD(0x218, PIN_OUTPUT_PULLDOWN, 0) /* (AB1) MMC0_CLK */
> AM62X_IOPAD(0x214, PIN_INPUT_PULLUP, 0) /* (AA2) MMC0_DAT0 */
> AM62X_IOPAD(0x210, PIN_INPUT_PULLUP, 0) /* (AA1) MMC0_DAT1 */
> AM62X_IOPAD(0x20c, PIN_INPUT_PULLUP, 0) /* (AA3) MMC0_DAT2 */
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH v4 12/15] arm64: dts: ti: k3-am625-tqma62xx: Set MMC clock pins as OUTPUT
2026-09-09 21:47 [PATCH v4 00/15] MMC pinmux configuration fixes Judith Mendez
` (10 preceding siblings ...)
2026-09-09 21:47 ` [PATCH v4 11/15] arm64: dts: ti: k3-am62-phycore-som: " Judith Mendez
@ 2026-09-09 21:47 ` Judith Mendez
2026-09-09 21:47 ` [PATCH v4 13/15] arm64: dts: ti: k3-am625-var-som-symphony: Set SD " Judith Mendez
` (4 subsequent siblings)
16 siblings, 0 replies; 26+ messages in thread
From: Judith Mendez @ 2026-09-09 21:47 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Wadim Egorov, Robert Nelson, Tony Lindgren,
Nora Schiffer, Stefano Radaelli, linux-arm-kernel, devicetree,
linux-kernel, Moteen Shah
MMC clock pins must be configured as OUTPUT-only per TRM [0] with
receiver disabled (RXACTIVE bit = 0). Change MMC CLK pins from
PIN_INPUT to PIN_OUTPUT to match documentation.
[0] https://www.ti.com/lit/pdf/spruiv7 Rev. C: Table 12-246. MMCSDi I/O Signals
Fixes: 5b36d8c72c91 ("arm64: dts: ti: Add TQ-Systems TQMa62xx SoM and MBa62xx carrier board Device Trees")
Cc: <stable@vger.kernel.org>
Signed-off-by: Judith Mendez <jm@ti.com>
---
arch/arm64/boot/dts/ti/k3-am625-tqma62xx-mba62xx.dts | 4 ++--
arch/arm64/boot/dts/ti/k3-am625-tqma62xx.dtsi | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am625-tqma62xx-mba62xx.dts b/arch/arm64/boot/dts/ti/k3-am625-tqma62xx-mba62xx.dts
index efc00d2a860a7..3859512c42628 100644
--- a/arch/arm64/boot/dts/ti/k3-am625-tqma62xx-mba62xx.dts
+++ b/arch/arm64/boot/dts/ti/k3-am625-tqma62xx-mba62xx.dts
@@ -735,7 +735,7 @@ main_mmc1_pins: main-mmc1-pins {
/* (A21) MMC1_CMD */
AM62X_IOPAD(0x23c, PIN_INPUT, 0)
/* (B22) MMC1_CLK */
- AM62X_IOPAD(0x234, PIN_INPUT, 0)
+ AM62X_IOPAD(0x234, PIN_OUTPUT, 0)
/* (A22) MMC1_DAT0 */
AM62X_IOPAD(0x230, PIN_INPUT, 0)
/* (B21) MMC1_DAT1 */
@@ -755,7 +755,7 @@ main_mmc2_pins: main-mmc2-pins {
/* (C24) MMC2_CMD */
AM62X_IOPAD(0x120, PIN_INPUT, 0)
/* (D25) MMC2_CLK */
- AM62X_IOPAD(0x118, PIN_INPUT, 0)
+ AM62X_IOPAD(0x118, PIN_OUTPUT, 0)
/* (B24) MMC2_DAT0 */
AM62X_IOPAD(0x114, PIN_INPUT, 0)
/* (C25) MMC2_DAT1 */
diff --git a/arch/arm64/boot/dts/ti/k3-am625-tqma62xx.dtsi b/arch/arm64/boot/dts/ti/k3-am625-tqma62xx.dtsi
index 72288678cd01f..a200a2afe16fa 100644
--- a/arch/arm64/boot/dts/ti/k3-am625-tqma62xx.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am625-tqma62xx.dtsi
@@ -299,7 +299,7 @@ main_mmc0_pins: main-mmc0-pins {
/* (Y3) MMC0_CMD */
AM62X_IOPAD(0x220, PIN_INPUT, 0)
/* (AB1) MMC0_CLK */
- AM62X_IOPAD(0x218, PIN_INPUT, 0)
+ AM62X_IOPAD(0x218, PIN_OUTPUT, 0)
/* (AA2) MMC0_DAT0 */
AM62X_IOPAD(0x214, PIN_INPUT, 0)
/* (AA1) MMC0_DAT1 */
--
2.34.1
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH v4 13/15] arm64: dts: ti: k3-am625-var-som-symphony: Set SD clock pins as OUTPUT
2026-09-09 21:47 [PATCH v4 00/15] MMC pinmux configuration fixes Judith Mendez
` (11 preceding siblings ...)
2026-09-09 21:47 ` [PATCH v4 12/15] arm64: dts: ti: k3-am625-tqma62xx: Set MMC clock pins " Judith Mendez
@ 2026-09-09 21:47 ` Judith Mendez
2026-09-14 15:00 ` Stefano Radaelli
2026-09-09 21:47 ` [PATCH v4 14/15] arm64: dts: ti: k3-am625-var-som: Set MMC " Judith Mendez
` (3 subsequent siblings)
16 siblings, 1 reply; 26+ messages in thread
From: Judith Mendez @ 2026-09-09 21:47 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Wadim Egorov, Robert Nelson, Tony Lindgren,
Nora Schiffer, Stefano Radaelli, linux-arm-kernel, devicetree,
linux-kernel, Moteen Shah
MMC clock pins must be configured as OUTPUT-only per TRM [0] with
receiver disabled (RXACTIVE bit = 0). Change MMC CLK pins from
PIN_INPUT to PIN_OUTPUT to match documentation.
[0] https://www.ti.com/lit/pdf/spruiv7 Rev. C: Table 12-246. MMCSDi I/O Signals
Fixes: acb4cd35dbd2 ("arm64: dts: ti: var-som-am62: Add support for Variscite Symphony Board")
Cc: <stable@vger.kernel.org>
Signed-off-by: Judith Mendez <jm@ti.com>
---
arch/arm64/boot/dts/ti/k3-am625-var-som-symphony.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am625-var-som-symphony.dts b/arch/arm64/boot/dts/ti/k3-am625-var-som-symphony.dts
index 7951d7ec952b5..082bf81143dcb 100644
--- a/arch/arm64/boot/dts/ti/k3-am625-var-som-symphony.dts
+++ b/arch/arm64/boot/dts/ti/k3-am625-var-som-symphony.dts
@@ -391,7 +391,7 @@ pinctrl_mmc1: main-mmc1-default-pins {
bootph-all;
pinctrl-single,pins = <
AM62X_IOPAD(0x23c, PIN_INPUT, 0) /* (A21) MMC1_CMD */
- AM62X_IOPAD(0x234, PIN_INPUT, 0) /* (B22) MMC1_CLK */
+ AM62X_IOPAD(0x234, PIN_OUTPUT, 0) /* (B22) MMC1_CLK */
AM62X_IOPAD(0x230, PIN_INPUT, 0) /* (A22) MMC1_DAT0 */
AM62X_IOPAD(0x22c, PIN_INPUT, 0) /* (B21) MMC1_DAT1 */
AM62X_IOPAD(0x228, PIN_INPUT, 0) /* (C21) MMC1_DAT2 */
--
2.34.1
^ permalink raw reply related [flat|nested] 26+ messages in thread* Re: [PATCH v4 13/15] arm64: dts: ti: k3-am625-var-som-symphony: Set SD clock pins as OUTPUT
2026-09-09 21:47 ` [PATCH v4 13/15] arm64: dts: ti: k3-am625-var-som-symphony: Set SD " Judith Mendez
@ 2026-09-14 15:00 ` Stefano Radaelli
0 siblings, 0 replies; 26+ messages in thread
From: Stefano Radaelli @ 2026-09-14 15:00 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Wadim Egorov, Robert Nelson, Tony Lindgren,
Nora Schiffer, linux-arm-kernel@lists.infradead.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
Moteen Shah
> MMC clock pins must be configured as OUTPUT-only per TRM [0] with
> receiver disabled (RXACTIVE bit = 0). Change MMC CLK pins from
> PIN_INPUT to PIN_OUTPUT to match documentation.
>
> [0] https://www.ti.com/lit/pdf/spruiv7 Rev. C: Table 12-246. MMCSDi I/O Signals
>
> Fixes: acb4cd35dbd2 ("arm64: dts: ti: var-som-am62: Add support for Variscite Symphony Board")
> Cc: <stable@vger.kernel.org>
> Signed-off-by: Judith Mendez <jm@ti.com>
> ---
> arch/arm64/boot/dts/ti/k3-am625-var-som-symphony.dts | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am625-var-som-symphony.dts b/arch/arm64/boot/dts/ti/k3-am625-var-som-symphony.dts
> index 7951d7ec952b5..082bf81143dcb 100644
> --- a/arch/arm64/boot/dts/ti/k3-am625-var-som-symphony.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am625-var-som-symphony.dts
> @@ -391,7 +391,7 @@ pinctrl_mmc1: main-mmc1-default-pins {
> bootph-all;
> pinctrl-single,pins = <
> AM62X_IOPAD(0x23c, PIN_INPUT, 0) /* (A21) MMC1_CMD */
> - AM62X_IOPAD(0x234, PIN_INPUT, 0) /* (B22) MMC1_CLK */
> + AM62X_IOPAD(0x234, PIN_OUTPUT, 0) /* (B22) MMC1_CLK */
> AM62X_IOPAD(0x230, PIN_INPUT, 0) /* (A22) MMC1_DAT0 */
> AM62X_IOPAD(0x22c, PIN_INPUT, 0) /* (B21) MMC1_DAT1 */
> AM62X_IOPAD(0x228, PIN_INPUT, 0) /* (C21) MMC1_DAT2 */
> --
> 2.34.1
Reviewed-by: Stefano Radaelli <stefano.r@variscite.com>
Thanks!
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH v4 14/15] arm64: dts: ti: k3-am625-var-som: Set MMC clock pins as OUTPUT
2026-09-09 21:47 [PATCH v4 00/15] MMC pinmux configuration fixes Judith Mendez
` (12 preceding siblings ...)
2026-09-09 21:47 ` [PATCH v4 13/15] arm64: dts: ti: k3-am625-var-som-symphony: Set SD " Judith Mendez
@ 2026-09-09 21:47 ` Judith Mendez
2026-09-14 15:04 ` Stefano Radaelli
2026-09-09 21:47 ` [PATCH v4 15/15] arm64: dts: ti: k3-am62x-phyboard-lyra: Set SD clock pin " Judith Mendez
` (2 subsequent siblings)
16 siblings, 1 reply; 26+ messages in thread
From: Judith Mendez @ 2026-09-09 21:47 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Wadim Egorov, Robert Nelson, Tony Lindgren,
Nora Schiffer, Stefano Radaelli, linux-arm-kernel, devicetree,
linux-kernel, Moteen Shah
MMC clock pins must be configured as OUTPUT-only per TRM [0] with
receiver disabled (RXACTIVE bit = 0). Change MMC CLK pins from
PIN_INPUT to PIN_OUTPUT to match documentation.
[0] https://www.ti.com/lit/pdf/spruiv7 Rev. C: Table 12-246. MMCSDi I/O Signals
Fixes: 887d9e03c487 ("arm64: dts: ti: Add support for Variscite VAR-SOM-AM62")
Cc: <stable@vger.kernel.org>
Signed-off-by: Judith Mendez <jm@ti.com>
---
arch/arm64/boot/dts/ti/k3-am625-var-som.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am625-var-som.dtsi b/arch/arm64/boot/dts/ti/k3-am625-var-som.dtsi
index c2daa56c35e17..c23b6a90b8b13 100644
--- a/arch/arm64/boot/dts/ti/k3-am625-var-som.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am625-var-som.dtsi
@@ -277,7 +277,7 @@ pinctrl_mmc0: main-mmc0-default-pins {
bootph-all;
pinctrl-single,pins = <
AM62X_IOPAD(0x220, PIN_INPUT, 0) /* (Y3) MMC0_CMD */
- AM62X_IOPAD(0x218, PIN_INPUT, 0) /* (AB1) MMC0_CLK */
+ AM62X_IOPAD(0x218, PIN_OUTPUT, 0) /* (AB1) MMC0_CLK */
AM62X_IOPAD(0x214, PIN_INPUT, 0) /* (AA2) MMC0_DAT0 */
AM62X_IOPAD(0x210, PIN_INPUT, 0) /* (AA1) MMC0_DAT1 */
AM62X_IOPAD(0x20c, PIN_INPUT, 0) /* (AA3) MMC0_DAT2 */
@@ -292,7 +292,7 @@ AM62X_IOPAD(0x1f8, PIN_INPUT, 0) /* (AC2) MMC0_DAT7 */
pinctrl_mmc2: main-mmc2-default-pins {
pinctrl-single,pins = <
AM62X_IOPAD(0x120, PIN_INPUT_PULLUP, 0) /* (C24) MMC2_CMD */
- AM62X_IOPAD(0x118, PIN_INPUT_PULLDOWN, 0) /* (D25) MMC2_CLK */
+ AM62X_IOPAD(0x118, PIN_OUTPUT_PULLDOWN, 0) /* (D25) MMC2_CLK */
AM62X_IOPAD(0x114, PIN_INPUT_PULLUP, 0) /* (B24) MMC2_DAT0 */
AM62X_IOPAD(0x110, PIN_INPUT_PULLUP, 0) /* (C25) MMC2_DAT1 */
AM62X_IOPAD(0x10c, PIN_INPUT_PULLUP, 0) /* (E23) MMC2_DAT2 */
--
2.34.1
^ permalink raw reply related [flat|nested] 26+ messages in thread* Re: [PATCH v4 14/15] arm64: dts: ti: k3-am625-var-som: Set MMC clock pins as OUTPUT
2026-09-09 21:47 ` [PATCH v4 14/15] arm64: dts: ti: k3-am625-var-som: Set MMC " Judith Mendez
@ 2026-09-14 15:04 ` Stefano Radaelli
0 siblings, 0 replies; 26+ messages in thread
From: Stefano Radaelli @ 2026-09-14 15:04 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Wadim Egorov, Robert Nelson, Tony Lindgren,
Nora Schiffer, linux-arm-kernel@lists.infradead.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
Moteen Shah
> MMC clock pins must be configured as OUTPUT-only per TRM [0] with
> receiver disabled (RXACTIVE bit = 0). Change MMC CLK pins from
> PIN_INPUT to PIN_OUTPUT to match documentation.
>
> [0] https://www.ti.com/lit/pdf/spruiv7 Rev. C: Table 12-246. MMCSDi I/O Signals
Fixes: 887d9e03c487 ("arm64: dts: ti: Add support for Variscite VAR-SOM-AM62")
Cc: <stable@vger.kernel.org>
Signed-off-by: Judith Mendez <jm@ti.com>
---
arch/arm64/boot/dts/ti/k3-am625-var-som.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am625-var-som.dtsi b/arch/arm64/boot/dts/ti/k3-am625-var-som.dtsi
index c2daa56c35e17..c23b6a90b8b13 100644
--- a/arch/arm64/boot/dts/ti/k3-am625-var-som.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am625-var-som.dtsi
@@ -277,7 +277,7 @@ pinctrl_mmc0: main-mmc0-default-pins {
bootph-all;
pinctrl-single,pins = <
AM62X_IOPAD(0x220, PIN_INPUT, 0) /* (Y3) MMC0_CMD */
- AM62X_IOPAD(0x218, PIN_INPUT, 0) /* (AB1) MMC0_CLK */
+ AM62X_IOPAD(0x218, PIN_OUTPUT, 0) /* (AB1) MMC0_CLK */
AM62X_IOPAD(0x214, PIN_INPUT, 0) /* (AA2) MMC0_DAT0 */
AM62X_IOPAD(0x210, PIN_INPUT, 0) /* (AA1) MMC0_DAT1 */
AM62X_IOPAD(0x20c, PIN_INPUT, 0) /* (AA3) MMC0_DAT2 */
@@ -292,7 +292,7 @@ AM62X_IOPAD(0x1f8, PIN_INPUT, 0) /* (AC2) MMC0_DAT7 */
pinctrl_mmc2: main-mmc2-default-pins {
pinctrl-single,pins = <
AM62X_IOPAD(0x120, PIN_INPUT_PULLUP, 0) /* (C24) MMC2_CMD */
- AM62X_IOPAD(0x118, PIN_INPUT_PULLDOWN, 0) /* (D25) MMC2_CLK */
+ AM62X_IOPAD(0x118, PIN_OUTPUT_PULLDOWN, 0) /* (D25) MMC2_CLK */
AM62X_IOPAD(0x114, PIN_INPUT_PULLUP, 0) /* (B24) MMC2_DAT0 */
AM62X_IOPAD(0x110, PIN_INPUT_PULLUP, 0) /* (C25) MMC2_DAT1 */
AM62X_IOPAD(0x10c, PIN_INPUT_PULLUP, 0) /* (E23) MMC2_DAT2 */
--
2.34.1
Reviewed-by: Stefano Radaelli <stefano.r@variscite.com>
Thanks!
^ permalink raw reply related [flat|nested] 26+ messages in thread
* [PATCH v4 15/15] arm64: dts: ti: k3-am62x-phyboard-lyra: Set SD clock pin as OUTPUT
2026-09-09 21:47 [PATCH v4 00/15] MMC pinmux configuration fixes Judith Mendez
` (13 preceding siblings ...)
2026-09-09 21:47 ` [PATCH v4 14/15] arm64: dts: ti: k3-am625-var-som: Set MMC " Judith Mendez
@ 2026-09-09 21:47 ` Judith Mendez
2026-09-15 16:54 ` Wadim Egorov
2026-09-16 4:49 ` [PATCH v4 00/15] MMC pinmux configuration fixes Moteen Shah
2026-09-22 14:24 ` Nishanth Menon
16 siblings, 1 reply; 26+ messages in thread
From: Judith Mendez @ 2026-09-09 21:47 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Wadim Egorov, Robert Nelson, Tony Lindgren,
Nora Schiffer, Stefano Radaelli, linux-arm-kernel, devicetree,
linux-kernel, Moteen Shah
MMC clock pins must be configured as OUTPUT-only per TRM [0] with
receiver disabled (RXACTIVE bit = 0). Change MMC CLK pins from
PIN_INPUT to PIN_OUTPUT to match documentation.
[0] https://www.ti.com/lit/pdf/spruiv7 Rev. C: Table 12-246. MMCSDi I/O Signals
Fixes: f1b3adade045 ("arm64: dts: ti: Add am62x-phyboard-lyra carrier board")
Cc: <stable@vger.kernel.org>
Signed-off-by: Judith Mendez <jm@ti.com>
---
arch/arm64/boot/dts/ti/k3-am62x-phyboard-lyra.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am62x-phyboard-lyra.dtsi b/arch/arm64/boot/dts/ti/k3-am62x-phyboard-lyra.dtsi
index d6e70ee15938e..c9b3e0518faf5 100644
--- a/arch/arm64/boot/dts/ti/k3-am62x-phyboard-lyra.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am62x-phyboard-lyra.dtsi
@@ -227,7 +227,7 @@ AM62X_IOPAD(0x068, PIN_INPUT, 3) /* (R21) GPMC0_AD11.MCASP2_AXR3 */
main_mmc1_pins_default: main-mmc1-default-pins {
pinctrl-single,pins = <
AM62X_IOPAD(0x23c, PIN_INPUT_PULLUP, 0) /* (A21) MMC1_CMD */
- AM62X_IOPAD(0x234, PIN_INPUT_PULLDOWN, 0) /* (B22) MMC1_CLK */
+ AM62X_IOPAD(0x234, PIN_OUTPUT_PULLDOWN, 0) /* (B22) MMC1_CLK */
AM62X_IOPAD(0x230, PIN_INPUT_PULLUP, 0) /* (A22) MMC1_DAT0 */
AM62X_IOPAD(0x22c, PIN_INPUT_PULLUP, 0) /* (B21) MMC1_DAT1 */
AM62X_IOPAD(0x228, PIN_INPUT_PULLUP, 0) /* (C21) MMC1_DAT2 */
--
2.34.1
^ permalink raw reply related [flat|nested] 26+ messages in thread* Re: [PATCH v4 15/15] arm64: dts: ti: k3-am62x-phyboard-lyra: Set SD clock pin as OUTPUT
2026-09-09 21:47 ` [PATCH v4 15/15] arm64: dts: ti: k3-am62x-phyboard-lyra: Set SD clock pin " Judith Mendez
@ 2026-09-15 16:54 ` Wadim Egorov
0 siblings, 0 replies; 26+ messages in thread
From: Wadim Egorov @ 2026-09-15 16:54 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Robert Nelson, Tony Lindgren, Nora Schiffer,
Stefano Radaelli, linux-arm-kernel, devicetree, linux-kernel,
Moteen Shah
On 9/10/26 12:47 AM, Judith Mendez wrote:
> MMC clock pins must be configured as OUTPUT-only per TRM [0] with
> receiver disabled (RXACTIVE bit = 0). Change MMC CLK pins from
> PIN_INPUT to PIN_OUTPUT to match documentation.
>
> [0] https://www.ti.com/lit/pdf/spruiv7 Rev. C: Table 12-246. MMCSDi I/O Signals
>
> Fixes: f1b3adade045 ("arm64: dts: ti: Add am62x-phyboard-lyra carrier board")
> Cc: <stable@vger.kernel.org>
> Signed-off-by: Judith Mendez <jm@ti.com>
Acked-by: Wadim Egorov <w.egorov@phytec.de>
> ---
> arch/arm64/boot/dts/ti/k3-am62x-phyboard-lyra.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am62x-phyboard-lyra.dtsi b/arch/arm64/boot/dts/ti/k3-am62x-phyboard-lyra.dtsi
> index d6e70ee15938e..c9b3e0518faf5 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62x-phyboard-lyra.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62x-phyboard-lyra.dtsi
> @@ -227,7 +227,7 @@ AM62X_IOPAD(0x068, PIN_INPUT, 3) /* (R21) GPMC0_AD11.MCASP2_AXR3 */
> main_mmc1_pins_default: main-mmc1-default-pins {
> pinctrl-single,pins = <
> AM62X_IOPAD(0x23c, PIN_INPUT_PULLUP, 0) /* (A21) MMC1_CMD */
> - AM62X_IOPAD(0x234, PIN_INPUT_PULLDOWN, 0) /* (B22) MMC1_CLK */
> + AM62X_IOPAD(0x234, PIN_OUTPUT_PULLDOWN, 0) /* (B22) MMC1_CLK */
> AM62X_IOPAD(0x230, PIN_INPUT_PULLUP, 0) /* (A22) MMC1_DAT0 */
> AM62X_IOPAD(0x22c, PIN_INPUT_PULLUP, 0) /* (B21) MMC1_DAT1 */
> AM62X_IOPAD(0x228, PIN_INPUT_PULLUP, 0) /* (C21) MMC1_DAT2 */
^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH v4 00/15] MMC pinmux configuration fixes
2026-09-09 21:47 [PATCH v4 00/15] MMC pinmux configuration fixes Judith Mendez
` (14 preceding siblings ...)
2026-09-09 21:47 ` [PATCH v4 15/15] arm64: dts: ti: k3-am62x-phyboard-lyra: Set SD clock pin " Judith Mendez
@ 2026-09-16 4:49 ` Moteen Shah
2026-09-22 14:24 ` Nishanth Menon
16 siblings, 0 replies; 26+ messages in thread
From: Moteen Shah @ 2026-09-16 4:49 UTC (permalink / raw)
To: Judith Mendez, Nishanth Menon
Cc: Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Garrett Giordano, Wadim Egorov, Robert Nelson, Tony Lindgren,
Nora Schiffer, Stefano Radaelli, linux-arm-kernel, devicetree,
linux-kernel
Hey Judith,
Thanks for the series.
On 10/09/26 03:17, Judith Mendez wrote:
> This series fixes MMC pin input/output and pullup/pulldown settings
> across AM62*, AM62AX, and AM62LX boards.
>
> All patches have been tested on their appropriate SK & EVM boards.
> BeaglePlay and Pocketbeagle boards have also been tested.
>
> Changes since v3:
> - Include PIN_OUTPUT fixes for missing third-party boards as per Sashiko
> review [0]
> - Split out squashed patches into separate commits as per Nishanth's
> review [1]
>
> [0] https://lore.kernel.org/linux-devicetree/20260805230256.41AA61F000E9@smtp.kernel.org/
> & https://lore.kernel.org/linux-devicetree/20260805230614.521AC1F000E9@smtp.kernel.org/
> [1] https://lore.kernel.org/linux-devicetree/20260904151335.fi5mvjpvziw2g3r2@exemption/
>
> Link to v1:
> https://lore.kernel.org/linux-devicetree/20260729195958.671147-1-jm@ti.com/
> Link to v2:
> https://lore.kernel.org/linux-devicetree/20260805004821.416613-1-jm@ti.com/
> Link to v3:
> https://lore.kernel.org/linux-devicetree/20260805225438.464184-1-jm@ti.com/
>
> Judith Mendez (15):
> arm64: dts: ti: k3-am62a: Fix MMC0_CLK register address
> arm64: dts: ti: k3-am62l-evm: Remove unnecessary internal pullups for
> MMC0
> arm64: dts: ti: k3-am62a7-sk: Set eMMC clock pin as OUTPUT
> arm64: dts: ti: k3-am62a7-sk: Set SD clock pin as OUTPUT
> arm64: dts: ti: k3-am62a-phycore-som: Set eMMC clock pin as OUTPUT
> arm64: dts: ti: k3-am62-lp-sk: Set eMMC clock pins as OUTPUT
> arm64: dts: ti: k3-am62-pocketbeagle2: Set SD clock pin as OUTPUT
> arm64: dts: ti: k3-am625-beagleplay: Set MMC clock pins as OUTPUT
> arm64: dts: ti: k3-am62x-sk-common: Set SD clock pin as OUTPUT
> arm64: dts: ti: k3-am625-sk-common: Set eMMC clock pin as OUTPUT
> arm64: dts: ti: k3-am62-phycore-som: Set eMMC clock pin as OUTPUT
> arm64: dts: ti: k3-am625-tqma62xx: Set MMC clock pins as OUTPUT
> arm64: dts: ti: k3-am625-var-som-symphony: Set SD clock pins as OUTPUT
> arm64: dts: ti: k3-am625-var-som: Set MMC clock pins as OUTPUT
> arm64: dts: ti: k3-am62x-phyboard-lyra: Set SD clock pin as OUTPUT
>
> arch/arm64/boot/dts/ti/k3-am62-lp-sk.dts | 2 +-
> arch/arm64/boot/dts/ti/k3-am62-phycore-som.dtsi | 2 +-
> arch/arm64/boot/dts/ti/k3-am62-pocketbeagle2.dts | 2 +-
> arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts | 6 +++---
> arch/arm64/boot/dts/ti/k3-am625-sk-common.dtsi | 2 +-
> arch/arm64/boot/dts/ti/k3-am625-tqma62xx-mba62xx.dts | 4 ++--
> arch/arm64/boot/dts/ti/k3-am625-tqma62xx.dtsi | 2 +-
> arch/arm64/boot/dts/ti/k3-am625-var-som-symphony.dts | 2 +-
> arch/arm64/boot/dts/ti/k3-am625-var-som.dtsi | 4 ++--
> arch/arm64/boot/dts/ti/k3-am62a-phycore-som.dtsi | 2 +-
> arch/arm64/boot/dts/ti/k3-am62a7-sk.dts | 5 ++---
> arch/arm64/boot/dts/ti/k3-am62l3-evm.dts | 4 ++--
> arch/arm64/boot/dts/ti/k3-am62x-phyboard-lyra.dtsi | 2 +-
> arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi | 2 +-
> 14 files changed, 20 insertions(+), 21 deletions(-)
For the whole series:
Reviewed-by: Moteen Shah <m-shah@ti.com>
Regards,
Moteen
^ permalink raw reply [flat|nested] 26+ messages in thread* Re: [PATCH v4 00/15] MMC pinmux configuration fixes
2026-09-09 21:47 [PATCH v4 00/15] MMC pinmux configuration fixes Judith Mendez
` (15 preceding siblings ...)
2026-09-16 4:49 ` [PATCH v4 00/15] MMC pinmux configuration fixes Moteen Shah
@ 2026-09-22 14:24 ` Nishanth Menon
16 siblings, 0 replies; 26+ messages in thread
From: Nishanth Menon @ 2026-09-22 14:24 UTC (permalink / raw)
To: Judith Mendez
Cc: Nishanth Menon, Tero Kristo, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Garrett Giordano, Wadim Egorov, Robert Nelson,
Tony Lindgren, Nora Schiffer, Stefano Radaelli, linux-arm-kernel,
devicetree, linux-kernel, Moteen Shah
Hi Judith Mendez,
On Wed, 09 Sep 2026 16:47:28 -0500, Judith Mendez wrote:
> This series fixes MMC pin input/output and pullup/pulldown settings
> across AM62*, AM62AX, and AM62LX boards.
>
> All patches have been tested on their appropriate SK & EVM boards.
> BeaglePlay and Pocketbeagle boards have also been tested.
>
> Changes since v3:
> - Include PIN_OUTPUT fixes for missing third-party boards as per Sashiko
> review [0]
> - Split out squashed patches into separate commits as per Nishanth's
> review [1]
>
> [...]
I have applied the following to branch ti-k3-dts-next on [1].
Just a future note for cover-letter $subject, use "arm64: dts: ti:" as prefix
Thank you!
[01/15] arm64: dts: ti: k3-am62a: Fix MMC0_CLK register address
commit: 9852f2badc438de82b6828c11c80ef0a656f1d4d
[02/15] arm64: dts: ti: k3-am62l-evm: Remove unnecessary internal pullups for MMC0
commit: dfba8329b8ebd49807ed1c1b336992e5791c7e48
[03/15] arm64: dts: ti: k3-am62a7-sk: Set eMMC clock pin as OUTPUT
commit: aa7d5b1efa75e58c9d440dfd720205045fe6a806
[04/15] arm64: dts: ti: k3-am62a7-sk: Set SD clock pin as OUTPUT
commit: cfad4f39124b218c2cbbbe17e742971abc9ac8b4
[05/15] arm64: dts: ti: k3-am62a-phycore-som: Set eMMC clock pin as OUTPUT
commit: 4834050197f205a649e36661ca150e6ac8295008
[06/15] arm64: dts: ti: k3-am62-lp-sk: Set eMMC clock pins as OUTPUT
commit: 156f4c2208c8e5e04002402494de28c21c0afcbc
[07/15] arm64: dts: ti: k3-am62-pocketbeagle2: Set SD clock pin as OUTPUT
commit: 843c4d171975059eaa1436c8403d9a29f677a75c
[08/15] arm64: dts: ti: k3-am625-beagleplay: Set MMC clock pins as OUTPUT
commit: c074abebc76f0e59d13dea499f582806c6a3d94d
[09/15] arm64: dts: ti: k3-am62x-sk-common: Set SD clock pin as OUTPUT
commit: 1e2e5016bda94b2bcd86258b5eb77fe3006639a0
[10/15] arm64: dts: ti: k3-am625-sk-common: Set eMMC clock pin as OUTPUT
commit: 8ef8a0e38d1377c6d51c06f95c64d87641b2c717
[11/15] arm64: dts: ti: k3-am62-phycore-som: Set eMMC clock pin as OUTPUT
commit: a3c2b6c52e02d58941c0b91bfba052788aeff890
[12/15] arm64: dts: ti: k3-am625-tqma62xx: Set MMC clock pins as OUTPUT
commit: f9a93fe78d8737860a8fb786d17c9dff20782b7d
[13/15] arm64: dts: ti: k3-am625-var-som-symphony: Set SD clock pins as OUTPUT
commit: dbe55d5f2930b27e9a3ce71f0c9b1bb0ad91a012
[14/15] arm64: dts: ti: k3-am625-var-som: Set MMC clock pins as OUTPUT
commit: b489fa1808ea807c4467def7947c83d896dfc355
[15/15] arm64: dts: ti: k3-am62x-phyboard-lyra: Set SD clock pin as OUTPUT
commit: 1d62ef8d47b535d31fb9cf38e5c36eb55b0189a4
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent up the chain during
the next merge window (or sooner if it is a relevant bug fix), however if
problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying
to this mail.
[1] https://git.kernel.org/pub/scm/linux/kernel/git/ti/linux.git
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
https://ti.com/opensource
^ permalink raw reply [flat|nested] 26+ messages in thread