public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Michael Hall <mhall119@gmail.com>
To: Hans Verkuil <hverkuil@xs4all.nl>,
	Steven Zakulec <spzakulec@gmail.com>,
	linux-media@vger.kernel.org
Subject: Re: Can the patch adding support for the Tasco USB microscope be queued up?
Date: Mon, 16 Feb 2015 11:01:16 -0500	[thread overview]
Message-ID: <54E2144C.7030206@gmail.com> (raw)
In-Reply-To: <54E20F64.5060506@xs4all.nl>

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.

Michael Hall
mhall119@gmail.com

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
>> mhall119@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.
>>>
>>> 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 majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>
> 

  reply	other threads:[~2015-02-16 16:01 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-16 15:08 Can the patch adding support for the Tasco USB microscope be queued up? Steven Zakulec
2015-02-16 15:31 ` Michael Hall
2015-02-16 15:40   ` Hans Verkuil
2015-02-16 16:01     ` Michael Hall [this message]
2015-02-16 16:08       ` Hans Verkuil
2015-02-16 18:35         ` Laurent Pinchart
2015-02-19  2:47           ` Steven Zakulec
2015-02-24  1:18             ` Laurent Pinchart

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=54E2144C.7030206@gmail.com \
    --to=mhall119@gmail.com \
    --cc=hverkuil@xs4all.nl \
    --cc=linux-media@vger.kernel.org \
    --cc=spzakulec@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox