Re: [GIT PATCHES FOR 2.6.40] Fixes

[Hans Verkuil <hverkuil@xs4all.nl>]

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