Halting the kernel on an unsupported stop command seems overkill, report
the error and say what we already did (due to autocmd12) instead.

Reviewed-by: Simon Horman <horms+rene...@verge.net.au>
Signed-off-by: Wolfram Sang <wsa+rene...@sang-engineering.com>
---
 drivers/mmc/host/tmio_mmc_pio.c | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/drivers/mmc/host/tmio_mmc_pio.c b/drivers/mmc/host/tmio_mmc_pio.c
index ad2840e1bfae51..b47dd9195fe3fe 100644
--- a/drivers/mmc/host/tmio_mmc_pio.c
+++ b/drivers/mmc/host/tmio_mmc_pio.c
@@ -553,10 +553,11 @@ void tmio_mmc_do_data_irq(struct tmio_mmc_host *host)
        }
 
        if (stop) {
-               if (stop->opcode == MMC_STOP_TRANSMISSION && !stop->arg)
-                       sd_ctrl_write16(host, CTL_STOP_INTERNAL_ACTION, 0);
-               else
-                       BUG();
+               if (stop->opcode != MMC_STOP_TRANSMISSION || stop->arg)
+                       dev_err(&host->pdev->dev, "unsupported stop: 
CMD%u,0x%x. We did CMD12,0\n",
+                               stop->opcode, stop->arg);
+
+               sd_ctrl_write16(host, CTL_STOP_INTERNAL_ACTION, 0);
        }
 
        schedule_work(&host->done);
-- 
2.11.0

Reply via email to