Re: capture-example.c crash on close_device()

2009-10-27 Thread Sean
I'm still having trouble with capture-example.c completely locking up my system. The same os image on other hardware works fine. Where do I start looking? What can I do to debug this issue? Sean Sean wrote: Hi, I have compiled kernel 2.6.30 from kernel.org, and I have also compiled capture

Re: Another gpsca kernel BUG when disconnecting camera while streaming with mmap

2009-11-07 Thread Sean
Thanks for your reply. That makes a lot of sense, because this is an embedded platform and there is no swap! But there seems to be 89544K free while capture-example is running. hmmm. Sean Lazar Hans de Goede wrote: Hi, On 11/04/2009 10:37 AM, Sean wrote: Hi, I am having a bug when

Re: libv4l2: error dequeuing buf: Input/output error

2009-11-15 Thread Sean
and libv4l2 from mercurial. http://linuxtv.org/hg/v4l-dvb Sean bifferos wrote: Hi all, Can anyone give any clue as to why I might get this error when capturing from a PAC207 webcam? It happens after a 6-7 second delay when capturing. I've seen this with 2.6.30.1, 2.6.30.5 and 2.6.32-rc7, however I

Re: [Bugme-new] [Bug 14564] New: capture-example sleeping function called from invalid context at arch/x86/mm/fault.c

2009-12-02 Thread Sean
Is there anything I can do to help? This is a show stopping bug for me. Thanks, Sean Lazar Alan Stern wrote: On Wed, 11 Nov 2009, Andrew Morton wrote: http://bugzilla.kernel.org/show_bug.cgi?id=14564 Summary: capture-example sleeping function called from invalid

Re: [Bugme-new] [Bug 14564] New: capture-example sleeping function called from invalid context at arch/x86/mm/fault.c

2009-12-16 Thread Sean
Sean Lazar --- ohci-mem.c.orig2009-12-16 22:57:49.0 + +++ ohci-mem.c2009-12-16 22:49:37.0 + @@ -103,8 +103,13 @@ { struct td**prev = hc-td_hash [TD_HASH_FUNC (td-td_dma)]; -while (*prev *prev != td) +while (*prev *prev != td

Re: [Bugme-new] [Bug 14564] New: capture-example sleeping function called from invalid context at arch/x86/mm/fault.c

2009-12-29 Thread Sean
return in td_free that is in the patch will trap the error. I changed the debug statements to ohci_dbg and I was able to capture the full output with klogd. It is attached. Sean Sean wrote: In looking at the log file it seems that there is a mismatch of td_alloc calls and td_free calls

Re: [Bugme-new] [Bug 14564] New: capture-example sleeping function called from invalid context at arch/x86/mm/fault.c

2009-12-29 Thread Sean
. That's because you didn't get the full output. Here's an enhanced version of the patch. It will provide more information and perhaps a smoking gun. Alan Stern Alan, Thanks for the debug patch. I'll send you the dmesg.log output in another email. It is 2MB. Sean -- To unsubscribe

Re: [Bugme-new] [Bug 14564] New: capture-example sleeping function called from invalid context at arch/x86/mm/fault.c

2010-01-02 Thread Sean
Alan, Thanks again. I was able to get the full dmesg output this time. I ran capture-example three times and each time removing the webcam before capture-example finished. On the third time I got the poisoned hash message and I captured the output to a file. Attached is the dmesg output. Sean

Re: [Bugme-new] [Bug 14564] New: capture-example sleeping function called from invalid context at arch/x86/mm/fault.c

2010-01-02 Thread Sean
a hardware memory error. What do you think? Attached is the latest dmesg output. Sean dmesg3.log.tar.gz Description: Unix tar archive

Re: [Bugme-new] [Bug 14564] New: capture-example sleeping function called from invalid context at arch/x86/mm/fault.c

2010-01-03 Thread Sean
to remove the camera. Attached are the serial console capture as well as the dmesg log in debug4.tar.gz. Did you want me to try to reproduce the poison message? Sean debug4.tar.gz Description: Unix tar archive

Re: [Bugme-new] [Bug 14564] New: capture-example sleeping function called from invalid context at arch/x86/mm/fault.c

2010-01-04 Thread Sean
; + } + td-ed-hwTailP = td-hwNextTD; } Alan, This last patch seems to do the job. Thanks so much for your help! Where do I donate/send beer? Sean -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org

Re: [Bug 14564] capture-example sleeping function called from invalid context at arch/x86/mm/fault.c

2010-01-04 Thread Sean
||moin...@free.fr --- Comment #22 from Jean-Francois Moine moin...@free.fr 2010-01-03 07:02:45 --- Hello Sean, Sorry to be a bit late. Looking at the dmesg, I found that the gspca version was 2.7.0. May you upgrade your linux media stuff from LinuxTv.org and check

Re: [Bugme-new] [Bug 14564] New: capture-example sleeping function called from invalid context at arch/x86/mm/fault.c

2010-01-04 Thread Sean
Alan Stern wrote: Um, when you say it does the job, what do you mean? It traps the error and prevents the kernel from crashing. The job it was _intended_ to do was to prove that your problems are caused by hardware errors rather than software bugs. If the patch causes the problems to stop,

Re: problem webcam gspca 2.6.32

2010-01-09 Thread Sean
What kind of errors or problems are you getting? Can you turn on debugging and give us some output? Sean --Original Message-- From: sacarde Sender: linux-media-ow...@vger.kernel.org To: linux-media@vger.kernel.org Subject: problem webcam gspca 2.6.32 Sent: Jan 9, 2010 12:32 AM hi

[PATCH] DVB USB should not depend on RC

2011-02-21 Thread Sean Young
on input which also has dependants. This can save a reasonable amount of memory: $ ./scripts/bloat-o-meter vmlinux vmlinux-no-rc add/remove: 0/909 grow/shrink: 1/20 up/down: 4/-159171 (-159167) Diff against current mainline. Signed-off-by: Sean Young s...@mess.org --- diff --git a/drivers/media

Re: [PATCH 2/5] MEDIA: ttusbir, fix double free

2013-04-09 Thread Sean Young
to NULL resulting in rc_free_device being a noop. Signed-off-by: Jiri Slaby jsl...@suse.cz Cc: Sean Young s...@mess.org Cc: Mauro Carvalho Chehab mche...@redhat.com Cc: linux-media@vger.kernel.org --- drivers/media/rc/ttusbir.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers

Re: [PATCH] [media] rc: ttusbir: fix potential double free in ttusbir_probe()

2013-04-09 Thread Sean Young
for that. However, some else has beaten you to it, I'm afraid: http://www.spinics.net/lists/linux-media/msg62058.html I guess it's up Mauro to decide which one to accept. Sean -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord

Re: [media] redrat3: remove memcpys and fix unaligned memory access

2013-04-10 Thread Sean Young
received the full packet: 689 if (rr3-bytes_read be16_to_cpu(rr3-irdata.header.length)) 690 /* we're still accumulating data */ 691 return 0; HTH and thanks for checking my changes. Sean -- To unsubscribe from this list: send the line unsubscribe linux

Re: Patch mceusb.c -- Kernel 3.9-rc6

2013-04-16 Thread Sean Young
: http://git.linuxtv.org/media_tree.git Lastly a better subject line would be helpful, this is the headline for the commit. Looks good otherwise. Sean --- a/drivers/media/rc/mceusb.c 2013-04-07 22:49:54.0 -0500 +++ b/drivers/media/rc/mceusb.c 2013-04-14 12:18:30.0

[PATCH] Minor cleanups for MCE USB.

2012-07-15 Thread Sean Young
Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/mceusb.c | 13 + 1 files changed, 5 insertions(+), 8 deletions(-) diff --git a/drivers/media/rc/mceusb.c b/drivers/media/rc/mceusb.c index e150a2e..1af0b0c 100644 --- a/drivers/media/rc/mceusb.c +++ b/drivers/media/rc

[PATCH] Add support for the IguanaWorks USB IR Transceiver.

2012-07-15 Thread Sean Young
Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/Kconfig| 11 + drivers/media/rc/Makefile |1 + drivers/media/rc/iguanair.c | 639 +++ 3 files changed, 651 insertions(+), 0 deletions(-) create mode 100644 drivers/media/rc

[PATCH] [media] winbond-cir: Fix initialization

2012-07-31 Thread Sean Young
-off-by: Sean Young s...@mess.org --- drivers/media/rc/winbond-cir.c | 21 - drivers/tty/serial/8250/8250.c | 1 + 2 files changed, 21 insertions(+), 1 deletion(-) diff --git a/drivers/media/rc/winbond-cir.c b/drivers/media/rc/winbond-cir.c index 54ee348..20a0bbb 100644

[PATCH] [media] nec-decoder: fix NEC decoding for Pioneer Laserdisc CU-700 remote

2012-07-31 Thread Sean Young
This remote sends a header pulse of 8150us followed by a space of 4000us. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/ir-nec-decoder.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/rc/ir-nec-decoder.c b/drivers/media/rc/ir-nec-decoder.c

[PATCH] [media] iguanair: various fixes

2012-07-31 Thread Sean Young
This fixes: - rx_overflow while holding down any down button on a nec remote - suspend/resume - stop receiver on rmmod - advertise rx_resolution and timeout properly - code simplify - ignore unsupported firmware versions Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/Kconfig

Re: Philips saa7134 IR remote problem with linux kernel v2.6.35

2012-08-10 Thread Sean Young
be surprised if the 15ms delay for processing in saa7134_raw_decode_irq (bottom of saa7134-input.c) needs increasing. Sean -- 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

[PATCH 2/6] [media] rc: transmit on device which does not support it should fail

2012-08-10 Thread Sean Young
Currently write() will return 0 if an IR device does not support sending. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/ir-lirc-codec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/rc/ir-lirc-codec.c b/drivers/media/rc/ir-lirc-codec.c index

[PATCH 5/6] [staging] lirc: remove lirc_ttusbir driver

2012-08-10 Thread Sean Young
We now have a proper driver which uses rc-core. Signed-off-by: Sean Young s...@mess.org --- drivers/staging/media/lirc/Kconfig| 6 - drivers/staging/media/lirc/Makefile | 1 - drivers/staging/media/lirc/lirc_ttusbir.c | 376 -- 3 files changed, 383

[PATCH 4/6] [media] rc: do not wake up rc thread unless there is something to do

2012-08-10 Thread Sean Young
The TechnoTrend USB IR Receiver sends 125 ISO URBs per second, even when there is no IR activity. Reduce the number of wake ups from the other drivers too. This saves about 0.25ms per second on a 2.4GHz Core 2 according to powertop. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc

[PATCH 3/6] [media] rc: Add support for the TechnoTrend USB IR Receiver

2012-08-10 Thread Sean Young
This driver is loosely based on the lirc_ttusbir driver in the staging area, but adds more accurate sampling and led control. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/Kconfig | 13 ++ drivers/media/rc/Makefile | 1 + drivers/media/rc/keymaps/rc-tt

[PATCH 6/6] [staging] lirc: lirc_ene0100.h is not referenced anywhere

2012-08-10 Thread Sean Young
There is a proper ene0100 driver anyway. Signed-off-by: Sean Young s...@mess.org --- drivers/staging/media/lirc/lirc_ene0100.h | 169 -- 1 file changed, 169 deletions(-) delete mode 100644 drivers/staging/media/lirc/lirc_ene0100.h diff --git a/drivers/staging/media

Re: [PATCH 1/6] [media] iguanair: Fix return value on transmit

2012-08-11 Thread Sean Young
On Fri, Aug 10, 2012 at 08:28:03PM +0100, Sean Young wrote: Transmit returned 0 after sending and failed to send anything if the amount exceeded its buffer size. Also fix some minor errors. Signed-off-by: Sean Young s...@mess.org I'm sorry, this patch series wasn't diffed against the right

Re: [PATCH] [media] iguanair: various fixes

2012-08-12 Thread Sean Young
On Sat, Aug 11, 2012 at 09:35:29PM -0300, Mauro Carvalho Chehab wrote: Em 31-07-2012 07:37, Sean Young escreveu: This fixes: - rx_overflow while holding down any down button on a nec remote - suspend/resume - stop receiver on rmmod - advertise rx_resolution and timeout properly

Re: Philips saa7134 IR remote problem with linux kernel v2.6.35

2012-08-13 Thread Sean Young
On Fri, Aug 10, 2012 at 11:41:23PM +0600, Partha Guha Roy wrote: On Fri, Aug 10, 2012 at 3:47 PM, Sean Young s...@mess.org wrote: Are you runnning the lircd user space process for input or relying on the in-kernel decoders? For my testing, I booted the vanilla kernel into ubuntu recovery

[PATCH 01/13] [media] iguanair: reuse existing urb callback for command responses

2012-08-13 Thread Sean Young
Rather than using usb_interrupt_msg() to receive responses, reuse the urb callback we already have in place. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/iguanair.c | 147 +--- 1 file changed, 56 insertions(+), 91 deletions(-) diff --git

[PATCH 02/13] [media] iguanair: ignore unsupported firmware versions

2012-08-13 Thread Sean Young
Firmware versions lower than 0x0205 use a different interface which is not supported. Also report the firmware version in the standard format. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/Kconfig| 8 ++-- drivers/media/rc/iguanair.c | 21 +++-- 2 files

[PATCH 05/13] [media] iguanair: support suspend and resume

2012-08-13 Thread Sean Young
Now unbind also stops the receiver. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/iguanair.c | 21 ++--- 1 file changed, 14 insertions(+), 7 deletions(-) diff --git a/drivers/media/rc/iguanair.c b/drivers/media/rc/iguanair.c index 4525107..a6a19eb 100644

[PATCH 04/13] [media] iguanair: fix return value for transmit

2012-08-13 Thread Sean Young
Also fix error codes returned from open. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/iguanair.c | 30 ++ 1 file changed, 18 insertions(+), 12 deletions(-) diff --git a/drivers/media/rc/iguanair.c b/drivers/media/rc/iguanair.c index 7eeabdb..4525107

[PATCH 06/13] [media] iguanair: reset the IR state after rx overflow or receiver enable

2012-08-13 Thread Sean Young
Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/iguanair.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/media/rc/iguanair.c b/drivers/media/rc/iguanair.c index a6a19eb..9810008 100644 --- a/drivers/media/rc/iguanair.c +++ b/drivers/media/rc/iguanair.c @@ -124,6

[PATCH 12/13] [staging] lirc: remove lirc_ttusbir driver

2012-08-13 Thread Sean Young
This has been replaced by the ttusbir driver. Signed-off-by: Sean Young s...@mess.org --- drivers/staging/media/lirc/Kconfig| 6 - drivers/staging/media/lirc/Makefile | 1 - drivers/staging/media/lirc/lirc_ttusbir.c | 376 -- 3 files changed, 383

[PATCH 13/13] [staging] lirc: lirc_ene0100.h is not referenced anywhere

2012-08-13 Thread Sean Young
There is a proper ene0100 driver anyway. Signed-off-by: Sean Young s...@mess.org --- drivers/staging/media/lirc/lirc_ene0100.h | 169 -- 1 file changed, 169 deletions(-) delete mode 100644 drivers/staging/media/lirc/lirc_ene0100.h diff --git a/drivers/staging/media

[PATCH 07/13] [media] iguanair: fix receiver overflow

2012-08-13 Thread Sean Young
handle at most 700 edges per second. The remote generates about 900. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/iguanair.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/media/rc/iguanair.c b/drivers/media/rc/iguanair.c index 9810008..6a09c2e

[PATCH 03/13] [media] iguanair: advertise the resolution and timeout properly

2012-08-13 Thread Sean Young
With the timeout supplied the interface can go idle. The keymap is the same one as other drivers which do not come with a remote. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/iguanair.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/media/rc

[PATCH 08/13] [media] rc: Add support for the TechnoTrend USB IR Receiver

2012-08-13 Thread Sean Young
This driver adds support for TechnoTrend USB IR Receiver. It is a complete rewrite of the staging/media/lirc/lirc_ttusbir driver. It adds more accurate sample reporting and led control. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/Kconfig | 13 ++ drivers/media/rc

[PATCH 09/13] [media] rc: do not wake up rc thread unless there is something to do

2012-08-13 Thread Sean Young
The TechnoTrend USB IR Receiver sends 125 ISO URBs per second, even when there is no IR activity. Reduce the number of wake ups from the other drivers too. This saves about 0.25ms/s on a 2.4GHz Core 2 according to powertop. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/fintek

[PATCH 10/13] [media] saa7134: simplify timer activation

2012-08-13 Thread Sean Young
tested only. Signed-off-by: Sean Young s...@mess.org --- drivers/media/video/saa7134/saa7134-input.c | 10 +++--- drivers/media/video/saa7134/saa7134.h | 1 - 2 files changed, 3 insertions(+), 8 deletions(-) diff --git a/drivers/media/video/saa7134/saa7134-input.c b/drivers/media/video

[PATCH 11/13] [media] rc: transmit on device which does not support it should fail

2012-08-13 Thread Sean Young
Currently write() will return 0 if an IR device does not support sending. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/ir-lirc-codec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/rc/ir-lirc-codec.c b/drivers/media/rc/ir-lirc-codec.c index

Re: [PATCHv2 1/2] media: rc: Introduce RX51 IR transmitter driver

2012-08-13 Thread Sean Young
On Fri, Aug 10, 2012 at 01:16:36PM +0300, Timo Kokkonen wrote: This is the driver for the IR transmitter diode found on the Nokia N900 (also known as RX51) device. The driver is mostly the same as found in the original 2.6.28 based kernel that comes with the device. The following

Re: [PATCH] [media] winbond-cir: Fix initialization

2012-08-16 Thread Sean Young
On Thu, Aug 16, 2012 at 09:19:09AM -0700, Greg KH wrote: On Tue, Jul 31, 2012 at 11:37:03AM +0100, Sean Young wrote: The serial driver will detect the winbond cir device as a serial port, since it looks exactly like a serial port unless you know what it is from the PNP ID. Winbond CIR

Re: [media] rc-core: move timeout and checks to lirc

2012-08-16 Thread Sean Young
(total_duration)); return count; } -- 1.7.11.2 Sean -- 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

Re: [media] rc-core: move timeout and checks to lirc

2012-08-21 Thread Sean Young
On Mon, Aug 20, 2012 at 07:02:47PM -0300, Mauro Carvalho Chehab wrote: Em 20-08-2012 18:36, David Härdeman escreveu: On Thu, Aug 16, 2012 at 08:12:34PM -0300, Mauro Carvalho Chehab wrote: Em 16-08-2012 19:15, Sean Young escreveu: Could we have a flag in rc_dev to signify whether a driver

Re: WPC8769L (WEC1020) support in winbond-cir?

2013-10-14 Thread Sean Young
On Mon, Oct 14, 2013 at 11:07:48AM -0300, Mauro Carvalho Chehab wrote: Em Mon, 14 Oct 2013 15:16:20 +0200 Tom Gundersen t...@jklm.no escreveu: Not sure about lirc_wpc8769l. David/Sean/Juan can have a better view. I've just had a cursory look at lirc_wpc8769l. The device does look similar

Re: [PATCHv2 06/29] iguanair: shut up a gcc warning on avr32 arch

2013-11-03 Thread Sean Young
to add a default line at the switch to shut it up. Mauro, I have a different way of solving this which also cleans up the code a little. I'll send a patch shortly. Sean -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org

[PATCH] [media] iguanair: simplify calculation of carrier delay cycles

2013-11-03 Thread Sean Young
Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/iguanair.c | 22 ++ 1 file changed, 6 insertions(+), 16 deletions(-) diff --git a/drivers/media/rc/iguanair.c b/drivers/media/rc/iguanair.c index 19632b1..7f05e19 100644 --- a/drivers/media/rc/iguanair.c +++ b

Re: IR devices refuses to generate events

2013-12-05 Thread Sean Young
set CONFIG_USB_DEBUG and see if you get more information? Sean -- 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

Re: IR devices refuses to generate events

2013-12-05 Thread Sean Young
On Thu, Dec 05, 2013 at 10:05:45PM +0530, Rajil Saraswat wrote: On 5 December 2013 18:54, Sean Young s...@mess.org wrote: I really don't know why that would happen. If I understand it correctly this is an error that is returned by the usb host controller driver. Could you set

Re: Patch mceusb: fix invalid urb interval

2013-12-11 Thread Sean Young
is this? Could you provide lsusb -vvv output please? Also what usb hub are you using? Another user is reporting problems with an xhci hub. Sean -- 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

[PATCH 1/3] [media] winbond-cir: fix idle mode

2012-10-24 Thread Sean Young
The receiver is never disabled by idle mode since rxstate never gets set to RXSTATE_ACTIVE, so we keep on getting interrupts after the first IR activity ends. Note that ir_raw_event_reset() already calls ir_raw_event_handle(). Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/winbond

[PATCH 2/3] [media] winbond-cir: increase IR receiver resolution

2012-10-24 Thread Sean Young
This is needed for carrier reporting. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/winbond-cir.c | 14 +- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/drivers/media/rc/winbond-cir.c b/drivers/media/rc/winbond-cir.c index 6f0f5ef..1ff47eb 100644

[PATCH 3/3] [media] winbond-cir: add carrier detection

2012-10-24 Thread Sean Young
The winbond hardware has a counter for leading edges, which increases as they are received. As we read raw IR from a fifo in an interrupt handler, we cannot correlate them to specific IR pulses so we simply count all pulses and edges until we go idle and disable the receiver. Signed-off-by: Sean

[PATCH] [media] Add MAINTAINERS entries for some RC devices

2012-11-19 Thread Sean Young
Signed-off-by: Sean Young s...@mess.org --- MAINTAINERS | 12 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index f4b3aa8..1a69d21 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -3751,6 +3751,12 @@ F: net/ieee802154/ F: net

Re: [PATCH 2/3] [media] winbond-cir: increase IR receiver resolution

2013-01-05 Thread Sean Young
On Thu, Jan 03, 2013 at 01:16:57AM +0100, David Härdeman wrote: On Wed, Oct 24, 2012 at 10:22:41PM +0100, Sean Young wrote: This is needed for carrier reporting. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/winbond-cir.c | 14 +- 1 file changed, 9 insertions

[PATCH 1/3] [media] winbond-cir: only enable higher sample resolution if needed

2013-01-06 Thread Sean Young
A sample resolution of 2us generates more than 300 interrupts per key and this resolution is not needed unless carrier reports are enabled. Revert to a resolution of 10us unless carrier reports are needed. This generates up to a fifth of the interrupts. Signed-off-by: Sean Young s...@mess.org

[PATCH 2/3] [media] iguanair: ensure transmission mask is initialized

2013-01-06 Thread Sean Young
Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/iguanair.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/rc/iguanair.c b/drivers/media/rc/iguanair.c index 5a9163d..a569c69 100644 --- a/drivers/media/rc/iguanair.c +++ b/drivers/media/rc/iguanair.c @@ -512,6 +512,7

[PATCH 3/3] [media] iguanair: intermittent initialization failure

2013-01-06 Thread Sean Young
Sometimes the first version request is sent before the device has fully initialized. This seems to happen on some hardware during boot when the iguanair is plugged into a root hub. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/iguanair.c | 19 ++- 1 file changed

Re: [PATCH] [RFC] video: exynos dp: Making Exynos DP Compliant with CDF

2013-01-11 Thread Sean Paul
On Fri, Jan 11, 2013 at 5:35 AM, Leela Krishna Amudala l.kris...@samsung.com wrote: The Exynos DP transmitter is treated as an end entity in the display pipeline and made this RFC patch compliant with CDF. Any suggestions are welcome. A few comments below. It's hard to get too much of an

Re: [PATCH 3/3] [media] iguanair: intermittent initialization failure

2013-01-12 Thread Sean Young
On Sun, Jan 06, 2013 at 05:19:45PM +, Sean Young wrote: Sometimes the first version request is sent before the device has fully initialized. This seems to happen on some hardware during boot when the iguanair is plugged into a root hub. Signed-off-by: Sean Young s...@mess.org Mauro

[PATCH v2] [media] iguanair: intermittent initialization failure

2013-01-14 Thread Sean Young
On cold boot the device does not initialize until the first packet is received, and that packet is not processed. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/iguanair.c | 25 +++-- 1 file changed, 15 insertions(+), 10 deletions(-) diff --git a/drivers/media

[PATCH] saa7134 can capture 720x480 when capturing NTSC

2010-02-06 Thread Sean Young
as in Windows. Tested on an AverMedia GO 007 FM Plus. Signed-off-by: Sean Young s...@mess.org -- diff --git a/drivers/media/video/saa7134/saa7134-video.c b/drivers/media/video/saa7134/saa7134-video.c index cb73264..31138d3 100644 --- a/drivers/media/video/saa7134/saa7134-video.c +++ b/drivers/media

[PATCH 1/5] [media] ttusbir: do not set led twice on resume

2013-01-29 Thread Sean Young
led_classdev_resume already sets the led. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/ttusbir.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/media/rc/ttusbir.c b/drivers/media/rc/ttusbir.c index 78be8a9..f9226b8 100644 --- a/drivers/media/rc

[PATCH 4/5] [media] mceusb: make transmit work on HP transceiver

2013-01-29 Thread Sean Young
This transceiver expects the set IR TX ports and IR data as seperate packets, like the Windows driver does. Remove unnecessary kzalloc. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/mceusb.c | 25 +++-- 1 file changed, 11 insertions(+), 14 deletions(-) diff

[PATCH 3/5] [media] mceusb: make transmit work on the Philips IR transceiver

2013-01-29 Thread Sean Young
The GET_REVISION command puts the device in an unresponsive state, although it continues to report any IR activity. Note that GET_REVISION command is not documented, nor is any possible response to it parsed. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/mceusb.c | 8 +++- 1

[PATCH 2/5] [media] ttusbir: add missing endian conversion

2013-01-29 Thread Sean Young
spotted by sparse. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/ttusbir.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/media/rc/ttusbir.c b/drivers/media/rc/ttusbir.c index f9226b8..cf0d47f 100644 --- a/drivers/media/rc/ttusbir.c +++ b

[PATCH 5/5] [media] redrat3: fix transmit return value and overrun

2013-01-29 Thread Sean Young
If more than 127 different lengths are transmitted then the driver causes an overrun on sample_lens. Try to send as much as possible and return the amount sent. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/redrat3.c | 18 ++ 1 file changed, 6 insertions(+), 12

Re: [PATCH v2 1/1] video: drm: exynos: Adds display-timing node parsing using video helper function

2013-01-30 Thread Sean Paul
) { + DRM_ERROR(no platform data specified\n); + return -EINVAL; + } + } + + pctrl = devm_pinctrl_get_select_default(dev); + if (IS_ERR(pctrl)) { Will this work for exynos5? AFAICT, there's no pinctrl setup for it. Sean

Re: [PATCH v3 1/1] video: drm: exynos: Adds display-timing node parsing using video helper function

2013-02-01 Thread Sean Paul
think this behavior is actually desired since there is no pinctrl for exynos5 and we don't want it to break. However I think your error should be more along the lines of pinctrl_get_select_default failed. You should also print and return PTR_RET(pctrl) instead of EINVAL. Sean

[PATCH 1/3] [media] redrat3: limit periods to hardware limits

2013-02-16 Thread Sean Young
The redrat hardware cannot handle periods of larger than 32767us, limit appropriately. Also fix memory leak in redrat3_get_timeout. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/redrat3.c | 53 1 files changed, 24 insertions(+), 29

[PATCH 3/3] [media] redrat3: missing endian conversions and warnings

2013-02-16 Thread Sean Young
Spotted by sparse. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/redrat3.c | 71 +++ 1 files changed, 12 insertions(+), 59 deletions(-) diff --git a/drivers/media/rc/redrat3.c b/drivers/media/rc/redrat3.c index ec655b8..12167a6 100644

[PATCH 0/3] [media] redrat3: cleanup driver

2013-02-16 Thread Sean Young
This driver has various minor issues and could be simpler. Sean Young (3): [media] redrat3: limit periods to hardware limits [media] redrat3: remove memcpys and fix unaligned memory access [media] redrat3: missing endian conversions and warnings drivers/media/rc/redrat3.c | 457

[PATCH 2/3] [media] redrat3: remove memcpys and fix unaligned memory access

2013-02-16 Thread Sean Young
In stead of doing a memcpy from #defined offset, declare structs which describe the incoming and outgoing data accurately. Tested on first generation RedRat. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/redrat3.c | 351 +--- 1 files

[PATCH] [media] rc: fix buffer overrun

2012-08-23 Thread Sean Young
[media] rc-core: move timeout and checks to lirc introduced a buffer overrun by passing the number of bytes, rather than the number of samples, to the transmit function. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/ir-lirc-codec.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH] [media] ttusbir: Add USB dependency

2012-08-23 Thread Sean Young
This patch fixes the error: ERROR: usb_speed_string [drivers/usb/core/usbcore.ko] undefined! Reported-by: Guenter Roeck li...@roeck-us.net Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/rc/Kconfig b/drivers

Re: [PATCH 2/8] ir-rx51: Handle signals properly

2012-08-24 Thread Sean Young
On Wed, Aug 22, 2012 at 10:50:35PM +0300, Timo Kokkonen wrote: The lirc-dev expects the ir-code to be transmitted when the write call returns back to the user space. We should not leave TX ongoing no matter what is the reason we return to the user space. Easiest solution for that is to simply

Re: [media] rc-core: move timeout and checks to lirc

2012-08-24 Thread Sean Young
On Sat, Aug 25, 2012 at 12:16:04AM +0200, David Härdeman wrote: On Thu, Aug 16, 2012 at 11:15:14PM +0100, Sean Young wrote: The lirc TX functionality expects the process which writes (TX) data to the lirc dev to sleep until the actual data has been transmitted by the hardware. Since

Re: [PATCH] [media] rc: do not sleep when the driver blocks on IR completion

2012-08-24 Thread Sean Young
On Sat, Aug 25, 2012 at 12:05:18AM +0200, David Härdeman wrote: On Thu, Aug 23, 2012 at 10:18:35PM +0100, Sean Young wrote: Some drivers wait for the IR device to complete sending before returning, so sleeping should not be done. I'm not quite sure what the purpose is. Even if a driver waits

Re: [PATCH] [media] rc: do not sleep when the driver blocks on IR completion

2012-08-25 Thread Sean Young
On Sat, Aug 25, 2012 at 11:25:26AM +0200, David Härdeman wrote: On Sat, Aug 25, 2012 at 12:26:25AM +0100, Sean Young wrote: On Sat, Aug 25, 2012 at 12:05:18AM +0200, David Härdeman wrote: On Thu, Aug 23, 2012 at 10:18:35PM +0100, Sean Young wrote: Some drivers wait for the IR device

[PATCH] [media] iguanair: do not modify transmit buffer

2012-08-25 Thread Sean Young
Since commit [media] rc-core: move timeout and checks to lirc, the incoming buffer is used after the driver transmits. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/iguanair.c | 51 +++-- 1 file changed, 21 insertions(+), 30 deletions

[PATCH] [media] ttusbir: support suspend and resume

2012-08-28 Thread Sean Young
The led is green, not yellow. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/ttusbir.c | 48 ++ 1 file changed, 44 insertions(+), 4 deletions(-) diff --git a/drivers/media/rc/ttusbir.c b/drivers/media/rc/ttusbir.c index 2151927..f0921b5

Re: [PATCHv3 2/9] ir-rx51: Handle signals properly

2012-09-03 Thread Sean Young
safety measure that this driver behaves well. In that case I think you should use wait_event_interruptible() instead. Well, that's what I had there in the first place. With interruptible wait we are left with problem with signals. I was told by Sean Young that the lirc API expects

Re: [RFC PATCH] [media] rc: filter out not allowed protocols when decoding

2012-09-03 Thread Sean Young
On Sat, Sep 01, 2012 at 09:57:09AM +0800, Du, Changbin wrote: From: Du, Changbin changbin...@gmail.com Each rc-raw device has a property allowed_protos stored in structure ir_raw_event_ctrl. But it didn't work because all decoders would be called when decoding. This path makes only allowed

Re: [RFC PATCH] [media] rc: filter out not allowed protocols when decoding

2012-09-04 Thread Sean Young
to different evnets since their protocol is different? At the moment, no. David Hardeman has sent a patch for this: http://patchwork.linuxtv.org/patch/11388/ Sean -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More

Re: [PATCHv3 2/9] ir-rx51: Handle signals properly

2012-09-04 Thread Sean Young
On Mon, Sep 03, 2012 at 11:41:55PM +0200, David Härdeman wrote: Hej, On Mon, Sep 03, 2012 at 01:36:53PM +0100, Sean Young wrote: On Sun, Sep 02, 2012 at 11:08:20PM +0300, Timo Kokkonen wrote: I guess the assumption is to avoid breaking the transmission in the middle in case the process

Re: [patch v2] [media] rc-core: prevent divide by zero bug in s_tx_carrier()

2012-09-09 Thread Sean Young
means disabling carrier modulation, which is used by the mceusb driver. So the check belongs in the individual drivers, as in the original patch. Sean -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo

Re: [PATCHv3 2/9] ir-rx51: Handle signals properly

2012-09-16 Thread Sean Young
together so I don't know when I or anyone else will have this finished. Sean -- 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

[PATCH 1/2] [media] iguanair: cannot send data from the stack

2012-09-28 Thread Sean Young
Note that the firmware already disables the receiver before transmit, there is no need to do this from the driver. Reported-by: Fengguang Wu fengguang...@intel.com Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/iguanair.c | 147 ++-- 1 file

[PATCH 2/2] [media] winbond: remove space from driver name

2012-09-28 Thread Sean Young
/rc_maps.cfg. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/winbond-cir.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/rc/winbond-cir.c b/drivers/media/rc/winbond-cir.c index 30ae1f2..7c9b5f3 100644 --- a/drivers/media/rc/winbond-cir.c +++ b/drivers

Re: [PATCH 2/2] [media] winbond: remove space from driver name

2012-10-12 Thread Sean Young
On Fri, Oct 12, 2012 at 01:16:36AM +0200, David Härdeman wrote: On Fri, Sep 28, 2012 at 09:44:33AM +0100, Sean Young wrote: [root@pequod ~]# udevadm test /sys/class/rc/rc0 -snip- ACTION=add DEVPATH=/devices/pnp0/00:04/rc/rc0 DRV_NAME=Winbond CIR NAME=rc-rc6-mce SUBSYSTEM=rc UDEV_LOG=6

[PATCH] [media] winbond-cir: do not rename input name

2012-10-17 Thread Sean Young
54fd321 [media] winbond: remove space from driver name inadvertently renamed the input device name. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/winbond-cir.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/rc/winbond-cir.c b/drivers/media/rc

Re: [PATCH] rc-core: add separate defines for protocol bitmaps and numbers

2012-10-19 Thread Sean Young
On Thu, Oct 18, 2012 at 11:59:21PM +0200, David Härdeman wrote: On Wed, Oct 17, 2012 at 05:18:56PM +0100, Sean Young wrote: On Fri, Oct 12, 2012 at 01:11:54AM +0200, David Härdeman wrote: The RC_TYPE_* defines are currently used both where a single protocol is expected and where a bitmap

[PATCH] [media] redrat3: errors on unplug

2013-07-08 Thread Sean Young
: 251, should be 0 Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/redrat3.c | 27 --- 1 file changed, 27 deletions(-) diff --git a/drivers/media/rc/redrat3.c b/drivers/media/rc/redrat3.c index 12167a6..3749443 100644 --- a/drivers/media/rc/redrat3.c +++ b

[PATCH] [media] lirc: validate transmission ir data

2013-07-08 Thread Sean Young
The lirc interface allows 255 u32 spaces and pulses, which are usec. If the driver can handle this (e.g. winbond-cir) you can produce hours of meaningless IR data and there is no method of interrupting it. Signed-off-by: Sean Young s...@mess.org --- drivers/media/rc/ir-lirc-codec.c | 11

  1   2   3   4   5   6   7   8   9   10   >