Hi, On 1/23/21 9:37 PM, Stefan Bosch wrote: > After issuing the switch command: Wait until 'current state' of the card > status becomes 'tran'. This prevents from response timeout at the next > command because of 'current state' = 'data'.> > Signed-off-by: Stefan Bosch <[email protected]> > --- > > drivers/mmc/mmc.c | 3 ++- > include/mmc.h | 1 + > 2 files changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c > index a47700e313..8ccd2058a9 100644 > --- a/drivers/mmc/mmc.c > +++ b/drivers/mmc/mmc.c > @@ -823,7 +823,8 @@ static int __mmc_switch(struct mmc *mmc, u8 set, u8 > index, u8 value, > value); > return -EIO; > } > - if (!ret && (status & MMC_STATUS_RDY_FOR_DATA)) > + if (!ret && (status & MMC_STATUS_RDY_FOR_DATA) && > + (status & MMC_STATUS_CURR_STATE) == MMC_STATE_TRANS)
AFAIK, MMC_STATUS_RDY_FOR_DATA bit means the signaling about "buffer is empty". Even though its bit was set, does it need to check TRANS state? Best Regards, Jaehoon Chung > return 0; > udelay(100); > } while (get_timer(start) < timeout_ms); > diff --git a/include/mmc.h b/include/mmc.h > index 1d377e0281..18402494c6 100644 > --- a/include/mmc.h > +++ b/include/mmc.h > @@ -178,6 +178,7 @@ static inline bool mmc_is_tuning_cmd(uint cmdidx) > #define MMC_STATUS_ERROR (1 << 19) > > #define MMC_STATE_PRG (7 << 9) > +#define MMC_STATE_TRANS (4 << 9) > > #define MMC_VDD_165_195 0x00000080 /* VDD voltage 1.65 - > 1.95 */ > #define MMC_VDD_20_21 0x00000100 /* VDD voltage 2.0 ~ > 2.1 */ >
