From: Brandon Philips <brandon@ifup.org>
To: Martin Rubli <v4l2-lists@rubli.info>,
linux1@rubli.info, Laurent Pinchart <laurent.pinchart@skynet.be>
Cc: Linux and Kernel Video <video4linux-list@redhat.com>
Subject: Re: [PATCH] Support for write-only controls
Date: Mon, 14 Apr 2008 17:44:16 -0700 [thread overview]
Message-ID: <20080415004416.GA11071@plankton.ifup.org> (raw)
In-Reply-To: <20080213231244.GA15895@plankton.ifup.org>
On 15:12 Wed 13 Feb 2008, Brandon Philips wrote:
> On 01:01 Tue 18 Dec 2007, Martin Rubli wrote:
> > Thanks a lot for all your feedback and the constructive discussion and sorry
> > for the delay while I was without Internet on the weekend. I'll try to
> > summarize what we have so far:
> >
> > Write-only controls:
> >
> > It seems, everybody likes EACCES. Michael, maybe we could get some feedback
> > from you on this? It would be nice to change the spec, so that EACCES also
> > becomes the error for writing read-only controls--it seems appropriate. But
> > if for some reason we can't change that we should probably make the
> > write-only controls consistent and return EINVAL as well.
> >
> > Unusable controls due to device communication error:
> >
> > The easiest solution seems to be to set the V4L2_CTRL_FLAG_DISABLED flag as
> > was suggested. The documentation currently says "permanently disabled and
> > should be ignored by the application" which I think is exactly what applies
> > to the situation. The V4L2_CTRL_FLAG_NEXT_CTRL would still be respected by
> > drivers supporting the extended control enumeration, so no need to the spec
> > is required. But I would still add a short paragraph about the first part as
> > a guide for future implementations and a witness of this thread. ;-)
> >
> > As soon as everyone agrees on this, I will propose new patches. Let me know
> > what you think ...
>
Ping. I never saw patches come across for this.
Thanks,
Brandon
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
next prev parent reply other threads:[~2008-04-15 0:49 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <op.t3hn72busxcvug@mrubli-nb.am.logitech.com>
2008-02-13 23:12 ` [PATCH] Support for write-only controls Brandon Philips
2008-04-15 0:44 ` Brandon Philips [this message]
2008-04-16 9:45 ` Martin Rubli
2008-04-21 23:15 ` [PATCH] v4l2-spec: " 'Brandon Philips'
[not found] ` <20080415001932.52039d0f@gaivota>
2008-04-16 22:41 ` [PATCH] Support for " Laurent Pinchart
2008-04-16 22:39 ` Mauro Carvalho Chehab
2008-04-22 2:22 ` Brandon Philips
2008-04-22 8:40 ` Laurent Pinchart
2008-04-22 14:25 ` Brandon Philips
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=20080415004416.GA11071@plankton.ifup.org \
--to=brandon@ifup.org \
--cc=laurent.pinchart@skynet.be \
--cc=linux1@rubli.info \
--cc=v4l2-lists@rubli.info \
--cc=video4linux-list@redhat.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 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.