* [PATCH 2/2] mmc: jz4740_mmc: Fix error check for dma_map_sg [not found] <20220825074008.33349-1-jinpu.wang@ionos.com> @ 2022-08-25 7:40 ` Jack Wang 2022-08-25 22:38 ` Paul Cercueil 0 siblings, 1 reply; 2+ messages in thread From: Jack Wang @ 2022-08-25 7:40 UTC (permalink / raw) To: ulf.hansson, linux-mmc; +Cc: Paul Cercueil, linux-mips, linux-kernel dma_map_sg return 0 on error. Cc: Paul Cercueil <paul@crapouillou.net> Cc: Ulf Hansson <ulf.hansson@linaro.org> Cc: linux-mips@vger.kernel.org Cc: linux-mmc@vger.kernel.org Cc: linux-kernel@vger.kernel.org Signed-off-by: Jack Wang <jinpu.wang@ionos.com> --- drivers/mmc/host/jz4740_mmc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/mmc/host/jz4740_mmc.c b/drivers/mmc/host/jz4740_mmc.c index b1d563b2ed1b..dc2db9c185ea 100644 --- a/drivers/mmc/host/jz4740_mmc.c +++ b/drivers/mmc/host/jz4740_mmc.c @@ -298,7 +298,7 @@ static int jz4740_mmc_prepare_dma_data(struct jz4740_mmc_host *host, { struct dma_chan *chan = jz4740_mmc_get_dma_chan(host, data); enum dma_data_direction dir = mmc_get_dma_dir(data); - int sg_count; + unsigned int sg_count; if (data->host_cookie == COOKIE_PREMAPPED) return data->sg_count; @@ -308,7 +308,7 @@ static int jz4740_mmc_prepare_dma_data(struct jz4740_mmc_host *host, data->sg_len, dir); - if (sg_count <= 0) { + if (!sg_count) { dev_err(mmc_dev(host->mmc), "Failed to map scatterlist for DMA operation\n"); return -EINVAL; -- 2.34.1 ^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH 2/2] mmc: jz4740_mmc: Fix error check for dma_map_sg 2022-08-25 7:40 ` [PATCH 2/2] mmc: jz4740_mmc: Fix error check for dma_map_sg Jack Wang @ 2022-08-25 22:38 ` Paul Cercueil 0 siblings, 0 replies; 2+ messages in thread From: Paul Cercueil @ 2022-08-25 22:38 UTC (permalink / raw) To: Jack Wang; +Cc: ulf.hansson, linux-mmc, linux-mips, linux-kernel Hi Jack, Le jeu., août 25 2022 at 09:40:08 +0200, Jack Wang <jinpu.wang@ionos.com> a écrit : > dma_map_sg return 0 on error. > > Cc: Paul Cercueil <paul@crapouillou.net> > Cc: Ulf Hansson <ulf.hansson@linaro.org> > Cc: linux-mips@vger.kernel.org > Cc: linux-mmc@vger.kernel.org > Cc: linux-kernel@vger.kernel.org > > Signed-off-by: Jack Wang <jinpu.wang@ionos.com> Acked-by: Paul Cercueil <paul@crapouillou.net> Cheers, -Paul > --- > drivers/mmc/host/jz4740_mmc.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/mmc/host/jz4740_mmc.c > b/drivers/mmc/host/jz4740_mmc.c > index b1d563b2ed1b..dc2db9c185ea 100644 > --- a/drivers/mmc/host/jz4740_mmc.c > +++ b/drivers/mmc/host/jz4740_mmc.c > @@ -298,7 +298,7 @@ static int jz4740_mmc_prepare_dma_data(struct > jz4740_mmc_host *host, > { > struct dma_chan *chan = jz4740_mmc_get_dma_chan(host, data); > enum dma_data_direction dir = mmc_get_dma_dir(data); > - int sg_count; > + unsigned int sg_count; > > if (data->host_cookie == COOKIE_PREMAPPED) > return data->sg_count; > @@ -308,7 +308,7 @@ static int jz4740_mmc_prepare_dma_data(struct > jz4740_mmc_host *host, > data->sg_len, > dir); > > - if (sg_count <= 0) { > + if (!sg_count) { > dev_err(mmc_dev(host->mmc), > "Failed to map scatterlist for DMA operation\n"); > return -EINVAL; > -- > 2.34.1 > ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-08-25 22:38 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20220825074008.33349-1-jinpu.wang@ionos.com>
2022-08-25 7:40 ` [PATCH 2/2] mmc: jz4740_mmc: Fix error check for dma_map_sg Jack Wang
2022-08-25 22:38 ` Paul Cercueil
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox