Hi, I am having issues with this patch (commit id:-785b215250310c690142abb1bdbb0767b8487c49 " mmc: sdhi: use maximum width for the sdbuf register") I am testing SDHI1 against linux-next-20170727 tag with iWave RZ/G1M board which is identical to R-Car M2.
Getting below SDHI1 errors during boot. This issue is fixed by reverting this patch. [ 11.846211] sh_mobile_sdhi ee140000.sd: timeout waiting for hardware interrupt (CMD51) [ 11.906243] mmc2: error -110 whilst initialising SD card [ 17.046328] sh_mobile_sdhi ee140000.sd: timeout waiting for hardware interrupt (CMD5) [ 22.166212] sh_mobile_sdhi ee140000.sd: timeout waiting for hardware interrupt (CMD5) [ 27.286205] sh_mobile_sdhi ee140000.sd: timeout waiting for hardware interrupt (CMD5) [ 32.406204] sh_mobile_sdhi ee140000.sd: timeout waiting for hardware interrupt (CMD5) [ 37.526209] sh_mobile_sdhi ee140000.sd: timeout waiting for hardware interrupt (CMD55) [ 42.646206] sh_mobile_sdhi ee140000.sd: timeout waiting for hardware interrupt (CMD55) [ 47.766203] sh_mobile_sdhi ee140000.sd: timeout waiting for hardware interrupt (CMD55) [ 52.886204] sh_mobile_sdhi ee140000.sd: timeout waiting for hardware interrupt (CMD55) [ 58.086208] sh_mobile_sdhi ee140000.sd: timeout waiting for hardware interrupt (CMD52) [ 63.206204] sh_mobile_sdhi ee140000.sd: timeout waiting for hardware interrupt (CMD5) [ 68.326205] sh_mobile_sdhi ee140000.sd: timeout waiting for hardware interrupt (CMD5) [ 73.446204] sh_mobile_sdhi ee140000.sd: timeout waiting for hardware interrupt (CMD5) Regards, Biju > -----Original Message----- > From: [email protected] [mailto:linux-mmc- > [email protected]] On Behalf Of Wolfram Sang > Sent: 30 June 2017 11:57 > To: [email protected] > Cc: [email protected]; Simon Horman <[email protected]>; > Yoshihiro Shimoda <[email protected]>; Wolfram Sang > <[email protected]> > Subject: [PATCH 0/4] tmio/sdhi: use max sd_buf size & clean header file before > > From: Wolfram Sang <[email protected]> > > The main patch is patch 4 which uses the maximum sd_buf size, i.e. 32 bit on > Gen2 and 64 bit on Gen3. Instead of a new flag, we use the bus_shift > information we already have. See the patch description for details. > > Before fixing this, I wanted to remove a magic value in that function which is > done in patch 3. > > When adding the new define, I noticed that some description is missing in the > header which is fixed with patch 2. > > When adding the description, I noticed this meanwhile obsolete define which > gets removed in patch 1. > > That's the way it went :) > > Tested on Salvator-X/M3-W and Lager/H2. > > Regards, > > Wolfram > > Wolfram Sang (4): > mmc: tmio: remove obsolete TMIO_BBS > mmc: tmio: add references to bit defines in the header > mmc: tmio: no magic values when enabling DMA > mmc: sdhi: use maximum width for the sdbuf register > > drivers/mmc/host/renesas_sdhi_core.c | 5 ++--- > drivers/mmc/host/tmio_mmc.h | 7 +++++-- > 2 files changed, 7 insertions(+), 5 deletions(-) > > -- > 2.11.0 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the > body > of a message to [email protected] More majordomo info at > http://vger.kernel.org/majordomo-info.html Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
