Re: [PATCH] Updated videobuf documentation

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

 



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>

thanks.
-- 
~Randy
--
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