Re: Can the patch adding support for the Tasco USB microscope be queued up?

2015-02-18 Thread Steven Zakulec
  bInterfaceNumber1
  bAlternateSetting   3
  bNumEndpoints   1
  bInterfaceClass14 Video
  bInterfaceSubClass  2 Video Streaming
  bInterfaceProtocol  0
  iInterface  0
  Endpoint Descriptor:
bLength 7
bDescriptorType 5
bEndpointAddress 0x83  EP 3 IN
bmAttributes5
  Transfer TypeIsochronous
  Synch Type   Asynchronous
  Usage Type   Data
wMaxPacketSize 0x03fc  1x 1020 bytes
bInterval   1
Interface Descriptor:
  bLength 9
  bDescriptorType 4
  bInterfaceNumber1
  bAlternateSetting   4
  bNumEndpoints   1
  bInterfaceClass14 Video
  bInterfaceSubClass  2 Video Streaming
  bInterfaceProtocol  0
  iInterface  0
  Endpoint Descriptor:
bLength 7
bDescriptorType 5
bEndpointAddress 0x83  EP 3 IN
bmAttributes5
  Transfer TypeIsochronous
  Synch Type   Asynchronous
  Usage Type   Data
wMaxPacketSize 0x0bfc  2x 1020 bytes
bInterval   1
Interface Descriptor:
  bLength 9
  bDescriptorType 4
  bInterfaceNumber1
  bAlternateSetting   5
  bNumEndpoints   1
  bInterfaceClass14 Video
  bInterfaceSubClass  2 Video Streaming
  bInterfaceProtocol  0
  iInterface  0
  Endpoint Descriptor:
bLength 7
bDescriptorType 5
bEndpointAddress 0x83  EP 3 IN
bmAttributes5
  Transfer TypeIsochronous
  Synch Type   Asynchronous
  Usage Type   Data
wMaxPacketSize 0x13fc  3x 1020 bytes
bInterval   1
Device Qualifier (for other device speed):
  bLength10
  bDescriptorType 6
  bcdUSB   2.00
  bDeviceClass  239 Miscellaneous Device
  bDeviceSubClass 2 ?
  bDeviceProtocol 1 Interface Association
  bMaxPacketSize064
  bNumConfigurations  1
Device Status: 0x
  (Bus Powered)

On Mon, Feb 16, 2015 at 1:35 PM, Laurent Pinchart
laurent.pinch...@ideasonboard.com wrote:
 Hi Hans,

 On Monday 16 February 2015 17:08:52 Hans Verkuil wrote:
 On 02/16/2015 05:01 PM, Michael Hall wrote:
  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 ?

  On 02/16/2015 10:40 AM, Hans Verkuil wrote:
  On 02/16/2015 04:31 PM, Michael Hall wrote:
  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,
 
 Hans
 
  Michael Hall
  mhall...@gmail.com
 
  On 02/16/2015 10:08 AM, Steven Zakulec wrote:
  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

Can the patch adding support for the Tasco USB microscope be queued up?

2015-02-16 Thread Steven Zakulec
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!
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html