On Tuesday, May 24, 2011 08:28:32 Hans Verkuil wrote:
> On Tuesday, May 24, 2011 03:42:32 Mauro Carvalho Chehab wrote:
> > Em 23-05-2011 08:06, Hans Verkuil escreveu:
> > > Hi Mauro,
> > >
> > > Here are a few fixes: the first fixes a bug in the wl12xx drivers (I hope
> > > Matti's
> > > email is still correct). The second fixes a few DocBook validation
> > > errors, and
> > > the last fixes READ_ONLY and GRABBED handling in the control framework.
> > >
> > > Regards,
> > >
> > > Hans
> > >
> > > The following changes since commit
> > > 87cf028f3aa1ed51fe29c36df548aa714dc7438f:
> > >
> > > [media] dm1105: GPIO handling added, I2C on GPIO added, LNB control
> > > through GPIO reworked (2011-05-21 11:10:28 -0300)
> > >
> > > are available in the git repository at:
> > > ssh://linuxtv.org/git/hverkuil/media_tree.git fixes
> > >
> > > Hans Verkuil (3):
> > > wl12xx: g_volatile_ctrl fix: wrong field set.
> > > Media DocBook: fix validation errors.
> >
> > The two above are fixes...
> >
> > > v4l2-ctrls: drivers should be able to ignore READ_ONLY and GRABBED
> > > flags
> >
> > But this one is a change at the behaviour. I need to analyse it better. The
> > idea
> > of a "read only" control that it is not read only seems too weird on my
> > tired eyes.
>
> It's read-only for *applications*. But if you have a read-only control that
> reflects a driver state, then it should be possible for a *driver* to change
> it. It's something that is needed for the upcoming Flash and HDMI APIs.
>
> The userspace behavior does not change.
>
> BTW, if you prefer to move this last patch to 2.6.41, then that's OK by me.
> It's not really necessary for 2.6.40.
I'm going to move this patch to 2.6.41, so there is no need for you to review
this.
I'll include it in another patch series I'm working on.
Regards,
Hans
>
> Regards,
>
> Hans
>
> >
> > I'll take a more careful look on it tomorrow.
> >
> > Thanks,
> > Mauro.
> >
> > >
> > > Documentation/DocBook/dvb/dvbproperty.xml | 5 ++-
> > > Documentation/DocBook/v4l/subdev-formats.xml | 10 ++--
> > > drivers/media/radio/radio-wl1273.c | 2 +-
> > > drivers/media/radio/wl128x/fmdrv_v4l2.c | 2 +-
> > > drivers/media/video/v4l2-ctrls.c | 59
> > > +++++++++++++++++---------
> > > 5 files changed, 50 insertions(+), 28 deletions(-)
> > > --
> > > 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
> >
> >
> --
> 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
>
>
--
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
