Re: [PATCH v2 2/2] dmaengine: s3c24xx-dma: Add cyclic transfer support

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

 



On Tue, May 20, 2014 at 11:31 AM, Andy Shevchenko
<andriy.shevchenko@xxxxxxxxxxxxxxx> wrote:
> On Mon, 2014-05-19 at 20:05 +0300, Vasily Khoruzhick wrote:
>> Many audio interface drivers require support of cyclic transfers to work
>> correctly, for example Samsung ASoC DMA driver. This patch adds support
>> for cyclic transfers to the s3c24xx-dma driver
>
> Sorry, not mentioned earlier, but couple of nitpicks below. (I think you
> may wait for more comments and then decide what to do with mine)
>
>> +     dev_dbg(&s3cdma->pdev->dev,
>> +             "prepare cyclic transaction of %d bytes with period %d from %s\n",
>> +             size, period, s3cchan->name);
>
> size and period are size_t, thus %zu.

Ok.

>> +             /* Check last period length */
>> +             if (i == (sg_len - 1))
>
> Redundant parenthesis.

Ok

>> +                     dsg->len = size - (period * i);
>> +             if (direction == DMA_MEM_TO_DEV) {
>> +                     dsg->src_addr = addr + (period * i);
>
> Ditto.

Ok

>> +                     dsg->dst_addr = slave_addr;
>> +             } else { /* DMA_DEV_TO_MEM */
>> +                     dsg->src_addr = slave_addr;
>> +                     dsg->dst_addr = addr + (period * i);
>
> Ditto.

Ok
--
To unsubscribe from this list: send the line "unsubscribe dmaengine" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[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