[PATCH 12/49] rc-core: simplify sysfs code

2014-04-03 Thread David Härdeman
Simplify and cleanup the sysfs code a bit. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/ir-raw.c |6 + drivers/media/rc/rc-main.c | 265 2 files changed, 150 insertions(+), 121 deletions(-) diff --git a/drivers/media/rc

[PATCH 10/49] [RFC] rc-core: use the full 32 bits for NEC scancodes

2014-04-03 Thread David Härdeman
later which makes the protocol explicit (and which expects all NEC scancodes to be 32 bit, thereby removing the need for guesswork). Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/pci/saa7134/saa7134-input.c |6 +- drivers/media/rc/img-ir/img-ir-nec.c | 79

[PATCH 14/49] rc-core: rename dev-scanmask to dev-scancode_mask

2014-04-03 Thread David Härdeman
We already have dev-scancode_filter and dev-scancode_wakeup_filter so rename dev-scanmask to dev-scancode_mask for consistency. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/pci/cx88/cx88-input.c |2 +- drivers/media/pci/ttpci/budget-ci.c |2 +- drivers

[PATCH 15/49] rc-core: merge rc5 and streamzap decoders

2014-04-03 Thread David Härdeman
that I've seen with e.g. Sony protocols). Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/Kconfig| 12 -- drivers/media/rc/Makefile |1 drivers/media/rc/ir-rc5-decoder.c | 72 -- drivers/media/rc/ir-rc5-sz-decoder.c

[PATCH 18/49] rc-core: allow chardev to be read

2014-04-03 Thread David Härdeman
will add further events. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-main.c | 154 +++- include/media/rc-core.h| 40 +++ 2 files changed, 189 insertions(+), 5 deletions(-) diff --git a/drivers/media/rc/rc-main.c b

[PATCH 16/49] rc-core: use an IDA rather than a bitmap

2014-04-03 Thread David Härdeman
This patch changes rc-core to use an IDA rather than a bitmap to assign unique numbers to each rc device. This is in preparation for introducing rc-core chardevs. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/ir-raw.c |2 +- drivers/media/rc/rc-main.c | 40

[PATCH 19/49] rc-core: use a kfifo for TX data

2014-04-03 Thread David Härdeman
and after this patch. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/ene_ir.c| 61 + drivers/media/rc/ene_ir.h|9 +-- drivers/media/rc/iguanair.c | 51 - drivers/media/rc/ir-lirc-codec.c | 100

[PATCH 17/49] rc-core: add chardev

2014-04-03 Thread David Härdeman
This patch lays the groundwork for adding a rc-core chardev. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/ir-raw.c |2 drivers/media/rc/rc-main.c | 284 +++- include/media/rc-core.h| 11 +- 3 files changed, 238

[PATCH 13/49] rc-core: remove protocol arrays

2014-04-03 Thread David Härdeman
= rc_allocate_device(); dev-x = a; rc_set_allowed_protocols(dev, RC_BIT_X); dev-z = c; rc_register_device(); There's no real need for the protocols to be an array, so change it back to be consistent (and in preparation for the following patches). Signed-off-by: David

[PATCH 20/49] rc-core: allow chardev to be written

2014-04-03 Thread David Härdeman
Add write functionality to the rc chardev (for use in transmitting remote control commands on capable hardware). The data format of the TX data is probably going to have to be dependent on the rc_driver_type. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-main.c | 71

[PATCH 21/49] rc-core: add ioctl support to the rc chardev

2014-04-03 Thread David Härdeman
. Signed-off-by: David Härdeman da...@hardeman.nu --- Documentation/ioctl/ioctl-number.txt |1 + drivers/media/rc/rc-main.c | 65 ++ include/media/rc-core.h | 19 ++ 3 files changed, 85 insertions(+) diff --git a/Documentation

[PATCH 22/49] rc-core: add an ioctl for getting IR RX settings

2014-04-03 Thread David Härdeman
the appropriate flag as now) or not (in which case it should always be on). Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-main.c | 28 +++ include/media/rc-core.h| 65 2 files changed, 93 insertions(+) diff --git

[PATCH 24/49] rc-core: add an ioctl for setting IR RX settings

2014-04-03 Thread David Härdeman
several operations in struct rc_dev to be deprecated. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/ir-lirc-codec.c | 53 +- drivers/media/rc/rc-core-priv.h |3 ++ drivers/media/rc/rc-main.c | 53

[PATCH 29/49] rc-loopback: add RCIOCSIRTX ioctl support

2014-04-03 Thread David Härdeman
As an example, this patch adds support for the new RCIOCSIRTX ioctl to rc-loopback and removes deprecated functions without a loss in functionality (as LIRC will automatically use the new functions). Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-loopback.c | 59

[PATCH 30/49] rc-core: leave the internals of rc_dev alone

2014-04-03 Thread David Härdeman
Just some debug statements to change drivers/media/video/tm6000/tm6000-input.c Not sure what the driver is trying to do, however, IR handling seems incomplete ATM so deleting the offending parts shouldn't affect functionality Signed-off-by: David Härdeman da

[PATCH 23/49] rc-loopback: add RCIOCGIRRX ioctl support

2014-04-03 Thread David Härdeman
As an example, this patch adds support for the new RCIOCGIRRX ioctl to rc-loopback. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-loopback.c | 22 ++ 1 file changed, 22 insertions(+) diff --git a/drivers/media/rc/rc-loopback.c b/drivers/media/rc

[PATCH 27/49] rc-loopback: add RCIOCGIRTX ioctl support

2014-04-03 Thread David Härdeman
As an example, this patch adds support for the new RCIOCGIRRX ioctl to rc-loopback. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-loopback.c | 26 ++ 1 file changed, 26 insertions(+) diff --git a/drivers/media/rc/rc-loopback.c b/drivers/media

[PATCH 26/49] rc-core: add an ioctl for getting IR TX settings

2014-04-03 Thread David Härdeman
. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-main.c | 12 include/media/rc-core.h| 38 ++ 2 files changed, 50 insertions(+) diff --git a/drivers/media/rc/rc-main.c b/drivers/media/rc/rc-main.c index 3ad565f..611d24d

[PATCH 31/49] rc-core: split rc-main.c into rc-main.c and rc-keytable.c

2014-04-03 Thread David Härdeman
with the absolute minimum changes (making some methods non-static) because it's nigh impossible to review code changes and this code of split in the same patch. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/Makefile |2 drivers/media/rc/rc-core-priv.h | 17 + drivers

[PATCH 28/49] rc-core: add an ioctl for setting IR TX settings

2014-04-03 Thread David Härdeman
functionality as an alternative to the old one and another bunch of operations in struct rc_dev are now deprecated. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/ir-lirc-codec.c | 42 +- drivers/media/rc/rc-main.c | 13

[PATCH 25/49] rc-loopback: add RCIOCSIRRX ioctl support

2014-04-03 Thread David Härdeman
As an example, this patch adds support for the new RCIOCSIRRX ioctl to rc-loopback and removes deprecated functions without a loss in functionality (as LIRC will automatically use the new functions). Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-loopback.c | 84

[PATCH 32/49] rc-core: prepare for multiple keytables

2014-04-03 Thread David Härdeman
to one physical remote controller, each with its own keymap and input device for reporting events to userspace. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-core-priv.h | 16 +- drivers/media/rc/rc-keytable.c | 341 ++- drivers/media

[PATCH 33/49] rc-core: make the keytable of rc_dev an array

2014-04-03 Thread David Härdeman
This is another step towards allowing multiple keytables per rc_dev. struct rc_dev is changed to hold an array of keytables (used later for indexed access to keytables) as well as a list of the same keytables (used for iteration in fast paths). Signed-off-by: David Härdeman da...@hardeman.nu

[PATCH 35/49] rc-core: remove redundant spinlock

2014-04-03 Thread David Härdeman
Remove a redundant spinlock from struct rc_map. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-keytable.c | 43 +--- include/media/rc-core.h|4 ++-- include/media/rc-map.h |1 - 3 files changed, 20 insertions

[PATCH 40/49] rc-ir-raw: simplify locking

2014-04-03 Thread David Härdeman
Simplify and improve the locking in rc-ir-raw by making better use of the existing kfifo functionality and by using RCU where possible. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-core-priv.h |6 +- drivers/media/rc/rc-ir-raw.c| 124

[PATCH 36/49] rc-core: make keytable RCU-friendly

2014-04-03 Thread David Härdeman
allocations can be avoided entirely at the cost of a couple of extra kmalloc() calls when changing a keytable (which is normally done once during boot). Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-keytable.c | 668 +++- include/media/rc

[PATCH 37/49] rc-core: allow empty keymaps

2014-04-03 Thread David Härdeman
Remove the RC_MAP_EMPTY hack and instead allow for empty keymaps. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/i2c/ir-kbd-i2c.c |4 +-- drivers/media/pci/cx88/cx88-input.c|6 - drivers/media/pci/ivtv/ivtv-i2c.c |2 +- drivers

[PATCH 42/49] rc-ir-raw: atomic reads of protocols

2014-04-03 Thread David Härdeman
Use atomic reads to avoid having to take a mutex when getting the bitmask of supported protocols. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-core-priv.h |2 +- drivers/media/rc/rc-ir-raw.c| 12 2 files changed, 5 insertions(+), 9 deletions

[PATCH 41/49] rc-core: rename mutex

2014-04-03 Thread David Härdeman
Having a mutex named lock is a bit misleading. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/img-ir/img-ir-hw.c |4 ++- drivers/media/rc/rc-main.c | 42 ++- include/media/rc-core.h |5 ++-- 3 files changed, 25

[PATCH 38/49] rc-core: rename ir-raw.c

2014-04-03 Thread David Härdeman
Move drivers/media/rc/ir-raw.c to drivers/media/rc/rc-ir-raw.c in preparation for the next patch. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/Makefile|2 +- drivers/media/rc/rc-ir-raw.c |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) rename drivers

[PATCH 39/49] rc-core: make IR raw handling a separate module

2014-04-03 Thread David Härdeman
to skip lots of unnecessary functionality. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/common/siano/smsir.c |5 +- drivers/media/common/siano/smsir.h |2 - drivers/media/i2c/cx25840/cx25840-ir.c |2 - drivers/media/pci/cx23885/cx23885-input.c

[PATCH 34/49] rc-core: add ioctls for adding/removing keytables from userspace

2014-04-03 Thread David Härdeman
As all the basics are now in place, we can finally add the ioctls for adding/removing keytables from userspace. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-core-priv.h |2 - drivers/media/rc/rc-keytable.c |5 ++ drivers/media/rc/rc-main.c | 81

[PATCH 47/49] rc-core: add keytable events

2014-04-03 Thread David Härdeman
Add separe rc device events on keytable addition/removal. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-main.c |2 ++ include/media/rc-core.h|2 ++ 2 files changed, 4 insertions(+) diff --git a/drivers/media/rc/rc-main.c b/drivers/media/rc/rc-main.c index

[PATCH 49/49] rc-core: make rc-core.h userspace friendly

2014-04-03 Thread David Härdeman
A few ifdef __KERNEL__ and some reorganisation to make rc-core.h usable from userspace programs. A split into include/uapi/ might be a good idea later. Signed-off-by: David Härdeman da...@hardeman.nu --- include/media/rc-core.h | 71 --- 1 file

[PATCH 43/49] rc-core: fix various sparse warnings

2014-04-03 Thread David Härdeman
Fix various sparse warnings under drivers/media/rc/*.c, mostly by making functions static. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/fintek-cir.c |4 ++-- drivers/media/rc/imon.c|8 drivers/media/rc/ite-cir.c |4 ++-- drivers/media/rc

[PATCH 45/49] rc-ir-raw: add various rc_events

2014-04-03 Thread David Härdeman
Reporting pulse/space events via the /dev/rc/rcX device node is an important step towards having feature parity with LIRC. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-ir-raw.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/drivers/media/rc/rc-ir

[PATCH 48/49] rc-core: move remaining keytable functions

2014-04-03 Thread David Härdeman
Move some more keytable related functionality over to rc-keytable.c which allows more implementational details to be obscured away. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-core-priv.h | 54 +++- drivers/media/rc/rc-keytable.c | 169

[PATCH 44/49] rc-core: don't report scancodes via input devices

2014-04-03 Thread David Härdeman
The scancode that is reported via the input device(s) is now incomplete (missing the protocol) and redundant. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-keytable.c | 21 + 1 file changed, 5 insertions(+), 16 deletions(-) diff --git a/drivers

Re: [ANNOUNCE] git web interface was changed to cgit

2014-04-02 Thread David Härdeman
On Wed, Apr 02, 2014 at 07:26:51PM -0300, Mauro Carvalho Chehab wrote: I changed today our git web interface from gitweb to cgit, due to seveal reasons: ... Please ping me if you fin any problems on it. Hi, one small thing I've noticed is that the repo links from http://linuxtv.org/cvs.php are

Re: [PATCH 10/11] [RFC] rc-core: use the full 32 bits for NEC scancodes

2014-03-31 Thread David Härdeman
On 2014-03-31 11:44, James Hogan wrote: On 29/03/14 16:11, David Härdeman wrote: Using the full 32 bits for all kinds of NEC scancodes simplifies rc-core and the nec decoder without any loss of functionality. In order to maintain backwards compatibility, some heuristics are added in rc

Re: [PATCH 10/11] [RFC] rc-core: use the full 32 bits for NEC scancodes

2014-03-31 Thread David Härdeman
On 2014-03-31 14:14, Mauro Carvalho Chehab wrote: Em Mon, 31 Mar 2014 12:19:10 +0200 David Härdeman da...@hardeman.nu escreveu: On 2014-03-31 11:44, James Hogan wrote: On 29/03/14 16:11, David Härdeman wrote: Using the full 32 bits for all kinds of NEC scancodes simplifies rc-core

Re: [PATCH 10/11] [RFC] rc-core: use the full 32 bits for NEC scancodes

2014-03-31 Thread David Härdeman
On 2014-03-31 12:56, James Hogan wrote: On 31/03/14 11:19, David Härdeman wrote: On 2014-03-31 11:44, James Hogan wrote: On 29/03/14 16:11, David Härdeman wrote: +/* raw encoding : ddDDaaAA - scan encoding: AAaaDDdd */ +*scancode = swab32((u32)raw); What's the point of the byte

Re: [PATCH 10/11] [RFC] rc-core: use the full 32 bits for NEC scancodes

2014-03-31 Thread David Härdeman
On 2014-03-31 15:15, Mauro Carvalho Chehab wrote: Em Mon, 31 Mar 2014 14:58:10 +0200 David Härdeman da...@hardeman.nu escreveu: On 2014-03-31 14:14, Mauro Carvalho Chehab wrote: The 24 or 32 bits variation is actually a violation of the NEC protocol. Violation is a misnomer. NEC created

Re: [PATCH 10/11] [RFC] rc-core: use the full 32 bits for NEC scancodes

2014-03-31 Thread David Härdeman
being the communication protocol between userspace - kernel if you like. There's no reason to complicate that with bitsizes and/or multiple protocols when a single 32 bit scancode describes exactly everything that the kernel and userspace needs to know. -- David Härdeman -- To unsubscribe from

Re: [PATCH 06/11] rc-core: remove generic scancode filter

2014-03-31 Thread David Härdeman
On Mon, Mar 31, 2014 at 10:29:53AM +0100, James Hogan wrote: On 29/03/14 16:11, David Härdeman wrote: The generic scancode filtering has questionable value and makes it impossible to determine from userspace if there is an actual scancode hw filter present or not. So revert the generic parts

Re: [PATCH 03/11] rc-core: document the protocol type

2014-03-31 Thread David Härdeman
On Mon, Mar 31, 2014 at 10:54:59AM +0100, James Hogan wrote: On 29/03/14 16:11, David Härdeman wrote: Right now the protocol information is not preserved, rc-core gets handed a scancode but has no idea which protocol it corresponds to. This patch (which required reading through the source

Re: [PATCH] rc-core: do not change 32bit NEC scancode format for now

2014-03-31 Thread David Härdeman
On Fri, Mar 28, 2014 at 11:17:09PM +, James Hogan wrote: On Friday 28 March 2014 01:08:56 David Härdeman wrote: On Thu, Mar 27, 2014 at 11:21:23PM +, James Hogan wrote: On Thursday 27 March 2014 22:00:37 David Härdeman wrote: This reverts 18bc17448147e93f31cc9b1a83be49f1224657b2

Re: lmedm04 NEC scancode question

2014-03-30 Thread David Härdeman
On Sun, Mar 30, 2014 at 07:18:20PM +0100, Malcolm Priestley wrote: On 28/03/14 00:38, David Härdeman wrote: I'm trying to make sure that the extended NEC parsing is consistent across drivers and I have a question regarding drivers/media/usb/dvb-usb-v2/lmedm04.c In commit 616a4b83 you changed

[PATCH 09/11] saa7134: NEC scancode fix

2014-03-29 Thread David Härdeman
hint that the change is correct). Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/pci/saa7134/saa7134-input.c |2 - drivers/media/rc/keymaps/rc-behold.c | 68 +++-- 2 files changed, 35 insertions(+), 35 deletions(-) diff --git a/drivers/media

[PATCH 04/11] rc-core: do not change 32bit NEC scancode format for now

2014-03-29 Thread David Härdeman
. Furthermore, I'm convinced that we have to stop playing games trying to decipher the meaning of NEC scancodes (what's the customer/vendor/address, which byte is the MSB, etc). This patch is in preparation for the next few patches in this series. Signed-off-by: David Härdeman da...@hardeman.nu

[PATCH 05/11] rc-core: split dev-s_filter

2014-03-29 Thread David Härdeman
Overloading dev-s_filter to do two different functions (set wakeup filters and generic hardware filters) makes it impossible to tell what the hardware actually supports, so create a separate dev-s_wakeup_filter and make the distinction explicit. Signed-off-by: David Härdeman da...@hardeman.nu

[PATCH 01/11] bt8xx: fixup RC5 decoding

2014-03-29 Thread David Härdeman
that this is perfectly backwards compatible). Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/pci/bt8xx/bttv-input.c | 62 ++- drivers/media/pci/bt8xx/bttvp.h |2 - drivers/media/rc/keymaps/rc-nebula.c | 112 +- 3 files changed, 88

[PATCH 08/11] lmedm04: NEC scancode cleanup

2014-03-29 Thread David Härdeman
= addr; ibuf[4] = ~cmd = not_cmd; ibuf[5] = ~not_cmd = cmd; And the scancode can then be understood as: addr 16 | not_addr 8 | cmd Except for when addr = 0x00 in which case the scancode is simply NEC16: 0x00 8 | cmd Signed-off-by: David Härdeman da

[PATCH 02/11] rc-core: improve ir-kbd-i2c get_key functions

2014-03-29 Thread David Härdeman
The arguments used for ir-kbd-i2c's get_key() functions are not really suited for rc-core and the ir_raw/ir_key distinction is just confusing. Convert all of them to return a protocol/scancode/toggle triple instead. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/i2c/ir-kbd

[PATCH 03/11] rc-core: document the protocol type

2014-03-29 Thread David Härdeman
and makes it easier to e.g. support multiple protocols with one decoder (think rc5 and rc-streamzap). The information isn't used yet so there should be no functional changes. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/i2c/ir-kbd-i2c.c |5 + drivers/media/pci

[PATCH 10/11] [RFC] rc-core: use the full 32 bits for NEC scancodes

2014-03-29 Thread David Härdeman
later which makes the protocol explicit (and which expects all NEC scancodes to be 32 bit, thereby removing the need for guesswork). Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/pci/saa7134/saa7134-input.c |6 +- drivers/media/rc/img-ir/img-ir-nec.c | 79

[PATCH 06/11] rc-core: remove generic scancode filter

2014-03-29 Thread David Härdeman
the valid sysfs files are created in the first place. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-main.c | 66 +--- include/media/rc-core.h|2 + 2 files changed, 45 insertions(+), 23 deletions(-) diff --git a/drivers/media

[PATCH 11/11] [RFC] rc-core: don't throw away protocol information

2014-03-29 Thread David Härdeman
to communicate the protocol to userspace. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-main.c | 184 include/media/rc-core.h| 20 + include/media/rc-map.h |8 +- 3 files changed, 155 insertions(+), 57 deletions

[PATCH 00/11] rc-core: My current patch queue

2014-03-29 Thread David Härdeman
run (by treating NEC16, NECX, NEC32 as NEC32 simply...and converting as necessary...also a new ioctl to make the protocol explicit, which has value for all protocols). --- David Härdeman (11): bt8xx: fixup RC5 decoding rc-core: improve ir-kbd-i2c get_key functions rc-core

Re: [PATCH] rc-core: do not change 32bit NEC scancode format for now

2014-03-29 Thread David Härdeman
On Fri, Mar 28, 2014 at 11:17:09PM +, James Hogan wrote: On Friday 28 March 2014 01:08:56 David Härdeman wrote: drivers/media/usb/dvb-usb-v2/az6007.c drivers/media/usb/dvb-usb-v2/af9035.c drivers/media/usb/dvb-usb-v2/rtl28xxu.c drivers/media/usb/dvb-usb-v2/af9015.c drivers/media/usb

dib0700 NEC scancode question

2014-03-27 Thread David Härdeman
). -- David Härdeman -- 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] rc-core: do not change 32bit NEC scancode format for now

2014-03-27 Thread David Härdeman
. Furthermore, I'm convinced that we have to stop playing games trying to decipher the meaning of NEC scancodes (what's the customer/vendor/address, which byte is the MSB, etc). I'll post separate proposals to that effect later. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/ir-nec

Re: dib0700 NEC scancode question

2014-03-27 Thread David Härdeman
On Thu, Mar 27, 2014 at 01:07:28PM +0100, David Härdeman wrote: Hi Patrick, a quick question regarding the dib0700 driver: in ./media/usb/dvb-usb/dib0700_core.c the RC RX packet is defined as: ... The NEC protocol transmits in the order: ... Does the dib0700 fw really reorder the bytes, or could

Re: dib0700 NEC scancode question

2014-03-27 Thread David Härdeman
On Thu, Mar 27, 2014 at 11:13:35PM +0100, Patrick Boettcher wrote: Hi David, On Thursday 27 March 2014 22:40:41 David Härdeman wrote: On Thu, Mar 27, 2014 at 01:07:28PM +0100, David Härdeman wrote: Hi Patrick, a quick question regarding the dib0700 driver: in ./media/usb/dvb-usb

Re: [PATCH] rc-core: do not change 32bit NEC scancode format for now

2014-03-27 Thread David Härdeman
On Thu, Mar 27, 2014 at 11:21:23PM +, James Hogan wrote: Hi David, On Thursday 27 March 2014 22:00:37 David Härdeman wrote: This reverts 18bc17448147e93f31cc9b1a83be49f1224657b2 The patch ignores the fact that NEC32 scancodes are generated not only in the NEC raw decoder but also

lmedm04 NEC scancode question

2014-03-27 Thread David Härdeman
in which case the scancode is simply NEC16: 0x00 8 | cmd Is my interpretation correct? -- David Härdeman -- 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

NECX scancode consistency

2014-03-27 Thread David Härdeman
is RC_MAP_PIXELVIEW_002T which is used in ./usb/cx231xx/cx231xx-cards.c, but that driver has a one-byte scancode filter so only the last byte is actually compared. -- David Härdeman -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More

Re: [PATCH v4 00/10] media: rc: ImgTec IR decoder driver

2014-03-26 Thread David Härdeman
On 2014-03-26 10:28, James Hogan wrote: On 25/03/14 23:53, David Härdeman wrote: One thing I just noticed...your copyright headers throughout the driver seems a bit...sparse? :) True, I can add the basic: ... to each of the files if you think it's necessary. I think doing so for each *.c

Re: [PATCH 1/5] rc-main: add generic scancode filtering

2014-03-26 Thread David Härdeman
On 2014-03-26 08:08, Antti Seppälä wrote: On 26 March 2014 01:21, David Härdeman da...@hardeman.nu wrote: On Tue, Mar 25, 2014 at 09:12:11AM +, James Hogan wrote: On Tuesday 25 March 2014 00:51:46 David Härdeman wrote: What's the purpose of providing the sw scancode filtering in the case

Re: [PATCH 1/5] rc-main: add generic scancode filtering

2014-03-25 Thread David Härdeman
On Tue, Mar 25, 2014 at 09:12:11AM +, James Hogan wrote: On Tuesday 25 March 2014 00:51:46 David Härdeman wrote: On Fri, Feb 28, 2014 at 11:17:02PM +, James Hogan wrote: Add generic scancode filtering of RC input events, and fall back to permitting any RC_FILTER_NORMAL scancode filter

[PATCH 0/3] Series short description

2014-03-25 Thread David Härdeman
The following series implements... --- David Härdeman (3): bt8xx: fixup RC5 decoding rc-core: improve ir-kbd-i2c get_key functions rc-core: document the protocol type drivers/media/i2c/ir-kbd-i2c.c | 91 +++--- drivers/media/pci/bt8xx/bttv

Re: [PATCH 0/3] Series short description

2014-03-25 Thread David Härdeman
On Wed, Mar 26, 2014 at 12:39:23AM +0100, David Härdeman wrote: The following series implements... Nothing...sorry, I'll send the real patchset soon :) -- 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: [PATCH v4 00/10] media: rc: ImgTec IR decoder driver

2014-03-25 Thread David Härdeman
On Fri, Feb 28, 2014 at 11:28:50PM +, James Hogan wrote: Add a driver for the ImgTec Infrared decoder block. Two separate rc input devices are exposed depending on kernel configuration. One uses the hardware decoder which is set up with timings for a specific protocol and supports mask/value

[PATCH 1/3] bt8xx: fixup RC5 decoding

2014-03-25 Thread David Härdeman
that this is perfectly backwards compatible). Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/pci/bt8xx/bttv-input.c | 62 ++- drivers/media/pci/bt8xx/bttvp.h |2 - drivers/media/rc/keymaps/rc-nebula.c | 112 +- 3 files changed, 88

[PATCH 0/3] rc-core: misc protocol related fixups

2014-03-25 Thread David Härdeman
entertaining) is currently not used in rc-core (but I plan to use it in the future). --- David Härdeman (3): bt8xx: fixup RC5 decoding rc-core: improve ir-kbd-i2c get_key functions rc-core: document the protocol type drivers/media/i2c/ir-kbd-i2c.c | 91

[PATCH 3/3] rc-core: document the protocol type

2014-03-25 Thread David Härdeman
and makes it easier to e.g. support multiple protocols with one decoder (think rc5 and rc-streamzap). The information isn't used yet so there should be no functional changes. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/i2c/ir-kbd-i2c.c |5 + drivers/media/pci

[PATCH 2/3] rc-core: improve ir-kbd-i2c get_key functions

2014-03-25 Thread David Härdeman
The arguments used for ir-kbd-i2c's get_key() functions are not really suited for rc-core and the ir_raw/ir_key distinction is just confusing. Convert all of them to return a protocol/scancode/toggle triple instead. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/i2c/ir-kbd

Re: [PATCH 1/5] rc-main: add generic scancode filtering

2014-03-24 Thread David Härdeman
the new filter */ *filter = local_filter; -- 1.8.3.2 -- 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 -- David Härdeman -- To unsubscribe from

Re: [PATCH v2 0/9] rc: Add IR encode based wakeup filtering

2014-03-24 Thread David Härdeman
-bit-command][8-bit-command] Thanks James -- David Härdeman -- 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: WPC8769L (WEC1020) support in winbond-cir?

2013-10-15 Thread David Härdeman
Juan is the one to talk to. I don't have the WEC1020 hardware and I don't have his experience of adding support for it... -- David Härdeman -- 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

Re: [PATCH] [media] winbond-cir: remove deprecated IRQF_DISABLED

2013-10-14 Thread David Härdeman
On 2013-10-13 08:11, Michael Opdenacker wrote: This patch proposes to remove the use of the IRQF_DISABLED flag It's a NOOP since 2.6.35 and it will be removed one day. Signed-off-by: Michael Opdenacker michael.opdenac...@free-electrons.com Acked-by: David Härdeman da...@hardeman.nu

Re: [PATCH 4/5] [media] winbond: wire up rc feedback led

2013-08-04 Thread David Härdeman
is suspended on suspend. Signed-off-by: Sean Young s...@mess.org Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/Kconfig | 1 - drivers/media/rc/winbond-cir.c | 38 ++ 2 files changed, 6 insertions(+), 33 deletions(-) diff --git a/drivers

[PATCH 0/3] rc-core fixes

2013-03-06 Thread David Härdeman
Three minor patches for rc-core...(against the for_v3.9 branch) --- David Härdeman (3): rc-core: don't treat dev-rc_map.rc_type as a bitmap rc-core: rename ir_input_class to rc_class rc-core: initialize rc-core earlier if built-in drivers/media/i2c/ir-kbd-i2c.c|1

[PATCH 1/3] rc-core: don't treat dev-rc_map.rc_type as a bitmap

2013-03-06 Thread David Härdeman
) as pointed out by James Hogan james.ho...@imgtec.com. Fix both issues by introducing a separate enabled_protocols member to struct rc_dev. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/i2c/ir-kbd-i2c.c|1 + drivers/media/rc/ir-jvc-decoder.c |2 +- drivers

[PATCH 2/3] rc-core: rename ir_input_class to rc_class

2013-03-06 Thread David Härdeman
The name is already misleading and will be more so in the future as the connection to the input subsystem is obscured away further. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-main.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git

[PATCH 3/3] rc-core: initialize rc-core earlier if built-in

2013-03-06 Thread David Härdeman
rc-core is a subsystem so it should be registered earlier if built into the kernel. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-main.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/rc/rc-main.c b/drivers/media/rc/rc-main.c index

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

2013-02-26 Thread David Härdeman
because it was enabled before */ -- 1.7.2.5 -- David Härdeman -- 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: [patch] [media] mceusb: move check earlier to make smatch happy

2013-02-19 Thread David Härdeman
; + /* All mce commands end with an empty packet (0x80) */ cmdbuf[cmdcount++] = MCE_IRDATA_TRAILER; -- David Härdeman -- 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

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

2013-01-08 Thread David Härdeman
of the interrupts. Signed-off-by: Sean Young s...@mess.org Thanks Sean! Acked-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/winbond-cir.c | 27 +++ 1 file changed, 19 insertions(+), 8 deletions(-) diff --git a/drivers/media/rc/winbond-cir.c b/drivers/media/rc

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

2013-01-02 Thread David Härdeman
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(+), 5 deletions(-) Using a resolution of 2us rather than 10us means that

Re: em28xx: msi Digivox ATSC board id [0db0:8810]

2013-01-02 Thread David Härdeman
On Tue, Dec 11, 2012 at 10:59:06PM +0200, Antti Palosaari wrote: Yes, that is. I have said it million times I would like to see that implemented as a one single 4 byte NEC, but it is currently what it is. What I understand David Härdeman has done some work toward that too, but it is not ready

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

2013-01-01 Thread David Härdeman
On Mon, Dec 17, 2012 at 03:15:27PM +, James Hogan wrote: On 12 October 2012 00:11, David Härdeman da...@hardeman.nu wrote: The RC_TYPE_* defines are currently used both where a single protocol is expected and where a bitmap of protocols is expected. This patch tries to separate the two

Re: [PATCH] rc-core: don't return from store_protocols without releasing device mutex

2012-12-25 Thread David Härdeman
hangs waiting on device mutex next time around. Signed-off-by: Sasha Levin sasha.le...@oracle.com Acked-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/rc-main.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/media/rc/rc-main.c b/drivers/media/rc/rc-main.c

[PATCH] siano: fix Kconfig

2012-11-06 Thread David Härdeman
expert). Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/common/siano/Kconfig |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/common/siano/Kconfig b/drivers/media/common/siano/Kconfig index 3cb7823..239b7ba 100644 --- a/drivers/media/common

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

2012-10-18 Thread David Härdeman
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 of protocols is expected. This patch tries to separate the two

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

2012-10-18 Thread David Härdeman
On Wed, Oct 17, 2012 at 04:38:21PM +0100, Sean Young wrote: 54fd321 [media] winbond: remove space from driver name inadvertently renamed the input device name. Signed-off-by: Sean Young s...@mess.org Acked-by: David Härdeman da...@hardeman.nu --- drivers/media/rc/winbond-cir.c | 4 ++-- 1 file

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

2012-10-11 Thread David Härdeman
*/ #define INVALID_SCANCODE 0x7FFF /* Invalid with all protos */ -- 1.7.11.4 -- David Härdeman -- 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

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

2012-09-20 Thread David Härdeman
On Tue, 4 Sep 2012 13:43:56 +0100, Sean Young s...@mess.org wrote: This interface is much better but it's also an ABI change. How should this be handled? Should rc-core expose it's own /dev/rc[0-9] device with its own ioctls? That was the plan yes. I've posted a patchbomb in the past to the

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

2012-09-03 Thread David Härdeman
by forcing it to drip feed long TX sequences. -- David Härdeman -- 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: [PATCH 0/8] rc-core: patches for 3.7

2012-09-03 Thread David Härdeman
On Thu, Aug 30, 2012 at 03:56:12PM -0400, Jarod Wilson wrote: On Sat, Aug 25, 2012 at 11:46:47PM +0200, David Härdeman wrote: This is two minor winbond-cir fixes as well as the first six patches from my previous patchbomb. The latter have been modified so that backwards compatibility

<    1   2   3   4   5   6   >