Re: Can the patch adding support for the Tasco USB microscope be queued up?
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Date: 2015-02-16 18:34:20
Hi Hans, On Monday 16 February 2015 17:08:52 Hans Verkuil wrote:
On 02/16/2015 05:01 PM, Michael Hall wrote:quoted
My apologies, the other emails were sent to linux-uvc-devel, not linux-media. Do you want an attached patch file, or simply a diff in the body of the email? I'm also not clear on what you mean by "correct Signed-off-by line", I have very little experience with git, I've mostly used bzr.This is a good link with the relevant information: http://linuxtv.org/wiki/index.php/Development:_How_to_submit_patches Anyway, I checked where the original patch came from, and Laurent Pinchart wrote it. Since he's kernel maintainer he knows all about well-formatted patches and it's best if he just posts and merges his own patch :-) Laurent, it's all yours!
I've sent the patch to linux-media and will include it in my next uvcvideo pull request. Steven, could you please send me the output of lsusb -v -d '1871:0516' (if possible running as root) on your system ?
quoted
On 02/16/2015 10:40 AM, Hans Verkuil wrote:quoted
On 02/16/2015 04:31 PM, Michael Hall wrote:quoted
This is now the 3rd or 4th email to this list requesting that this patch be merged in. If there is something wrong with the patch that needs fixing, please let me know and I will work on the fix. Otherwise I've lost interest in pushing to get it into upstream.I can't remember ever seeing a patch for that posted to the linux-media mailinglist. The best way is just to post the patch to this mailinglist, check that it appears in patchwork (https://patchwork.linuxtv.org/project/linux-media/list/), make sure you keep the author and correct Signed-off-by line and it's *guaranteed* that someone will look at it, and merge it or reply to it if there are problems. Mails like 'please pick up a patch from some other git repo' are very likely to be forgotten due to volume of other postings. Patchwork won't pick them up and that's what we all rely on. So if either of you can just post this as a properly formatted patch, then it will be taken care of. Regards, Hansquoted
Michael Hall mhall119@gmail.com On 02/16/2015 10:08 AM, Steven Zakulec wrote:quoted
Hi, as an owner of a Tasco/Aveo USB microscope detected but not working under Linux, I'd really like to see the patch adding this variant added to the kernel. I've copied the patch's author on the email. The people on the linux-uvc-devel list directed me over here. The patch here: http://sourceforge.net/p/linux-uvc/mailman/message/32434617/ , itself an update of an earlier patch: http://sourceforge.net/p/linux-uvc/mailman/message/29835445/ works. The patch does make the USB microscope work where it didn't work at all before. Thank you!
-- Regards, Laurent Pinchart