From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from smtp2.dnainternet.fi ([87.94.96.112]) by www.linuxtv.org with esmtp (Exim 4.63) (envelope-from ) id 1K26UR-00008H-JP for linux-dvb@linuxtv.org; Fri, 30 May 2008 17:26:03 +0200 Received: from localhost.localdomain (dyn3-82-128-187-102.psoas.suomi.net [82.128.187.102]) by smtp2.dnainternet.fi (Postfix) with ESMTP id E4253CD76 for ; Fri, 30 May 2008 18:25:23 +0300 (EEST) Message-ID: <48401C63.1010601@iki.fi> Date: Fri, 30 May 2008 18:25:23 +0300 From: Antti Palosaari MIME-Version: 1.0 CC: linux-dvb@linuxtv.org References: <1212079844.26238.22.camel@rommel.snap.tv> <483EED5A.7080200@iki.fi> <48400833.60909@gmail.com> <48401099.7040908@iki.fi> <484014AC.3090603@gmail.com> In-Reply-To: <484014AC.3090603@gmail.com> Subject: Re: [linux-dvb] Oops in tda10023 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: e9hack wrote: > Antti Palosaari schrieb: >>> I think the oops occurs, because tda10023_writereg() fails in tda10023_attach(). If >>> tda10023_writereg fails, an error message is printed. In this case, >>> state->frontend.dvb->num is accessed, but it isn't initialized yet. >> hmm, I see the problem now. Originally state was initialized before >> tda10023_writereg() was called but after I did some changes this is not >> done anymore. And when writereg() fails in attach some reason it oops. > > It wasn't introduced with your modifications. The frontend.dvb part is initialized after > the attach call. tda10023_writereg() must check, if state->frontend.dvb is initialized or > not. yeah, you are correct. I looked through all frontend drivers and only TDA10021 and TDA10023 was using dvb->num in writereg. Anyhow, TDA10021 is not affected because it does not write in attach. Antti -- http://palosaari.fi/ _______________________________________________ linux-dvb mailing list linux-dvb@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb