On Fri, Jun 16, 2023 at 9:39 AM Biju Das <biju.das.jz@xxxxxxxxxxxxxx> wrote: > > Hi Adam, > > > -----Original Message----- > > From: Adam Ford <aford173@xxxxxxxxx> > > Sent: Friday, June 16, 2023 3:35 PM > > To: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx>; Adam Ford > > <aford173@xxxxxxxxx>; linux-mmc@xxxxxxxxxxxxxxx; linux-renesas- > > soc@xxxxxxxxxxxxxxx; Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx> > > Subject: Re: [RFC/RFT PATCH] mmc: renesas_sdhi: add delay between tuning > > cmds > > > > On Wed, May 31, 2023 at 12:45 PM Wolfram Sang <wsa+renesas@sang- > > engineering.com> wrote: > > > > > > > > > > I am testing this on a 5.10 kernel based on the Renesas RZ linux > > > > kernel branch. The good news is that the card doesn't hang like it > > > > used to on the older 4.19 kernel, but it still does not enumerate to > > > > SDR104 speeds. > > > > > > U-Boot has a 1ms delay. Could you increase the delay to 1ms, please? > > > > Sorry for the delay, but I increased this to usleep_range(1000, 2500) and > > it still hangs. > > > > I don't know if the issue is dependent on the variant. For what it's > > worth, I am testing an RZ/G2M, but I can do the same thing on an RZ/G2N or > > RZ/G2H if you want. > > I am doing 6.1.y testing[1] with all RZ board variants, > [1] https://git.kernel.org/pub/scm/linux/kernel/git/cip/linux-cip.git/log/?h=linux-6.1.y-cip > > I have done SDHI testing with RZ/G2M 1.3 HiHope board and I didn't face this issue. > > Can you please let me know how to reproduce this issue? It's not easy to reproduce, and it only happens on a small subset of cards. I sent one to Wolfram sometime last year. I can also take these same cards that hang when booting Linux, and they enumerate just fine in U-Boot at SRD104 speeds. I can use the same RZ/G2 board with other cards, and other cards work fine. I then tested these same 'failing' cards on other platforms and they work fine, so it seems like there is a tuning combination in Linux that is different from U-Boot that is unique to these certain cards which prevents them from working. adam > > Cheers, > Biju
