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@xxxxxxxxxxxxxxx>
> > Signed-off-by: Siddharth Vadapalli <s-vadapalli@xxxxxx>
> > ---
> >
> > 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.