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.

Could you drop me that firmware privately as I don't wish to install Windows drivers in order to extract it.


Tested-by: Antti Palosaari <cr...@iki.fi>

Signed-off-by: Malcolm Priestley <tvbox...@gmail.com>
---
  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



--
http://palosaari.fi/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to