Re: [PATCH v8 0/2] sDMA descriptor autoloading feature

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

 



On Wed, May 5, 2010 at 4:45 AM, Madhusudhan <madhu.cr@xxxxxx> wrote:
>> -----Original Message-----
>> From: svenkatr@xxxxxxxxx [mailto:svenkatr@xxxxxxxxx] On Behalf Of
>> Venkatraman S
>> Sent: Thursday, April 29, 2010 12:35 PM
>> To: linux-omap@xxxxxxxxxxxxxxx; linux-mmc@xxxxxxxxxxxxxxx; linux-arm-
>> kernel@xxxxxxxxxxxxxxxxxxx
>> Cc: Madhusudhan Chikkature; Tony Lindgren; Adrian Hunter; Santosh
>> Shilimkar
>> Subject: [PATCH v8 0/2] sDMA descriptor autoloading feature
>>
>>     This patch series is the introduction of new sDMA feature: descriptor
>> autoloading (v8) and it's adoption by the OMAP HSMMC driver.
>>
>>    The previous version (v7) was blocked on achieving proper interrupt
>> syncronisation
>> between the MMC and DMA callback in the HSMMC driver.
>>   Thanks to Adrian Hunter's patch,
>> https://patchwork.kernel.org/patch/94670/
>> this has been cleaned up.
>>
>> Current version is based on for-next branch + Adrian Hunter's
>> interrupt syncronisation patch
>> (https://patchwork.kernel.org/patch/94670)
>>
>> Changes since v7:
>>   * Removed the 'fix race condition between DMA and HSMMC callback'
>> part of the series
>>   * Added memory barries in start and resume sglist transfer for cpu
>> buffer syncronization
>>   * Cleanup and comments
>>
>> This has been tested on OMAP3430, OMAP3630 and OMAP4430 SDP
>> for MMC file transfer and as boot file system.
>>
>
> The HSMMC driver changes look good to me. I can ACK that but am not a DMA
> expert to review your changes to the DMA driver.
>
Thanks. Can you please add your acked-by to 2/2 of this series so that
patchworks can pick it up.

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

[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux