Re: [V2 PATCH] dmaengine: axi-dmac: assign `copy_align` property

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 27-02-19, 08:12, Ardelean, Alexandru wrote:
> On Tue, 2019-02-26 at 10:52 +0200, Alexandru Ardelean wrote:
> > The `copy_align` property is a generic property that describes alignment
> > for DMA memcpy & sg ops.
> > It serves mostly an informational purpose, and can be used in DMA tests,
> > to
> > pass the info to know what alignment to expect.
> > 
> 
> build bot said this doesn't compile.
> I think this patch needs to come later after some more updates have been
> pushed for this driver.

Well build bot has no idea about that (like me), so better to put them
in a series and send (that way bot and me know which should come first)

> 
> > Signed-off-by: Alexandru Ardelean <alexandru.ardelean@xxxxxxxxxx>
> > ---
> > 
> > Changelog v1 -> v2:
> > * re-applied & moved the `copy_align` assignment; apparently it was
> >   conflicting with another patch from another series
> > 
> >  drivers/dma/dma-axi-dmac.c | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/drivers/dma/dma-axi-dmac.c b/drivers/dma/dma-axi-dmac.c
> > index ffc0adc2f6ce..5ec223ff7e1a 100644
> > --- a/drivers/dma/dma-axi-dmac.c
> > +++ b/drivers/dma/dma-axi-dmac.c
> > @@ -666,6 +666,7 @@ static int axi_dmac_probe(struct platform_device
> > *pdev)
> >  	dma_dev->dst_addr_widths = BIT(dmac->chan.dest_width);
> >  	dma_dev->directions = BIT(dmac->chan.direction);
> >  	dma_dev->residue_granularity = DMA_RESIDUE_GRANULARITY_DESCRIPTOR;
> > +	dma_dev->copy_align = (dmac->chan.address_align_mask + 1);
> >  	INIT_LIST_HEAD(&dma_dev->channels);
> >  
> >  	dmac->chan.vchan.desc_free = axi_dmac_desc_free;

-- 
~Vinod



[Index of Archives]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux PCI]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux