Re: [PATCH] Updated videobuf documentation

[Mauro Carvalho Chehab <mchehab@infradead.org>]

Randy Dunlap wrote:
> On 02/22/10 12:47, Jonathan Corbet wrote:
>> On Fri, 19 Feb 2010 04:09:19 -0200
>> Mauro Carvalho Chehab <mchehab@infradead.org> wrote:
>>
>>> Not anymore: there's a patch that added USERPTR support for videobuf-dma-contig:
>>>
>>> commit 720b17e759a50635c429ccaa2ec3d01edb4f92d6
>>> Author: Magnus Damm <damm@igel.co.jp>
>>> Date:   Tue Jun 16 15:32:36 2009 -0700
>>>
>>>     videobuf-dma-contig: zero copy USERPTR support
>> Now *that* is a special-purpose hack.  But it's worth mentioning, so
>> I've updated the text accordingly.
>>
>>> In terms of memory types, there's a possibility that weren't mentioned: the OVERLAY mode.
>>>
>>> Maybe a small paragraph may be added just for the completeness of the doc.
>> Yeah, I try to ignore overlay whenever I can.  I've added a brief bit.
>>
>> New version of the patch attached; look better?
> 
> Looks good to me (other than a few straggling lines that end with whitespace).
> 
> Reviewed-by: Randy Dunlap <rdunlap@xenotime.net>

Seems OK to me also. I'll apply it (running a script to remove trailing whitespaces ;) )


-- 

Cheers,
Mauro