]> www.infradead.org Git - users/dwmw2/linux.git/commitdiff
mmc: meson-gx: Remove setting of CMD_CFG_ERROR
authorRong Chen <rong.chen@amlogic.com>
Thu, 26 Oct 2023 07:31:56 +0000 (15:31 +0800)
committerUlf Hansson <ulf.hansson@linaro.org>
Fri, 27 Oct 2023 09:59:14 +0000 (11:59 +0200)
For the t7 and older SoC families, the CMD_CFG_ERROR has no effect.
Starting from SoC family C3, setting this bit without SG LINK data
address will cause the controller to generate an IRQ and stop working.

To fix it, don't set the bit CMD_CFG_ERROR anymore.

Fixes: 18f92bc02f17 ("mmc: meson-gx: make sure the descriptor is stopped on errors")
Signed-off-by: Rong Chen <rong.chen@amlogic.com>
Reviewed-by: Jerome Brunet <jbrunet@baylibre.com>
Cc: stable@vger.kernel.org
Link: https://lore.kernel.org/r/20231026073156.2868310-1-rong.chen@amlogic.com
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/mmc/host/meson-gx-mmc.c

index 9837dab096e640962b1efcdc5993cbc7ecd0b64b..c7c067b9415a41db7ea30223e15b772a6a02a058 100644 (file)
@@ -801,7 +801,6 @@ static void meson_mmc_start_cmd(struct mmc_host *mmc, struct mmc_command *cmd)
 
        cmd_cfg |= FIELD_PREP(CMD_CFG_CMD_INDEX_MASK, cmd->opcode);
        cmd_cfg |= CMD_CFG_OWNER;  /* owned by CPU */
-       cmd_cfg |= CMD_CFG_ERROR; /* stop in case of error */
 
        meson_mmc_set_response_bits(cmd, &cmd_cfg);