IR: few fixes, additions and ENE driver
Subsystems (4)
v4 cover IR: few fixes, additions and ENE driver
Hi,
4th revision of my patches below:
Changes:
* more carefull repeat support in NECX protocol
* added documentation for wide band mode ioctl
* fix for 64 bit divide
* updated summary of patches, and preserved few
* Acked/Reviewed by tags you gave me.
Best regards,
Maxim Levitsky
PATCH 1/13 IR: Kconfig fixes
Move IR drives below separate menu.
This allows to disable them.
Also correct a typo.
--- drivers/media/IR/Kconfig | 10 +++++++--- 1 files changed, 7 insertions(+), 3 deletions(-)
drivers/media/IR/Kconfig +7 -3
@@ -1,8 +1,10 @@ -config IR_CORE - tristate +menuconfig IR_CORE + tristate "Infrared remote controller adapters" depends on INPUT default INPUT +if IR_CORE + config VIDEO_IR tristate depends on IR_CORE
@@ -16,7 +18,7 @@ config LIRC Enable this option to build the Linux Infrared Remote Control (LIRC) core device interface driver. The LIRC interface passes raw IR to and from userspace, where the - LIRC daemon handles protocol decoding for IR reception ann + LIRC daemon handles protocol decoding for IR reception and encoding for IR transmitting (aka "blasting"). source "drivers/media/IR/keymaps/Kconfig"
@@ -102,3 +104,5 @@ config IR_MCEUSB To compile this driver as a module, choose M here: the module will be called mceusb. + +endif #IR_CORE
Signature · 1 line
1.7.0.4
PATCH 2/13 IR: minor fixes:
In reply to Maxim Levitsky* lirc: Don't propagate reset event to userspace
* lirc: Remove strange logic from lirc that would make first sample always be pulse
* Make TO_US macro actualy print what it should.
--- drivers/media/IR/ir-core-priv.h | 4 +--- drivers/media/IR/ir-lirc-codec.c | 14 ++++++++------ drivers/media/IR/ir-raw-event.c | 3 +++ 3 files changed, 12 insertions(+), 9 deletions(-)
drivers/media/IR/ir-core-priv.h +1 -3
@@ -76,7 +76,6 @@ struct ir_raw_event_ctrl {struct lirc_codec {struct ir_input_dev *ir_dev;struct lirc_driver *drv; -int lircdata; } lirc; };
@@ -104,10 +103,9 @@ static inline void decrease_duration(struct ir_raw_event *ev, unsigned duration) ev->duration -= duration; } -#define TO_US (duration) (((duration) + 500) / 1000) +#define TO_US (duration) DIV_ROUND_CLOSEST((duration), 1000)#define TO_STR (is_pulse) ((is_pulse) ? "pulse" : "space")#define IS_RESET (ev) (ev.duration == 0) -/* * Routines from ir-sysfs.c - Meant to be called only internally inside * ir-core
drivers/media/IR/ir-lirc-codec.c +8 -6
@@ -32,6 +32,7 @@static int ir_lirc_decode (struct input_dev *input_dev,struct ir_raw_event ev) {struct ir_input_dev *ir_dev =input_get_drvdata (input_dev); +int sample;if (!(ir_dev->raw->enabled_protocols &IR_TYPE_LIRC ))return 0 ;
@@ -39,18 +40,21 @@ static int ir_lirc_decode(struct input_dev *input_dev, struct ir_raw_event ev)if (!ir_dev->raw->lirc.drv || !ir_dev->raw->lirc.drv->rbuf)return -EINVAL ; +if (IS_RESET (ev)) +return 0 ; +IR_dprintk (2 ,"LIRC data transfer started (%uus %s)\n" ,TO_US (ev.duration),TO_STR (ev.pulse)); - ir_dev->raw->lirc.lircdata += ev.duration /1000 ; + + sample = ev.duration /1000 ;if (ev.pulse) - ir_dev->raw->lirc.lircdata |=PULSE_BIT ; + sample |=PULSE_BIT ;lirc_buffer_write (ir_dev->raw->lirc.drv->rbuf, - (unsigned char *) &ir_dev->raw->lirc.lircdata); + (unsigned char *) &sample);wake_up (&ir_dev->raw->lirc.drv->rbuf->wait_poll); - ir_dev->raw->lirc.lircdata =0 ;return 0 ; }
@@ -224,8 +228,6 @@ static int ir_lirc_register(struct input_dev *input_dev) ir_dev->raw->lirc.drv = drv; ir_dev->raw->lirc.ir_dev = ir_dev; - ir_dev->raw->lirc.lircdata =PULSE_MASK ; -return 0 ; lirc_register_failed:
drivers/media/IR/ir-raw-event.c +3 -0
@@ -66,6 +66,9 @@ int ir_raw_event_store(struct input_dev *input_dev, struct ir_raw_event *ev)if (!ir->raw)return -EINVAL ; +IR_dprintk (2 ,"sample: (05%dus %s)\n" , +TO_US (ev->duration),TO_STR (ev->pulse)); +if (kfifo_in (&ir->raw->kfifo, ev,sizeof (*ev)) !=sizeof (*ev))return -ENOMEM ;
Signature · 1 line
1.7.0.4
PATCH 3/13 IR: replace spinlock with mutex.
In reply to Maxim LevitskySome handlers (lirc for example) allocates memory on initialization, doing so in atomic context is cumbersome.
Fixes warning about sleeping function in atomic context.
--- drivers/media/IR/ir-raw-event.c | 28 ++++++++++++++-------------- 1 files changed, 14 insertions(+), 14 deletions(-)
drivers/media/IR/ir-raw-event.c +14 -14
@@ -13,7 +13,7 @@*/ #include <linux/workqueue.h> -#include <linux/spinlock.h> +#include <linux/mutex.h> #include <linux/sched.h> #include "ir-core-priv.h"
@@ -24,7 +24,7 @@static LIST_HEAD (ir_raw_client_list);/* Used to handle IR raw handler extensions */ -static DEFINE_SPINLOCK (ir_raw_handler_lock); +static DEFINE_MUTEX (ir_raw_handler_lock);static LIST_HEAD (ir_raw_handler_list);static u64 available_protocols;
@@ -41,10 +41,10 @@ static void ir_raw_event_work(struct work_struct *work) container_of(work, struct ir_raw_event_ctrl, rx_work);while (kfifo_out (&raw->kfifo, &ev,sizeof (ev)) ==sizeof (ev)) { -spin_lock (&ir_raw_handler_lock); +mutex_lock (&ir_raw_handler_lock);list_for_each_entry (handler, &ir_raw_handler_list, list) handler->decode (raw->input_dev, ev); -spin_unlock (&ir_raw_handler_lock); +mutex_unlock (&ir_raw_handler_lock); raw->prev_ev = ev; } }
@@ -150,9 +150,9 @@ u64 ir_raw_get_allowed_protocols() {u64 protocols; -spin_lock (&ir_raw_handler_lock); +mutex_lock (&ir_raw_handler_lock); protocols = available_protocols; -spin_unlock (&ir_raw_handler_lock); +mutex_unlock (&ir_raw_handler_lock);return protocols; }
@@ -180,12 +180,12 @@ int ir_raw_event_register(struct input_dev *input_dev)return rc; } -spin_lock (&ir_raw_handler_lock); +mutex_lock (&ir_raw_handler_lock);list_add_tail (&ir->raw->list, &ir_raw_client_list);list_for_each_entry (handler, &ir_raw_handler_list, list)if (handler->raw_register) handler->raw_register (ir->raw->input_dev); -spin_unlock (&ir_raw_handler_lock); +mutex_unlock (&ir_raw_handler_lock);return 0 ; }
@@ -200,12 +200,12 @@ void ir_raw_event_unregister(struct input_dev *input_dev)cancel_work_sync (&ir->raw->rx_work); -spin_lock (&ir_raw_handler_lock); +mutex_lock (&ir_raw_handler_lock);list_del (&ir->raw->list);list_for_each_entry (handler, &ir_raw_handler_list, list)if (handler->raw_unregister) handler->raw_unregister (ir->raw->input_dev); -spin_unlock (&ir_raw_handler_lock); +mutex_unlock (&ir_raw_handler_lock);kfifo_free (&ir->raw->kfifo);kfree (ir->raw);
@@ -220,13 +220,13 @@ int ir_raw_handler_register(struct ir_raw_handler *ir_raw_handler) {struct ir_raw_event_ctrl *raw; -spin_lock (&ir_raw_handler_lock); +mutex_lock (&ir_raw_handler_lock);list_add_tail (&ir_raw_handler->list, &ir_raw_handler_list);if (ir_raw_handler->raw_register)list_for_each_entry (raw, &ir_raw_client_list, list) ir_raw_handler->raw_register (raw->input_dev); available_protocols |= ir_raw_handler->protocols; -spin_unlock (&ir_raw_handler_lock); +mutex_unlock (&ir_raw_handler_lock);return 0 ; }
@@ -236,13 +236,13 @@ void ir_raw_handler_unregister(struct ir_raw_handler *ir_raw_handler) {struct ir_raw_event_ctrl *raw; -spin_lock (&ir_raw_handler_lock); +mutex_lock (&ir_raw_handler_lock);list_del (&ir_raw_handler->list);if (ir_raw_handler->raw_unregister)list_for_each_entry (raw, &ir_raw_client_list, list) ir_raw_handler->raw_unregister (raw->input_dev); available_protocols &= ~ir_raw_handler->protocols; -spin_unlock (&ir_raw_handler_lock); +mutex_unlock (&ir_raw_handler_lock); }EXPORT_SYMBOL (ir_raw_handler_unregister);
Signature · 1 line
1.7.0.4
PATCH 4/13 IR: replace workqueue with kthread
In reply to Maxim LevitskyIt is perfectly possible to have ir_raw_event_work running concurently on two cpus, thus we must protect it from that situation.
This stems from the fact that if hardware sends short packets of samples we might end up queueing the work item more times that nessesary.
Such job isn't well suited for a workqueue, so use a kernel thread.
--- drivers/media/IR/ir-core-priv.h | 2 +- drivers/media/IR/ir-raw-event.c | 42 ++++++++++++++++++++++++++++---------- 2 files changed, 32 insertions(+), 12 deletions(-)
drivers/media/IR/ir-core-priv.h +1 -1
@@ -32,7 +32,7 @@ struct ir_raw_handler {struct ir_raw_event_ctrl {struct list_head list;/* to keep track of raw clients */ -struct work_struct rx_work;/* for the rx decoding workqueue */ +struct task_struct *thread;struct kfifo kfifo;/* fifo for the pulse/space durations */ ktime_t last_event;/* when last event occurred */ enum raw_event_type last_type;/* last event type */
drivers/media/IR/ir-raw-event.c +31 -11
87 lines not on this page · raw message
Signature · 1 line
1.7.0.4
PATCH 5/13 IR: JVC: make repeat work
In reply to Maxim LevitskyCurrently, jvc decoder will attempt misdetect next press as a repeat of last keypress, therefore second keypress isn't detected.
--- drivers/media/IR/ir-jvc-decoder.c | 14 +++++++++++++- 1 files changed, 13 insertions(+), 1 deletions(-)
drivers/media/IR/ir-jvc-decoder.c +13 -1
@@ -32,6 +32,7 @@ enum jvc_state {STATE_BIT_SPACE ,STATE_TRAILER_PULSE ,STATE_TRAILER_SPACE , + STATE_CHECK_REPEAT, };/**
@@ -60,6 +61,7 @@ static int ir_jvc_decode(struct input_dev *input_dev, struct ir_raw_event ev)IR_dprintk (2 ,"JVC decode started at state %d (%uus %s)\n" , data->state,TO_US (ev.duration),TO_STR (ev.pulse)); +again:switch (data->state) {case STATE_INACTIVE :
@@ -149,8 +151,18 @@ static int ir_jvc_decode(struct input_dev *input_dev, struct ir_raw_event ev) } data->count =0 ; - data->state =STATE_BIT_PULSE ; + data->state =STATE_CHECK_REPEAT ;return 0 ; + +case STATE_CHECK_REPEAT : +if (!ev.pulse) +break ; + +if (eq_margin (ev.duration,JVC_HEADER_PULSE ,JVC_UNIT /2 )) + data->state =STATE_INACTIVE ; +else + data->state =STATE_BIT_PULSE ; + goto again; } out:
Signature · 1 line
1.7.0.4
PATCH 6/13 IR: nec decoder: fix repeat.
In reply to Maxim LevitskyRepeat space is 4 units, not 8.
Current code would never trigger a repeat.
However that isn't true for NECX, so repeat there must be handled differently.
--- drivers/media/IR/ir-nec-decoder.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
drivers/media/IR/ir-nec-decoder.c +1 -1
@@ -20,7 +20,7 @@#define NEC_HEADER_PULSE (16 * NEC_UNIT)#define NECX_HEADER_PULSE (8 * NEC_UNIT)/* Less common NEC variant */ #define NEC_HEADER_SPACE (8 * NEC_UNIT) -#define NEC_REPEAT_SPACE (8 * NEC_UNIT) +#define NEC_REPEAT_SPACE (4 * NEC_UNIT)#define NEC_BIT_PULSE (1 * NEC_UNIT)#define NEC_BIT_0_SPACE (1 * NEC_UNIT)#define NEC_BIT_1_SPACE (3 * NEC_UNIT)
Signature · 1 line
1.7.0.4
PATCH 7/13 IR: NECX: support repeat
In reply to Maxim LevitskyThis adds support for repeat detecting for NECX variant Tested with uneversal remote
--- drivers/media/IR/ir-core-priv.h | 2 ++ drivers/media/IR/ir-nec-decoder.c | 23 +++++++++++++++++++++-- 2 files changed, 23 insertions(+), 2 deletions(-)
drivers/media/IR/ir-core-priv.h +2 -0
@@ -45,6 +45,8 @@ struct ir_raw_event_ctrl {int state;unsigned count;u32 bits; +bool is_nec_x; +bool necx_repeat; } nec;struct rc5_dec {int state;
drivers/media/IR/ir-nec-decoder.c +21 -2
@@ -26,6 +26,7 @@#define NEC_BIT_1_SPACE (3 * NEC_UNIT)#define NEC_TRAILER_PULSE (1 * NEC_UNIT)#define NEC_TRAILER_SPACE (10 * NEC_UNIT)/* even longer in reality */ +#define NECX_REPEAT_BITS 1enum nec_state {STATE_INACTIVE ,
@@ -67,8 +68,12 @@ static int ir_nec_decode(struct input_dev *input_dev, struct ir_raw_event ev)if (!ev.pulse)break ; -if (!eq_margin (ev.duration,NEC_HEADER_PULSE ,NEC_UNIT /2 ) && - !eq_margin (ev.duration,NECX_HEADER_PULSE ,NEC_UNIT /2 )) +if (eq_margin (ev.duration,NEC_HEADER_PULSE ,NEC_UNIT /2 )) { + data->is_nec_x = false; + data->necx_repeat = false; + }else if (eq_margin (ev.duration,NECX_HEADER_PULSE ,NEC_UNIT /2 )) + data->is_nec_x = true; +else break ; data->count =0 ;
@@ -105,6 +110,17 @@ static int ir_nec_decode(struct input_dev *input_dev, struct ir_raw_event ev)if (ev.pulse)break ; +if (data->necx_repeat && data->count ==NECX_REPEAT_BITS && +geq_margin (ev.duration, +NEC_TRAILER_SPACE ,NEC_UNIT /2 )) { +IR_dprintk (1 ,"Repeat last key\n" ); +ir_repeat (input_dev); + data->state =STATE_INACTIVE ; +return 0 ; + + }else if (data->count >NECX_REPEAT_BITS ) + data->necx_repeat = false; + data->bits <<=1 ;if (eq_margin (ev.duration,NEC_BIT_1_SPACE ,NEC_UNIT /2 )) data->bits |=1 ;
@@ -159,6 +175,9 @@ static int ir_nec_decode(struct input_dev *input_dev, struct ir_raw_event ev)IR_dprintk (1 ,"NEC scancode 0x%04x\n" , scancode); } +if (data->is_nec_x) + data->necx_repeat = true; +ir_keydown (input_dev, scancode,0 ); data->state =STATE_INACTIVE ;return 0 ;
Signature · 1 line
1.7.0.4
PATCH 8/13 IR: Allow not to compile keymaps in.
In reply to Maxim LevitskyCurrently, ir device registration fails if keymap requested by driver is not found. Fix that by always compiling in the empty keymap, and using it as a failback.
--- drivers/media/IR/ir-core-priv.h | 3 +- drivers/media/IR/ir-sysfs.c | 2 + drivers/media/IR/keymaps/Makefile | 1 - drivers/media/IR/keymaps/rc-empty.c | 44 ----------------------------------- drivers/media/IR/rc-map.c | 23 ++++++++++++++++++ include/media/ir-core.h | 8 ++++- 6 files changed, 33 insertions(+), 48 deletions(-) delete mode 100644 drivers/media/IR/keymaps/rc-empty.c
drivers/media/IR/ir-core-priv.h +2 -1
@@ -126,7 +126,8 @@ int ir_raw_handler_register(struct ir_raw_handler *ir_raw_handler);void ir_raw_handler_unregister (struct ir_raw_handler *ir_raw_handler);void ir_raw_init (void ); - +int ir_rcmap_init (void ); +void ir_rcmap_cleanup (void );/* * Decoder initialization code *
drivers/media/IR/ir-sysfs.c +2 -0
@@ -341,6 +341,7 @@ static int __init ir_core_init(void)/* Initialize/load the decoders/keymap code that will be used */ ir_raw_init (); +ir_rcmap_init ();return 0 ; }
@@ -348,6 +349,7 @@ static int __init ir_core_init(void) static void __exit ir_core_exit(void) {class_unregister (&ir_input_class); +ir_rcmap_cleanup (); }module_init (ir_core_init);
drivers/media/IR/keymaps/Makefile +0 -1
@@ -17,7 +17,6 @@ obj-$(CONFIG_RC_MAP) += rc-adstech-dvb-t-pci.o \ rc-dm1105-nec.o \ rc-dntv-live-dvb-t.o \ rc-dntv-live-dvbt-pro.o \ - rc-empty.o \ rc-em-terratec.o \ rc-encore-enltv2.o \ rc-encore-enltv.o \
drivers/media/IR/keymaps/rc-empty.c deleted +0 -44
44 lines not on this page · raw message
drivers/media/IR/rc-map.c +23 -0
@@ -82,3 +82,26 @@ void ir_unregister_map(struct rc_keymap *map) }EXPORT_SYMBOL_GPL (ir_unregister_map); + +static struct ir_scancode empty[] = { + {0x2a ,KEY_COFFEE }, +}; + +static struct rc_keymap empty_map = { + .map = { + .scan = empty, + .size =ARRAY_SIZE (empty), + .ir_type =IR_TYPE_UNKNOWN ,/* Legacy IR type */ + .name =RC_MAP_EMPTY , + } +}; + +int ir_rcmap_init (void ) +{ +return ir_register_map (&empty_map); +} + +void ir_rcmap_cleanup (void ) +{ +ir_unregister_map (&empty_map); +}
include/media/ir-core.h +6 -2
@@ -110,8 +110,12 @@ static inline int ir_input_register(struct input_dev *dev,return -EINVAL ; ir_codes =get_rc_map (map_name); -if (!ir_codes) -return -EINVAL ; +if (!ir_codes) { + ir_codes =get_rc_map (RC_MAP_EMPTY ); + +if (!ir_codes) +return -EINVAL ; + } rc =__ir_input_register (dev, ir_codes, props, driver_name);if (rc <0 )
Signature · 1 line
1.7.0.4
PATCH 9/13 IR: add helper function for hardware with small o/b buffer.
In reply to Maxim LevitskySome ir input devices have small buffer, and interrupt the host each time it is full (or half full)
Add a helper that automaticly handles timeouts, and also automaticly merges samples of same time (space-space) Such samples might be placed by hardware because size of sample in the buffer is small (a byte for example).
Also remove constness from ir_dev_props, because it now contains timeout settings that driver might want to change
--- drivers/media/IR/ir-core-priv.h | 1 + drivers/media/IR/ir-keytable.c | 2 +- drivers/media/IR/ir-raw-event.c | 84 +++++++++++++++++++++++++++++++++++++++ include/media/ir-core.h | 23 +++++++++- 4 files changed, 106 insertions(+), 4 deletions(-)
drivers/media/IR/ir-core-priv.h +1 -0
@@ -41,6 +41,7 @@ struct ir_raw_event_ctrl {/* raw decoder state follows */ struct ir_raw_event prev_ev; +struct ir_raw_event this_ev;struct nec_dec {int state;unsigned count;
drivers/media/IR/ir-keytable.c +1 -1
@@ -428,7 +428,7 @@ static void ir_close(struct input_dev *input_dev)*/ int __ir_input_register (struct input_dev *input_dev,const struct ir_scancode_table *rc_tab, -const struct ir_dev_props *props, +struct ir_dev_props *props,const char *driver_name) {struct ir_input_dev *ir_dev;
drivers/media/IR/ir-raw-event.c +84 -0
90 lines not on this page · raw message
include/media/ir-core.h +20 -3
@@ -41,6 +41,9 @@ enum rc_driver_type {* anything with it. Yet, as the same keycode table can be used with other * devices, a mask is provided to allow its usage. Drivers should generally * leave this field in blank +* @timeout: optional time after which device stops sending data +* @min_timeout: minimum timeout supported by device +* @max_timeout: maximum timeout supported by device * @priv: driver-specific data, to be used on the callbacks * @change_protocol: allow changing the protocol used on hardware decoders * @open: callback to allow drivers to enable polling/irq when IR input device
@@ -50,11 +53,19 @@ enum rc_driver_type {* @s_tx_mask: set transmitter mask (for devices with multiple tx outputs) * @s_tx_carrier: set transmit carrier frequency * @tx_ir: transmit IR +* @s_idle: optional: enable/disable hardware idle mode, upon which, +* device doesn't interrupt host untill it sees IR data */ struct ir_dev_props {enum rc_driver_type driver_type;unsigned long allowed_protos;u32 scanmask; + +u32 timeout; +u32 min_timeout; +u32 max_timeout; + +void *priv;int (*change_protocol)(void *priv,u64 ir_type);int (*open)(void *priv);
@@ -62,6 +73,7 @@ struct ir_dev_props {int (*s_tx_mask)(void *priv,u32 mask);int (*s_tx_carrier)(void *priv,u32 carrier);int (*tx_ir)(void *priv,int *txbuf,u32 n); +void (*s_idle)(void *priv,int enable); };struct ir_input_dev {
@@ -69,9 +81,10 @@ struct ir_input_dev {char *driver_name;/* Name of the driver module */ struct ir_scancode_table rc_tab;/* scan/key table */ unsigned long devno;/* device number */ -const struct ir_dev_props *props;/* Device properties */ +struct ir_dev_props *props;/* Device properties */ struct ir_raw_event_ctrl *raw;/* for raw pulse/space events */ struct input_dev *input_dev;/* the input device associated with this device */ +bool idle;/* key info - needed by IR keycode handlers */ spinlock_t keylock;/* protects the below members */
@@ -95,12 +108,12 @@ enum raw_event_type {/* From ir-keytable.c */ int __ir_input_register (struct input_dev *dev,const struct ir_scancode_table *ir_codes, -const struct ir_dev_props *props, +struct ir_dev_props *props,const char *driver_name);static inline int ir_input_register (struct input_dev *dev,const char *map_name, -const struct ir_dev_props *props, +struct ir_dev_props *props,const char *driver_name) {struct ir_scancode_table *ir_codes;struct ir_input_dev *ir_dev;
@@ -148,6 +161,10 @@ struct ir_raw_event {void ir_raw_event_handle (struct input_dev *input_dev);int ir_raw_event_store (struct input_dev *input_dev,struct ir_raw_event *ev);int ir_raw_event_store_edge (struct input_dev *input_dev,enum raw_event_type type); +int ir_raw_event_store_with_filter (struct input_dev *input_dev, +struct ir_raw_event *ev); +void ir_raw_event_set_idle (struct input_dev *input_dev,int idle); +static inline void ir_raw_event_reset (struct input_dev *input_dev) {struct ir_raw_event ev = { .pulse = false, .duration =0 };
Signature · 1 line
1.7.0.4
Re: [PATCH 09/13] IR: add helper function for hardware with small o/b buffer.
Quoted 8 lines · Maxim Levitsky
Some ir input devices have small buffer, and interrupt the host each time it is full (or half full)
Add a helper that automaticly handles timeouts, and also automaticly merges samples of same time (space-space) Such samples might be placed by hardware because size of sample in the buffer is small (a byte for example).
Quoted 5 lines
Also remove constness from
ir_dev_props, because it now contains timeout settings that driver might want to changeSigned-off-by Maxim Levitsky <[email protected]>Acked-by Jarod Wilson <[email protected]>
Hi Maxim and Jarod,
I've started looking at this patch, and patch 10/13, to work with them and build upon them. I have some comments, below:
Quoted 39 lines
--- drivers/media/IR/ir-core-priv.h | 1 + drivers/media/IR/ir-keytable.c | 2 +- drivers/media/IR/ir-raw-event.c | 84 +++++++++++++++++++++++++++++++++++++++ include/media/ir-core.h | 23 +++++++++- 4 files changed, 106 insertions(+), 4 deletions(-)drivers/media/IR/ir-core-priv.h +1
-0@@ -41,6 +41,7 @@ struct ir_raw_event_ctrl {/* raw decoder state follows */ struct ir_raw_event prev_ev; +struct ir_raw_event this_ev;struct nec_dec {int state;unsigned count;drivers/media/IR/ir-keytable.c +1
-1@@ -428,7 +428,7 @@ static void ir_close(struct input_dev *input_dev)*/ int __ir_input_register (struct input_dev *input_dev,const struct ir_scancode_table *rc_tab, -const struct ir_dev_props *props, +struct ir_dev_props *props,const char *driver_name) {struct ir_input_dev *ir_dev;drivers/media/IR/ir-raw-event.c +0
-0@@ -140,6 +140,90 @@ int ir_raw_event_store_edge(struct input_dev *input_dev, enum raw_event_type typEXPORT_SYMBOL_GPL (ir_raw_event_store_edge);
drivers/media/IR/ir-raw-event.c int ir_raw_event_store_edge(struct input_dev *input_dev, enum raw_event_type typ · line 151
Quoted 10 lines
drivers/media/IR/ir-raw-event.c +9
-0/** +* ir_raw_event_store_with_filter() - pass next pulse/space to decoders with some processing +* @input_dev: the struct input_dev device descriptor +* @type: the type of the event that has occurred +* +* This routine (which may be called from an interrupt context) works +* in similiar manner to ir_raw_event_store_edge. +* This routine is intended for devices with limited internal buffer +* It automerges samples of same type, and handles timeouts +*/
I think this comment might need to explain the filtering being performed a little more explicitly, because "handles timeouts" wasn't enough to go on.
Quoted 1 line
From what I can tell, it performs
a. space aggrregation
b. auto "idle" of the receiver and some state c. gap measurment and gap space event generation d, decoder reset at the end of the gap
(For my needs, c. is very useful, and a. & d. don't hurt.)
What is idle supposed to do for hardware that provides interrupts?
Aside from asking the hardware driver to do something, idle otherwise appears to be used to keep track of being in a gap or not. Did I get that all right?
Quoted 113 lines
drivers/media/IR/ir-raw-event.c +75
-0+int ir_raw_event_store_with_filter (struct input_dev *input_dev, +struct ir_raw_event *ev) +{ +struct ir_input_dev *ir =input_get_drvdata (input_dev); +struct ir_raw_event_ctrl *raw = ir->raw; + +if (!raw || !ir->props) +return -EINVAL ; + +/* Ignore spaces in idle mode */ +if (ir->idle && !ev->pulse) +return 0 ; +else if (ir->idle) +ir_raw_event_set_idle (input_dev,0 ); + +if (!raw->this_ev.duration) { + raw->this_ev = *ev; + }else if (ev->pulse == raw->this_ev.pulse) { + raw->this_ev.duration += ev->duration; + }else { +ir_raw_event_store (input_dev, &raw->this_ev); + raw->this_ev = *ev; + } + +/* Enter idle mode if nessesary */ +if (!ev->pulse && ir->props->timeout && + raw->this_ev.duration >= ir->props->timeout) +ir_raw_event_set_idle (input_dev,1 ); +return 0 ; +} +EXPORT_SYMBOL_GPL (ir_raw_event_store_with_filter); + +void ir_raw_event_set_idle (struct input_dev *input_dev,int idle) +{ +struct ir_input_dev *ir =input_get_drvdata (input_dev); +struct ir_raw_event_ctrl *raw = ir->raw; +ktime_t now; +u64 delta; + +if (!ir->props) +return ; + +if (!ir->raw) + goto out; + +if (idle) { +IR_dprintk (2 ,"enter idle mode\n" ); + raw->last_event =ktime_get (); + }else { +IR_dprintk (2 ,"exit idle mode\n" ); + + now =ktime_get (); + delta =ktime_to_ns (ktime_sub (now, ir->raw->last_event)); + +WARN_ON (raw->this_ev.pulse); + + raw->this_ev.duration = +min (raw->this_ev.duration + delta, + (u64 )IR_MAX_DURATION ); + +ir_raw_event_store (input_dev, &raw->this_ev); + +if (raw->this_ev.duration ==IR_MAX_DURATION ) +ir_raw_event_reset (input_dev); + + raw->this_ev.duration =0 ; + } +out: +if (ir->props->s_idle) + ir->props->s_idle (ir->props->priv, idle); + ir->idle = idle; +} +EXPORT_SYMBOL_GPL (ir_raw_event_set_idle); + +/** * ir_raw_event_handle() - schedules the decoding of stored ir data * @input_dev: the struct input_dev device descriptor * include/media/ir-core.h +11
-0@@ -41,6 +41,9 @@ enum rc_driver_type {* anything with it. Yet, as the same keycode table can be used with other * devices, a mask is provided to allow its usage. Drivers should generally * leave this field in blank +* @timeout: optional time after which device stops sending data +* @min_timeout: minimum timeout supported by device +* @max_timeout: maximum timeout supported by device * @priv: driver-specific data, to be used on the callbacks * @change_protocol: allow changing the protocol used on hardware decoders * @open: callback to allow drivers to enable polling/irq when IR input device @@ -50,11 +53,19 @@ enum rc_driver_type {* @s_tx_mask: set transmitter mask (for devices with multiple tx outputs) * @s_tx_carrier: set transmit carrier frequency * @tx_ir: transmit IR +* @s_idle: optional: enable/disable hardware idle mode, upon which, +* device doesn't interrupt host untill it sees IR data */ struct ir_dev_props {enum rc_driver_type driver_type;unsigned long allowed_protos;u32 scanmask; + +u32 timeout; +u32 min_timeout; +u32 max_timeout; + +void *priv;int (*change_protocol)(void *priv,u64 ir_type);int (*open)(void *priv);
include/media/ir-core.h struct ir_dev_props { · line 77
Quoted 5 lines
include/media/ir-core.h +1
-0@@ -62,6 +73,7 @@ struct ir_dev_props {int (*s_tx_mask)(void *priv,u32 mask);int (*s_tx_carrier)(void *priv,u32 carrier);int (*tx_ir)(void *priv,int *txbuf,u32 n); +void (*s_idle)(void *priv,int enable); };
As I begin to look at using these changes for setting parameters and storing them, I realize I've been down this road before with the struct v4l2_subdev_ir_ops found here:
http://git.linuxtv.org/awalls/v4l-dvb.git?a=blob;f=include/media/v4l2-subdev.h;h=08880dd15d2fe6582865633f297a5e0293becf82;hb=wilson-levitsky#l391
http://git.linuxtv.org/awalls/v4l-dvb.git?a=blob;f=include/media/v4l2-subdev.h;h=08880dd15d2fe6582865633f297a5e0293becf82;hb=wilson-levitsky#l339
and struct v4l2_subdev_ir_params found here:
I had originally started with an approach of adding a specialized callback for everything. However, besides being large and clumsy, Hans and Mauro shot it down.
In retrospect I'm glad they did - the final implementation was 7 simple callbacks (3 for Rx, 3 for Tx and 1 for IRQ handling) and one parameter form.
Example parameter setting can be found here:
http://git.linuxtv.org/awalls/v4l-dvb.git?a=blob;f=drivers/media/video/cx23885/cx23885-input.c;h=bb61870b8d6ed39d25c11aa676b55bd0a94dc235;hb=wilson-levitsky#l105
http://git.linuxtv.org/awalls/v4l-dvb.git?a=blob;f=drivers/media/video/cx23885/cx23885-input.c;h=bb61870b8d6ed39d25c11aa676b55bd0a94dc235;hb=wilson-levitsky#l135
The parameter form can also be used to hold the state of the transmitter and receiver internal to the driver:
include/media/ir-core.h struct ir_input_dev { · line 84
Quoted 7 lines
include/media/ir-core.h +1
-1struct ir_input_dev {@@ -69,9 +81,10 @@ struct ir_input_dev {char *driver_name;/* Name of the driver module */ struct ir_scancode_table rc_tab;/* scan/key table */ unsigned long devno;/* device number */ -const struct ir_dev_props *props;/* Device properties */ +struct ir_dev_props *props;/* Device properties */
So I don't think the struct ir_dev_props structure is the right place to be storing current operating parameters. IMO, operating parameters stored in the ir_dev_props are "too far" from the lower level driver, and are essentially mirroring what the low level driver should be tracking internally as it's own state anyway.
So in summary, I think we need to keep the opertions in struct ir_dev_props simple using ,get_parameters() and .set_parameters() to contol the lower level driver and to fetch, retrieve, and store parameters.
Regards,
Andy
Quoted 32 lines
include/media/ir-core.h +7
-2struct ir_raw_event_ctrl *raw;/* for raw pulse/space events */ struct input_dev *input_dev;/* the input device associated with this device */ +bool idle;/* key info - needed by IR keycode handlers */ spinlock_t keylock;/* protects the below members */ @@ -95,12 +108,12 @@ enum raw_event_type {/* From ir-keytable.c */ int __ir_input_register (struct input_dev *dev,const struct ir_scancode_table *ir_codes, -const struct ir_dev_props *props, +struct ir_dev_props *props,const char *driver_name);static inline int ir_input_register (struct input_dev *dev,const char *map_name, -const struct ir_dev_props *props, +struct ir_dev_props *props,const char *driver_name) {struct ir_scancode_table *ir_codes;struct ir_input_dev *ir_dev;@@ -148,6 +161,10 @@ struct ir_raw_event {void ir_raw_event_handle (struct input_dev *input_dev);int ir_raw_event_store (struct input_dev *input_dev,struct ir_raw_event *ev);int ir_raw_event_store_edge (struct input_dev *input_dev,enum raw_event_type type); +int ir_raw_event_store_with_filter (struct input_dev *input_dev, +struct ir_raw_event *ev); +void ir_raw_event_set_idle (struct input_dev *input_dev,int idle); +static inline void ir_raw_event_reset (struct input_dev *input_dev) {struct ir_raw_event ev = { .pulse = false, .duration =0 };
Re: [PATCH 09/13] IR: add helper function for hardware with small o/b buffer.
Quoted 77 lines · Andy Walls
Quoted 8 lines · Maxim Levitsky
Some ir input devices have small buffer, and interrupt the host each time it is full (or half full)
Add a helper that automaticly handles timeouts, and also automaticly merges samples of same time (space-space) Such samples might be placed by hardware because size of sample in the buffer is small (a byte for example).
Quoted 5 lines
Also remove constness from
ir_dev_props, because it now contains timeout settings that driver might want to changeSigned-off-by Maxim Levitsky <[email protected]>Acked-by Jarod Wilson <[email protected]>Hi Maxim and Jarod,
I've started looking at this patch, and patch 10/13, to work with them and build upon them. I have some comments, below:
Quoted 39 lines
--- drivers/media/IR/ir-core-priv.h | 1 + drivers/media/IR/ir-keytable.c | 2 +- drivers/media/IR/ir-raw-event.c | 84 +++++++++++++++++++++++++++++++++++++++ include/media/ir-core.h | 23 +++++++++- 4 files changed, 106 insertions(+), 4 deletions(-)drivers/media/IR/ir-core-priv.h +1
-0@@ -41,6 +41,7 @@ struct ir_raw_event_ctrl {/* raw decoder state follows */ struct ir_raw_event prev_ev; +struct ir_raw_event this_ev;struct nec_dec {int state;unsigned count;drivers/media/IR/ir-keytable.c +1
-1@@ -428,7 +428,7 @@ static void ir_close(struct input_dev *input_dev)*/ int __ir_input_register (struct input_dev *input_dev,const struct ir_scancode_table *rc_tab, -const struct ir_dev_props *props, +struct ir_dev_props *props,const char *driver_name) {struct ir_input_dev *ir_dev;drivers/media/IR/ir-raw-event.c +0
-0@@ -140,6 +140,90 @@ int ir_raw_event_store_edge(struct input_dev *input_dev, enum raw_event_type typEXPORT_SYMBOL_GPL (ir_raw_event_store_edge);drivers/media/IR/ir-raw-event.c int ir_raw_event_store_edge(struct input_dev *input_dev, enum raw_event_type typ · line 151
Quoted 10 lines
drivers/media/IR/ir-raw-event.c +9
-0/** +* ir_raw_event_store_with_filter() - pass next pulse/space to decoders with some processing +* @input_dev: the struct input_dev device descriptor +* @type: the type of the event that has occurred +* +* This routine (which may be called from an interrupt context) works +* in similiar manner to ir_raw_event_store_edge. +* This routine is intended for devices with limited internal buffer +* It automerges samples of same type, and handles timeouts +*/ I think this comment might need to explain the filtering being performed a little more explicitly, because "handles timeouts" wasn't enough to go on.
Quoted 1 line
From what I can tell, it performs
Quoted 8 lines
a. space aggrregation
b. auto "idle" of the receiver and some state c. gap measurment and gap space event generation d, decoder reset at the end of the gap(For my needs, c. is very useful, and a. & d. don't hurt.)
What is idle supposed to do for hardware that provides interrupts?
In my case, I can program hardware not to interrupt host until it has a sample that isn't a overflow sample
Exaplanation what overflow sample is:
Hardware stores samples in small 8 bytes buffer, and has a flag which half of this buffer to read.
Each sample is 8 bits, out of which 7 are timing and 8th is pulse/space. So larger sample is sample period * 127.
If hardware wants to send larger sample, it sends this sample, and another.
When I enable idle mode, it won't send sample with 127 within them. Note that this is only supported on my receiver. Newer receivers don't have that function, but rather stop sending samples on their own after about 250 ms.
Quoted 4 lines
Aside from asking the hardware driver to do something, idle otherwise appears to be used to keep track of being in a gap or not. Did I get that all right?
Yes of course.
Quoted 165 lines
Quoted 114 lines
drivers/media/IR/ir-raw-event.c +75
-0+int ir_raw_event_store_with_filter (struct input_dev *input_dev, +struct ir_raw_event *ev) +{ +struct ir_input_dev *ir =input_get_drvdata (input_dev); +struct ir_raw_event_ctrl *raw = ir->raw; + +if (!raw || !ir->props) +return -EINVAL ; + +/* Ignore spaces in idle mode */ +if (ir->idle && !ev->pulse) +return 0 ; +else if (ir->idle) +ir_raw_event_set_idle (input_dev,0 ); + +if (!raw->this_ev.duration) { + raw->this_ev = *ev; + }else if (ev->pulse == raw->this_ev.pulse) { + raw->this_ev.duration += ev->duration; + }else { +ir_raw_event_store (input_dev, &raw->this_ev); + raw->this_ev = *ev; + } + +/* Enter idle mode if nessesary */ +if (!ev->pulse && ir->props->timeout && + raw->this_ev.duration >= ir->props->timeout) +ir_raw_event_set_idle (input_dev,1 ); +return 0 ; +} +EXPORT_SYMBOL_GPL (ir_raw_event_store_with_filter); + +void ir_raw_event_set_idle (struct input_dev *input_dev,int idle) +{ +struct ir_input_dev *ir =input_get_drvdata (input_dev); +struct ir_raw_event_ctrl *raw = ir->raw; +ktime_t now; +u64 delta; + +if (!ir->props) +return ; + +if (!ir->raw) + goto out; + +if (idle) { +IR_dprintk (2 ,"enter idle mode\n" ); + raw->last_event =ktime_get (); + }else { +IR_dprintk (2 ,"exit idle mode\n" ); + + now =ktime_get (); + delta =ktime_to_ns (ktime_sub (now, ir->raw->last_event)); + +WARN_ON (raw->this_ev.pulse); + + raw->this_ev.duration = +min (raw->this_ev.duration + delta, + (u64 )IR_MAX_DURATION ); + +ir_raw_event_store (input_dev, &raw->this_ev); + +if (raw->this_ev.duration ==IR_MAX_DURATION ) +ir_raw_event_reset (input_dev); + + raw->this_ev.duration =0 ; + } +out: +if (ir->props->s_idle) + ir->props->s_idle (ir->props->priv, idle); + ir->idle = idle; +} +EXPORT_SYMBOL_GPL (ir_raw_event_set_idle); + +/** * ir_raw_event_handle() - schedules the decoding of stored ir data * @input_dev: the struct input_dev device descriptor * include/media/ir-core.h +11
-0@@ -41,6 +41,9 @@ enum rc_driver_type {* anything with it. Yet, as the same keycode table can be used with other * devices, a mask is provided to allow its usage. Drivers should generally * leave this field in blank +* @timeout: optional time after which device stops sending data +* @min_timeout: minimum timeout supported by device +* @max_timeout: maximum timeout supported by device * @priv: driver-specific data, to be used on the callbacks * @change_protocol: allow changing the protocol used on hardware decoders * @open: callback to allow drivers to enable polling/irq when IR input device @@ -50,11 +53,19 @@ enum rc_driver_type {* @s_tx_mask: set transmitter mask (for devices with multiple tx outputs) * @s_tx_carrier: set transmit carrier frequency * @tx_ir: transmit IR +* @s_idle: optional: enable/disable hardware idle mode, upon which, +* device doesn't interrupt host untill it sees IR data */ struct ir_dev_props {enum rc_driver_type driver_type;unsigned long allowed_protos;u32 scanmask; + +u32 timeout; +u32 min_timeout; +u32 max_timeout; + +void *priv;int (*change_protocol)(void *priv,u64 ir_type);int (*open)(void *priv);include/media/ir-core.h struct ir_dev_props { · line 77
Quoted 5 lines
include/media/ir-core.h +1
-0@@ -62,6 +73,7 @@ struct ir_dev_props {int (*s_tx_mask)(void *priv,u32 mask);int (*s_tx_carrier)(void *priv,u32 carrier);int (*tx_ir)(void *priv,int *txbuf,u32 n); +void (*s_idle)(void *priv,int enable); };As I begin to look at using these changes for setting parameters and storing them, I realize I've been down this road before with the struct
v4l2_subdev_ir_opsfound here:http://git.linuxtv.org/awalls/v4l-dvb.git?a=blob;f=include/media/v4l2-subdev.h;h=08880dd15d2fe6582865633f297a5e0293becf82;hb=wilson-levitsky#l391
http://git.linuxtv.org/awalls/v4l-dvb.git?a=blob;f=include/media/v4l2-subdev.h;h=08880dd15d2fe6582865633f297a5e0293becf82;hb=wilson-levitsky#l339and struct
v4l2_subdev_ir_paramsfound here:I had originally started with an approach of adding a specialized callback for everything. However, besides being large and clumsy, Hans and Mauro shot it down.
In retrospect I'm glad they did - the final implementation was 7 simple callbacks (3 for Rx, 3 for Tx and 1 for IRQ handling) and one parameter form.
Example parameter setting can be found here:
http://git.linuxtv.org/awalls/v4l-dvb.git?a=blob;f=drivers/media/video/cx23885/cx23885-input.c;h=bb61870b8d6ed39d25c11aa676b55bd0a94dc235;hb=wilson-levitsky#l105
http://git.linuxtv.org/awalls/v4l-dvb.git?a=blob;f=drivers/media/video/cx23885/cx23885-input.c;h=bb61870b8d6ed39d25c11aa676b55bd0a94dc235;hb=wilson-levitsky#l135The parameter form can also be used to hold the state of the transmitter and receiver internal to the driver:
include/media/ir-core.h struct ir_input_dev { · line 84
Quoted 7 lines
include/media/ir-core.h +1
-1struct ir_input_dev {@@ -69,9 +81,10 @@ struct ir_input_dev {char *driver_name;/* Name of the driver module */ struct ir_scancode_table rc_tab;/* scan/key table */ unsigned long devno;/* device number */ -const struct ir_dev_props *props;/* Device properties */ +struct ir_dev_props *props;/* Device properties */ So I don't think the struct
ir_dev_propsstructure is the right place to be storing current operating parameters. IMO, operating parameters stored in their_dev_propsare "too far" from the lower level driver, and are essentially mirroring what the low level driver should be tracking internally as it's own state anyway.
Quoted 4 lines
So in summary, I think we need to keep the opertions in struct
ir_dev_propssimple using ,get_parameters() and .set_parameters() to contol the lower level driver and to fetch, retrieve, and store parameters.
There is one thing that callbacks are better that one large .set_param. if callback is set to NULL, it obvious that driver doesn't support the feature....
Otherwise we will need some 'feature' bitmask, will need to translate it into lirc features...
Mario, Hans, what objections did you have against that?
Best regards,
Maxim Levitsky
Quoted 40 lines
Regards,
AndyQuoted 32 lines
include/media/ir-core.h +7
-2struct ir_raw_event_ctrl *raw;/* for raw pulse/space events */ struct input_dev *input_dev;/* the input device associated with this device */ +bool idle;/* key info - needed by IR keycode handlers */ spinlock_t keylock;/* protects the below members */ @@ -95,12 +108,12 @@ enum raw_event_type {/* From ir-keytable.c */ int __ir_input_register (struct input_dev *dev,const struct ir_scancode_table *ir_codes, -const struct ir_dev_props *props, +struct ir_dev_props *props,const char *driver_name);static inline int ir_input_register (struct input_dev *dev,const char *map_name, -const struct ir_dev_props *props, +struct ir_dev_props *props,const char *driver_name) {struct ir_scancode_table *ir_codes;struct ir_input_dev *ir_dev;@@ -148,6 +161,10 @@ struct ir_raw_event {void ir_raw_event_handle (struct input_dev *input_dev);int ir_raw_event_store (struct input_dev *input_dev,struct ir_raw_event *ev);int ir_raw_event_store_edge (struct input_dev *input_dev,enum raw_event_type type); +int ir_raw_event_store_with_filter (struct input_dev *input_dev, +struct ir_raw_event *ev); +void ir_raw_event_set_idle (struct input_dev *input_dev,int idle); +static inline void ir_raw_event_reset (struct input_dev *input_dev) {struct ir_raw_event ev = { .pulse = false, .duration =0 };
Re: [PATCH 09/13] IR: add helper function for hardware with small o/b buffer.
In reply to Andy WallsQuoted 1 line · Andy Walls
On Sat, 2010-07-31 at 17:59 +0300, Maxim Levitsky wrote:
...
Quoted 15 lines
include/media/ir-core.h struct ir_input_dev { · line 84
Quoted 7 lines
struct
ir_input_dev{include/media/ir-core.h +1
-1@@ -69,9 +81,10 @@ struct ir_input_dev {char *driver_name;/* Name of the driver module */ struct ir_scancode_table rc_tab;/* scan/key table */ unsigned long devno;/* device number */ -const struct ir_dev_props *props;/* Device properties */ +struct ir_dev_props *props;/* Device properties */ So I don't think the struct
ir_dev_propsstructure is the right place to be storing current operating parameters. IMO, operating parameters stored in their_dev_propsare "too far" from the lower level driver, and are essentially mirroring what the low level driver should be tracking internally as it's own state anyway.
Quoted 4 lines
So in summary, I think we need to keep the opertions in struct
ir_dev_propssimple using ,get_parameters() and .set_parameters() to contol the lower level driver and to fetch, retrieve, and store parameters.
Yeah, I neglected to consider this change the first pass through an earlier revision. Making props modifiable on the fly does feel like we're mixing up hardware features with hardware state, and perhaps the on-the-fly-modifiable state bits should be stored somewhere else.
Signature · 2 lines
Jarod Wilson [email protected]
PATCH 10/13 IR: extend interfaces to support more device settings
In reply to Maxim LevitskyLIRC: add new IOCTL that enables learning mode (wide band receiver) Still missing features: carrier report & timeout reports. Will need to pack these into ir_raw_event
--- .../DocBook/v4l/lirc_device_interface.xml | 16 +++ drivers/media/IR/ir-core-priv.h | 1 + drivers/media/IR/ir-lirc-codec.c | 112 ++++++++++++++++---- include/media/ir-core.h | 12 ++- include/media/lirc.h | 5 +- 5 files changed, 125 insertions(+), 21 deletions(-)
Documentation/DocBook/v4l/lirc_device_interface.xml +16 -0
@@ -229,6 +229,22 @@ on working with the default settings initially.</para> and LIRC_SETUP_END. Drivers can also choose to ignore these ioctls.</para> </listitem> </varlistentry> + <varlistentry> + <term>LIRC_SET_WIDEBAND_RECEIVER</term> + <listitem> + <para>Some receivers are equipped with special wide band receiver which is intended + to be used to learn output of existing remote. + Calling that ioctl with (1) will enable it, and with (0) disable it. + This might be useful of receivers that have otherwise narrow band receiver + that prevents them to be used with some remotes. + Wide band receiver might also be more precise + On the other hand its disadvantage it usually reduced range of reception. + Note: wide band receiver might be implictly enabled if you enable + carrier reports. In that case it will be disabled as soon as you disable + carrier reports. Trying to disable wide band receiver while carrier + reports are active will do nothing.</para> + </listitem> + </varlistentry> </variablelist> </section>
drivers/media/IR/ir-core-priv.h +1 -0
@@ -79,6 +79,7 @@ struct ir_raw_event_ctrl {struct lirc_codec {struct ir_input_dev *ir_dev;struct lirc_driver *drv; +int carrier_low; } lirc; };
drivers/media/IR/ir-lirc-codec.c +93 -19
173 lines not on this page · raw message
include/media/ir-core.h +11 -1
@@ -44,6 +44,8 @@ enum rc_driver_type {* @timeout: optional time after which device stops sending data * @min_timeout: minimum timeout supported by device * @max_timeout: maximum timeout supported by device +* @rx_resolution : resolution (in ns) of input sampler +* @tx_resolution: resolution (in ns) of output sampler * @priv: driver-specific data, to be used on the callbacks * @change_protocol: allow changing the protocol used on hardware decoders * @open: callback to allow drivers to enable polling/irq when IR input device
@@ -52,9 +54,12 @@ enum rc_driver_type {* is opened. * @s_tx_mask: set transmitter mask (for devices with multiple tx outputs) * @s_tx_carrier: set transmit carrier frequency +* @s_tx_duty_cycle: set transmit duty cycle (0% - 100%) +* @s_rx_carrier: inform driver about carrier it is expected to handle * @tx_ir: transmit IR * @s_idle: optional: enable/disable hardware idle mode, upon which, -* device doesn't interrupt host untill it sees IR data +device doesn't interrupt host until it sees IR pulses +* @s_learning_mode: enable wide band receiver used for learning */ struct ir_dev_props {enum rc_driver_type driver_type;
@@ -65,6 +70,8 @@ struct ir_dev_props {u32 min_timeout;u32 max_timeout; +u32 rx_resolution; +u32 tx_resolution;void *priv;int (*change_protocol)(void *priv,u64 ir_type);
@@ -72,8 +79,11 @@ struct ir_dev_props {void (*close)(void *priv);int (*s_tx_mask)(void *priv,u32 mask);int (*s_tx_carrier)(void *priv,u32 carrier); +int (*s_tx_duty_cycle)(void *priv,u32 duty_cycle); +int (*s_rx_carrier_range)(void *priv,u32 min,u32 max);int (*tx_ir)(void *priv,int *txbuf,u32 n);void (*s_idle)(void *priv,int enable); +int (*s_learning_mode)(void *priv,int enable); };struct ir_input_dev {
include/media/lirc.h +4 -1
@@ -77,6 +77,7 @@#define LIRC_CAN_SET_REC_FILTER 0x08000000#define LIRC_CAN_MEASURE_CARRIER 0x02000000 +#define LIRC_CAN_USE_WIDEBAND_RECEIVER 0x04000000#define LIRC_CAN_SEND (x) ((x)&LIRC_CAN_SEND_MASK)#define LIRC_CAN_REC (x) ((x)&LIRC_CAN_REC_MASK)
@@ -145,7 +146,7 @@* if enabled from the next key press on the driver will send * LIRC_MODE2_FREQUENCY packets */ -#define LIRC_SET_MEASURE_CARRIER_MODE _IOW('i', 0x0000001d, __u32) +#define LIRC_SET_MEASURE_CARRIER_MODE _IOW('i', 0x0000001d, __u32)/* * to set a range use
@@ -162,4 +163,6 @@#define LIRC_SETUP_START _IO('i', 0x00000021)#define LIRC_SETUP_END _IO('i', 0x00000022) +#define LIRC_SET_WIDEBAND_RECEIVER _IOW('i', 0x00000023, __u32) +#endif
Signature · 1 line
1.7.0.4
PATCH 11/13 IR: report unknown scancodes the in-kernel decoders found.
In reply to Maxim LevitskyThis way it is possible to use evtest to create keymap for unknown remote.
--- drivers/media/IR/ir-keytable.c | 8 ++++++++ 1 files changed, 8 insertions(+), 0 deletions(-)
drivers/media/IR/ir-keytable.c +8 -0
@@ -339,6 +339,8 @@ void ir_repeat(struct input_dev *dev)spin_lock_irqsave (&ir->keylock, flags); +input_event (dev,EV_MSC ,MSC_SCAN , ir->last_scancode); +if (!ir->keypressed) goto out;
@@ -370,6 +372,8 @@ void ir_keydown(struct input_dev *dev, int scancode, u8 toggle)spin_lock_irqsave (&ir->keylock, flags); +input_event (dev,EV_MSC ,MSC_SCAN , scancode); +/* Repeat event? */ if (ir->keypressed && ir->last_scancode == scancode &&
@@ -383,9 +387,11 @@ void ir_keydown(struct input_dev *dev, int scancode, u8 toggle) ir->last_toggle = toggle; ir->last_keycode = keycode; +if (keycode ==KEY_RESERVED ) goto out; +/* Register a keypress */ ir->keypressed = true; IR_dprintk(1, "%s: key down event, key 0x%04x, scancode 0x%04x\n",
@@ -480,6 +486,8 @@ int __ir_input_register(struct input_dev *input_dev,set_bit (EV_KEY , input_dev->evbit);set_bit (EV_REP , input_dev->evbit); +set_bit (EV_MSC , input_dev->evbit); +set_bit (MSC_SCAN , input_dev->mscbit);if (ir_setkeytable (input_dev, &ir_dev->rc_tab, rc_tab)) { rc = -ENOMEM ;
Signature · 1 line
1.7.0.4
PATCH 12/13 STAGING: remove lirc_ene0100 driver.
In reply to Maxim LevitskyAdd latest unported version of this driver to media/IR. Next patch will port it to ir core.
--- drivers/media/IR/ene_ir.c | 1176 +++++++++++++++++++++++++++++++++++ drivers/media/IR/ene_ir.h | 240 +++++++ drivers/staging/lirc/Kconfig | 8 - drivers/staging/lirc/Makefile | 1 - drivers/staging/lirc/lirc_ene0100.c | 646 ------------------- drivers/staging/lirc/lirc_ene0100.h | 169 ----- 6 files changed, 1416 insertions(+), 824 deletions(-) create mode 100644 drivers/media/IR/ene_ir.c create mode 100644 drivers/media/IR/ene_ir.h delete mode 100644 drivers/staging/lirc/lirc_ene0100.c delete mode 100644 drivers/staging/lirc/lirc_ene0100.h
drivers/media/IR/ene_ir.c new file +1176 -0
@@ -0,0 +1,1176 @@ +/* +* driver for ENE KB3926 B/C/D CIR (also known as ENE0100/ENE0200/ENE0201) +* +* Copyright (C) 2010 Maxim Levitsky <[email protected]> +* +* This program is free software; you can redistribute it and/or +* modify it under the terms of the GNU General Public License as +* published by the Free Software Foundation; either version 2 of the +* License, or (at your option) any later version. +* +* This program is distributed in the hope that it will be useful, but +* WITHOUT ANY WARRANTY; without even the implied warranty of +* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU +* General Public License for more details. +* +* You should have received a copy of the GNU General Public License +* along with this program; if not, write to the Free Software +* Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 +* USA +*/ + +#include <linux/kernel.h> +#include <linux/module.h> +#include <linux/pnp.h> +#include <linux/io.h> +#include <linux/interrupt.h> +#include <linux/sched.h> +#include <linux/uaccess.h> +#include "lirc_ene0100.h" + + +static int sample_period =-1 ; +static int enable_idle =1 ; +static int enable_duty_carrier; +static int input =1 ; +static int debug; +static int txsim; + +static void ene_rx_set_idle (struct ene_device *dev,int idle); +static int ene_irq_status (struct ene_device *dev); +static void ene_send_sample (struct ene_device *dev,unsigned long sample); + +/* read a hardware register */ +static u8 ene_hw_read_reg (struct ene_device *dev,u16 reg) +{ +u8 retval; +outb (reg >>8 , dev->hw_io +ENE_ADDR_HI ); +outb (reg &0xFF , dev->hw_io +ENE_ADDR_LO ); + retval =inb (dev->hw_io +ENE_IO ); + +ene_dbg_verbose ("reg %04x == %02x" , reg, retval); +return retval; +} + +/* write a hardware register */ +static void ene_hw_write_reg (struct ene_device *dev,u16 reg,u8 value) +{ +outb (reg >>8 , dev->hw_io +ENE_ADDR_HI ); +outb (reg &0xFF , dev->hw_io +ENE_ADDR_LO ); +outb (value, dev->hw_io +ENE_IO ); + +ene_dbg_verbose ("reg %04x <- %02x" , reg, value); +} + +/* change specific bits in hardware register */ +static void ene_hw_write_reg_mask (struct ene_device *dev, +u16 reg,u8 value,u8 mask) +{ +u8 regvalue; + +outb (reg >>8 , dev->hw_io +ENE_ADDR_HI ); +outb (reg &0xFF , dev->hw_io +ENE_ADDR_LO ); + + regvalue =inb (dev->hw_io +ENE_IO ) & ~mask; + regvalue |= (value & mask); +outb (regvalue, dev->hw_io +ENE_IO ); + +ene_dbg_verbose ("reg %04x <- %02x (mask=%02x)" , reg, value, mask); +} + +/* detect hardware features */ +static int ene_hw_detect (struct ene_device *dev) +{ +u8 chip_major, chip_minor; +u8 hw_revision, old_ver; +u8 tmp; +u8 fw_capabilities; + + tmp =ene_hw_read_reg (dev,ENE_HW_UNK ); +ene_hw_write_reg (dev,ENE_HW_UNK , tmp & ~ENE_HW_UNK_CLR ); + + chip_major =ene_hw_read_reg (dev,ENE_HW_VER_MAJOR ); + chip_minor =ene_hw_read_reg (dev,ENE_HW_VER_MINOR ); + +ene_hw_write_reg (dev,ENE_HW_UNK , tmp); + hw_revision =ene_hw_read_reg (dev,ENE_HW_VERSION ); + old_ver =ene_hw_read_reg (dev,ENE_HW_VER_OLD ); + +if (hw_revision ==0xFF ) { + +ene_printk (KERN_WARNING ,"device seems to be disabled\n" ); +ene_printk (KERN_WARNING , +"send a mail to [email protected]\n" ); +ene_printk (KERN_WARNING ,"please attach output of acpidump\n" ); +return -ENODEV ; + } + +if (chip_major ==0x33 ) { +ene_printk (KERN_WARNING ,"chips 0x33xx aren't supported\n" ); +return -ENODEV ; + } + +if (chip_major ==0x39 && chip_minor ==0x26 && hw_revision ==0xC0 ) { + dev->hw_revision =ENE_HW_C ; + }else if (old_ver ==0x24 && hw_revision ==0xC0 ) { + dev->hw_revision =ENE_HW_B ; +ene_printk (KERN_NOTICE ,"KB3926B detected\n" ); + }else { + dev->hw_revision =ENE_HW_D ; +ene_printk (KERN_WARNING , +"unknown ENE chip detected, assuming KB3926D\n" ); +ene_printk (KERN_WARNING , +"driver support might be not complete" ); + + } + +ene_printk (KERN_DEBUG , +"chip is 0x%02x%02x - kbver = 0x%02x, rev = 0x%02x\n" , + chip_major, chip_minor, old_ver, hw_revision); + +/* detect features hardware supports */ +if (dev->hw_revision <ENE_HW_C ) +return 0 ; + + fw_capabilities =ene_hw_read_reg (dev,ENE_FW2 ); +ene_dbg ("Firmware capabilities: %02x" , fw_capabilities); + + dev->hw_gpio40_learning = fw_capabilities &ENE_FW2_GP40_AS_LEARN ; + dev->hw_learning_and_tx_capable = fw_capabilities &ENE_FW2_LEARNING ; + + dev->hw_fan_as_normal_input = dev->hw_learning_and_tx_capable && + (fw_capabilities &ENE_FW2_FAN_AS_NRML_IN ); + +ene_printk (KERN_NOTICE ,"hardware features:\n" ); +ene_printk (KERN_NOTICE , +"learning and transmit %s, gpio40_learn %s, fan_in %s\n" , + dev->hw_learning_and_tx_capable ?"on" :"off" , + dev->hw_gpio40_learning ?"on" :"off" , + dev->hw_fan_as_normal_input ?"on" :"off" ); + +if (dev->hw_learning_and_tx_capable) { +ene_printk (KERN_WARNING , +"Device supports transmitting, but that support is\n" ); +ene_printk (KERN_WARNING , +"lightly tested. Please test it and mail\n" ); +ene_printk (KERN_WARNING , +"[email protected]\n" ); + } +return 0 ; +} + +/* this enables/disables IR input via gpio40*/ +static void ene_enable_gpio40_recieve (struct ene_device *dev,int enable) +{ +ene_hw_write_reg_mask (dev,ENE_CIR_CONF2 , enable ? +0 :ENE_CIR_CONF2_GPIO40DIS , +ENE_CIR_CONF2_GPIO40DIS ); +} + +/* this enables/disables IR via standard input */ +static void ene_enable_normal_recieve (struct ene_device *dev,int enable) +{ +ene_hw_write_reg (dev,ENE_CIR_CONF1 , enable ?ENE_CIR_CONF1_RX_ON :0 ); +} + +/* this enables/disables IR input via unused fan tachtometer input */ +static void ene_enable_fan_recieve (struct ene_device *dev,int enable) +{ +if (!enable) +ene_hw_write_reg (dev,ENE_FAN_AS_IN1 ,0 ); +else { +ene_hw_write_reg (dev,ENE_FAN_AS_IN1 ,ENE_FAN_AS_IN1_EN ); +ene_hw_write_reg (dev,ENE_FAN_AS_IN2 ,ENE_FAN_AS_IN2_EN ); + } + dev->rx_fan_input_inuse = enable; +} + + +/* Sense current recieved carrier */ +static int ene_rx_sense_carrier (struct ene_device *dev) +{ +int period =ene_hw_read_reg (dev,ENE_RX_CARRIER ); +int carrier; +ene_dbg ("RX: hardware carrier period = %02x" , period); + +if (!(period &ENE_RX_CARRIER_VALID )) +return 0 ; + + period &= ~ENE_RX_CARRIER_VALID ; + +if (!period) +return 0 ; + + carrier =2000000 / period; +ene_dbg ("RX: sensed carrier = %d Hz" , carrier); +return carrier; +} + +/* determine which input to use*/ +static void ene_rx_set_inputs (struct ene_device *dev) +{ +int learning_mode = dev->learning_enabled || dev->rx_carrier_sense; + +ene_dbg ("RX: setup reciever, learning mode = %d" , learning_mode); + +ene_enable_normal_recieve (dev,1 ); + +/* old hardware doesn't support learning mode for sure */ +if (dev->hw_revision <=ENE_HW_B ) +return ; + +/* reciever not learning capable, still set gpio40 correctly */ +if (!dev->hw_learning_and_tx_capable) { +ene_enable_gpio40_recieve (dev, !dev->hw_gpio40_learning); +return ; + } + +/* enable learning mode */ +if (learning_mode) { +ene_enable_gpio40_recieve (dev, dev->hw_gpio40_learning); + +/* fan input is not used for learning */ +if (dev->hw_fan_as_normal_input) +ene_enable_fan_recieve (dev,0 ); + +/* disable learning mode */ + }else { +if (dev->hw_fan_as_normal_input) { +ene_enable_fan_recieve (dev,1 ); +ene_enable_normal_recieve (dev,0 ); + }else +ene_enable_gpio40_recieve (dev, + !dev->hw_gpio40_learning); + } + +/* set few additional settings for this mode */ +ene_hw_write_reg_mask (dev,ENE_CIR_CONF1 , learning_mode ? +ENE_CIR_CONF1_LEARN1 :0 ,ENE_CIR_CONF1_LEARN1 ); + +ene_hw_write_reg_mask (dev,ENE_CIR_CONF2 , learning_mode ? +ENE_CIR_CONF2_LEARN2 :0 ,ENE_CIR_CONF2_LEARN2 ); +} + +/* Enable the device for receive */ +static void ene_rx_enable (struct ene_device *dev) +{ +u8 reg_value; + +if (dev->hw_revision <ENE_HW_C ) { +ene_hw_write_reg (dev,ENEB_IRQ , dev->irq <<1 ); +ene_hw_write_reg (dev,ENEB_IRQ_UNK1 ,0x01 ); + }else { + reg_value =ene_hw_read_reg (dev,ENEC_IRQ ) &0xF0 ; + reg_value |=ENEC_IRQ_UNK_EN ; + reg_value &= ~ENEC_IRQ_STATUS ; + reg_value |= (dev->irq &ENEC_IRQ_MASK ); +ene_hw_write_reg (dev,ENEC_IRQ , reg_value); +ene_hw_write_reg (dev,ENE_TX_UNK1 ,0x63 ); + } + +ene_hw_write_reg (dev,ENE_CIR_CONF2 ,0x00 ); +ene_rx_set_inputs (dev); + +/* set sampling period */ +ene_hw_write_reg (dev,ENE_CIR_SAMPLE_PERIOD , sample_period); + +/* ack any pending irqs - just in case */ +ene_irq_status (dev); + +/* enter idle mode */ +ene_rx_set_idle (dev,1 ); + +/* enable firmware bits */ +ene_hw_write_reg_mask (dev,ENE_FW1 , +ENE_FW1_ENABLE |ENE_FW1_IRQ , +ENE_FW1_ENABLE |ENE_FW1_IRQ ); +} + +/* Disable the device reciever */ +static void ene_rx_disable (struct ene_device *dev) +{ +/* disable inputs */ +ene_enable_normal_recieve (dev,0 ); + +if (dev->hw_fan_as_normal_input) +ene_enable_fan_recieve (dev,0 ); + +/* disable hardware IRQ and firmware flag */ +ene_hw_write_reg_mask (dev,ENE_FW1 ,0 ,ENE_FW1_ENABLE |ENE_FW1_IRQ ); + +ene_rx_set_idle (dev,1 ); +} + +/* send current sample to the user */ +static void ene_rx_flush (struct ene_device *dev,int timeout) +{ +unsigned long value; + + value = dev->rx_sample_pulse ?LIRC_PULSE (dev->rx_sample) : +LIRC_SPACE (dev->rx_sample); +ene_send_sample (dev, value); + dev->rx_sample =0 ; + dev->rx_sample_pulse =0 ; +} + +/* recieve new sample and process it */ +static void ene_rx_sample (struct ene_device *dev,int sample,int is_pulse) +{ +ene_dbg ("RX: sample %8d (%s)" , sample, is_pulse ?"pulse" :"space" ); + +/* ignore spaces in idle mode, can get them on revC */ +/* also ignore a space in front of first pulse */ +if (dev->rx_idle && !is_pulse) +return ; + +/* get out of idle mode now */ +if (dev->rx_idle) +ene_rx_set_idle (dev,0 ); + +if (!dev->rx_sample) { + dev->rx_sample = sample; + dev->rx_sample_pulse = is_pulse; + }else if (is_pulse == dev->rx_sample_pulse) + dev->rx_sample += sample; +else { +ene_rx_flush (dev,0 ); + dev->rx_sample = sample; + dev->rx_sample_pulse = is_pulse; + } + +if (is_pulse) +return ; + +/* overflow sample from fan input recieved, enable idle mode */ +if (dev->rx_fan_input_inuse && + sample ==ENE_FAN_VALUE_MASK *ENE_SAMPLE_PERIOD_FAN ) { +ene_rx_set_idle (dev,1 ); +return ; + } + +if (!dev->rx_fan_input_inuse) { +/* Report timeout if enabled */ +if (dev->rx_timeout && dev->rx_send_timeout_packet && + !dev->rx_timeout_sent && + dev->rx_sample > dev->rx_timeout) { +ene_dbg ("RX: sending timeout sample" ); +ene_send_sample (dev,LIRC_TIMEOUT (dev->rx_sample)); + dev->rx_timeout_sent =1 ; + } + +/* too large sample accumulated via normal input. +note that on revC, hardware idle mode turns on automaticly, +so max gap should be less that the gap after which +hw stops sending samples */ +if (dev->rx_sample >ENE_MAXGAP ) { +ene_rx_set_idle (dev,1 ); +return ; + } + } +} + +/* enable or disable idle mode */ +static void ene_rx_set_idle (struct ene_device *dev,int idle) +{ +struct timeval now; +int disable_sampler =0 ; + + +/* Also put hardware sampler in 'idle' mode on revB*/ +/* revC and higher do that automaticly (firmware does?) */ +if ((dev->hw_revision <ENE_HW_C ) && enable_idle) +if (idle) + disable_sampler =1 ; + +ene_hw_write_reg_mask (dev,ENE_CIR_SAMPLE_PERIOD , + disable_sampler ?0 :ENE_CIR_SAMPLE_OVERFLOW , +ENE_CIR_SAMPLE_OVERFLOW ); + dev->rx_idle = idle; + +/* remember when we have entered the idle mode */ +if (idle) { +ene_dbg ("RX: going into idle mode" ); +do_gettimeofday (&dev->rx_gap_start); +return ; + } + +ene_dbg ("RX: back from idle mode" ); + +/* send the gap between keypresses now */ +do_gettimeofday (&now); + +if (dev->rx_sample_pulse) { +ene_dbg ("RX: somehow we recieved a pulse before idle mode???" ); +return ; + } + +/* manually calculate and recieve the gap between keypresses */ +if (now.tv_sec - dev->rx_gap_start.tv_sec >16 ) + dev->rx_sample =LIRC_SPACE (LIRC_VALUE_MASK ); +else + dev->rx_sample += +1000000ull * (now.tv_sec - dev->rx_gap_start.tv_sec) + + now.tv_usec - dev->rx_gap_start.tv_usec; + +if (dev->rx_sample >LIRC_SPACE (LIRC_VALUE_MASK )) + dev->rx_sample =LIRC_SPACE (LIRC_VALUE_MASK ); + +ene_rx_flush (dev,0 ); + dev->rx_timeout_sent =0 ; +} + +/* prepare transmission */ +static void ene_tx_prepare (struct ene_device *dev) +{ +u8 conf1; + + conf1 =ene_hw_read_reg (dev,ENE_CIR_CONF1 ); + dev->saved_conf1 = conf1; + +if (dev->hw_revision ==ENE_HW_C ) + conf1 &= ~ENE_CIR_CONF1_TX_CLEAR ; + +/* Enable TX engine */ + conf1 |=ENE_CIR_CONF1_TX_ON ; + +/* Set carrier */ +if (dev->tx_period) { + +int tx_period_in500ns = dev->tx_period *2 ; + +int tx_pulse_width_in_500ns = + tx_period_in500ns / (100 / dev->tx_duty_cycle); + +if (!tx_pulse_width_in_500ns) + tx_pulse_width_in_500ns =1 ; + +ene_dbg ("TX: pulse distance = %d * 500 ns" , tx_period_in500ns); +ene_dbg ("TX: pulse width = %d * 500 ns" , + tx_pulse_width_in_500ns); + +ene_hw_write_reg (dev,ENE_TX_PERIOD ,ENE_TX_PERIOD_UNKBIT | + tx_period_in500ns); + +ene_hw_write_reg (dev,ENE_TX_PERIOD_PULSE , + tx_pulse_width_in_500ns); + + conf1 |=ENE_CIR_CONF1_TX_CARR ; + }else + conf1 &= ~ENE_CIR_CONF1_TX_CARR ; + +ene_hw_write_reg (dev,ENE_CIR_CONF1 , conf1); + dev->tx_underway =1 ; + +} + +/* end transmission */ +static void ene_tx_complete (struct ene_device *dev) +{ +ene_hw_write_reg (dev,ENE_CIR_CONF1 , dev->saved_conf1); + dev->tx_underway =0 ; +} + +/* set transmit mask */ +static void ene_tx_set_transmiter_mask (struct ene_device *dev) +{ +u8 txport1 =ene_hw_read_reg (dev,ENE_TX_PORT1 ) & ~ENE_TX_PORT1_EN ; +u8 txport2 =ene_hw_read_reg (dev,ENE_TX_PORT2 ) & ~ENE_TX_PORT2_EN ; + +if (dev->transmitter_mask &0x01 ) + txport1 |=ENE_TX_PORT1_EN ; + +if (dev->transmitter_mask &0x02 ) + txport2 |=ENE_TX_PORT2_EN ; + +ene_hw_write_reg (dev,ENE_TX_PORT1 , txport1); +ene_hw_write_reg (dev,ENE_TX_PORT2 , txport2); +} + +/* TX one sample - must be called with dev->hw_lock*/ +static void ene_tx_sample (struct ene_device *dev) +{ +u8 raw_tx; +u32 sample; + +if (!dev->tx_underway) { +ene_dbg ("TX: attempt to transmit while hw isn't setup" ); +return ; + } + +/* Grab next TX sample */ +if (!dev->tx_sample) { +again: +if (dev->tx_pos == dev->tx_len +1 ) { +if (!dev->tx_done) { +ene_dbg ("TX: no more data to send" ); + dev->tx_done =1 ; + goto exit; + }else { +ene_dbg ("TX: last sample sent by hardware" ); +ene_tx_complete (dev); +complete (&dev->tx_complete); +return ; + } + } + + sample = dev->tx_buffer[dev->tx_pos++]; + dev->tx_sample_pulse = !dev->tx_sample_pulse; + +ene_dbg ("TX: sample %8d (%s)" , sample, dev->tx_sample_pulse ? +"pulse" :"space" ); + + dev->tx_sample =DIV_ROUND_CLOSEST (sample,ENE_TX_SMPL_PERIOD ); + +/* guard against too short samples */ +if (!dev->tx_sample) + goto again; + } + + raw_tx =min (dev->tx_sample , (unsigned int )ENE_TX_SMLP_MASK ); + dev->tx_sample -= raw_tx; + +if (dev->tx_sample_pulse) + raw_tx |=ENE_TX_PULSE_MASK ; + +ene_hw_write_reg (dev,ENE_TX_INPUT1 + dev->tx_reg, raw_tx); + dev->tx_reg = !dev->tx_reg; +exit: +/* simulate TX done interrupt */ +if (txsim) +mod_timer (&dev->tx_sim_timer, jiffies +HZ /500 ); +} + +/* timer to simulate tx done interrupt */ +static void ene_tx_irqsim (unsigned long data) +{ +struct ene_device *dev = (struct ene_device *)data; +unsigned long flags; + +spin_lock_irqsave (&dev->hw_lock, flags); +ene_tx_sample (dev); +spin_unlock_irqrestore (&dev->hw_lock, flags); +} + + +/* read irq status and ack it */ +static int ene_irq_status (struct ene_device *dev) +{ +u8 irq_status; +u8 fw_flags1, fw_flags2; +int cur_rx_pointer; +int retval =0 ; + + fw_flags2 =ene_hw_read_reg (dev,ENE_FW2 ); + cur_rx_pointer = !!(fw_flags2 &ENE_FW2_BUF_HIGH ); + +if (dev->hw_revision <ENE_HW_C ) { + irq_status =ene_hw_read_reg (dev,ENEB_IRQ_STATUS ); + +if (!(irq_status &ENEB_IRQ_STATUS_IR )) +return 0 ; + +ene_hw_write_reg (dev,ENEB_IRQ_STATUS , + irq_status & ~ENEB_IRQ_STATUS_IR ); + dev->rx_pointer = cur_rx_pointer; +return ENE_IRQ_RX ; + } + + irq_status =ene_hw_read_reg (dev,ENEC_IRQ ); + +if (!(irq_status &ENEC_IRQ_STATUS )) +return 0 ; + +/* original driver does that twice - a workaround ? */ +ene_hw_write_reg (dev,ENEC_IRQ , irq_status & ~ENEC_IRQ_STATUS ); +ene_hw_write_reg (dev,ENEC_IRQ , irq_status & ~ENEC_IRQ_STATUS ); + +/* clear unknown flag in F8F9 */ +if (fw_flags2 &ENE_FW2_IRQ_CLR ) +ene_hw_write_reg (dev,ENE_FW2 , fw_flags2 & ~ENE_FW2_IRQ_CLR ); + +/* check if this is a TX interrupt */ + fw_flags1 =ene_hw_read_reg (dev,ENE_FW1 ); +if (fw_flags1 &ENE_FW1_TXIRQ ) { +ene_hw_write_reg (dev,ENE_FW1 , fw_flags1 & ~ENE_FW1_TXIRQ ); + retval |=ENE_IRQ_TX ; + } + +/* Check if this is RX interrupt */ +if (dev->rx_pointer != cur_rx_pointer) { + retval |=ENE_IRQ_RX ; + dev->rx_pointer = cur_rx_pointer; + + }else if (!(retval &ENE_IRQ_TX )) { +ene_dbg ("RX: interrupt without change in RX pointer(%d)" , + dev->rx_pointer); + retval |=ENE_IRQ_RX ; + } + +if ((retval &ENE_IRQ_RX ) && (retval &ENE_IRQ_TX )) +ene_dbg ("both RX and TX interrupt at same time" ); + +return retval; +} + +/* interrupt handler */ +static irqreturn_t ene_isr (int irq,void *data) +{ +u16 hw_value; +int i, hw_sample; +int pulse; +int irq_status; +unsigned long flags; +int carrier =0 ; +irqreturn_t retval =IRQ_NONE ; +struct ene_device *dev = (struct ene_device *)data; + + +spin_lock_irqsave (&dev->hw_lock, flags); + irq_status =ene_irq_status (dev); + +if (!irq_status) + goto unlock; + + retval =IRQ_HANDLED ; + +if (irq_status &ENE_IRQ_TX ) { + +if (!dev->hw_learning_and_tx_capable) { +ene_dbg ("TX interrupt on unsupported device!" ); + goto unlock; + } +ene_tx_sample (dev); + } + +if (!(irq_status &ENE_IRQ_RX )) + goto unlock; + + +if ((debug && dev->learning_enabled) || dev->rx_carrier_sense) + carrier =ene_rx_sense_carrier (dev); + +if (dev->rx_carrier_sense && carrier) +ene_send_sample (dev,LIRC_FREQUENCY (carrier)); + + +for (i =0 ; i <ENE_SAMPLES_SIZE ; i++) { + hw_value =ene_hw_read_reg (dev, +ENE_SAMPLE_BUFFER + dev->rx_pointer *4 + i); + +if (dev->rx_fan_input_inuse) { +/* read high part of the sample */ + hw_value |=ene_hw_read_reg (dev, +ENE_SAMPLE_BUFFER_FAN + + dev->rx_pointer *4 + i) <<8 ; + pulse = hw_value &ENE_FAN_SMPL_PULS_MSK ; + +/* clear space bit, and other unused bits */ + hw_value &=ENE_FAN_VALUE_MASK ; + hw_sample = hw_value *ENE_SAMPLE_PERIOD_FAN ; + + }else { + pulse = !(hw_value &ENE_SAMPLE_SPC_MASK ); + hw_value &=ENE_SAMPLE_VALUE_MASK ; + hw_sample = hw_value * sample_period; + } +/* no more data */ +if (!(hw_value)) +break ; + +ene_rx_sample (dev, hw_sample, pulse); + } +unlock: +spin_unlock_irqrestore (&dev->hw_lock, flags); +return retval; +} + +/* Initialize default settings */ +static void ene_setup_settings (struct ene_device *dev) +{ + dev->rx_send_timeout_packet =0 ; + dev->rx_timeout =ENE_MAXGAP ; + dev->tx_period =32 ; + dev->tx_duty_cycle =25 ;/*%*/ + dev->transmitter_mask =3 ; + +/* Force learning mode if (input == 2), otherwise +let user set it with LIRC_SET_REC_CARRIER */ + dev->learning_enabled = + (input ==2 && dev->hw_learning_and_tx_capable); + +/* Clear accumulated sample bufer */ + dev->rx_sample =0 ; + dev->rx_sample_pulse =0 ; + dev->rx_pointer =-1 ; + dev->rx_carrier_sense =0 ; + +} + +/* outside interface: called on first open*/ +static int ene_open (void *data) +{ +struct ene_device *dev = (struct ene_device *)data; +unsigned long flags; + +spin_lock_irqsave (&dev->hw_lock, flags); + dev->in_use =1 ; +ene_setup_settings (dev); +ene_rx_enable (dev); +spin_unlock_irqrestore (&dev->hw_lock, flags); +return 0 ; +} + +/* outside interface: called on device close*/ +static void ene_close (void *data) +{ +struct ene_device *dev = (struct ene_device *)data; +unsigned long flags; +spin_lock_irqsave (&dev->hw_lock, flags); + +ene_rx_disable (dev); + dev->in_use =0 ; +spin_unlock_irqrestore (&dev->hw_lock, flags); +} + +/* outside interface for settings */ +static int ene_ioctl (struct inode *node,struct file *file, +unsigned int cmd,unsigned long arg) +{ +int lvalue =0 , retval, tmp; +unsigned long flags; +struct ene_device *dev =lirc_get_pdata (file); + + +switch (cmd) { +case LIRC_SET_SEND_CARRIER : +case LIRC_SET_SEND_DUTY_CYCLE : +case LIRC_SET_TRANSMITTER_MASK : +case LIRC_SET_MEASURE_CARRIER_MODE : +case LIRC_SET_REC_CARRIER : +/* All these aren't possible without this */ +if (!dev->hw_learning_and_tx_capable) +return -ENOSYS ; +/* Fall through */ +case LIRC_SET_REC_TIMEOUT : +case LIRC_SET_REC_TIMEOUT_REPORTS : + retval =get_user (lvalue, (unsigned int *) arg); +if (retval) +return retval; + } + +switch (cmd) { +case LIRC_SET_SEND_CARRIER : +ene_dbg ("TX: attempt to set tx carrier to %d kHz" , lvalue); + tmp =1000000 / lvalue;/* (1 / freq) (* # usec in 1 sec) */ + +if (tmp && (tmp >ENE_TX_PERIOD_MAX || + tmp <ENE_TX_PERIOD_MIN )) { + +ene_dbg ("TX: out of range %d-%d carrier, " +"falling back to 32 kHz" , +1000 /ENE_TX_PERIOD_MIN , +1000 /ENE_TX_PERIOD_MAX ); + + tmp =32 ;/* this is just a coincidence!!! */ + } +ene_dbg ("TX: set carrier to %d kHz" , lvalue); + +spin_lock_irqsave (&dev->hw_lock, flags); + dev->tx_period = tmp; +spin_unlock_irqrestore (&dev->hw_lock, flags); +break ; +case LIRC_SET_SEND_DUTY_CYCLE : +ene_dbg ("TX: attempt to set duty cycle to %d%%" , lvalue); + +if ((lvalue >=100 ) || (lvalue <=0 )) { + retval = -EINVAL ; +break ; + } +spin_lock_irqsave (&dev->hw_lock, flags); + dev->tx_duty_cycle = lvalue; +spin_unlock_irqrestore (&dev->hw_lock, flags); +break ; +case LIRC_SET_TRANSMITTER_MASK : +ene_dbg ("TX: attempt to set transmitter mask %02x" , lvalue); + +/* invalid txmask */ +if (!lvalue || lvalue & ~0x3 ) { +ene_dbg ("TX: invalid mask" ); +/* this supposed to return num of transmitters */ + retval =2 ; +break ; + } +spin_lock_irqsave (&dev->hw_lock, flags); + dev->transmitter_mask = lvalue; +spin_unlock_irqrestore (&dev->hw_lock, flags); +break ; +case LIRC_SET_REC_CARRIER : + tmp = (lvalue >ENE_NORMAL_RX_HI || lvalue <ENE_NORMAL_RX_LOW ); + +if (tmp != dev->learning_enabled) { +spin_lock_irqsave (&dev->hw_lock, flags); + dev->learning_enabled = tmp; +ene_rx_set_inputs (dev); +spin_unlock_irqrestore (&dev->hw_lock, flags); + } +break ; +case LIRC_SET_REC_TIMEOUT : +spin_lock_irqsave (&dev->hw_lock, flags); + dev->rx_timeout = lvalue; +spin_unlock_irqrestore (&dev->hw_lock, flags); +ene_dbg ("RX: set rx report timeout to %d" , dev->rx_timeout); +break ; +case LIRC_SET_REC_TIMEOUT_REPORTS : +spin_lock_irqsave (&dev->hw_lock, flags); + dev->rx_send_timeout_packet = lvalue; +spin_unlock_irqrestore (&dev->hw_lock, flags); +ene_dbg ("RX: %sable timeout reports" , + dev->rx_send_timeout_packet ?"en" :"dis" ); +break ; +case LIRC_SET_MEASURE_CARRIER_MODE : +if (dev->rx_carrier_sense == lvalue) +break ; +spin_lock_irqsave (&dev->hw_lock, flags); + dev->rx_carrier_sense = lvalue; +ene_rx_set_inputs (dev); +spin_unlock_irqrestore (&dev->hw_lock, flags); +break ; +case LIRC_GET_REC_RESOLUTION : + tmp = dev->rx_fan_input_inuse ? +ENE_SAMPLE_PERIOD_FAN : sample_period; + retval =put_user (tmp, (unsigned long *) arg); +break ; +default : + retval = -ENOIOCTLCMD ; +break ; + } + +return retval; +} + +/* outside interface: transmit */ +static ssize_t ene_transmit (struct file *file,const char *buf, +size_t n,loff_t *ppos) +{ +struct ene_device *dev =lirc_get_pdata (file); +unsigned long flags; + +if (!dev) +return -EFAULT ; + +if (!dev->hw_learning_and_tx_capable) +return -ENODEV ; + +if (n %sizeof (int )) +return -EINVAL ; + +if (n >ENE_TXBUF_SIZE *sizeof (int )) +return -ENOMEM ; + +if (copy_from_user (dev->tx_buffer, buf, n)) +return -EFAULT ; + + dev->tx_len = n /sizeof (int ); + dev->tx_pos =0 ; + dev->tx_reg =0 ; + dev->tx_done =0 ; + dev->tx_sample =0 ; + dev->tx_sample_pulse =0 ; + +ene_dbg ("TX: %d samples" , dev->tx_len); + +spin_lock_irqsave (&dev->hw_lock, flags); + +ene_tx_set_transmiter_mask (dev); +ene_tx_prepare (dev); + +/* Transmit first two samples */ +ene_tx_sample (dev); +ene_tx_sample (dev); + +spin_unlock_irqrestore (&dev->hw_lock, flags); + +if (wait_for_completion_timeout (&dev->tx_complete,2 *HZ ) ==0 ) { +ene_dbg ("TX: timeout" ); +spin_lock_irqsave (&dev->hw_lock, flags); +ene_tx_complete (dev); +spin_unlock_irqrestore (&dev->hw_lock, flags); + }else +ene_dbg ("TX: done" ); + +return n; +} + +/* Sends one sample to the user */ +static void ene_send_sample (struct ene_device *dev,unsigned long sample) +{ +if (!lirc_buffer_full (dev->lirc_driver->rbuf)) { +lirc_buffer_write (dev->lirc_driver->rbuf, (void *)&sample); +wake_up (&dev->lirc_driver->rbuf->wait_poll); + } +} + + +static const struct file_operations ene_fops = { + .owner =THIS_MODULE , + .write = ene_transmit, + .ioctl = ene_ioctl, +}; + +/* main load function */ +static int ene_probe (struct pnp_dev *pnp_dev, +const struct pnp_device_id *dev_id) +{ +struct ene_device *dev; +struct lirc_driver *lirc_driver; +int error = -ENOMEM ; + + dev =kzalloc (sizeof (struct ene_device ),GFP_KERNEL ); + +if (!dev) + goto err1; + + dev->pnp_dev = pnp_dev; +pnp_set_drvdata (pnp_dev, dev); + +/* prepare lirc interface */ + error = -ENOMEM ; + lirc_driver =kzalloc (sizeof (struct lirc_driver ),GFP_KERNEL ); + +if (!lirc_driver) + goto err2; + + dev->lirc_driver = lirc_driver; + +strcpy (lirc_driver->name,ENE_DRIVER_NAME ); + lirc_driver->minor =-1 ; + lirc_driver->code_length =sizeof (int ) *8 ; + lirc_driver->features =LIRC_CAN_REC_MODE2 | +LIRC_CAN_GET_REC_RESOLUTION | +LIRC_CAN_SET_REC_TIMEOUT ; + lirc_driver->data = dev; + lirc_driver->set_use_inc = ene_open; + lirc_driver->set_use_dec = ene_close; + lirc_driver->dev = &pnp_dev->dev; + lirc_driver->owner =THIS_MODULE ; + lirc_driver->fops = &ene_fops; + lirc_driver->min_timeout =ENE_MINGAP ; + lirc_driver->max_timeout =ENE_MAXGAP ; + lirc_driver->rbuf =kzalloc (sizeof (struct lirc_buffer ),GFP_KERNEL ); + +if (!lirc_driver->rbuf) + goto err3; + +if (lirc_buffer_init (lirc_driver->rbuf,sizeof (int ),sizeof (int ) *512 )) + goto err4; + +/* validate resources */ +if (!pnp_port_valid (pnp_dev,0 ) || +pnp_port_len (pnp_dev,0 ) <ENE_MAX_IO ) + goto err5; + +if (!pnp_irq_valid (pnp_dev,0 )) + goto err5; + + dev->hw_io =pnp_port_start (pnp_dev,0 ); + dev->irq =pnp_irq (pnp_dev,0 ); +spin_lock_init (&dev->hw_lock); + +/* claim the resources */ + error = -EBUSY ; +if (!request_region (dev->hw_io,ENE_MAX_IO ,ENE_DRIVER_NAME )) + goto err5; + +if (request_irq (dev->irq, ene_isr, +IRQF_SHARED ,ENE_DRIVER_NAME , (void *)dev)) + goto err6; + +/* detect hardware version and features */ + error =ene_hw_detect (dev); +if (error) + goto err7; + +ene_setup_settings (dev); + +if (!dev->hw_learning_and_tx_capable && txsim) { + dev->hw_learning_and_tx_capable =1 ; +setup_timer (&dev->tx_sim_timer, ene_tx_irqsim, + (long unsigned int )dev); +ene_printk (KERN_WARNING , +"Simulation of TX activated\n" ); + } + +if (dev->hw_learning_and_tx_capable) { + lirc_driver->features |=LIRC_CAN_SEND_PULSE | +LIRC_CAN_SET_SEND_CARRIER | +LIRC_CAN_SET_TRANSMITTER_MASK ; + +if (enable_duty_carrier) + lirc_driver->features |=LIRC_CAN_SET_SEND_DUTY_CYCLE ; + +if (input ==0 ) + lirc_driver->features |=LIRC_CAN_SET_REC_CARRIER ; + +init_completion (&dev->tx_complete); + } + +/* don't allow too short/long sample periods */ +if (sample_period <5 || sample_period >0x7F ) + sample_period =-1 ; + +/* choose default sample period */ +if (sample_period ==-1 ) { + + sample_period =50 ; + +/* on revB, hardware idle mode eats first sample +if we set too low sample period */ +if (dev->hw_revision ==ENE_HW_B && enable_idle) + sample_period =75 ; + } + +device_set_wakeup_capable (&pnp_dev->dev,1 ); +device_set_wakeup_enable (&pnp_dev->dev,1 ); + + error = -ENODEV ; +if (lirc_register_driver (lirc_driver)) + goto err7; + +ene_printk (KERN_NOTICE ,"driver has been succesfully loaded\n" ); +return 0 ; + +err7: +free_irq (dev->irq, dev); +err6: +release_region (dev->hw_io,ENE_MAX_IO ); +err5: +lirc_buffer_free (lirc_driver->rbuf); +err4: +kfree (lirc_driver->rbuf); +err3: +kfree (lirc_driver); +err2: +kfree (dev); +err1: +return error; +} + +/* main unload function */ +static void ene_remove (struct pnp_dev *pnp_dev) +{ +struct ene_device *dev =pnp_get_drvdata (pnp_dev); +unsigned long flags; + +spin_lock_irqsave (&dev->hw_lock, flags); +ene_rx_disable (dev); +spin_unlock_irqrestore (&dev->hw_lock, flags); + +free_irq (dev->irq, dev); +release_region (dev->hw_io,ENE_MAX_IO ); +lirc_unregister_driver (dev->lirc_driver->minor); +lirc_buffer_free (dev->lirc_driver->rbuf); +kfree (dev->lirc_driver); +kfree (dev); +} + +/* enable wake on IR (wakes on specific button on original remote) */ +static void ene_enable_wake (struct ene_device *dev,int enable) +{ + enable = enable &&device_may_wakeup (&dev->pnp_dev->dev); + +ene_dbg ("wake on IR %s" , enable ?"enabled" :"disabled" ); + +ene_hw_write_reg_mask (dev,ENE_FW1 , enable ? +ENE_FW1_WAKE :0 ,ENE_FW1_WAKE ); +} + +#ifdef CONFIG_PM +static int ene_suspend (struct pnp_dev *pnp_dev,pm_message_t state) +{ +struct ene_device *dev =pnp_get_drvdata (pnp_dev); +ene_enable_wake (dev,1 ); +return 0 ; +} + +static int ene_resume (struct pnp_dev *pnp_dev) +{ +struct ene_device *dev =pnp_get_drvdata (pnp_dev); +if (dev->in_use) +ene_rx_enable (dev); + +ene_enable_wake (dev,0 ); +return 0 ; +} +#endif + +static void ene_shutdown (struct pnp_dev *pnp_dev) +{ +struct ene_device *dev =pnp_get_drvdata (pnp_dev); +ene_enable_wake (dev,1 ); +} + +static const struct pnp_device_id ene_ids[] = { + {.id ="ENE0100" ,}, + {.id ="ENE0200" ,}, + {.id ="ENE0201" ,}, + {}, +}; + +static struct pnp_driver ene_driver = { + .name =ENE_DRIVER_NAME , + .id_table = ene_ids, + .flags =PNP_DRIVER_RES_DO_NOT_CHANGE , + + .probe = ene_probe, + .remove =__devexit_p (ene_remove), +#ifdef CONFIG_PM + .suspend = ene_suspend, + .resume = ene_resume, +#endif + .shutdown = ene_shutdown, +}; + +static int __init ene_init (void ) +{ +return pnp_register_driver (&ene_driver); +} + +static void ene_exit (void ) +{ +pnp_unregister_driver (&ene_driver); +} + +module_param (sample_period, int,S_IRUGO ); +MODULE_PARM_DESC (sample_period,"Hardware sample period (50 us default)" ); + +module_param (enable_idle, bool,S_IRUGO |S_IWUSR ); +MODULE_PARM_DESC (enable_idle, +"Enables turning off signal sampling after long inactivity time; " +"if disabled might help detecting input signal (default: enabled)" +" (KB3926B only)" ); + +module_param (input, bool,S_IRUGO ); +MODULE_PARM_DESC (input,"select which input to use " +"0 - auto, 1 - standard, 2 - wideband(KB3926C+)" ); + +module_param (debug, int,S_IRUGO |S_IWUSR ); +MODULE_PARM_DESC (debug,"Enable debug (debug=2 verbose debug output)" ); + +module_param (txsim, bool,S_IRUGO ); +MODULE_PARM_DESC (txsim, +"Simulate TX features on unsupported hardware (dangerous)" ); + +module_param (enable_duty_carrier, bool,S_IRUGO ); +MODULE_PARM_DESC (enable_duty_carrier, +"Enable a code that might allow to to set TX carrier duty cycle" ); + +MODULE_DEVICE_TABLE (pnp, ene_ids); +MODULE_DESCRIPTION + ("LIRC driver for KB3926B/KB3926C/KB3926D " +"(aka ENE0100/ENE0200/ENE0201) CIR port" ); + +MODULE_AUTHOR ("Maxim Levitsky" ); +MODULE_LICENSE ("GPL" ); + +module_init (ene_init); +module_exit (ene_exit);
drivers/media/IR/ene_ir.h new file +240 -0
240 lines not on this page · raw message
drivers/staging/lirc/Kconfig +0 -8
@@ -17,14 +17,6 @@ config LIRC_BT829 help Driver for the IR interface on BT829-based hardware -config LIRC_ENE0100 - tristate "ENE KB3924/ENE0100 CIR Port Reciever" - depends on LIRC_STAGING - help - This is a driver for CIR port handled by ENE KB3924 embedded - controller found on some notebooks. - It appears on PNP list as ENE0100. - config LIRC_I2C tristate "I2C Based IR Receivers" depends on LIRC_STAGING
drivers/staging/lirc/Makefile +0 -1
@@ -4,7 +4,6 @@ # Each configuration option enables a list of files. obj-$(CONFIG_LIRC_BT829) += lirc_bt829.o -obj-$(CONFIG_LIRC_ENE0100) += lirc_ene0100.o obj-$(CONFIG_LIRC_I2C) += lirc_i2c.o obj-$(CONFIG_LIRC_IGORPLUGUSB) += lirc_igorplugusb.o obj-$(CONFIG_LIRC_IMON) += lirc_imon.o
drivers/staging/lirc/lirc_ene0100.c deleted +0 -646
646 lines not on this page · raw message
drivers/staging/lirc/lirc_ene0100.h deleted +0 -169
169 lines not on this page · raw message
Signature · 1 line
1.7.0.4
PATCH 13/13 IR: Port ene driver to new IR subsystem and enable it.
In reply to Maxim Levitsky--- MAINTAINERS | 6 + drivers/media/IR/Kconfig | 14 + drivers/media/IR/Makefile | 1 + drivers/media/IR/ene_ir.c | 595 +++++++++++++++++---------------------------- drivers/media/IR/ene_ir.h | 51 ++--- 5 files changed, 265 insertions(+), 402 deletions(-)
MAINTAINERS +6 -0
@@ -2188,6 +2188,12 @@ F: drivers/misc/cb710/ F: drivers/mmc/host/cb710-mmc.* F: include/linux/cb710.h +ENE KB2426 (ENE0100/ENE020XX) INFRARED RECEIVER +M: Maxim Levitsky <[email protected]> +S: Maintained +F: drivers/media/IR/ene_ir.c +F: drivers/media/IR/ene_ir.h + EPSON 1355 FRAMEBUFFER DRIVER M: Christopher Hoover <[email protected]> M: Christopher Hoover <[email protected]>
drivers/media/IR/Kconfig +14 -0
@@ -105,4 +105,18 @@ config IR_MCEUSB To compile this driver as a module, choose M here: the module will be called mceusb. +config IR_ENE + tristate "ENE eHome Receiver/Transciever (pnp id: ENE0100/ENE02xxx)" + depends on PNP + depends on IR_CORE + ---help--- + Say Y here to enable support for integrated infrared receiver + /transciever made by ENE. + + You can see if you have it by looking at lspnp output. + Output should include ENE0100 ENE0200 or something similiar. + + To compile this driver as a module, choose M here: the + module will be called ene_ir. + endif #IR_CORE
drivers/media/IR/Makefile +1 -0
@@ -16,3 +16,4 @@ obj-$(CONFIG_IR_LIRC_CODEC) += ir-lirc-codec.o # stand-alone IR receivers/transmitters obj-$(CONFIG_IR_IMON) += imon.o obj-$(CONFIG_IR_MCEUSB) += mceusb.o +obj-$(CONFIG_IR_ENE) += ene_ir.o
drivers/media/IR/ene_ir.c +221 -374
880 lines not on this page · raw message
drivers/media/IR/ene_ir.h +23 -28
121 lines not on this page · raw message
Signature · 1 line
1.7.0.4