On Mon, 2012-08-06 at 21:46 +0300, Antti Palosaari wrote:
> On 08/03/2012 02:31 AM, Malcolm Priestley wrote:
> > On Thu, 2012-08-02 at 23:54 +0300, Antti Palosaari wrote:
> >> Moi Malcolm,
> >> Any idea why this seems to crash Kernel just when device is plugged?
> >>
> > Hi Antti
> >
> > Yes, there missing error handling when no firmware file found.
> >
> > It seems that this is more of a problem with udev-182+.
> >
> > However, so far udev-182 is only a problem on first ever plug.
> >
> > Regards
> >
> >
> > Malcolm
>
>
> Aug 6 20:56:34 localhost kernel: [19094.248540] LME2510(C): Firmware
> Status: 6 (44)
> Aug 6 20:56:34 localhost kernel: [19094.251541] LME2510(C): FRM No
> Firmware Found - please install
> Aug 6 20:56:34 localhost kernel: [19094.251559] usbcore: registered new
> interface driver LME2510C_DVB-S
>
> It is good to print needed fw name. I found it from the documentation,
> Documentation/dvb/lmedm04.txt.
Hi Antti,
Yes, this is a good idea to print the firmware it finds and then
selects.
>
> Could you drop me that firmware privately as I don't wish to install
> Windows drivers in order to extract it.
>
It would be interesting to see if your firmware is newer, my two boxes
are over a year old.
I as sure the firmware has a bug.
Regards
Malcolm
>
> Tested-by: Antti Palosaari <crope@xxxxxx>
>
> > Signed-off-by: Malcolm Priestley <tvboxspy@xxxxxxxxx>
> > ---
> > drivers/media/dvb/dvb-usb/lmedm04.c | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/drivers/media/dvb/dvb-usb/lmedm04.c b/drivers/media/dvb/dvb-usb/lmedm04.c
> > index 25d1031..26ba5bc 100644
> > --- a/drivers/media/dvb/dvb-usb/lmedm04.c
> > +++ b/drivers/media/dvb/dvb-usb/lmedm04.c
> > @@ -878,6 +878,10 @@ static int lme_firmware_switch(struct usb_device *udev, int cold)
> > fw_lme = fw_c_rs2000;
> > ret = request_firmware(&fw, fw_lme, &udev->dev);
> > dvb_usb_lme2510_firmware = TUNER_RS2000;
> > + if (ret == 0)
> > + break;
> > + info("FRM No Firmware Found - please install");
> > + cold_fw = 0;
> > break;
> > default:
> > fw_lme = fw_c_s7395;
> >
>
>
> regards
> Antti
>
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
