Re: A second easycap driver implementation

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

 



Hi,

On 03/06/2012 10:04 PM, Ezequiel García wrote:
Hello,

After some research on v4l2 and videbuf2, and considering that easycap
driver is pretty
outdated I've decided to start writing a new driver from scratch.

I am using the excellent vivi driver and some usb video capture drivers as
a starting point. And of course, I'm using the current easycap implementation
as a reference (it works pretty well).

I have a couple of doubts regarding the development itself (how to
trace properly,
where to allocate urbs, and such) but perhaps the maintainers prefer
to take a look
at the code.

However, currently the driver is just a skeleton: it does all v4l2 and
videobuf2 intialization
but it doesn't actually stream video or submit urbs.

So,
1. Should I try to have something more finished before submit or can I
submit as it is?
2. In any case, how should I submit it? (Considering there is already
a working driver).

Have you considered instead slowly moving the existing easycap driver
over to all the new infrastructure we have now. For starters replace
its buffer management with videobuf2, then in another patch replace
some other bits, etc. ?  See what I've done to the pwc driver :)

OTOH if you already have a new more modern driver ready, then I say
go ahead and submit it. I would suggest to add it to staging too,
and make the 2 kconfig options conflict, so enabling one would allow
the user to no longer select the other (with a note about this in
the help text). And then hopefully soon we will see a follow up
patch removing the old driver, and then moving the new one out
of staging.

Which ever path you choose: Thanks for working on this!

Regards,

Hans
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux