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

[Siddharth Vadapalli <s-vadapalli@ti.com>]

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.