Re: [PATCH] [resent] cx88-dvb: Fix Oops in case i2c bus failed to register

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

 



On Freitag, 24. Oktober 2008, Darron Broad wrote:
> In message <200810241247.08480.zzam@xxxxxxxxxx>, Matthias Schwarzott wrote:
>
> Hiya.
>
> >Hi!
> >
> >@Mauro: Please pull this patch
> >There already is an report at kernel bugzilla about this issue:
> >http://bugzilla.kernel.org/show_bug.cgi?id=9455
> >
> >When enabling extra checks for the i2c-bus of cx88 based cards by
> >loading i2c_algo_bit with bit_test=1 this may trigger an oops
> >when loading cx88_dvb.
> >
> >This is caused by the extra check code that detects that the
> >sda-line is stuck high and thus does not register the i2c-bus.
> >
> >cx88-dvb however does not check if the i2c-bus is valid and just
> >uses core->i2c_adap to attach dvb frontend modules.
> >This leads to an oops at the first call to i2c_transfer:
> >
> ># modprobe i2c_algo_bit bit_test=1
> ># modprobe cx8802
>
> Thanks for this information. If you don't mind, would you let
> me utilise this fix in another way?
>
> I have been auditing MFE additions and I need to change
> something elsewhere and can see an alternative to
> to bailing out in dvb_register?
>
> Thanks

I think Mauro already applied it, but well, if you want it in another place, 
the check could also be placed in cx8802_dvb_probe around the line where 
vp3054_i2c_probe is called.

Regards
Matthias


_______________________________________________
linux-dvb mailing list
linux-dvb@xxxxxxxxxxx
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb

[Index of Archives]     [Linux Media]     [Video 4 Linux]     [Asterisk]     [Samba]     [Xorg]     [Xfree86]     [Linux USB]

  Powered by Linux