From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mx3.redhat.com (mx3.redhat.com [172.16.48.32]) by int-mx1.corp.redhat.com (8.13.1/8.13.1) with ESMTP id m2KKgbJk017413 for ; Thu, 20 Mar 2008 16:42:37 -0400 Received: from mailout09.sul.t-online.de (mailout09.sul.t-online.de [194.25.134.84]) by mx3.redhat.com (8.13.8/8.13.8) with ESMTP id m2KKfiBY017955 for ; Thu, 20 Mar 2008 16:41:44 -0400 Message-ID: <47E2CBEF.3090609@t-online.de> Date: Thu, 20 Mar 2008 21:41:19 +0100 From: Hartmut Hackmann MIME-Version: 1.0 To: Mauro Carvalho Chehab References: <47E060EB.5040207@t-online.de> <47E190CF.9050904@t-online.de> <20080319193832.643bf8a0@gaivota> <47E1BCAF.80208@t-online.de> <20080319224222.581d7b85@gaivota> In-Reply-To: <20080319224222.581d7b85@gaivota> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: Michael Krufky , Linux and Kernel Video , LInux DVB Subject: Re: [RFC] TDA8290 / TDA827X with LNA: testers wanted List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: video4linux-list-bounces@redhat.com Errors-To: video4linux-list-bounces@redhat.com List-ID: HI, Mauro Mauro Carvalho Chehab schrieb: > On Thu, 20 Mar 2008 02:23:59 +0100 > Hartmut Hackmann wrote: > >>> On your patch, you're just returning, if dev=NULL, at saa7134 callback function. IMO, the correct would be to >>> print an error message and return. Also, we should discover why dev is being >>> null there (I'll try to identify here - the reason - yet, I can't really test, >>> since the saa7134 boards I have don't need any callback. >> That's not the point. In the call in tda827x.c tda827xa_lna_gain(), the argument >> did not point to the saa7134_dev structure as the function expected. I added >> the check for NULL because only at the very first call, the pointer is still >> not valid. I did not check this carefully but i guess this is a matter of the >> initilization sequence of the data structures. IMHO yes, we should understand this >> sometime but this does not have priority because i am sure that the NULL pointer >> occurs only during initialization. > > This is caused by a patch conflict between hybrid redesign and the merge of > xc3028 support. The enclosed experimental patch fixes the tuner_callback > argument, on linux/drivers/media/dvb/frontends/tda827x.c. > It should also fix the priv argument on saa7134_tuner_callback(). I can't test > the saa7134 part here, due to the lack of a saa7134 hardware that needs a > callback. > > The patch also intends to make xc3028 easier to use. That part is still not > fully working. I should finish this patch tomorrow. > >>>>> I still need to send a patchset to Linus, after testing compilation >>>>> (unfortunately, I had to postpone, since I need first to free some >>>>> hundreds of Mb on my HD on my /home, to allow kernel compilation). >>>>> Hopefully, I'll have some time tomorrow for doing a "housekeeping". >>>>> >>>> Unfortunately, i deleted you mails describing what went to linux and i don't >>>> have the RC source here :-( >>> You may take a look on master branch on my git tree. I'm about to forward him a >>> series of patches. Hopefully, 2GB free space will be enough for a full kernel >>> compilation. I'll discover soon... >>> >> Jep. Meanwhile Michael confirmed that the problem is not in mainstream, >> so there is no reason to hurry. > > Yes. > >> But we should have a bigger audience for my latest changes, so i will send >> you a pull request in a minute. > > Could you please test my patch first? Having the same arguments for all > callback functions avoid future mistakes. > > --- > [RFC] Fix tuner_callback for tda827x > > Signed-off-by Mauro Carvalho Chehab > Your patch does not completely apply for me, it fails in cx88-dvb.c I had a close look and found that we are going in the same direction. - The change in tda827x is the same as i did. - In saa7134-cards.c your patch is right. My version just worked by accident. I corrected this in my repository. By the way: the dev pointer is NULL during initialization is gone. I tested again and things work for me. I would recommend the following: - You pull from my repository (sent you the request yesterday) - You apply the patch *except* the changes in tda827x.c, saa7134-cards.c and saa7134-dvb.c. Afterwards we should be fine. My other changes to tda827x and saa7134-dvb.c are not only cosmetic. It merged the _lna_gain functions for analog and dvb and adapt the data structures. What do you think? I will be out from friday to monday. Best regards Hartmut -- video4linux-list mailing list Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe https://www.redhat.com/mailman/listinfo/video4linux-list