Re: [PATCH] cxd2820r: fix gpio null pointer dereference

2017-01-17 Thread Chris Rankin
On 16 January 2017 at 23:40, Antti Palosaari <cr...@iki.fi> wrote: > Chris and Håkan, test please without Kconfig CONFIG_GPIOLIB option. I cannot > test it properly as there seems to quite many drivers selecting this option > by default. Works here :-) Tested-by: Chris Rankin <

[OOPS] EM28xx with 4.9.x kernel

2017-01-14 Thread Chris Rankin
Hi, This oops has appeared for my PCTV T2 290e adapter when I switched to the 4.9.x kernel. It does not happen with the 4.8.x kernel. I believe the problem is that "priv" is NULL in cxd2820r_gpio(). Jan 13 11:19:45 endgame kernel: em28174 #0: EEPROM ID = 26 00 01 00, EEPROM hash = 0x1eb936d2

Oops with 4.9.x kernel and em28xx DVB device

2017-01-14 Thread Chris Rankin
Hi, This oops has appeared for my PCTV T2 290e adapter when I switched to the 4.9.x kernel. It does not happen with the 4.8.x kernel. Jan 13 11:19:45 endgame kernel: em28174 #0: EEPROM ID = 26 00 01 00, EEPROM hash = 0x1eb936d2 Jan 13 11:19:45 endgame kernel: em28174 #0: EEPROM info: Jan 13

Hi linux

2016-09-29 Thread Chris Rankin
hiya linux http://wdentertainmentinc.com/manner.php?sets=xkwqz2nzt03eb59 Chris -- 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

VRC-1100 Vista MCE Remote Control: 05a4:9881

2015-11-22 Thread Chris Rankin
Hi, I've bought a VRC-110 MCE IR remote control (USB IDs 05a4:9881) to replace a broken Hauppauge RC5 remote, and have *finally* managed to get it working with VDR. However, this has involved doing things like writing a userspace "muxing" device to unify the IR's separate keyboard and mouse

Trying to enable RC6 IR for PCTV T2 290e

2015-11-15 Thread Chris Rankin
Hi, My Hauppauge RC5 remote control finally broke, and the PCTV T2 290e's native RC5 remote control isn't suitable for VDR, and so I bought a cheap RC6 remote as a replacement. The unit I chose was the Ortek VRC-1100 Vista MCE Remote Control, USB ID 05a4:9881. I've been able to switch the PCTV

Re: Trying to enable RC6 IR for PCTV T2 290e

2015-11-15 Thread Chris Rankin
Hi, I've dug a bit deeper and discovered that the reason the em28xx_info(...) lines that I'd added to em2874_polling_getkey() weren't appearing is because I'd loaded the wrong version of the em28xx-rc module! (Doh!) The polling function *is* being called regularly, but em28xx_ir_handle_key()

Re: Trying to enable RC6 IR for PCTV T2 290e

2015-11-15 Thread Chris Rankin
> How are you "switching back to RC5"? I use the command "ir-keytable -p rc-5" or "ir-keytable -p rc-6" to switch between IR protocols, which does seem to invoke the em2874_ir_change_protocol() function. I'm not sure that I have a suitable RC6 keymap for this IR, and was expecting to have to

Re: Trying to enable RC6 IR for PCTV T2 290e

2015-11-15 Thread Chris Rankin
ease? Thanks, Chris On Sun, Nov 15, 2015 at 10:46 PM, Mauro Carvalho Chehab <mche...@osg.samsung.com> wrote: > Em Sun, 15 Nov 2015 22:18:48 + > Chris Rankin <ranki...@googlemail.com> escreveu: > >> > How are you "switching back to RC5"? >>

Re: Updated DVB-T tables - where to send them?

2014-03-18 Thread Chris Rankin
your distro hasn't updated. I did a pull request into this github repo https://github.com/oliv3r/dtv-scan-tables  Si. On 17 March 2014 23:44, Chris Rankin ranki...@yahoo.com wrote: Hi, The DVB-T initial tuning information for Crystal Palace in the UK is completely obsolete - despite my

Updated DVB-T tables - where to send them?

2014-03-17 Thread Chris Rankin
Hi, The DVB-T initial tuning information for Crystal Palace in the UK is completely obsolete - despite my two attempts to submit an updated version over the YEARS. Where is the best place to send this information, please? Thanks, Chris -- To unsubscribe from this list: send the line

Re: PWC webcam and setpwc tool no longer working with 3.12.11 kernel

2014-02-21 Thread Chris Rankin
On Friday, 21 February 2014, 9:47, Hans de Goede hdego...@redhat.com wrote: This is likely caused by the camera being plugged into a usb-bus which already is used by other reserved-bandwidth devices such as mice, keyboard, usb soundcards, etc. If possible USB-3 ports are preferred over

PWC webcam and setpwc tool no longer working with 3.12.11 kernel

2014-02-20 Thread Chris Rankin
Hi, I have an old Logitech webcam, with USB IDs 046d:08b3. When I try to use this camera now, I see this error in the dmesg log: [ 2883.852464] pwc: isoc_init() submit_urb 0 failed with error -28 This error is apparently ENOSPC, which made me suspect that I was trying to use a mode that

[3.12.5] Regression with PCTV 290e DVB-T2 adapter.

2013-12-15 Thread Chris Rankin
Hi, I have just tested my PCTV 290e DVB-T2 adapter with 3.12.5 and discovered that it fails with logs of messages like these: [11720.780975] __tda18271_write_regs: [7-0060|M] ERROR: idx = 0x5, len = 1, i2c_transfer returned: -19 [11720.788726] tda18271_init: [7-0060|M] error -19 on line 832

Very verbose message about em28174 chip.

2013-07-28 Thread Chris Rankin
Hi, I plugged my PCTV 290e device into my newly compiled 3.10.3 kernel today, and found this message in the dmesg log. [  511.041412] usb 10-4: new high-speed USB device number 3 using ehci-pci [  511.216218] em28xx: New device PCTV Systems PCTV 290e @ 480 Mbps (2013:024f, interface 0, class

Re: Very verbose message about em28174 chip.

2013-07-28 Thread Chris Rankin
- Original Message - From: Devin Heitmueller dheitmuel...@kernellabs.com The amount of output is not inconsistent with most other linuxtv drivers though. It's the EEPROM dump that really caught my eye: 16+ lines of pure WTF?. Cheers, Chris -- To unsubscribe from this list: send the

Re: 3.9.2 kernel - IR / em28xx_rc broken?

2013-05-20 Thread Chris Rankin
-EINVAL; This is against 3.9.3. Signed-off-by: Chris Rankin ranki...@yahoo.com No, this patch is wrong. Updating ir-rc_type with the new value of *rc_type is correct. Well, it restores 3.8 behaviour, i.e. em28xx not clobbering its RC5 configuration when RC core subsequently calls

Re: 3.9.2 kernel - IR / em28xx_rc broken?

2013-05-20 Thread Chris Rankin
- Original Message - And this is me calling ir-keytable: [ 2183.812407] em28xx #0: Changing protocol: rc_type=1 So with 3.8 the same happens as with 3.9. Yes, that does appear to be part of the RC core ABI. Well, if ir-keycode / the RC core requests RC_BIT_UNKNOWN, they get

Re: 3.9.2 kernel - IR / em28xx_rc broken?

2013-05-20 Thread Chris Rankin
- Original Message - If I had to guess, I would say you should check your rc_maps.cfg / keytable. ;) This is unchanged between 3.8.x and 3.9.x, and so is correct by definition. Kernel Upgrades Do Not Break Userspace. Cheers, Chris -- To unsubscribe from this list: send the line

Re: 3.9.2 kernel - IR / em28xx_rc broken?

2013-05-20 Thread Chris Rankin
- Original Message - No, just because it didn't change it isn't automatically correct. ;) It's Fedora 18's haupp keytable - so it must be correct :-). -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More

Re: 3.9.2 kernel - IR / em28xx_rc broken?

2013-05-19 Thread Chris Rankin
- Original Message - em28xx_ir_change_protocol() should be called at least twice: First from em28xx_ir_init() with RC_BIT_UNKNOWN (initial configuration) and later from the RC core with RC_BIT_RC5. Can you confirm that ? Yes, it does appear to be called twice: first with *rc_type=1

Re: 3.9.2 kernel - IR / em28xx_rc broken?

2013-05-19 Thread Chris Rankin
- Original Message - Hmm... that's weird. Are you sure about that ? Is this really a 3.9.2 vanilla kernel ? Quite sure, although it turns out that there's a bit more to it. Here is the dmesg output with my debugging messages in: [ 6263.496794] em28174 #0: Calling

Re: 3.9.2 kernel - IR / em28xx_rc broken?

2013-05-19 Thread Chris Rankin
- Original Message - And this seems to reset the protocol back to unknown. (But I need to use this other remote control to use VDR - the PCTV one just doesn't have enough buttons). Ok, then it seems to be no em28xx issue. What happens with kernel 3.8 ? Does ir-keytable trigger an

Re: 3.9.2 kernel - IR / em28xx_rc broken?

2013-05-19 Thread Chris Rankin
- Original Message - What happens with kernel 3.8 ? Does ir-keytable trigger an em28xx_ir_change_protocol() call there, too, but with type=8 ? Or is this call missing ? This is the dmesg output from 3.8, with an extra ex28xx_info() call at the start of em28xx_ir_change_protocol():

Re: 3.9.2 kernel - IR / em28xx_rc broken?

2013-05-19 Thread Chris Rankin
; +    return 0; } else {     *rc_type = ir-rc_type;     return -EINVAL; This is against 3.9.3. Signed-off-by: Chris Rankin ranki...@yahoo.com -- 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

3.9.2 kernel - IR / em28xx_rc broken?

2013-05-18 Thread Chris Rankin
Hi, I have a PCTV 290e DVB2 adapter (em28xx, em28xx_dvb, em28xx_rc, cxd2820r), and I have just discovered that the IR remote control has stopped working with VDR when using a vanilla 3.9.2 kernel. Downgrading the kernel to 3.8.12 fixes things again. (Switching to my old DVB NOVA-T2 device

[3.9.2] Broken IR with em28xx?

2013-05-18 Thread Chris Rankin
Hi, Further to my original email, here is the dmesg log from 3.9.2 when inserting my 290e adapter: [ 1273.581835] usb 10-4: new high-speed USB device number 6 using ehci-pci [ 1273.704508] em28xx: New device PCTV Systems PCTV 290e @ 480 Mbps (2013:024f, interface 0, class 0) [ 1273.712167]

Re: 3.9.2 kernel - IR / em28xx_rc broken?

2013-05-18 Thread Chris Rankin
- Original Message - Am 18.05.2013 15:57, schrieb Chris Rankin: I have a PCTV 290e DVB2 adapter (em28xx, em28xx_dvb, em28xx_rc, cxd2820r), and I have just discovered that the IR remote control has stopped working with VDR when using a vanilla 3.9.2 kernel. Downgrading the kernel

Re: 3.9.2 kernel - IR / em28xx_rc broken?

2013-05-18 Thread Chris Rankin
- Original Message - For the em28xx driver: em28xx-input.c: em28xx_ir_work() is called every 100ms     calls em28xx_ir_handle_key()         - calls ir-get_key() which is em2874_polling_getkey() in case of your device         - reports the detected key via rc_keydown() through the RC

Re: 3.9.2 kernel - IR / em28xx_rc broken?

2013-05-18 Thread Chris Rankin
- Original Message - For the em28xx driver: em28xx-input.c: em28xx_ir_work() is called every 100ms     calls em28xx_ir_handle_key()         - calls ir-get_key() which is em2874_polling_getkey() in case of your device         - reports the detected key via rc_keydown() through the

Tuning file for Crystal Palace, UK (post digital switch-over)

2012-04-18 Thread Chris Rankin
Hi, The Crystal Palace transmitter has now completed its digital switch-over, so here's the updated tuning file. Cheers, Chris #-- # file automatically generated by w_scan #

DVB ioctl FE_GET_EVENT behaviour broken in 3.3

2012-04-06 Thread Chris Rankin
The reason that DVB playback with xine is broken in 3.3 is that the userspace semantics of FE_GET_EVENT have changed. Xine tunes into a DVB channel as follows: * discards stale frontend events by calling FE_GET_EVENT until there is none left. * calls FE_SET_FRONTEND with the new frequency. *

Re: DVB ioctl FE_GET_EVENT behaviour broken in 3.3

2012-04-06 Thread Chris Rankin
Before LOCK you cannot know many parameters at all and frequency also can be changed a little bit during tuning process (ZigZag tuning algo). But surely the point of calling poll() on the front end's descriptor is either to be notified once the tuning algorithm has locked, or to be told that

Re: DVB ioctl FE_GET_EVENT behaviour broken in 3.3

2012-04-06 Thread Chris Rankin
The problem is that the following line was deleted from the FE_SET_FRONTEND ioctl logic: fepriv-parameters_out = fepriv-parameters_in; The following dirty little patch restores the correct behaviour: --- dvb_frontend.c.orig 2012-04-06 13:28:43.0 +0100 +++ dvb_frontend.c

[REGRESSION] Linux 3.3 DVB userspace ABI broken for xine (FE_SET_FRONTEND)

2012-04-06 Thread Chris Rankin
Hi, The following commit has broken the DVB ABI for xine: http://git.linuxtv.org/linux-2.6.git/commitdiff/e399ce77e6e8f0ff2e0b8ef808cbb88fc824c610 author: Mauro Carvalho Chehab mche...@redhat.com Sun, 1 Jan 2012 19:11:16 + (16:11 -0300) committer: Mauro Carvalho Chehab mche...@redhat.com

[PATCH] Linux 3.3 DVB userspace ABI broken for xine (FE_SET_FRONTEND)

2012-04-06 Thread Chris Rankin
In fact, the following patch works for me. Signed-off-by: Chris Rankin ranki...@yahoo.com --- linux-3.3/drivers/media/dvb/dvb-core/dvb_frontend.c.orig 2012-04-06 20:16:02.0 +0100 +++ linux-3.3/drivers/media/dvb/dvb-core/dvb_frontend.c 2012-04-06 21:17:38.0 +0100 @@ -1831,6

[PATCH v2] Linux 3.3 DVB userspace ABI broken for xine (FE_SET_FRONTEND)

2012-04-06 Thread Chris Rankin
I've had a closer look at the commit which caused the regression and it looks like there were two places where fepriv-parameters_in was assigned to fepriv-parameters_out. So I've updated my patch accordingly. Cheers, Chris Signed-off-by: Chris Rankin ranki...@yahoo.com --- linux-3.3/drivers

Re: UVC video output problem with 3.3.1 kernel

2012-04-05 Thread Chris Rankin
Does this happen in any resolution ? All resolutions except 960x720: in this resolution I only get a tiny strip of image across the top of the window. Cheers, Chris -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org

Is xine DVB broken with Linux 3.3.1?

2012-04-04 Thread Chris Rankin
 Hi, Is anyone else having difficulty watching DVB-T with xine and a 3.3.1 kernel? I have tried the following with several different USB adapters: 1) Plug in the device 2) Launch xine. 3) Select DVB-T channel. and each time, the results are the same. Namely, xine refuses to acknowledge any

Is xine DVB broken with Linux 3.3.1?

2012-04-04 Thread Chris Rankin
I have just checked the 3.2.14 kernel, and it seems unaffected by this issue. So I'm guessing it was introduced recently in the 3.3 kernel. Cheers, Chris -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo

UVC video output problem with 3.3.1 kernel

2012-04-04 Thread Chris Rankin
Hi, I have a UVC video device, which lsusb describes as: 046d:0992 Logitech, Inc. QuickCam Communicate Deluxe With the 3.3.1 kernel, the bottom 3rd of the video window displayed by guvcview is completely black. This happens whenever I select either BGR3 or RGB3 as the video output format.

[PATCH] em28xx: pre-allocate DVB isoc transfer buffers

2012-02-19 Thread Chris Rankin
Gianluca, One quick comment about your patch; I've noticed that you've declared two new GPL only symbols: EXPORT_SYMBOL_GPL(em28xx_capture_start); EXPORT_SYMBOL_GPL(em28xx_alloc_isoc); I'm not sure what the exact policy is with GPL symbols, but I do know what Al Viro posted recently on the

Updated tuning file for Crystal Palace transmitter, UK

2012-02-11 Thread Chris Rankin
Hi, The UK's Crystal Palace transmitter supports DVB-T2, so here's an updated tuning file. Cheers, Chris #-- # file automatically generated by w_scan # (http://wirbel.htpc-forum.de/w_scan/index2.html) #! w_scan

Fwd: em28xx leaks

2012-01-28 Thread Chris Rankin
Subject: em28xx leaks Date: Fri, 27 Jan 2012 13:21:50 -0600 From: Todd Squires squir...@ct0.com Organisation: Core Technologies To: ranki...@yahoo.com Hi Chris, I've recently started using an em28xx, and have run into a memory leak in the 3.2.1 kernel. Poking around the Internet, I found that

Re: Support for RC-6 in em28xx driver?

2012-01-05 Thread Chris Rankin
Hi, As someone who also owns a PCTV 290e device, I must agree that the remote control that it ships with is useless for VDR. Its biggest flaw is a lack of red, green, yellow and blue buttons, unlike the very nice remote control that ships with the Hauppauge NOVA-T2. Are you suggesting that

Re: [PATCH v3] EM28xx - fix deadlock when unplugging and replugging a DVB adapter

2011-09-25 Thread Chris Rankin
On 25/09/11 13:49, Mauro Carvalho Chehab wrote: Ok, I've applied it, but it doesn't sound a good idea to me to do: + mutex_unlock(dev-lock); em28xx_init_extension(dev); + mutex_lock(dev-lock); I'll later test it with some hardware and see how well this behaves in practice.

Re: [PATCH v3] EM28xx - fix deadlock when unplugging and replugging a DVB adapter

2011-09-25 Thread Chris Rankin
On 25/09/11 14:00, Mauro Carvalho Chehab wrote: Hmm... This would probably work better (not tested). Could you please test it on your hardware? Hmm, I don't understand this. The deadlock isn't about taking em28xx_devlist_mutex, but happens because em28xx_dvb_init() tries to retake dev-lock

[PATCH] EM28xx - remove unused prototypes

2011-09-25 Thread Chris Rankin
Mauro, This patch just removes the prototypes for the two functions that I've already deleted in my previous patches. Cheers, Chris Signed-off-by: Chris Rankin ranki...@yahoo.com --- linux/drivers/media/video/em28xx/em28xx.h.orig 2011-09-25 14:46:02.0 +0100 +++ linux/drivers/media

[PATCH] EM28xx - replug locking cleanup

2011-09-25 Thread Chris Rankin
the case anyway. Signed-off-by: Chris Rankin ranki...@yahoo.com Cheers, Chris --- linux/drivers/media/video/em28xx/em28xx-cards.c.orig 2011-09-25 22:51:59.0 +0100 +++ linux/drivers/media/video/em28xx/em28xx-cards.c 2011-09-25 23:24:06.0 +0100 @@ -3005,10 +3005,6 @@ goto fail

[PATCH 0/2] EM28xx patches for 3.2

2011-09-24 Thread Chris Rankin
Mauro, I am resending the two patches for the em28xx driver which seem to have been missed for 3.2. The first one simply removes a stray bit operation on em28xx_devused, whereas the second tidies up the device/extension list handling. It is possible that the first patch has been applied

[PATCH 1/2] use atomic bit operations for devices-in-use mask

2011-09-24 Thread Chris Rankin
This line was probably missed in the merge. Signed-off-by: Chris Rankin ranki...@yahoo.com --- linux/drivers/media/video/em28xx/em28xx-cards.c.orig.0 2011-09-24 15:30:02.0 +0100 +++ linux/drivers/media/video/em28xx/em28xx-cards.c 2011-09-24 15:19:28.0 +0100 @@ -3114,7 +3114,6

[PATCH 2/2] EM28xx - fix race on disconnect

2011-09-24 Thread Chris Rankin
. Signed-off-by: Chris Rankin ranki...@yahoo.com --- linux/drivers/media/video/em28xx/em28xx-core.c.orig 2011-09-24 14:56:05.0 +0100 +++ linux/drivers/media/video/em28xx/em28xx-core.c 2011-09-24 15:01:23.0 +0100 @@ -1184,18 +1184,6 @@ static DEFINE_MUTEX(em28xx_devlist_mutex

Re: [PATCH 0/2] EM28xx patches for 3.2

2011-09-24 Thread Chris Rankin
- Original Message - BTW, did you implement a different solution for the DVB module trying to retake the dev-lock mutex? Because it looks as if both em28xx_dvb_init() and em28xx_usb_probe() are still calling mutex_lock(). No, I didn't find any time to look into it. Too much work

[PATCH v2] EM28xx - fix deadlock when unplugging and replugging a DVB adapter

2011-09-24 Thread Chris Rankin
either a device or an extension to the respective lists. (Because em28xx_init_dvb() will want to take the lock instead). Conversely, for Audio-Only devices, the device lock *must* be held when adding/removing either a device or an extension to the respective lists. Signed-off-by: Chris Rankin ranki

[PATCH v3] EM28xx - fix deadlock when unplugging and replugging a DVB adapter

2011-09-24 Thread Chris Rankin
adding/removing either a device or an extension to the respective lists. Signed-off-by: Chris Rankin ranki...@yahoo.com Cheers, Chris --- linux/drivers/media/video/em28xx/em28xx-cards.c.orig 2011-09-24 21:42:43.0 +0100 +++ linux/drivers/media/video/em28xx/em28xx-cards.c 2011-09-24 21:48

Re: em28xx PCTV 290e patches

2011-09-23 Thread Chris Rankin
Hi Stuart, On 23/09/11 09:52, Stuart Morris wrote: I have a PCTV 290e and have been watching closely the updates to the Linux media tree for this device. Thanks for addressing the issues with the 290e driver, I am now able to use my 290e for watching UK FreeviewHD with a good degree of

Re: [PATCH 1/1] EM28xx - fix deadlock when unplugging and replugging a DVB adapter

2011-09-13 Thread Chris Rankin
On 13/09/11 21:04, Antti Palosaari wrote: On 08/21/2011 03:32 PM, Chris Rankin wrote: It occurred to me this morning that since we're no longer supposed to be holding the device lock when taking the device list lock, then the em28xx_usb_disconnect() function needs changing too. Signed-off

Re: ERROR: em28xx_add_into_devlist [drivers/media/video/em28xx/em28xx.ko] undefined!

2011-09-05 Thread Chris Rankin
On 05/09/11 14:57, Mauro Carvalho Chehab wrote: Could you please provide me a patch for it? I'll merge with your original one when submitting it upstream. Here you go. Did you also pick up the other merge fix and the (separate) memory leak fix, please? Cheers, Chris Signed-off-by: Chris

Re: [git:v4l-dvb/for_v3.2] [media] em28xx: use atomic bit operations for devices-in-use mask

2011-09-04 Thread Chris Rankin
On 04/09/11 00:49, Mauro Carvalho Chehab wrote: This is an automatic generated email to let you know that the following patch were queued at the http://git.linuxtv.org/media_tree.git tree: Subject: [media] em28xx: use atomic bit operations for devices-in-use mask Author: Chris

[PATCH 1/1] EM28xx - Fix memory leak on disconnect or error.

2011-09-04 Thread Chris Rankin
Mauro, This patch seems to have been missed, so I'm resending it. Release the dev-alt_max_pkt_size buffer in all cases. Signed-off-by: Chris Rankin ranki...@yahoo.com Cheers, Chris --- linux/drivers/media/video/em28xx/em28xx-cards.c.orig 2011-09-04 20:30:14.0 +0100 +++ linux/drivers

Re: ERROR: em28xx_add_into_devlist [drivers/media/video/em28xx/em28xx.ko] undefined!

2011-09-04 Thread Chris Rankin
On 05/09/11 00:46, Antti Palosaari wrote: Moikka, Current linux-media make gives error. Any idea what's wrong? Kernel: arch/x86/boot/bzImage is ready (#1) Building modules, stage 2. MODPOST 1907 modules ERROR: em28xx_add_into_devlist [drivers/media/video/em28xx/em28xx.ko] undefined! WARNING:

Re: ERROR: em28xx_add_into_devlist [drivers/media/video/em28xx/em28xx.ko] undefined!

2011-09-04 Thread Chris Rankin
On 05/09/11 00:46, Antti Palosaari wrote: Moikka, Current linux-media make gives error. Any idea what's wrong? Kernel: arch/x86/boot/bzImage is ready (#1) Building modules, stage 2. MODPOST 1907 modules ERROR: em28xx_add_into_devlist [drivers/media/video/em28xx/em28xx.ko] undefined! WARNING:

Re: ERROR: em28xx_add_into_devlist [drivers/media/video/em28xx/em28xx.ko] undefined!

2011-09-04 Thread Chris Rankin
On 05/09/11 01:24, Antti Palosaari wrote: If you select em28xx-cards.c blob link you give you can see it is there still for some reason. It's a merge issue. This lingering reference must have been added after I posted my original patch. Fortunately, it's easily fixed: the

Re: Is DVB ioctl FE_SET_FRONTEND broken?

2011-08-26 Thread Chris Rankin
--- On Fri, 26/8/11, Andreas Oberritter o...@linuxtv.org wrote: can you please test whether https://patchwork.kernel.org/patch/1036132/ restores the old behaviour? These three pending patches are also related to frontend events: https://patchwork.kernel.org/patch/1036112/

Re: Is DVB ioctl FE_SET_FRONTEND broken?

2011-08-26 Thread Chris Rankin
--- On Fri, 26/8/11, Andreas Oberritter o...@linuxtv.org wrote: I first thought that you were talking about a regression in Linux 3.0.x. Heh, yes and no. I am talking about a regression that I am definitely seeing in 3.0.x. However, I cannot say which kernel the problem first appeared in.

Is DVB ioctl FE_SET_FRONTEND broken?

2011-08-25 Thread Chris Rankin
Hi, As far as I understand it, the FE_SET_FRONTEND ioctl is supposed to tell a DVB device to tune itself, and will send a poll() event when it completes. The frequency parameter of this event will be the frequency of the newly tuned channel, or 0 if tuning failed.

Re: [PATCH 1/1] EM28xx - fix deadlock when unplugging and replugging a DVB adapter

2011-08-21 Thread Chris Rankin
It occurred to me this morning that since we're no longer supposed to be holding the device lock when taking the device list lock, then the em28xx_usb_disconnect() function needs changing too. Signed-off-by: Chris Rankin ranki...@yahoo.com --- linux-3.0/drivers/media/video/em28xx/em28xx

[PATCH 1/6 ] EM28xx - pass correct buffer size to snprintf

2011-08-20 Thread Chris Rankin
snprintf()'s size parameter includes space for the terminating '\0' character. Signed-off-by: Chris Rankin ranki...@yahoo.com --- linux-3.0/drivers/media/video/em28xx/em28xx-cards.c.orig 2011-08-17 08:52:19.0 +0100 +++ linux-3.0/drivers/media/video/em28xx/em28xx-cards.c 2011-08-18 22:03

[PATCH 2/6] Fix memory leak on disconnect or error.

2011-08-20 Thread Chris Rankin
Release the dev-alt_max_pkt_size buffer in all cases. Signed-off-by: Chris Rankin ranki...@yahoo.com --- linux-3.0/drivers/media/video/em28xx/em28xx-video.c.orig 2011-08-18 17:20:10.0 +0100 +++ linux-3.0/drivers/media/video/em28xx/em28xx-video.c 2011-08-18 17:20:33.0 +0100

[PATCH 3/6] EM28xx - use atomic bit operations for devices-in-use mask

2011-08-20 Thread Chris Rankin
Use atomic bit operations for the em28xx_devused mask, to prevent an unlikely race condition should two adapters be plugged in simultaneously. The operations also clearer than explicit bit manipulation anyway. Signed-off-by: Chris Rankin ranki...@yahoo.com --- linux-3.0/drivers/media/video

[PATCH 4/6] EM28xx - clean up resources should init fail

2011-08-20 Thread Chris Rankin
This patch ensures that the em28xx_init_dev() function cleans up after itself, in the event that it fails. This isimportant because the struct em28xx will be deallocated if em28xx_init_dev() returns an error. Signed-off-by: Chris Rankin ranki...@yahoo.com --- linux-3.0/drivers/media/video

[PATCH 5/6] EM28xx - move printk lines outside mutex lock

2011-08-20 Thread Chris Rankin
There's no reason to still be holding the device list mutex for either of these printk statements. Signed-off-by: Chris Rankin ranki...@yahoo.com --- linux-3.0/drivers/media/video/em28xx/em28xx-core.c.orig 2011-08-18 23:05:50.0 +0100 +++ linux-3.0/drivers/media/video/em28xx/em28xx

[PATCH 6/6] EM28xx - don't sleep on disconnect

2011-08-20 Thread Chris Rankin
write failures with error -19 from appearing occasionally in the dmesg log. Signed-off-by: Chris Rankin ranki...@yahoo.com --- linux-3.0/drivers/media/common/tuners/tda18271-fe.c.orig 2011-08-18 16:55:53.0 +0100 +++ linux-3.0/drivers/media/common/tuners/tda18271-fe.c 2011-08-18 23:12

[PATCH 1/2] EM28xx - fix race on disconnect

2011-08-20 Thread Chris Rankin
been initialised for every extension in the extension list. Signed-of-by: Chris Rankin ranki...@yahoo.com --- linux-3.0/drivers/media/video/em28xx/em28xx-cards.c.orig 2011-08-19 00:23:17.0 +0100 +++ linux-3.0/drivers/media/video/em28xx/em28xx-cards.c 2011-08-19 00:32:40.0 +0100

[PATCH 2/2] EM28xx - fix deadlock when unplugging and replugging a DVB adapter

2011-08-20 Thread Chris Rankin
situation is a definite improvement :-). Signed-off-by: Chris Rankin ranki...@yahoo.com --- linux-3.0/drivers/media/video/em28xx/em28xx-dvb.c.orig 2011-08-19 00:50:41.0 +0100 +++ linux-3.0/drivers/media/video/em28xx/em28xx-dvb.c 2011-08-19 00:51:03.0 +0100 @@ -542,7 +542,6 @@ dev

Re: [PATCH 6/6] EM28xx - don't sleep on disconnect

2011-08-20 Thread Chris Rankin
--- On Sat, 20/8/11, Mauro Carvalho Chehab mche...@redhat.com wrot This will cause an OOPS if dvb-fe[n] == NULL. OK, that's trivially fixable. I'll send you an updated patch. Is it safe to assume that dvb-fe[0] at least will always be non-NULL? Cheers, Chris -- To unsubscribe from this

Re: [PATCH 2/2] EM28xx - fix deadlock when unplugging and replugging a DVB adapter

2011-08-20 Thread Chris Rankin
--- On Sat, 20/8/11, Mauro Carvalho Chehab mche...@redhat.com wrote: No. The extension load can happen after the usb probe phase. In practice, the only case where the extension init will happen together with the usb probe phase is when the em28xx modules are compiled builtin It also happens

Re: [PATCH 6/6] EM28xx - don't sleep on disconnect

2011-08-20 Thread Chris Rankin
Here's the updated patch that checks whether the frontend exists before disabling the sleep operations. Cheers, Chris --- linux-3.0/drivers/media/common/tuners/tda18271-fe.c.orig2011-08-20 18:53:48.0 +0100 +++ linux-3.0/drivers/media/common/tuners/tda18271-fe.c 2011-08-20

[PATCH 1/1] EM28xx - fix deadlock when unplugging and replugging a DVB adapter

2011-08-20 Thread Chris Rankin
Here's the new patch for the deadlock problem, which releases the device mutex before calling em28xx_init_extension() and then reacquires it afterwards. The locking in dvb_init() is now left alone. Signed-off-by: Chris Rankin ranki...@yahoo.com --- linux-3.0/drivers/media/video/em28xx/em28xx

[PATCH] Latest version of em28xx / em28xx-dvb patch for PCTV 290e

2011-08-18 Thread Chris Rankin
Hi, Here's my latest patch for the em28xx / em28xx-dvb modules, which addresses the following problems: a) Locking problem when unplugging and then replugging USB adapter. b) Race conditions between adding/removing devices from the devlist, while simultaneously loading/unloading extension

Re: [PATCH] Latest version of em28xx / em28xx-dvb patch for PCTV 290e

2011-08-18 Thread Chris Rankin
On 18/08/11 19:43, Devin Heitmueller wrote: You would be well served to break this into a patch series, as it tends to be difficult to review a whole series of changes in a single patch. You seem to be mixed in a bunch of useless changes alongside functional changes. For example, if you're

Re: [PATCH] Latest version of em28xx / em28xx-dvb patch for PCTV 290e

2011-08-18 Thread Chris Rankin
for em28xx_devused. - use the correct amount of memory for snprintf(). Cheers, Chris Signed-of-by: Chris Rankin ranki...@yahoo.com --- linux-3.0/drivers/media/video/em28xx/em28xx-cards.c.orig 2011-08-18 22:24:12.0 +0100 +++ linux-3.0/drivers/media/video/em28xx/em28xx-cards.c 2011-08-18 22:22

Re: [PATCH] Latest version of em28xx / em28xx-dvb patch for PCTV 290e

2011-08-18 Thread Chris Rankin
Two more patches: a) clean up resources more reliably if em28xx_init_dev() fails, b) move two printk() statements outside the mutex lock Cheers, Chris Signed-off-by: Chris Rankin ranki...@yahoo.com --- linux-3.0/drivers/media/video/em28xx/em28xx-cards.c.orig 2011-08-18 22:42:03.0

Re: [PATCH] Latest version of em28xx / em28xx-dvb patch for PCTV 290e

2011-08-18 Thread Chris Rankin
by nulling out the function pointers that the DVB framework would otherwise try to call. I have therefore declared the structs in the tda18271 and cxd2820r modules to be const, so that we know that they are supposed only to be templates. Signed-off-by: Chris Rankin ranki...@yahoo.com --- linux-3.0

Re: [PATCH] Latest version of em28xx / em28xx-dvb patch for PCTV 290e

2011-08-18 Thread Chris Rankin
. Signed-of-by: Chris Rankin ranki...@yahoo.com --- linux-3.0/drivers/media/video/em28xx/em28xx-cards.c.orig 2011-08-19 00:23:17.0 +0100 +++ linux-3.0/drivers/media/video/em28xx/em28xx-cards.c 2011-08-19 00:32:40.0 +0100 @@ -2738,9 +2738,9 @@ #endif /* CONFIG_MODULES

Re: [PATCH] Latest version of em28xx / em28xx-dvb patch for PCTV 290e

2011-08-18 Thread Chris Rankin
situation is a definite improvement :-). Signed-off-by: Chris Rankin ranki...@yahoo.com --- linux-3.0/drivers/media/video/em28xx/em28xx-dvb.c.orig 2011-08-19 00:50:41.0 +0100 +++ linux-3.0/drivers/media/video/em28xx/em28xx-dvb.c 2011-08-19 00:51:03.0 +0100 @@ -542,7 +542,6 @@ dev

[PATCH] Fix locking and resource problems for PCTV 290e (em28xx / em28xx-dvb)

2011-08-17 Thread Chris Rankin
Hi, This is my third draft at fixing the em28xx modules for the PCTV 290e DVB adapter. The highlights are: - resolve the locking issue when replugging the USB adapter, - remove race condition when initialising an adapter and simultaneously loading the em28xx-dvb module, - more reliable

[PATCH] Fix locking problem between em28xx and em28xx-dvb modules

2011-08-16 Thread Chris Rankin
bits should be as brief as possible and neither printk() does anything that needs the lock. A new patch is attached, for review. Signed-off-by: Chris Rankin ranki...@yahoo.com --- linux-3.0/drivers/media/video/em28xx/em28xx-core.c.orig 2011-08-16 09:15:46.0 +0100 +++ linux-3.0/drivers

Locking problem between em28xx and em28xx-dvb modules - Part 2

2011-08-16 Thread Chris Rankin
Hi, I've been looking deeper into the em28xx and em28xx-dvb modules, and I'm concerned that there are some races and resource leaks inherent in the current code: a) Shouldn't em28xx_init_extension() and em28xx_add_into_devlist() be unified into a single function? Otherwise, consider someone

[PATCH] Fix locking problem, race conditions and resource leaks in em28xx, em28xx-dvb

2011-08-16 Thread Chris Rankin
Hi, Here's the latest draft of my patch (against 3.0). Any feedback greatly appreciated. Cheers, Chris --- linux-3.0/drivers/media/video/em28xx/em28xx-core.c.orig 2011-08-16 19:49:58.0 +0100 +++ linux-3.0/drivers/media/video/em28xx/em28xx-core.c 2011-08-16 21:46:35.0 +0100 @@

Re: PCTV 290e - assorted problems

2011-08-15 Thread Chris Rankin
--- On Mon, 15/8/11, Steve Kerrison st...@stevekerrison.com wrote: It depends on whether the problem is a weak signal or a too strong signal. I suspect that the HD signal is too weak where I am, because I am in the transmitter's green area. So it's possible that I'll have to wait until April

Possible cause of replug lockup in em28xx-dvb

2011-08-15 Thread Chris Rankin
Hi, I've been plugging away at the locking issue in the em28xx / em28xx-dvb modules, and I think I've found the reason. Basically, we're trying to acquire the dev-lock mutex again in dvb_init() when we've already acquired it in em28xx_init_dev(). The exact sequence is:

[PATCH] Add missing OK key to PCTV IR keymap

2011-08-14 Thread Chris Rankin
Hi, The following patch adds the IR code for the missing OK key to the Pinnacle PCTV HD map. This map is now used by the PCTV 290e DVB-T2 device, whose remote control has 26 buttons. Cheers, Chris Signed-off-by: Chris Rankin ranki...@yahoo.com --- linux-3.0/drivers/media/rc/keymaps/rc

PCTV 290e - assorted problems

2011-08-14 Thread Chris Rankin
Hi, I've been experimenting with my new PCTV 290e DVB-T2 device this weekend, and have a couple of observations. For example, the device sometimes has trouble initialising itself: usb 4-2: new high speed USB device number 4 using ehci_hcd em28xx: New device PCTV Systems PCTV 290e @ 480 Mbps

PCTV 290e - assorted problems (cont)

2011-08-14 Thread Chris Rankin
Another thing I've noticed - a successful hotplug outputs the following messages into the dmesg log: DVB: registering new adapter (em28xx #0) DVB: registering adapter 0 frontend 0 (Sony CXD2820R (DVB-T/T2))... DVB: registering adapter 0 frontend 1 (Sony CXD2820R (DVB-C))... em28xx #0:

Re: PCTV 290e - assorted problems

2011-08-14 Thread Chris Rankin
--- On Mon, 15/8/11, Antti Palosaari cr...@iki.fi wrote: T 55400 8MHz + auto auto auto etc. is enough. Hmm, not here it isn't: $ scandvb -u -vvv uk-CrystalPalace scanning uk-CrystalPalace using '/dev/dvb/adapter0/frontend0' and '/dev/dvb/adapter0/demux0' initial transponder 55400 0 9

Re: PCTV 290e - assorted problems

2011-08-14 Thread Chris Rankin
INFO: task khubd:1100 blocked for more than 120 seconds. echo 0 /proc/sys/kernel/hung_task_timeout_secs disables this message. khubd D 0 1100 2 0x 8801a694e930 0046 8801a691ffd8 8162b020 00010280

PCTV 290e nanostick and remote control support

2011-08-13 Thread Chris Rankin
Hi, I've just acquired a PCTV 290e nanostick, and have successfully tuned it into a DVB-T2 MUX. Yay! :-). However, before declaring total victory, I have noticed that no-one has yet wired up the device's IR support in the em28xx driver. The adapter ships with a tiny RC with only 26 buttons,

Re: PCTV 290e nanostick and remote control support

2011-08-13 Thread Chris Rankin
--- On Sat, 13/8/11, Antti Palosaari cr...@iki.fi wrote: Remote is already supported, but from the 3.1 or maybe 3.2 (I am not sure if Mauro was hurry to sent it 3.1). Hi, This appears to be the diff from 3.1 that adds RC support: --- linux-3.0/drivers/media/video/em28xx/em28xx-cards.c.orig

Re: PCTV 290e nanostick and remote control support

2011-08-13 Thread Chris Rankin
Hi, The rc-pinnacle-pctv-hd keymap is missing the definition of the OK key: --- linux-3.0/drivers/media/rc/keymaps/rc-pinnacle-pctv-hd.c.orig 2011-08-14 02:42:01.0 +0100 +++ linux-3.0/drivers/media/rc/keymaps/rc-pinnacle-pctv-hd.c2011-08-14 02:12:45.0 +0100 @@ -20,6

  1   2   >