Re: V4L1 compatibility broken for VIDIOCGTUNER with radio
On Thu, Dec 3, 2009 at 8:10 PM, hermann pitton wrote: > > Am Donnerstag, den 03.12.2009, 21:04 -0200 schrieb Herton Ronaldo > Krzesinski: >> 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. >> > > Yes, for sure. It's our fault. > > Seems we had an "internal server error" :( > > I came across it by accident. > >> The only other issue I'm aware of is that radio is broken since guessed >> 8 weeks on my tuners, only realized when testing on enabling external >> active antenna voltage for DVB-T on a/some 310i. > > I did the bisect with some delay and Hans marked the fix with priority > "high", but we missed to point Mike at it for stable explicitly. > > Mike, please review and forward. > > Sorry, > Hermann Already done. It's queued for 2.6.31.7 The delay was due to the standard bureaucracy ... nothing we haven't seen before -- hopefully the distros will take in in once it's merged to stable, which should probably be within the next week or two. Thanks, Mike -- 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
Re: V4L1 compatibility broken for VIDIOCGTUNER with radio
Am Donnerstag, den 03.12.2009, 21:04 -0200 schrieb Herton Ronaldo Krzesinski: > 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. > Yes, for sure. It's our fault. Seems we had an "internal server error" :( I came across it by accident. > The only other issue I'm aware of is that radio is broken since guessed > 8 weeks on my tuners, only realized when testing on enabling external > active antenna voltage for DVB-T on a/some 310i. I did the bisect with some delay and Hans marked the fix with priority "high", but we missed to point Mike at it for stable explicitly. Mike, please review and forward. Sorry, Hermann -- 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
Re: V4L1 compatibility broken for VIDIOCGTUNER with radio
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 majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: V4L1 compatibility broken for VIDIOCGTUNER with radio
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 Maybe it was not ported to stable? Cheers, Hermann -- 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
V4L1 compatibility broken for VIDIOCGTUNER with radio
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 -- 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