On Fri, May 05, 2023 at 09:37:10PM +0200, Pali Rohár wrote: > On Wednesday 03 May 2023 13:14:56 Tom Rini wrote: > > On Wed, May 03, 2023 at 11:18:39AM +0200, Stefan Roese wrote: > > > > > Hi Tom, > > > > > > please pull this next batch of mostly Marvell related patches: > > > > NAK. With commit: > > commit 461fa17970de418a93832f734a595031c0b72128 > > Author: Pali Rohár <p...@kernel.org> > > Date: Thu Apr 13 22:57:48 2023 +0200 > > > > mmc: Read eMMC partition access bits before card reset > > > > eMMC specification in section "Access partitions" says that all reset > > events will restore the access bits in PARTITION_CONFIG CSD register to > > default User Data Area value (0b000). > > > > So read partition access bits from PARTITION_CONFIG CSD register before > > issuing card reset. This allows SPL/U-Boot to get information which eMMC > > partition was in use before SPL/U-Boot was booted. For some platforms > > this > > is the way how to determinate boot partition from which BootROM loaded > > SPL. > > > > Signed-off-by: Pali Rohár <p...@kernel.org> > > > > My am335x_evm now fails to boot with: > > > > U-Boot SPL 2023.07-rc1-00021-g461fa17970de (May 03 2023 - 13:10:10 -0400) > > Trying to boot from MMC1 > > omap_hsmmc_send_cmd: timedout waiting on cmd inhibit to clear > > spl: mmc init failed with error: -110 > > SPL: failed to boot from all boot devices > > ### ERROR ### Please RESET the board ### > > > > I can provide more details / test patches as needed. > > > > -- > > Tom > > I do not know what to do with this... The only idea is to hide this code > behind CONFIG symbol and enable it only for mvebu. For example by this:
Well, maybe the problem is we're trying this on uSD cards? The failure I reported was uSD and not eMMC. I see a failure with this commit on rpi_3_32b, also from uSD boot. This time it's: Loading Environment from FAT... fsm 0, hsts 00000000 fsm 0, hsts 00000000 ... once in U-Boot itself. Going to the commit prior to the above one and the board is fine again. -- Tom
signature.asc
Description: PGP signature