From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail.work.de ([212.12.32.20]) by www.linuxtv.org with esmtp (Exim 4.63) (envelope-from ) id 1KuXyV-0001X0-GM for linux-dvb@linuxtv.org; Mon, 27 Oct 2008 20:42:28 +0100 Message-ID: <49061983.3070800@gmail.com> Date: Mon, 27 Oct 2008 23:41:55 +0400 From: Manu Abraham MIME-Version: 1.0 To: Matthias Schwarzott References: <200810272023.23513.zzam@gentoo.org> In-Reply-To: <200810272023.23513.zzam@gentoo.org> Cc: linux-dvb@linuxtv.org Subject: Re: [linux-dvb] commit 9344:aa3a67b658e8 (DVB-Core update) breaks tuning of cx24123 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: linux-dvb-bounces@linuxtv.org Errors-To: linux-dvb-bounces+mchehab=infradead.org@linuxtv.org List-ID: Hi Mathias, Matthias Schwarzott wrote: > Hi Manu, hi Steven! > > It seems an update of dvb-core breaks tuning of cx24123. > After updating to latest v4l-dvb the nova-s plus card just did no longer lock > to any channel. So I bisected it, and found this commit: > > changeset: 9344:aa3a67b658e8 > parent: 9296:e2a8b9b9c294 > user: Manu Abraham > date: Tue Oct 14 23:34:07 2008 +0400 > summary: DVB-Core update > > http://linuxtv.org/hg/v4l-dvb/rev/aa3a67b658e8 > > It basically did update the dvb-kernel-thread and enhanced the code using > get_frontend_algo. > > The codepath when get_frontend_algo returns *_ALGO_HW stayed the same, only > one line got removed: params = &fepriv->parameter > > Just re-adding that line made my card working again. Either this was lost, or > the last two lines using "params" should also be converted to directly > use "&fepriv->parameters". True. In the port, the one line got missed out. Thanks for taking the time to look at it. BTW, i don't see any reason why cx24123 should be using HW_ALGO as it is a standard demodulator. When we have a dedicated microcontroller employed to do that check, we might like to use HW_ALGO, since it would simply handle it. Not in the case of standard demodulators. As an example i could say cinergyT2, dst etc would be candidates for HW_ALGO, where tuning is offloaded to a onboard microcontroller. But overall, the fix looks fine though, for the devices that make use of HW_ALGO Please do have a Signed-off-by line so that it can be applied. Reviewed-by: Manu Abraham Thanks, Manu > ------------------------------------------------------------------------ > > --- v4l-dvb.orig/linux/drivers/media/dvb/dvb-core/dvb_frontend.c > +++ v4l-dvb/linux/drivers/media/dvb/dvb-core/dvb_frontend.c > @@ -584,6 +584,7 @@ restart: > > if (fepriv->state & FESTATE_RETUNE) { > dprintk("%s: Retune requested, FESTATE_RETUNE\n", __func__); > + params = &fepriv->parameters; > fepriv->state = FESTATE_TUNED; > } > _______________________________________________ linux-dvb mailing list linux-dvb@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb