From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Date: Wed, 21 Jan 2015 11:07:59 +0000 Subject: Re: [PATCH] dmaengine: rcar-hpbdma: tidyup residue_granularity Message-Id: <33068117.HPgfh8e0Et@avalon> List-Id: References: <87vbk0n3q3.wl%kuninori.morimoto.gx@renesas.com> In-Reply-To: <87vbk0n3q3.wl%kuninori.morimoto.gx@renesas.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-sh@vger.kernel.org Hi Morimoto-san, Thank you for the patch. On Wednesday 21 January 2015 04:42:22 Kuninori Morimoto wrote: > From: Kuninori Morimoto > > The driver doesn't support residue reporting at all. > residue_granularity should be set to DMA_RESIDUE_GRANULARITY_DESCRIPTOR. > Special thanks to Laurent > > Reported-by: Laurent Pinchart > Signed-off-by: Kuninori Morimoto I thought the plan was to fix this in v4 of "[PATCH v3] dmaengine:: rcar- hpbdma: add dma_dev->directions" ? > --- > drivers/dma/sh/rcar-hpbdma.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/dma/sh/rcar-hpbdma.c b/drivers/dma/sh/rcar-hpbdma.c > index 6fef1b9..749f26e 100644 > --- a/drivers/dma/sh/rcar-hpbdma.c > +++ b/drivers/dma/sh/rcar-hpbdma.c > @@ -600,7 +600,7 @@ static int hpb_dmae_probe(struct platform_device *pdev) > dma_dev->src_addr_widths = widths; > dma_dev->dst_addr_widths = widths; > dma_dev->directions = BIT(DMA_MEM_TO_DEV) | BIT(DMA_DEV_TO_MEM); > - dma_dev->residue_granularity = DMA_RESIDUE_GRANULARITY_BURST; > + dma_dev->residue_granularity = DMA_RESIDUE_GRANULARITY_DESCRIPTOR; > > hpbdev->shdma_dev.ops = &hpb_dmae_ops; > hpbdev->shdma_dev.desc_size = sizeof(struct hpb_desc); -- Regards, Laurent Pinchart