Re: V4L1 compatibility broken for VIDIOCGTUNER with radio

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Em Qui 03 Dez 2009, às 19:54:29, hermann pitton escreveu:
> Hi,
> 
> Am Donnerstag, den 03.12.2009, 16:56 -0200 schrieb Herton Ronaldo
> Krzesinski:
> > Hi,
> > 
> > After commit 9bedc7f ("V4L/DVB (12429): v4l2-ioctl: fix G_STD and G_PARM 
> > default handlers"), radio software using V4L1 stopped to work on a saa7134 
> > card, a git bisect pointed to this commit introducing the regression. All 
> > VIDIOCGTUNER calls on a v4l1 application are returning -EINVAL after this 
> > commit.
> > 
> > Investigating the issue, it turns out that v4l1_compat_get_tuner calls 
> > VIDIOC_G_STD ioctl, but as it is a radio device (saa7134-radio) it now is 
> > returning -EINVAL to user space applications which are being confused about 
> > this.
> > 
> > May be VIDIOC_G_STD change in the commit above should be reverted, or 
> > v4l1_compat_get_tuner changed to not return error with G_STD, or not call 
> > G_STD ioctl for a radio device?
> > 
> > --
> > []'s
> > Herton
> 
> it was fixed here.
> 
> http://linuxtv.org/hg/v4l-dvb/rev/58ecda742a70

Indeed, thanks for the pointer. I forgot to check latest v4l1-compat.c /o\

> 
> Maybe it was not ported to stable?

Not on latest stable (2.6.31.6), perhaps it should be forwarded.

> 
> Cheers,
> Hermann

--
[]'s
Herton
--
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

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux