[PATCH] arm64: dts: ti: k3-am62a7-sk: Fix pinmux for pin M19 used by sdhci1

[PATCH] arm64: dts: ti: k3-am62a7-sk: Fix pinmux for pin M19 used by sdhci1

[Siddharth Vadapalli]

According to the datasheet for the AM62Ax SoC [0], pin M19 has the address
0x000F40A8. Therefore, the offset to be passed to the AM62AX_IOPAD macro is
0xa8 and not 0x07c. With the existing incorrect offset, the following error
is seen when Linux boots:
	fa00000.mmc: deferred probe pending: platform: supplier regulator-5 not ready
with the SD Card being unusable and the boot process halting due to the root
filesystem in the SD Card being inaccessible.

Hence, fix it.

[0]: https://www.ti.com/lit/ds/symlink/am62a7.pdf

Fixes: 8f023012eb4a ("arm64: dts: ti: k3-am62a: Enable UHS mode support for SD cards")
Cc: <stable@vger.kernel.org>
Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---

Hello,

This patch is based on commit
37a93dd5c49b Merge tag 'net-next-7.0' of git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net-next
of Mainline Linux.

Regards,
Siddharth.

 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 e99bdbc2e0cb..9cfe7e7b317b 100644
--- a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
+++ b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
@@ -398,7 +398,7 @@ AM62AX_IOPAD(0x01d4, PIN_INPUT, 7) /* (C15) UART0_RTSn.GPIO1_23 */
 
 	vddshv_sdio_pins_default: vddshv-sdio-default-pins {
 		pinctrl-single,pins = <
-			AM62AX_IOPAD(0x07c, PIN_OUTPUT, 7) /* (M19) GPMC0_CLK.GPIO0_31 */
+			AM62AX_IOPAD(0x0a8, PIN_OUTPUT, 7) /* (M19) GPMC0_CLK.GPIO0_31 */
 		>;
 	};
 
-- 
2.51.1

Re: [PATCH] arm64: dts: ti: k3-am62a7-sk: Fix pinmux for pin M19 used by sdhci1

[Judith Mendez]

Hi Siddharth,

On 2/12/26 7:06 AM, Siddharth Vadapalli wrote:
> According to the datasheet for the AM62Ax SoC [0], pin M19 has the address
> 0x000F40A8. Therefore, the offset to be passed to the AM62AX_IOPAD macro is
> 0xa8 and not 0x07c. With the existing incorrect offset, the following error
> is seen when Linux boots:
> 	fa00000.mmc: deferred probe pending: platform: supplier regulator-5 not ready
> with the SD Card being unusable and the boot process halting due to the root
> filesystem in the SD Card being inaccessible.
> 
> Hence, fix it.
> 
> [0]: https://www.ti.com/lit/ds/symlink/am62a7.pdf
> 
> Fixes: 8f023012eb4a ("arm64: dts: ti: k3-am62a: Enable UHS mode support for SD cards")
> Cc: <stable@vger.kernel.org>
> Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
> ---
> 
> Hello,
> 
> This patch is based on commit
> 37a93dd5c49b Merge tag 'net-next-7.0' of git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net-next
> of Mainline Linux.
> 
> Regards,
> Siddharth.
> 
>   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 e99bdbc2e0cb..9cfe7e7b317b 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
> @@ -398,7 +398,7 @@ AM62AX_IOPAD(0x01d4, PIN_INPUT, 7) /* (C15) UART0_RTSn.GPIO1_23 */
>   
>   	vddshv_sdio_pins_default: vddshv-sdio-default-pins {
>   		pinctrl-single,pins = <
> -			AM62AX_IOPAD(0x07c, PIN_OUTPUT, 7) /* (M19) GPMC0_CLK.GPIO0_31 */
> +			AM62AX_IOPAD(0x0a8, PIN_OUTPUT, 7) /* (M19) GPMC0_CLK.GPIO0_31 */

What! I don't think this is right.

Looking at device tree, regulator-5 is using main_gpio0 31 to control SD
ENA with PMIC. Which is GPMC0_CLK (N22 pad) and VSEL_SD_SOC. Which is
0x000F407C address in the device datasheet. So as far as I can see, the
original address is correct and just the (M19) name is wrong. Did you
test this patch to see if that fixed the failure?

~ Judith

Re: [PATCH] arm64: dts: ti: k3-am62a7-sk: Fix pinmux for pin M19 used by sdhci1

[Siddharth Vadapalli]

On 03/03/26 22:06, Judith Mendez wrote:
> Hi Siddharth,
> 
> On 2/12/26 7:06 AM, Siddharth Vadapalli wrote:
>> According to the datasheet for the AM62Ax SoC [0], pin M19 has the address
>> 0x000F40A8. Therefore, the offset to be passed to the AM62AX_IOPAD macro is
>> 0xa8 and not 0x07c. With the existing incorrect offset, the following error
>> is seen when Linux boots:
>>     fa00000.mmc: deferred probe pending: platform: supplier regulator-5 
>> not ready
>> with the SD Card being unusable and the boot process halting due to the root
>> filesystem in the SD Card being inaccessible.
>>
>> Hence, fix it.
>>
>> [0]: https://www.ti.com/lit/ds/symlink/am62a7.pdf
>>
>> Fixes: 8f023012eb4a ("arm64: dts: ti: k3-am62a: Enable UHS mode support 
>> for SD cards")
>> Cc: <stable@vger.kernel.org>
>> Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
>> ---
>>
>> Hello,
>>
>> This patch is based on commit
>> 37a93dd5c49b Merge tag 'net-next-7.0' of git://git.kernel.org/pub/scm/ 
>> linux/kernel/git/netdev/net-next
>> of Mainline Linux.
>>
>> Regards,
>> Siddharth.
>>
>>   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 e99bdbc2e0cb..9cfe7e7b317b 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
>> +++ b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
>> @@ -398,7 +398,7 @@ AM62AX_IOPAD(0x01d4, PIN_INPUT, 7) /* (C15) 
>> UART0_RTSn.GPIO1_23 */
>>       vddshv_sdio_pins_default: vddshv-sdio-default-pins {
>>           pinctrl-single,pins = <
>> -            AM62AX_IOPAD(0x07c, PIN_OUTPUT, 7) /* (M19) 
>> GPMC0_CLK.GPIO0_31 */
>> +            AM62AX_IOPAD(0x0a8, PIN_OUTPUT, 7) /* (M19) 
>> GPMC0_CLK.GPIO0_31 */
> 
> What! I don't think this is right.
> 
> Looking at device tree, regulator-5 is using main_gpio0 31 to control SD
> ENA with PMIC. Which is GPMC0_CLK (N22 pad) and VSEL_SD_SOC. Which is
> 0x000F407C address in the device datasheet. So as far as I can see, the
> original address is correct and just the (M19) name is wrong. Did you
> test this patch to see if that fixed the failure?
Yes, without this patch I saw the following:
[    2.108345] Waiting for root device PARTUUID=076c4a2a-02...
[   12.261669] platform fa00000.mmc: deferred probe pending: platform: 
supplier regulator-5

Since the pin was named 'M19' in the comment, I corrected the offset to 
match that of M19 and the issue was fixed. So it seems that although it 
fixed the issue, it isn't really a fix.

Regards,
Siddharth.

Re: [PATCH] arm64: dts: ti: k3-am62a7-sk: Fix pinmux for pin M19 used by sdhci1

[Vignesh Raghavendra]

On 04/03/26 10:24, Siddharth Vadapalli wrote:
> On 03/03/26 22:06, Judith Mendez wrote:
>> Hi Siddharth,
>>
>> On 2/12/26 7:06 AM, Siddharth Vadapalli wrote:
>>> According to the datasheet for the AM62Ax SoC [0], pin M19 has the
>>> address
>>> 0x000F40A8. Therefore, the offset to be passed to the AM62AX_IOPAD
>>> macro is
>>> 0xa8 and not 0x07c. With the existing incorrect offset, the following
>>> error
>>> is seen when Linux boots:
>>>     fa00000.mmc: deferred probe pending: platform: supplier
>>> regulator-5 not ready
>>> with the SD Card being unusable and the boot process halting due to
>>> the root
>>> filesystem in the SD Card being inaccessible.
>>>
>>> Hence, fix it.
>>>
>>> [0]: https://www.ti.com/lit/ds/symlink/am62a7.pdf
>>>
>>> Fixes: 8f023012eb4a ("arm64: dts: ti: k3-am62a: Enable UHS mode
>>> support for SD cards")
>>> Cc: <stable@vger.kernel.org>
>>> Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
>>> ---
>>>
>>> Hello,
>>>
>>> This patch is based on commit
>>> 37a93dd5c49b Merge tag 'net-next-7.0' of git://git.kernel.org/pub/
>>> scm/ linux/kernel/git/netdev/net-next
>>> of Mainline Linux.
>>>
>>> Regards,
>>> Siddharth.
>>>
>>>   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 e99bdbc2e0cb..9cfe7e7b317b 100644
>>> --- a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
>>> +++ b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
>>> @@ -398,7 +398,7 @@ AM62AX_IOPAD(0x01d4, PIN_INPUT, 7) /* (C15)
>>> UART0_RTSn.GPIO1_23 */
>>>       vddshv_sdio_pins_default: vddshv-sdio-default-pins {
>>>           pinctrl-single,pins = <
>>> -            AM62AX_IOPAD(0x07c, PIN_OUTPUT, 7) /* (M19)
>>> GPMC0_CLK.GPIO0_31 */
>>> +            AM62AX_IOPAD(0x0a8, PIN_OUTPUT, 7) /* (M19)
>>> GPMC0_CLK.GPIO0_31 */
>>
>> What! I don't think this is right.
>>
>> Looking at device tree, regulator-5 is using main_gpio0 31 to control SD
>> ENA with PMIC. Which is GPMC0_CLK (N22 pad) and VSEL_SD_SOC. Which is
>> 0x000F407C address in the device datasheet. So as far as I can see, the
>> original address is correct and just the (M19) name is wrong. Did you
>> test this patch to see if that fixed the failure?
> Yes, without this patch I saw the following:
> [    2.108345] Waiting for root device PARTUUID=076c4a2a-02...
> [   12.261669] platform fa00000.mmc: deferred probe pending: platform:
> supplier regulator-5
> 
> Since the pin was named 'M19' in the comment, I corrected the offset to
> match that of M19 and the issue was fixed. So it seems that although it
> fixed the issue, it isn't really a fix.
> 

Refer to the schematics and not datasheet for pin routing. Please submit
patch fixing the comment to say N22

> Regards,
> Siddharth.

-- 
Regards
Vignesh
https://ti.com/opensource

Re: [PATCH] arm64: dts: ti: k3-am62a7-sk: Fix pinmux for pin M19 used by sdhci1

[Siddharth Vadapalli]

On 09/03/26 10:05, Vignesh Raghavendra wrote:
> 
> 
> On 04/03/26 10:24, Siddharth Vadapalli wrote:
>> On 03/03/26 22:06, Judith Mendez wrote:
>>> Hi Siddharth,
>>>
>>> On 2/12/26 7:06 AM, Siddharth Vadapalli wrote:
>>>> According to the datasheet for the AM62Ax SoC [0], pin M19 has the
>>>> address
>>>> 0x000F40A8. Therefore, the offset to be passed to the AM62AX_IOPAD
>>>> macro is
>>>> 0xa8 and not 0x07c. With the existing incorrect offset, the following
>>>> error
>>>> is seen when Linux boots:
>>>>      fa00000.mmc: deferred probe pending: platform: supplier
>>>> regulator-5 not ready
>>>> with the SD Card being unusable and the boot process halting due to
>>>> the root
>>>> filesystem in the SD Card being inaccessible.
>>>>
>>>> Hence, fix it.
>>>>
>>>> [0]: https://www.ti.com/lit/ds/symlink/am62a7.pdf
>>>>
>>>> Fixes: 8f023012eb4a ("arm64: dts: ti: k3-am62a: Enable UHS mode
>>>> support for SD cards")
>>>> Cc: <stable@vger.kernel.org>
>>>> Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
>>>> ---
>>>>
>>>> Hello,
>>>>
>>>> This patch is based on commit
>>>> 37a93dd5c49b Merge tag 'net-next-7.0' of git://git.kernel.org/pub/
>>>> scm/ linux/kernel/git/netdev/net-next
>>>> of Mainline Linux.
>>>>
>>>> Regards,
>>>> Siddharth.
>>>>
>>>>    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 e99bdbc2e0cb..9cfe7e7b317b 100644
>>>> --- a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
>>>> +++ b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
>>>> @@ -398,7 +398,7 @@ AM62AX_IOPAD(0x01d4, PIN_INPUT, 7) /* (C15)
>>>> UART0_RTSn.GPIO1_23 */
>>>>        vddshv_sdio_pins_default: vddshv-sdio-default-pins {
>>>>            pinctrl-single,pins = <
>>>> -            AM62AX_IOPAD(0x07c, PIN_OUTPUT, 7) /* (M19)
>>>> GPMC0_CLK.GPIO0_31 */
>>>> +            AM62AX_IOPAD(0x0a8, PIN_OUTPUT, 7) /* (M19)
>>>> GPMC0_CLK.GPIO0_31 */
>>>
>>> What! I don't think this is right.
>>>
>>> Looking at device tree, regulator-5 is using main_gpio0 31 to control SD
>>> ENA with PMIC. Which is GPMC0_CLK (N22 pad) and VSEL_SD_SOC. Which is
>>> 0x000F407C address in the device datasheet. So as far as I can see, the
>>> original address is correct and just the (M19) name is wrong. Did you
>>> test this patch to see if that fixed the failure?
>> Yes, without this patch I saw the following:
>> [    2.108345] Waiting for root device PARTUUID=076c4a2a-02...
>> [   12.261669] platform fa00000.mmc: deferred probe pending: platform:
>> supplier regulator-5
>>
>> Since the pin was named 'M19' in the comment, I corrected the offset to
>> match that of M19 and the issue was fixed. So it seems that although it
>> fixed the issue, it isn't really a fix.
>>
> 
> Refer to the schematics and not datasheet for pin routing. Please submit
> patch fixing the comment to say N22
I have posted the v2 patch fixing the pin name at:
https://lore.kernel.org/r/20260309045539.2070793-1-s-vadapalli@ti.com/