mmc: block: blk-mq: Check error bits and save the exception bit when polling card busy
Check error bits and save the exception bit when polling card busy. Signed-off-by: Adrian Hunter <adrian.hunter@intel.com> Acked-by: Linus Walleij <linus.walleij@linaro.org> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org> Tested-by: Linus Walleij <linus.walleij@linaro.org>
This commit is contained in:
committed by
Ulf Hansson
parent
c89b4851c6
commit
f47a1fe346
@@ -1457,15 +1457,18 @@ static inline void mmc_apply_rel_rw(struct mmc_blk_request *brq,
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
#define CMD_ERRORS \
|
#define CMD_ERRORS_EXCL_OOR \
|
||||||
(R1_OUT_OF_RANGE | /* Command argument out of range */ \
|
(R1_ADDRESS_ERROR | /* Misaligned address */ \
|
||||||
R1_ADDRESS_ERROR | /* Misaligned address */ \
|
|
||||||
R1_BLOCK_LEN_ERROR | /* Transferred block length incorrect */\
|
R1_BLOCK_LEN_ERROR | /* Transferred block length incorrect */\
|
||||||
R1_WP_VIOLATION | /* Tried to write to protected block */ \
|
R1_WP_VIOLATION | /* Tried to write to protected block */ \
|
||||||
R1_CARD_ECC_FAILED | /* Card ECC failed */ \
|
R1_CARD_ECC_FAILED | /* Card ECC failed */ \
|
||||||
R1_CC_ERROR | /* Card controller error */ \
|
R1_CC_ERROR | /* Card controller error */ \
|
||||||
R1_ERROR) /* General/unknown error */
|
R1_ERROR) /* General/unknown error */
|
||||||
|
|
||||||
|
#define CMD_ERRORS \
|
||||||
|
(CMD_ERRORS_EXCL_OOR | \
|
||||||
|
R1_OUT_OF_RANGE) /* Command argument out of range */ \
|
||||||
|
|
||||||
static void mmc_blk_eval_resp_error(struct mmc_blk_request *brq)
|
static void mmc_blk_eval_resp_error(struct mmc_blk_request *brq)
|
||||||
{
|
{
|
||||||
u32 val;
|
u32 val;
|
||||||
@@ -2157,24 +2160,40 @@ static inline bool mmc_blk_rq_error(struct mmc_blk_request *brq)
|
|||||||
brq->data.error || brq->cmd.resp[0] & CMD_ERRORS;
|
brq->data.error || brq->cmd.resp[0] & CMD_ERRORS;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static inline bool mmc_blk_oor_valid(struct mmc_blk_request *brq)
|
||||||
|
{
|
||||||
|
return !!brq->mrq.sbc;
|
||||||
|
}
|
||||||
|
|
||||||
|
static inline u32 mmc_blk_stop_err_bits(struct mmc_blk_request *brq)
|
||||||
|
{
|
||||||
|
return mmc_blk_oor_valid(brq) ? CMD_ERRORS : CMD_ERRORS_EXCL_OOR;
|
||||||
|
}
|
||||||
|
|
||||||
static int mmc_blk_card_busy(struct mmc_card *card, struct request *req)
|
static int mmc_blk_card_busy(struct mmc_card *card, struct request *req)
|
||||||
{
|
{
|
||||||
struct mmc_queue_req *mqrq = req_to_mmc_queue_req(req);
|
struct mmc_queue_req *mqrq = req_to_mmc_queue_req(req);
|
||||||
bool gen_err = false;
|
u32 status = 0;
|
||||||
int err;
|
int err;
|
||||||
|
|
||||||
if (mmc_host_is_spi(card->host) || rq_data_dir(req) == READ)
|
if (mmc_host_is_spi(card->host) || rq_data_dir(req) == READ)
|
||||||
return 0;
|
return 0;
|
||||||
|
|
||||||
err = card_busy_detect_err(card, MMC_BLK_TIMEOUT_MS, false, req,
|
err = card_busy_detect(card, MMC_BLK_TIMEOUT_MS, false, req, &status);
|
||||||
&gen_err);
|
|
||||||
|
|
||||||
/* Copy the general error bit so it will be seen later on */
|
/*
|
||||||
if (gen_err) {
|
* Do not assume data transferred correctly if there are any error bits
|
||||||
mqrq->brq.stop.resp[0] |= R1_ERROR;
|
* set.
|
||||||
|
*/
|
||||||
|
if (status & mmc_blk_stop_err_bits(&mqrq->brq)) {
|
||||||
|
mqrq->brq.data.bytes_xfered = 0;
|
||||||
err = err ? err : -EIO;
|
err = err ? err : -EIO;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/* Copy the exception bit so it will be seen later on */
|
||||||
|
if (mmc_card_mmc(card) && status & R1_EXCEPTION_EVENT)
|
||||||
|
mqrq->brq.cmd.resp[0] |= R1_EXCEPTION_EVENT;
|
||||||
|
|
||||||
return err;
|
return err;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user