Re: [PATCH] Updated videobuf documentation

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

 



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@xxxxxxxxxxxxx> wrote:
>>
>>> Not anymore: there's a patch that added USERPTR support for videobuf-dma-contig:
>>>
>>> commit 720b17e759a50635c429ccaa2ec3d01edb4f92d6
>>> Author: Magnus Damm <damm@xxxxxxxxxx>
>>> 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@xxxxxxxxxxxx>

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


-- 

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

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux