All of lore.kernel.org
 help / color / mirror / Atom feed
From: Hans de Goede <hdegoede@redhat.com>
To: "Németh Márton" <nm127@freemail.hu>
Cc: Jean-Francois Moine <moinejf@free.fr>,
	Richard Purdie <rpurdie@rpsys.net>,
	V4L Mailing List <linux-media@vger.kernel.org>
Subject: Re: [PATCH 1/2] gspca pac7302: allow controlling LED separately
Date: Sat, 27 Feb 2010 08:53:16 +0100	[thread overview]
Message-ID: <4B88CF6C.2070703@redhat.com> (raw)
In-Reply-To: <4B886566.8000600@freemail.hu>

Hi,

On 02/27/2010 01:20 AM, Németh Márton wrote:
> From: Márton Németh<nm127@freemail.hu>
>
> On Labtec Webcam 2200 there is a feedback LED which can be controlled
> independent from the streaming.

This is true for a lot of cameras, so if we are going to add a way to
support control of the LED separate of the streaming state, we
should do that at the gspca_main level, and let sub drivers which
support this export a set_led callback function.

I must say I personally don't see much of a use case for this feature,
but I believe JF Moine does, so I'll leave further comments and
review of this to JF. I do believe it is important that if we go this
way we do so add the gspca_main level.

Regards,

Hans

  reply	other threads:[~2010-02-27  7:46 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-02-24  6:52 [PATCH 1/2] gspca pac7302: add LED control Németh Márton
2010-02-24  7:22 ` Jean-Francois Moine
2010-02-27  0:20   ` [PATCH 1/2] gspca pac7302: allow controlling LED separately Németh Márton
2010-02-27  7:53     ` Hans de Goede [this message]
2010-02-27  8:22       ` Németh Márton
2010-02-27  9:04       ` Jean-Francois Moine
2010-02-27 15:35         ` Németh Márton
2010-02-27 19:12           ` Hans de Goede
2010-02-27 20:00             ` Jean-Francois Moine
2010-02-27  8:17     ` Németh Márton

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=4B88CF6C.2070703@redhat.com \
    --to=hdegoede@redhat.com \
    --cc=linux-media@vger.kernel.org \
    --cc=moinejf@free.fr \
    --cc=nm127@freemail.hu \
    --cc=rpurdie@rpsys.net \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.