Em Wed, 29 Jul 2009 08:09:31 +0200 Hans Verkuil <hverkuil@xxxxxxxxx> escreveu: > On Wednesday 29 July 2009 06:57:30 Mauro Carvalho Chehab wrote: > > Em Tue, 28 Jul 2009 20:07:53 -0400 > > > > acano@xxxxxxxxxxx escreveu: > > > On Mon, Jul 27, 2009 at 09:28:11PM -0300, Mauro Carvalho Chehab wrote: > > > > Hi Acano, > > > > > > Tested-by: Angelo Cano <acano@xxxxxxxxxxx> > > > > > > works great > > > > Good! > > > > > > > + /*FIXME hack to unmute usb audio stream */ > > > > > + em28xx_set_ctrl(dev, ctrl); > > > > > > > > Hmm... this function were removed. In thesis, you shouldn't need to > > > > do anything to unmute. > > > > > > I still need it, see attachment. > > > > > > > Could you please try the enclosed patch and see if this is enough to > > > > fix for Plextor? If so, please send me a Tested-by: tag for me to add > > > > it at 2.6.31 fix patches. > > > > > > Like I said the patch works great, and also solves my audio volume > > > problem. With your patch the volume is set to a sane value > > > (presumably 0db) and the distortion/clipping is gone. > > > > > > Thanks man. The volume problem was driving me crazy. > > > > Ah, yes, there's a missing mute/unmute issue there. Instead of using your > > code, I opted to duplicate part of ac97_set_ctrl code there. > > > > I opted to have a small duplicated code, but, IMO, it is now clearer to > > see why we still need to call em28xx_audio_analog_set(). You will notice > > that I've rearranged the place where I update volume and mute. The > > rationale is that v4l2_device_call_all() might eventually change a value > > for volume/mute. > > > > Another reason is that, IMO, v4l2_device_call_all() should return values. > > In the specific case of volume/mute, if the user tries to specify a value > > outside the range, the -ERANGE should be returned. > > > > I've already committed the patches at the tree. Please double-check. > > > > Hans, > > > > we need to fix the returned error value for v4l2_device_call_all(). I > > know that this is an old issue that weren't changed by v4l dev/subdev > > conversion, but now it is easier for us to fix. The idea here is to be > > sure that, if a sub-driver with a proper handling for a function returns > > an error value, this would be returned by v4l2_device_call_all(). Maybe > > we'll need to adjust some things at the sub-drivers. > > Use v4l2_device_call_until_err instead of v4l2_device_call_all. That macro > checks for errors returned from the subdevs. It doesn't work as expected. If I use it for queryctl, for example, it returns an empty set of controls. If I use it for g_ctrl, it returns: error 22 getting ctrl Brightness error 22 getting ctrl Contrast error 22 getting ctrl Saturation error 22 getting ctrl Hue error 22 getting ctrl Volume error 22 getting ctrl Balance error 22 getting ctrl Bass error 22 getting ctrl Treble error 22 getting ctrl Mute error 22 getting ctrl Loudness The issue here is we need something that discards errors for non-implemented controls. As the sub-drivers are returning -EINVAL for non-implemented controls (and probably other stuff that aren't implemented there), the function will not work for some ioctls. The proper fix seems to elect an error condition to be returned by driver when a function is not implemented, and such errors to be discarded by the macro. It seems that the proper error code for such case is this one: #define ENOSYS 38 /* Function not implemented */ Cheers, Mauro -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
