Re: [PATCH] mmc: sh_mmcif: Correct TX DMA channel allocation
On Wednesday 10 February 2016 14:07:01 Chris Paterson wrote: > Commit 27cbd7e815a8 ("mmc: sh_mmcif: rework dma channel handling") > introduced a typo causing the TX DMA channel allocation to be overwritten > by the requested RX DMA channel. > > Signed-off-by: Chris Paterson > --- > Acked-by: Arnd Bergmann Sorry about the trouble.
Re: [PATCH] mmc: sh_mmcif: Correct TX DMA channel allocation
Hi Chris, Thank you for the patch. On Wednesday 10 February 2016 14:07:01 Chris Paterson wrote: > Commit 27cbd7e815a8 ("mmc: sh_mmcif: rework dma channel handling") > introduced a typo causing the TX DMA channel allocation to be overwritten > by the requested RX DMA channel. > > Signed-off-by: Chris Paterson Acked-by: Laurent Pinchart > --- > drivers/mmc/host/sh_mmcif.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/mmc/host/sh_mmcif.c b/drivers/mmc/host/sh_mmcif.c > index e0c076a..8d870ce 100644 > --- a/drivers/mmc/host/sh_mmcif.c > +++ b/drivers/mmc/host/sh_mmcif.c > @@ -445,7 +445,7 @@ static void sh_mmcif_request_dma(struct sh_mmcif_host > *host) pdata->slave_id_rx); > } else { > host->chan_tx = dma_request_slave_channel(dev, "tx"); > - host->chan_tx = dma_request_slave_channel(dev, "rx"); > + host->chan_rx = dma_request_slave_channel(dev, "rx"); > } > dev_dbg(dev, "%s: got channel TX %p RX %p\n", __func__, host->chan_tx, > host->chan_rx); -- Regards, Laurent Pinchart
Re: [PATCH] mmc: sh_mmcif: Correct TX DMA channel allocation
On 10 February 2016 at 15:07, Chris Paterson wrote: > Commit 27cbd7e815a8 ("mmc: sh_mmcif: rework dma channel handling") > introduced a typo causing the TX DMA channel allocation to be overwritten > by the requested RX DMA channel. > > Signed-off-by: Chris Paterson Thanks, applied for fixes and I also added a corresponding fixes tag. Kind regards Uffe > --- > drivers/mmc/host/sh_mmcif.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/mmc/host/sh_mmcif.c b/drivers/mmc/host/sh_mmcif.c > index e0c076a..8d870ce 100644 > --- a/drivers/mmc/host/sh_mmcif.c > +++ b/drivers/mmc/host/sh_mmcif.c > @@ -445,7 +445,7 @@ static void sh_mmcif_request_dma(struct sh_mmcif_host > *host) > pdata->slave_id_rx); > } else { > host->chan_tx = dma_request_slave_channel(dev, "tx"); > - host->chan_tx = dma_request_slave_channel(dev, "rx"); > + host->chan_rx = dma_request_slave_channel(dev, "rx"); > } > dev_dbg(dev, "%s: got channel TX %p RX %p\n", __func__, host->chan_tx, > host->chan_rx); > -- > 1.9.1 >
[PATCH] mmc: sh_mmcif: Correct TX DMA channel allocation
Commit 27cbd7e815a8 ("mmc: sh_mmcif: rework dma channel handling") introduced a typo causing the TX DMA channel allocation to be overwritten by the requested RX DMA channel. Signed-off-by: Chris Paterson --- drivers/mmc/host/sh_mmcif.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mmc/host/sh_mmcif.c b/drivers/mmc/host/sh_mmcif.c index e0c076a..8d870ce 100644 --- a/drivers/mmc/host/sh_mmcif.c +++ b/drivers/mmc/host/sh_mmcif.c @@ -445,7 +445,7 @@ static void sh_mmcif_request_dma(struct sh_mmcif_host *host) pdata->slave_id_rx); } else { host->chan_tx = dma_request_slave_channel(dev, "tx"); - host->chan_tx = dma_request_slave_channel(dev, "rx"); + host->chan_rx = dma_request_slave_channel(dev, "rx"); } dev_dbg(dev, "%s: got channel TX %p RX %p\n", __func__, host->chan_tx, host->chan_rx); -- 1.9.1