Re: [PATCH] [BUG] Re: dvb_usb_lmedm04 crash Kernel (rs2000)

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

 



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 <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



--
http://palosaari.fi/
--
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


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux