diff options
author | Ulf Hansson <ulf.hansson@linaro.org> | 2019-03-06 09:04:52 -0500 |
---|---|---|
committer | Ulf Hansson <ulf.hansson@linaro.org> | 2019-04-15 05:55:54 -0400 |
commit | c21aa7a804160f58e391c9fcb2aff16e7d30c80f (patch) | |
tree | a1aeac561c45693e541e932c0bdc98e8c72595fb | |
parent | 6aa35ce74399f70b88e4c0e17c486da8ad6ef33d (diff) |
mmc: mmci: Drop unnecessary clear of variant->qcom_dml flag
There's no point clearing the variant flag in case the qcom variant fails
to setup DMA. This is because if mmci_dma_setup() fails, then the use_dma
flag remains set to false, which leads to mmci using PIO mode and not DMA.
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
-rw-r--r-- | drivers/mmc/host/mmci_qcom_dml.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/mmc/host/mmci_qcom_dml.c b/drivers/mmc/host/mmci_qcom_dml.c index 25d0a75533ea..b942d4aeeb77 100644 --- a/drivers/mmc/host/mmci_qcom_dml.c +++ b/drivers/mmc/host/mmci_qcom_dml.c | |||
@@ -133,7 +133,6 @@ static int qcom_dma_setup(struct mmci_host *host) | |||
133 | producer_id = of_get_dml_pipe_index(np, "rx"); | 133 | producer_id = of_get_dml_pipe_index(np, "rx"); |
134 | 134 | ||
135 | if (producer_id < 0 || consumer_id < 0) { | 135 | if (producer_id < 0 || consumer_id < 0) { |
136 | host->variant->qcom_dml = false; | ||
137 | mmci_dmae_release(host); | 136 | mmci_dmae_release(host); |
138 | return -EINVAL; | 137 | return -EINVAL; |
139 | } | 138 | } |