From: Christian Lütke-Stetzkamp Date: Tue, 24 Apr 2018 18:01:24 +0000 (+0200) Subject: staging: mt7621-mmc: Fix dma_map_sg may map to fewer entries X-Git-Tag: v4.18-rc1~83^2~553 X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=c302214c9d7d8c4f4f1ac82715ce952155355648;p=linux.git staging: mt7621-mmc: Fix dma_map_sg may map to fewer entries The dma_map_sg function may merge several sglist entries into one, the return value has to be saved to consider that. The data->sg_count field is the position, where it should be saved, like it is done in other mmc host drivers. Also the count of mapped entries is needed for the dma setup. Signed-off-by: Christian Lütke-Stetzkamp Reviewed-by: NeilBrown Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/mt7621-mmc/sd.c b/drivers/staging/mt7621-mmc/sd.c index d464e2a321da..f11eb1fe3f6a 100644 --- a/drivers/staging/mt7621-mmc/sd.c +++ b/drivers/staging/mt7621-mmc/sd.c @@ -1423,8 +1423,10 @@ static int msdc_do_request(struct mmc_host *mmc, struct mmc_request *mrq) goto done; dir = read ? DMA_FROM_DEVICE : DMA_TO_DEVICE; - (void)dma_map_sg(mmc_dev(mmc), data->sg, data->sg_len, dir); - msdc_dma_setup(host, &host->dma, data->sg, data->sg_len); + data->sg_count = dma_map_sg(mmc_dev(mmc), data->sg, + data->sg_len, dir); + msdc_dma_setup(host, &host->dma, data->sg, + data->sg_count); /* then wait command done */ if (msdc_command_resp(host, cmd, 1, CMD_TIMEOUT) != 0)