* em28xx: commit aab3125c43d8fecc7134e5f1e729fabf4dd196da broke HVR 900
@ 2013-03-21 8:33 Hans Verkuil
2013-03-21 10:03 ` Mauro Carvalho Chehab
0 siblings, 1 reply; 8+ messages in thread
From: Hans Verkuil @ 2013-03-21 8:33 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: linux-media
I tried to use my HVR 900 stick today and discovered that it no longer worked.
I traced it to commit aab3125c43d8fecc7134e5f1e729fabf4dd196da: "em28xx: add
support for registering multiple i2c buses".
The kernel messages for when it fails are:
Mar 21 09:26:54 telek kernel: [ 1393.446606] em28xx: New device WinTV HVR-900 @ 480 Mbps (2040:6502, interface 0, class 0)
Mar 21 09:26:54 telek kernel: [ 1393.446610] em28xx: Audio interface 0 found (Vendor Class)
Mar 21 09:26:54 telek kernel: [ 1393.446611] em28xx: Video interface 0 found: isoc
Mar 21 09:26:54 telek kernel: [ 1393.446612] em28xx: DVB interface 0 found: isoc
Mar 21 09:26:54 telek kernel: [ 1393.446979] em28xx: chip ID is em2882/3
Mar 21 09:26:54 telek kernel: [ 1393.587885] em2882/3 #0: i2c eeprom 00: 1a eb 67 95 40 20 02 65 d0 12 5c 03 82 1e 6a 18
Mar 21 09:26:54 telek kernel: [ 1393.587896] em2882/3 #0: i2c eeprom 10: 00 00 24 57 66 07 01 00 00 00 00 00 00 00 00 00
Mar 21 09:26:54 telek kernel: [ 1393.587905] em2882/3 #0: i2c eeprom 20: 46 00 01 00 f0 10 02 00 b8 00 00 00 5b e0 00 00
Mar 21 09:26:54 telek kernel: [ 1393.587913] em2882/3 #0: i2c eeprom 30: 00 00 20 40 20 6e 02 20 10 01 01 01 00 00 00 00
Mar 21 09:26:54 telek kernel: [ 1393.587921] em2882/3 #0: i2c eeprom 40: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
Mar 21 09:26:54 telek kernel: [ 1393.587929] em2882/3 #0: i2c eeprom 50: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
Mar 21 09:26:54 telek kernel: [ 1393.587937] em2882/3 #0: i2c eeprom 60: 00 00 00 00 00 00 00 00 00 00 18 03 34 00 30 00
Mar 21 09:26:54 telek kernel: [ 1393.587946] em2882/3 #0: i2c eeprom 70: 32 00 37 00 36 00 37 00 38 00 36 00 33 00 39 00
Mar 21 09:26:54 telek kernel: [ 1393.587954] em2882/3 #0: i2c eeprom 80: 00 00 1e 03 57 00 69 00 6e 00 54 00 56 00 20 00
Mar 21 09:26:54 telek kernel: [ 1393.587962] em2882/3 #0: i2c eeprom 90: 48 00 56 00 52 00 2d 00 39 00 30 00 30 00 00 00
Mar 21 09:26:54 telek kernel: [ 1393.587970] em2882/3 #0: i2c eeprom a0: 84 12 00 00 05 50 1a 7f d4 78 23 fa fd d0 28 89
Mar 21 09:26:54 telek kernel: [ 1393.587979] em2882/3 #0: i2c eeprom b0: ff 00 00 00 04 84 0a 00 01 01 20 77 00 40 af 7f
Mar 21 09:26:54 telek kernel: [ 1393.587987] em2882/3 #0: i2c eeprom c0: 11 f0 74 02 01 00 01 79 0b 00 00 00 00 00 00 00
Mar 21 09:26:54 telek kernel: [ 1393.587995] em2882/3 #0: i2c eeprom d0: 84 12 00 00 05 50 1a 7f d4 78 23 fa fd d0 28 89
Mar 21 09:26:54 telek kernel: [ 1393.588011] em2882/3 #0: i2c eeprom e0: ff 00 00 00 04 84 0a 00 01 01 20 77 00 40 af 7f
Mar 21 09:26:54 telek kernel: [ 1393.588020] em2882/3 #0: i2c eeprom f0: 11 f0 74 02 01 00 01 79 0b 00 00 00 00 00 00 00
Mar 21 09:26:54 telek kernel: [ 1393.588029] em2882/3 #0: EEPROM ID = 1a eb 67 95, EEPROM hash = 0x323f39dd
Mar 21 09:26:54 telek kernel: [ 1393.588031] em2882/3 #0: EEPROM info:
Mar 21 09:26:54 telek kernel: [ 1393.588032] em2882/3 #0: No audio on board.
Mar 21 09:26:54 telek kernel: [ 1393.588033] em2882/3 #0: 500mA max power
Mar 21 09:26:54 telek kernel: [ 1393.588034] em2882/3 #0: Table at offset 0x00, strings=0x0000, 0x0000, 0x0000
Mar 21 09:26:54 telek kernel: [ 1393.588036] em2882/3 #0: Identified as Hauppauge WinTV HVR 900 (R2) (card=18)
Mar 21 09:26:54 telek kernel: [ 1393.591653] tveeprom 12-0050: Hauppauge model 65018, rev B2C0, serial# 1146799
Mar 21 09:26:54 telek kernel: [ 1393.591655] tveeprom 12-0050: tuner model is Xceive XC3028 (idx 120, type 71)
Mar 21 09:26:54 telek kernel: [ 1393.591657] tveeprom 12-0050: TV standards PAL(B/G) PAL(I) PAL(D/D1/K) ATSC/DVB Digital (eeprom 0xd4)
Mar 21 09:26:54 telek kernel: [ 1393.591658] tveeprom 12-0050: audio processor is None (idx 0)
Mar 21 09:26:54 telek kernel: [ 1393.591659] tveeprom 12-0050: has radio
Mar 21 09:26:54 telek kernel: [ 1393.646629] tvp5150 12-005c: chip found @ 0xb8 (em2882/3 #0)
Mar 21 09:26:54 telek kernel: [ 1393.646632] tvp5150 12-005c: tvp5150am1 detected.
Mar 21 09:26:54 telek kernel: [ 1393.677877] tuner 12-0061: Tuner -1 found with type(s) Radio TV.
Mar 21 09:26:54 telek kernel: [ 1393.683854] xc2028 12-0061: creating new instance
Mar 21 09:26:54 telek kernel: [ 1393.683858] xc2028 12-0061: type set to XCeive xc2028/xc3028 tuner
Mar 21 09:26:54 telek kernel: [ 1393.684170] xc2028 12-0061: Loading 97 firmware images from xc3028-v27.fw, type: xc2028 firmware, ver 2.7
Mar 21 09:26:54 telek kernel: [ 1393.685287] em2882/3 #0: Config register raw data: 0xd0
Mar 21 09:26:54 telek kernel: [ 1393.686766] em2882/3 #0: AC97 vendor ID = 0xffffffff
Mar 21 09:26:54 telek kernel: [ 1393.687823] em2882/3 #0: AC97 features = 0x6a90
Mar 21 09:26:54 telek kernel: [ 1393.687826] em2882/3 #0: Empia 202 AC97 audio processor detected
Mar 21 09:26:54 telek kernel: [ 1393.882147] em2882/3 #0: v4l2 driver version 0.2.0
Mar 21 09:26:54 telek kernel: [ 1393.882772] xc2028 12-0061: Loading firmware for type=BASE F8MHZ MTS (7), id 0000000000000000.
Mar 21 09:26:56 telek kernel: [ 1395.953247] MTS (4), id 00000000000000ff:
Mar 21 09:26:56 telek kernel: [ 1395.953262] xc2028 12-0061: Loading firmware for type=MTS (4), id 0000000100000007.
Mar 21 09:26:57 telek kernel: [ 1396.308931] em2882/3 #0: V4L2 video device registered as video0
Mar 21 09:26:57 telek kernel: [ 1396.308935] em2882/3 #0: V4L2 VBI device registered as vbi0
Mar 21 09:26:57 telek kernel: [ 1396.310318] em2882/3 #0: analog set to isoc mode.
Mar 21 09:26:57 telek kernel: [ 1396.310320] em2882/3 #0: dvb set to isoc mode.
Mar 21 09:26:57 telek kernel: [ 1396.311485] usbcore: registered new interface driver em28xx
Mar 21 09:26:57 telek kernel: [ 1396.317648] em28xx-audio.c: probing for em28xx Audio Vendor Class
Mar 21 09:26:57 telek kernel: [ 1396.317651] em28xx-audio.c: Copyright (C) 2006 Markus Rechberger
Mar 21 09:26:57 telek kernel: [ 1396.317652] em28xx-audio.c: Copyright (C) 2007-2011 Mauro Carvalho Chehab
Mar 21 09:26:57 telek kernel: [ 1396.328976] Em28xx: Initialized (Em28xx Audio Extension) extension
Mar 21 09:26:57 telek kernel: [ 1396.542517] xc2028 12-0061: attaching existing instance
Mar 21 09:26:57 telek kernel: [ 1396.542521] xc2028 12-0061: type set to XCeive xc2028/xc3028 tuner
Mar 21 09:26:57 telek kernel: [ 1396.542523] em2882/3 #0: em2882/3 #0/2: xc3028 attached
Mar 21 09:26:57 telek kernel: [ 1396.542525] DVB: registering new adapter (em2882/3 #0)
Mar 21 09:26:57 telek kernel: [ 1396.542548] usb 5-2: DVB: registering adapter 0 frontend 0 (Micronas DRXD DVB-T)...
Mar 21 09:26:57 telek kernel: [ 1396.546126] em2882/3 #0: Successfully loaded em28xx-dvb
Mar 21 09:26:57 telek kernel: [ 1396.546131] Em28xx: Initialized (Em28xx dvb Extension) extension
Mar 21 09:26:57 telek kernel: [ 1396.547833] xc2028 12-0061: Error on line 1293: -19
Mar 21 09:26:57 telek kernel: [ 1396.592046] Registered IR keymap rc-hauppauge
Mar 21 09:26:57 telek kernel: [ 1396.594254] input: em28xx IR (em2882/3 #0) as /devices/pci0000:00/0000:00:1c.2/0000:07:00.0/usb5/5-2/rc/rc0/input27
Mar 21 09:26:57 telek kernel: [ 1396.595200] rc0: em28xx IR (em2882/3 #0) as /devices/pci0000:00/0000:00:1c.2/0000:07:00.0/usb5/5-2/rc/rc0
Mar 21 09:26:57 telek kernel: [ 1396.595594] Em28xx: Initialized (Em28xx Input Extension) extension
Mar 21 09:26:59 telek kernel: [ 1398.314392] xc2028 12-0061: i2c input error: rc = -19 (should be 2)
Mar 21 09:26:59 telek kernel: [ 1398.316642] xc2028 12-0061: i2c input error: rc = -19 (should be 2)
Mar 21 09:26:59 telek kernel: [ 1398.336509] xc2028 12-0061: i2c input error: rc = -19 (should be 2)
Mar 21 09:26:59 telek kernel: [ 1398.337631] xc2028 12-0061: i2c input error: rc = -19 (should be 2)
Mar 21 09:27:02 telek kernel: [ 1401.460224] xc2028 12-0061: Loading firmware for type=BASE F8MHZ MTS (7), id 0000000000000000.
Mar 21 09:27:02 telek kernel: [ 1401.461603] xc2028 12-0061: i2c output error: rc = -19 (should be 64)
Mar 21 09:27:02 telek kernel: [ 1401.461612] xc2028 12-0061: -19 returned from send
Mar 21 09:27:02 telek kernel: [ 1401.461617] xc2028 12-0061: Error -22 while loading base firmware
Mar 21 09:27:02 telek kernel: [ 1401.512052] xc2028 12-0061: Loading firmware for type=BASE F8MHZ MTS (7), id 0000000000000000.
Mar 21 09:27:02 telek kernel: [ 1401.512639] xc2028 12-0061: i2c output error: rc = -19 (should be 64)
Mar 21 09:27:02 telek kernel: [ 1401.512642] xc2028 12-0061: -19 returned from send
Mar 21 09:27:02 telek kernel: [ 1401.512643] xc2028 12-0061: Error -22 while loading base firmware
Mar 21 09:27:02 telek kernel: [ 1401.563027] xc2028 12-0061: Loading firmware for type=BASE F8MHZ MTS (7), id 0000000000000000.
Mar 21 09:27:02 telek kernel: [ 1401.563587] xc2028 12-0061: i2c output error: rc = -19 (should be 64)
Mar 21 09:27:02 telek kernel: [ 1401.563590] xc2028 12-0061: -19 returned from send
Mar 21 09:27:02 telek kernel: [ 1401.563592] xc2028 12-0061: Error -22 while loading base firmware
Mar 21 09:27:02 telek kernel: [ 1401.614035] xc2028 12-0061: Loading firmware for type=BASE F8MHZ MTS (7), id 0000000000000000.
Mar 21 09:27:02 telek kernel: [ 1401.614599] xc2028 12-0061: i2c output error: rc = -19 (should be 64)
Mar 21 09:27:02 telek kernel: [ 1401.614602] xc2028 12-0061: -19 returned from send
Mar 21 09:27:02 telek kernel: [ 1401.614603] xc2028 12-0061: Error -22 while loading base firmware
Mar 21 09:27:02 telek kernel: [ 1401.665047] xc2028 12-0061: Loading firmware for type=BASE F8MHZ MTS (7), id 0000000000000000.
Mar 21 09:27:02 telek kernel: [ 1401.665608] xc2028 12-0061: i2c output error: rc = -19 (should be 64)
Mar 21 09:27:02 telek kernel: [ 1401.665610] xc2028 12-0061: -19 returned from send
Mar 21 09:27:02 telek kernel: [ 1401.665612] xc2028 12-0061: Error -22 while loading base firmware
Mar 21 09:27:02 telek kernel: [ 1401.716048] xc2028 12-0061: Loading firmware for type=BASE F8MHZ MTS (7), id 0000000000000000.
Mar 21 09:27:02 telek kernel: [ 1401.716580] xc2028 12-0061: i2c output error: rc = -19 (should be 64)
Mar 21 09:27:02 telek kernel: [ 1401.716583] xc2028 12-0061: -19 returned from send
Mar 21 09:27:02 telek kernel: [ 1401.716584] xc2028 12-0061: Error -22 while loading base firmware
Mar 21 09:27:02 telek kernel: [ 1401.767050] xc2028 12-0061: Loading firmware for type=BASE F8MHZ MTS (7), id 0000000000000000.
Mar 21 09:27:02 telek kernel: [ 1401.767659] xc2028 12-0061: i2c output error: rc = -19 (should be 64)
Mar 21 09:27:02 telek kernel: [ 1401.767662] xc2028 12-0061: -19 returned from send
Mar 21 09:27:02 telek kernel: [ 1401.767664] xc2028 12-0061: Error -22 while loading base firmware
Mar 21 09:27:02 telek kernel: [ 1401.818018] xc2028 12-0061: Loading firmware for type=BASE F8MHZ MTS (7), id 0000000000000000.
Mar 21 09:27:02 telek kernel: [ 1401.818523] xc2028 12-0061: i2c output error: rc = -19 (should be 64)
Mar 21 09:27:02 telek kernel: [ 1401.818525] xc2028 12-0061: -19 returned from send
Mar 21 09:27:02 telek kernel: [ 1401.818527] xc2028 12-0061: Error -22 while loading base firmware
Mar 21 09:27:02 telek kernel: [ 1401.869048] xc2028 12-0061: Loading firmware for type=BASE F8MHZ MTS (7), id 0000000000000000.
Mar 21 09:27:02 telek kernel: [ 1401.869625] xc2028 12-0061: i2c output error: rc = -19 (should be 64)
Mar 21 09:27:02 telek kernel: [ 1401.869628] xc2028 12-0061: -19 returned from send
Mar 21 09:27:02 telek kernel: [ 1401.869629] xc2028 12-0061: Error -22 while loading base firmware
Mar 21 09:27:06 telek kernel: [ 1405.202225] xc2028 12-0061: Error on line 1293: -19
And this is how it should look like:
Mar 21 09:24:49 telek kernel: [ 1268.313138] em28xx: New device WinTV HVR-900 @ 480 Mbps (2040:6502, interface 0, class 0)
Mar 21 09:24:49 telek kernel: [ 1268.313141] em28xx: Audio interface 0 found (Vendor Class)
Mar 21 09:24:49 telek kernel: [ 1268.313143] em28xx: Video interface 0 found: isoc
Mar 21 09:24:49 telek kernel: [ 1268.313144] em28xx: DVB interface 0 found: isoc
Mar 21 09:24:49 telek kernel: [ 1268.313483] em28xx: chip ID is em2882/3
Mar 21 09:24:49 telek kernel: [ 1268.451504] em2882/3 #0: i2c eeprom 00: 1a eb 67 95 40 20 02 65 d0 12 5c 03 82 1e 6a 18
Mar 21 09:24:49 telek kernel: [ 1268.451515] em2882/3 #0: i2c eeprom 10: 00 00 24 57 66 07 01 00 00 00 00 00 00 00 00 00
Mar 21 09:24:49 telek kernel: [ 1268.451523] em2882/3 #0: i2c eeprom 20: 46 00 01 00 f0 10 02 00 b8 00 00 00 5b e0 00 00
Mar 21 09:24:49 telek kernel: [ 1268.451531] em2882/3 #0: i2c eeprom 30: 00 00 20 40 20 6e 02 20 10 01 01 01 00 00 00 00
Mar 21 09:24:49 telek kernel: [ 1268.451539] em2882/3 #0: i2c eeprom 40: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
Mar 21 09:24:49 telek kernel: [ 1268.451547] em2882/3 #0: i2c eeprom 50: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
Mar 21 09:24:49 telek kernel: [ 1268.451556] em2882/3 #0: i2c eeprom 60: 00 00 00 00 00 00 00 00 00 00 18 03 34 00 30 00
Mar 21 09:24:49 telek kernel: [ 1268.451564] em2882/3 #0: i2c eeprom 70: 32 00 37 00 36 00 37 00 38 00 36 00 33 00 39 00
Mar 21 09:24:49 telek kernel: [ 1268.451572] em2882/3 #0: i2c eeprom 80: 00 00 1e 03 57 00 69 00 6e 00 54 00 56 00 20 00
Mar 21 09:24:49 telek kernel: [ 1268.451580] em2882/3 #0: i2c eeprom 90: 48 00 56 00 52 00 2d 00 39 00 30 00 30 00 00 00
Mar 21 09:24:49 telek kernel: [ 1268.451588] em2882/3 #0: i2c eeprom a0: 84 12 00 00 05 50 1a 7f d4 78 23 fa fd d0 28 89
Mar 21 09:24:49 telek kernel: [ 1268.451596] em2882/3 #0: i2c eeprom b0: ff 00 00 00 04 84 0a 00 01 01 20 77 00 40 af 7f
Mar 21 09:24:49 telek kernel: [ 1268.451605] em2882/3 #0: i2c eeprom c0: 11 f0 74 02 01 00 01 79 0b 00 00 00 00 00 00 00
Mar 21 09:24:49 telek kernel: [ 1268.451613] em2882/3 #0: i2c eeprom d0: 84 12 00 00 05 50 1a 7f d4 78 23 fa fd d0 28 89
Mar 21 09:24:49 telek kernel: [ 1268.451621] em2882/3 #0: i2c eeprom e0: ff 00 00 00 04 84 0a 00 01 01 20 77 00 40 af 7f
Mar 21 09:24:49 telek kernel: [ 1268.451629] em2882/3 #0: i2c eeprom f0: 11 f0 74 02 01 00 01 79 0b 00 00 00 00 00 00 00
Mar 21 09:24:49 telek kernel: [ 1268.451638] em2882/3 #0: EEPROM ID = 1a eb 67 95, EEPROM hash = 0x323f39dd
Mar 21 09:24:49 telek kernel: [ 1268.451640] em2882/3 #0: EEPROM info:
Mar 21 09:24:49 telek kernel: [ 1268.451641] em2882/3 #0: No audio on board.
Mar 21 09:24:49 telek kernel: [ 1268.451642] em2882/3 #0: 500mA max power
Mar 21 09:24:49 telek kernel: [ 1268.451644] em2882/3 #0: Table at offset 0x00, strings=0x0000, 0x0000, 0x0000
Mar 21 09:24:49 telek kernel: [ 1268.451645] em2882/3 #0: Identified as Hauppauge WinTV HVR 900 (R2) (card=18)
Mar 21 09:24:49 telek kernel: [ 1268.455336] tveeprom 12-0050: Hauppauge model 65018, rev B2C0, serial# 1146799
Mar 21 09:24:49 telek kernel: [ 1268.455338] tveeprom 12-0050: tuner model is Xceive XC3028 (idx 120, type 71)
Mar 21 09:24:49 telek kernel: [ 1268.455340] tveeprom 12-0050: TV standards PAL(B/G) PAL(I) PAL(D/D1/K) ATSC/DVB Digital (eeprom 0xd4)
Mar 21 09:24:49 telek kernel: [ 1268.455342] tveeprom 12-0050: audio processor is None (idx 0)
Mar 21 09:24:49 telek kernel: [ 1268.455343] tveeprom 12-0050: has radio
Mar 21 09:24:49 telek kernel: [ 1268.511612] tvp5150 12-005c: chip found @ 0xb8 (em2882/3 #0)
Mar 21 09:24:49 telek kernel: [ 1268.511616] tvp5150 12-005c: tvp5150am1 detected.
Mar 21 09:24:49 telek kernel: [ 1268.543048] tuner 12-0061: Tuner -1 found with type(s) Radio TV.
Mar 21 09:24:49 telek kernel: [ 1268.550908] xc2028 12-0061: creating new instance
Mar 21 09:24:49 telek kernel: [ 1268.550911] xc2028 12-0061: type set to XCeive xc2028/xc3028 tuner
Mar 21 09:24:49 telek kernel: [ 1268.551206] xc2028 12-0061: Loading 97 firmware images from xc3028-v27.fw, type: xc2028 firmware, ver 2.7
Mar 21 09:24:49 telek kernel: [ 1268.552265] em2882/3 #0: Config register raw data: 0xd0
Mar 21 09:24:49 telek kernel: [ 1268.553675] em2882/3 #0: AC97 vendor ID = 0xffffffff
Mar 21 09:24:49 telek kernel: [ 1268.554349] em2882/3 #0: AC97 features = 0x6a90
Mar 21 09:24:49 telek kernel: [ 1268.554351] em2882/3 #0: Empia 202 AC97 audio processor detected
Mar 21 09:24:49 telek kernel: [ 1268.739185] em2882/3 #0: v4l2 driver version 0.2.0
Mar 21 09:24:49 telek kernel: [ 1268.773016] xc2028 12-0061: Loading firmware for type=BASE F8MHZ MTS (7), id 0000000000000000.
Mar 21 09:24:51 telek kernel: [ 1270.903855] MTS (4), id 00000000000000ff:
Mar 21 09:24:51 telek kernel: [ 1270.903870] xc2028 12-0061: Loading firmware for type=MTS (4), id 0000000100000007.
Mar 21 09:24:52 telek kernel: [ 1271.252677] em2882/3 #0: V4L2 video device registered as video0
Mar 21 09:24:52 telek kernel: [ 1271.252681] em2882/3 #0: V4L2 VBI device registered as vbi0
Mar 21 09:24:52 telek kernel: [ 1271.253425] em2882/3 #0: analog set to isoc mode.
Mar 21 09:24:52 telek kernel: [ 1271.253427] em2882/3 #0: dvb set to isoc mode.
Mar 21 09:24:52 telek kernel: [ 1271.254699] usbcore: registered new interface driver em28xx
Mar 21 09:24:52 telek kernel: [ 1271.260336] em28xx-audio.c: probing for em28xx Audio Vendor Class
Mar 21 09:24:52 telek kernel: [ 1271.260339] em28xx-audio.c: Copyright (C) 2006 Markus Rechberger
Mar 21 09:24:52 telek kernel: [ 1271.260340] em28xx-audio.c: Copyright (C) 2007-2011 Mauro Carvalho Chehab
Mar 21 09:24:52 telek kernel: [ 1271.263753] xc2028 12-0061: Error on line 1293: -19
Mar 21 09:24:52 telek kernel: [ 1271.267562] Em28xx: Initialized (Em28xx Audio Extension) extension
Mar 21 09:24:52 telek kernel: [ 1271.337649] xc2028 12-0061: attaching existing instance
Mar 21 09:24:52 telek kernel: [ 1271.337653] xc2028 12-0061: type set to XCeive xc2028/xc3028 tuner
Mar 21 09:24:52 telek kernel: [ 1271.337654] em2882/3 #0: em2882/3 #0/2: xc3028 attached
Mar 21 09:24:52 telek kernel: [ 1271.337657] DVB: registering new adapter (em2882/3 #0)
Mar 21 09:24:52 telek kernel: [ 1271.337676] usb 5-2: DVB: registering adapter 0 frontend 0 (Micronas DRXD DVB-T)...
Mar 21 09:24:52 telek kernel: [ 1271.341549] em2882/3 #0: Successfully loaded em28xx-dvb
Mar 21 09:24:52 telek kernel: [ 1271.341553] Em28xx: Initialized (Em28xx dvb Extension) extension
Mar 21 09:24:52 telek kernel: [ 1271.384073] Registered IR keymap rc-hauppauge
Mar 21 09:24:52 telek kernel: [ 1271.385939] input: em28xx IR (em2882/3 #0) as /devices/pci0000:00/0000:00:1c.2/0000:07:00.0/usb5/5-2/rc/rc0/input26
Mar 21 09:24:52 telek kernel: [ 1271.386941] rc0: em28xx IR (em2882/3 #0) as /devices/pci0000:00/0000:00:1c.2/0000:07:00.0/usb5/5-2/rc/rc0
Mar 21 09:24:52 telek kernel: [ 1271.387303] Em28xx: Initialized (Em28xx Input Extension) extension
Mar 21 09:24:54 telek kernel: [ 1273.974437] xc2028 12-0061: i2c input error: rc = -19 (should be 2)
Mar 21 09:24:54 telek kernel: [ 1273.975531] xc2028 12-0061: i2c input error: rc = -19 (should be 2)
Mar 21 09:24:54 telek kernel: [ 1273.997453] xc2028 12-0061: i2c input error: rc = -19 (should be 2)
Mar 21 09:24:54 telek kernel: [ 1273.998584] xc2028 12-0061: i2c input error: rc = -19 (should be 2)
Mar 21 09:24:59 telek kernel: [ 1278.657044] xc2028 12-0061: Loading firmware for type=BASE F8MHZ MTS (7), id 0000000000000000.
Mar 21 09:25:01 telek kernel: [ 1280.764698] MTS (4), id 00000000000000ff:
Mar 21 09:25:01 telek kernel: [ 1280.764713] xc2028 12-0061: Loading firmware for type=MTS (4), id 0000000100000007.
(Note: I've omitted irrelevant call stack traces due to a WARN_ON in vb2_queue)
Regards,
Hans
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: em28xx: commit aab3125c43d8fecc7134e5f1e729fabf4dd196da broke HVR 900
2013-03-21 8:33 em28xx: commit aab3125c43d8fecc7134e5f1e729fabf4dd196da broke HVR 900 Hans Verkuil
@ 2013-03-21 10:03 ` Mauro Carvalho Chehab
2013-03-21 10:19 ` Hans Verkuil
2013-03-21 15:34 ` Hans Verkuil
0 siblings, 2 replies; 8+ messages in thread
From: Mauro Carvalho Chehab @ 2013-03-21 10:03 UTC (permalink / raw)
To: Hans Verkuil; +Cc: linux-media
Em Thu, 21 Mar 2013 09:33:41 +0100
Hans Verkuil <hverkuil@xs4all.nl> escreveu:
> I tried to use my HVR 900 stick today and discovered that it no longer worked.
> I traced it to commit aab3125c43d8fecc7134e5f1e729fabf4dd196da: "em28xx: add
> support for registering multiple i2c buses".
>
> The kernel messages for when it fails are:
...
> Mar 21 09:26:57 telek kernel: [ 1396.542517] xc2028 12-0061: attaching existing instance
> Mar 21 09:26:57 telek kernel: [ 1396.542521] xc2028 12-0061: type set to XCeive xc2028/xc3028 tuner
> Mar 21 09:26:57 telek kernel: [ 1396.542523] em2882/3 #0: em2882/3 #0/2: xc3028 attached
...
> Mar 21 09:26:57 telek kernel: [ 1396.547833] xc2028 12-0061: Error on line 1293: -19
Probably, the I2C speed is wrong. I noticed a small bug on this patch.
The following patch should fix it. Could you please test?
--
Regards,
Mauro
[PATCH] em28xx: fix I2C write to register 06
Signed-off-by: Mauro Carvalho Chehab
diff --git a/drivers/media/usb/em28xx/em28xx-i2c.c b/drivers/media/usb/em28xx/em28xx-i2c.c
index de9b208..dd1f2de 100644
--- a/drivers/media/usb/em28xx/em28xx-i2c.c
+++ b/drivers/media/usb/em28xx/em28xx-i2c.c
@@ -295,7 +295,9 @@ static int em28xx_i2c_xfer(struct i2c_adapter *i2c_adap,
dev->cur_i2c_bus |= EM2874_I2C_SECONDARY_BUS_SELECT;
else
dev->cur_i2c_bus &= ~EM2874_I2C_SECONDARY_BUS_SELECT;
- em28xx_write_reg(dev, EM28XX_R06_I2C_CLK, dev->cur_i2c_bus);
+ em28xx_write_reg_bits(dev, EM28XX_R06_I2C_CLK,
+ dev->cur_i2c_bus,
+ EM2874_I2C_SECONDARY_BUS_SELECT);
dev->cur_i2c_bus = bus;
}
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: em28xx: commit aab3125c43d8fecc7134e5f1e729fabf4dd196da broke HVR 900
2013-03-21 10:03 ` Mauro Carvalho Chehab
@ 2013-03-21 10:19 ` Hans Verkuil
2013-03-21 15:34 ` Hans Verkuil
1 sibling, 0 replies; 8+ messages in thread
From: Hans Verkuil @ 2013-03-21 10:19 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: linux-media
On Thu 21 March 2013 11:03:27 Mauro Carvalho Chehab wrote:
> Em Thu, 21 Mar 2013 09:33:41 +0100
> Hans Verkuil <hverkuil@xs4all.nl> escreveu:
>
> > I tried to use my HVR 900 stick today and discovered that it no longer worked.
> > I traced it to commit aab3125c43d8fecc7134e5f1e729fabf4dd196da: "em28xx: add
> > support for registering multiple i2c buses".
> >
> > The kernel messages for when it fails are:
> ...
> > Mar 21 09:26:57 telek kernel: [ 1396.542517] xc2028 12-0061: attaching existing instance
> > Mar 21 09:26:57 telek kernel: [ 1396.542521] xc2028 12-0061: type set to XCeive xc2028/xc3028 tuner
> > Mar 21 09:26:57 telek kernel: [ 1396.542523] em2882/3 #0: em2882/3 #0/2: xc3028 attached
> ...
> > Mar 21 09:26:57 telek kernel: [ 1396.547833] xc2028 12-0061: Error on line 1293: -19
>
> Probably, the I2C speed is wrong. I noticed a small bug on this patch.
> The following patch should fix it. Could you please test?
>
>
I'll try to test this later today, otherwise it will be tomorrow.
Regards,
Hans
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: em28xx: commit aab3125c43d8fecc7134e5f1e729fabf4dd196da broke HVR 900
2013-03-21 10:03 ` Mauro Carvalho Chehab
2013-03-21 10:19 ` Hans Verkuil
@ 2013-03-21 15:34 ` Hans Verkuil
2013-03-21 16:23 ` Mauro Carvalho Chehab
2013-03-21 18:21 ` Frank Schäfer
1 sibling, 2 replies; 8+ messages in thread
From: Hans Verkuil @ 2013-03-21 15:34 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: linux-media
On Thu March 21 2013 11:03:27 Mauro Carvalho Chehab wrote:
> Em Thu, 21 Mar 2013 09:33:41 +0100
> Hans Verkuil <hverkuil@xs4all.nl> escreveu:
>
> > I tried to use my HVR 900 stick today and discovered that it no longer worked.
> > I traced it to commit aab3125c43d8fecc7134e5f1e729fabf4dd196da: "em28xx: add
> > support for registering multiple i2c buses".
> >
> > The kernel messages for when it fails are:
> ...
> > Mar 21 09:26:57 telek kernel: [ 1396.542517] xc2028 12-0061: attaching existing instance
> > Mar 21 09:26:57 telek kernel: [ 1396.542521] xc2028 12-0061: type set to XCeive xc2028/xc3028 tuner
> > Mar 21 09:26:57 telek kernel: [ 1396.542523] em2882/3 #0: em2882/3 #0/2: xc3028 attached
> ...
> > Mar 21 09:26:57 telek kernel: [ 1396.547833] xc2028 12-0061: Error on line 1293: -19
>
> Probably, the I2C speed is wrong. I noticed a small bug on this patch.
> The following patch should fix it. Could you please test?
No luck, it didn't help.
Regards,
Hans
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: em28xx: commit aab3125c43d8fecc7134e5f1e729fabf4dd196da broke HVR 900
2013-03-21 15:34 ` Hans Verkuil
@ 2013-03-21 16:23 ` Mauro Carvalho Chehab
2013-03-22 10:20 ` Hans Verkuil
2013-03-21 18:21 ` Frank Schäfer
1 sibling, 1 reply; 8+ messages in thread
From: Mauro Carvalho Chehab @ 2013-03-21 16:23 UTC (permalink / raw)
To: Hans Verkuil; +Cc: linux-media
Em Thu, 21 Mar 2013 16:34:13 +0100
Hans Verkuil <hverkuil@xs4all.nl> escreveu:
> On Thu March 21 2013 11:03:27 Mauro Carvalho Chehab wrote:
> > Em Thu, 21 Mar 2013 09:33:41 +0100
> > Hans Verkuil <hverkuil@xs4all.nl> escreveu:
> >
> > > I tried to use my HVR 900 stick today and discovered that it no longer worked.
> > > I traced it to commit aab3125c43d8fecc7134e5f1e729fabf4dd196da: "em28xx: add
> > > support for registering multiple i2c buses".
> > >
> > > The kernel messages for when it fails are:
> > ...
> > > Mar 21 09:26:57 telek kernel: [ 1396.542517] xc2028 12-0061: attaching existing instance
> > > Mar 21 09:26:57 telek kernel: [ 1396.542521] xc2028 12-0061: type set to XCeive xc2028/xc3028 tuner
> > > Mar 21 09:26:57 telek kernel: [ 1396.542523] em2882/3 #0: em2882/3 #0/2: xc3028 attached
> > ...
> > > Mar 21 09:26:57 telek kernel: [ 1396.547833] xc2028 12-0061: Error on line 1293: -19
> >
> > Probably, the I2C speed is wrong. I noticed a small bug on this patch.
> > The following patch should fix it. Could you please test?
>
> No luck, it didn't help.
On a first glance, I've no idea what else is different for devices with
just one I2C bus, like HVR-900.
Could you send me an USB sniff dump with the kernel that works and with
the broken kernel, with this patch applied?
The parsing tools are under v4l-utils contrib/ dir.
The first step is to check the usbmon interface for capture, with:
$ ./parse_tcpdump_log.pl --list-devices
usbmon4 ==> 020f (level 2)
usbmon2 ==> USB2.0 Hub (level 1)
usbmon1 ==> WinTV HVR-930C (level 7)
Then, you can run the parser to capture the data:
# ./parse_tcpdump_log.pl --device usbmon1 | ./em28xx/parse_em28xx.pl
Please load the em28xx driver only after starting the parser, as we want
to see what the driver is doing during the initialization.
Thanks!
Mauro
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: em28xx: commit aab3125c43d8fecc7134e5f1e729fabf4dd196da broke HVR 900
2013-03-21 15:34 ` Hans Verkuil
2013-03-21 16:23 ` Mauro Carvalho Chehab
@ 2013-03-21 18:21 ` Frank Schäfer
2013-03-21 19:05 ` Mauro Carvalho Chehab
1 sibling, 1 reply; 8+ messages in thread
From: Frank Schäfer @ 2013-03-21 18:21 UTC (permalink / raw)
To: Hans Verkuil; +Cc: Mauro Carvalho Chehab, Linux Media Mailing List
Am 21.03.2013 16:34, schrieb Hans Verkuil:
> On Thu March 21 2013 11:03:27 Mauro Carvalho Chehab wrote:
>> Em Thu, 21 Mar 2013 09:33:41 +0100
>> Hans Verkuil <hverkuil@xs4all.nl> escreveu:
>>
>>> I tried to use my HVR 900 stick today and discovered that it no longer worked.
>>> I traced it to commit aab3125c43d8fecc7134e5f1e729fabf4dd196da: "em28xx: add
>>> support for registering multiple i2c buses".
>>>
>>> The kernel messages for when it fails are:
>> ...
>>> Mar 21 09:26:57 telek kernel: [ 1396.542517] xc2028 12-0061: attaching existing instance
>>> Mar 21 09:26:57 telek kernel: [ 1396.542521] xc2028 12-0061: type set to XCeive xc2028/xc3028 tuner
>>> Mar 21 09:26:57 telek kernel: [ 1396.542523] em2882/3 #0: em2882/3 #0/2: xc3028 attached
>> ...
>>> Mar 21 09:26:57 telek kernel: [ 1396.547833] xc2028 12-0061: Error on line 1293: -19
>> Probably, the I2C speed is wrong. I noticed a small bug on this patch.
>> The following patch should fix it. Could you please test?
> No luck, it didn't help.
I can't test it at the moment, but I have the same device and also
tested at least two other (non-em28xx) devices with the xc3028 during
the last weeks.
With all devices, I'm often getting i2c errors like this, too. Usually
only a few not affecting the device operation, but sometimes the log is
full of them and I have to unplugg the device and plug it in again to
make it work.
So something might be wrong with the xc3028 driver...
Maybe you just had bad luck when you tested this patch ?
Regards,
Frank
>
> Regards,
>
> Hans
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: em28xx: commit aab3125c43d8fecc7134e5f1e729fabf4dd196da broke HVR 900
2013-03-21 18:21 ` Frank Schäfer
@ 2013-03-21 19:05 ` Mauro Carvalho Chehab
0 siblings, 0 replies; 8+ messages in thread
From: Mauro Carvalho Chehab @ 2013-03-21 19:05 UTC (permalink / raw)
To: Frank Schäfer; +Cc: Hans Verkuil, Linux Media Mailing List
Em Thu, 21 Mar 2013 19:21:02 +0100
Frank Schäfer <fschaefer.oss@googlemail.com> escreveu:
> Am 21.03.2013 16:34, schrieb Hans Verkuil:
> > On Thu March 21 2013 11:03:27 Mauro Carvalho Chehab wrote:
> >> Em Thu, 21 Mar 2013 09:33:41 +0100
> >> Hans Verkuil <hverkuil@xs4all.nl> escreveu:
> >>
> >>> I tried to use my HVR 900 stick today and discovered that it no longer worked.
> >>> I traced it to commit aab3125c43d8fecc7134e5f1e729fabf4dd196da: "em28xx: add
> >>> support for registering multiple i2c buses".
> >>>
> >>> The kernel messages for when it fails are:
> >> ...
> >>> Mar 21 09:26:57 telek kernel: [ 1396.542517] xc2028 12-0061: attaching existing instance
> >>> Mar 21 09:26:57 telek kernel: [ 1396.542521] xc2028 12-0061: type set to XCeive xc2028/xc3028 tuner
> >>> Mar 21 09:26:57 telek kernel: [ 1396.542523] em2882/3 #0: em2882/3 #0/2: xc3028 attached
> >> ...
> >>> Mar 21 09:26:57 telek kernel: [ 1396.547833] xc2028 12-0061: Error on line 1293: -19
> >> Probably, the I2C speed is wrong. I noticed a small bug on this patch.
> >> The following patch should fix it. Could you please test?
> > No luck, it didn't help.
>
> I can't test it at the moment, but I have the same device and also
> tested at least two other (non-em28xx) devices with the xc3028 during
> the last weeks.
> With all devices, I'm often getting i2c errors like this, too. Usually
> only a few not affecting the device operation, but sometimes the log is
> full of them and I have to unplugg the device and plug it in again to
> make it work.
> So something might be wrong with the xc3028 driver...
I doubt that this is the cause. It could be some race condition between
the main em28xx driver and some other one (we had this issue in the past).
Also, there's no situation at xc3028 driver where ENODEV is returned.
So, this is likely at em28xx-i2c, where this error is returned when the
physical device doesn't answer to an I2C xfer.
Regards,
Mauro
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: em28xx: commit aab3125c43d8fecc7134e5f1e729fabf4dd196da broke HVR 900
2013-03-21 16:23 ` Mauro Carvalho Chehab
@ 2013-03-22 10:20 ` Hans Verkuil
0 siblings, 0 replies; 8+ messages in thread
From: Hans Verkuil @ 2013-03-22 10:20 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: linux-media
[-- Attachment #1: Type: Text/Plain, Size: 2567 bytes --]
Hi Mauro,
I've attached the dumps:
ok.txt is tested with commit aab3125c43d8fecc7134e5f1e729fabf4dd196da^ (i.e.
the commit right before the failing commit).
fail.txt is tested with commit aab3125c43d8fecc7134e5f1e729fabf4dd196da and
fail-with-patch.txt is tested with the same commit + your i2c speed patch.
Test sequence: start the sniffer, insert the stick, run v4l2-ctl -f 61.25 to
change the frequency. All on the same PC. Note that this PC is different from
the one I used yesterday, so it is not specific to one unlucky hardware
combination.
Regards,
Hans
On Thu March 21 2013 17:23:44 Mauro Carvalho Chehab wrote:
> Em Thu, 21 Mar 2013 16:34:13 +0100
> Hans Verkuil <hverkuil@xs4all.nl> escreveu:
>
> > On Thu March 21 2013 11:03:27 Mauro Carvalho Chehab wrote:
> > > Em Thu, 21 Mar 2013 09:33:41 +0100
> > > Hans Verkuil <hverkuil@xs4all.nl> escreveu:
> > >
> > > > I tried to use my HVR 900 stick today and discovered that it no longer worked.
> > > > I traced it to commit aab3125c43d8fecc7134e5f1e729fabf4dd196da: "em28xx: add
> > > > support for registering multiple i2c buses".
> > > >
> > > > The kernel messages for when it fails are:
> > > ...
> > > > Mar 21 09:26:57 telek kernel: [ 1396.542517] xc2028 12-0061: attaching existing instance
> > > > Mar 21 09:26:57 telek kernel: [ 1396.542521] xc2028 12-0061: type set to XCeive xc2028/xc3028 tuner
> > > > Mar 21 09:26:57 telek kernel: [ 1396.542523] em2882/3 #0: em2882/3 #0/2: xc3028 attached
> > > ...
> > > > Mar 21 09:26:57 telek kernel: [ 1396.547833] xc2028 12-0061: Error on line 1293: -19
> > >
> > > Probably, the I2C speed is wrong. I noticed a small bug on this patch.
> > > The following patch should fix it. Could you please test?
> >
> > No luck, it didn't help.
>
> On a first glance, I've no idea what else is different for devices with
> just one I2C bus, like HVR-900.
>
> Could you send me an USB sniff dump with the kernel that works and with
> the broken kernel, with this patch applied?
>
> The parsing tools are under v4l-utils contrib/ dir.
>
> The first step is to check the usbmon interface for capture, with:
>
> $ ./parse_tcpdump_log.pl --list-devices
> usbmon4 ==> 020f (level 2)
> usbmon2 ==> USB2.0 Hub (level 1)
> usbmon1 ==> WinTV HVR-930C (level 7)
>
> Then, you can run the parser to capture the data:
>
> # ./parse_tcpdump_log.pl --device usbmon1 | ./em28xx/parse_em28xx.pl
>
> Please load the em28xx driver only after starting the parser, as we want
> to see what the driver is doing during the initialization.
>
> Thanks!
> Mauro
>
[-- Attachment #2: usbdump.tar.bz2 --]
[-- Type: application/x-bzip-compressed-tar, Size: 12331 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2013-03-22 10:20 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-03-21 8:33 em28xx: commit aab3125c43d8fecc7134e5f1e729fabf4dd196da broke HVR 900 Hans Verkuil
2013-03-21 10:03 ` Mauro Carvalho Chehab
2013-03-21 10:19 ` Hans Verkuil
2013-03-21 15:34 ` Hans Verkuil
2013-03-21 16:23 ` Mauro Carvalho Chehab
2013-03-22 10:20 ` Hans Verkuil
2013-03-21 18:21 ` Frank Schäfer
2013-03-21 19:05 ` Mauro Carvalho Chehab
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.