* [PATCH] Add EV_IR bit @ 2009-08-12 22:33 David Härdeman 2009-08-13 6:49 ` Dmitry Torokhov 0 siblings, 1 reply; 5+ messages in thread From: David Härdeman @ 2009-08-12 22:33 UTC (permalink / raw) To: linux-input This patch adds an EV_IR bit to allow input drivers to let userspace know that the hardware is an infrared remote control transceiver. No dev->irbit is defined yet but I plan to add that later (which would be used to indicate whether the hardware supports RX, TX, etc). If the patch is accepted I'll send followup patches adding the appropriate input_set_capability call to the relevant drivers. Signed-off-by: David Härdeman <david@hardeman.nu> Index: linux-2.6/drivers/input/input.c =================================================================== --- linux-2.6.orig/drivers/input/input.c 2009-08-12 19:45:10.000000000 +0200 +++ linux-2.6/drivers/input/input.c 2009-08-12 19:54:56.000000000 +0200 @@ -1375,6 +1375,10 @@ /* do nothing */ break; + case EV_IR: + /* do nothing */ + break; + default: printk(KERN_ERR "input_set_capability: unknown type %u (code %u)\n", Index: linux-2.6/include/linux/input.h =================================================================== --- linux-2.6.orig/include/linux/input.h 2009-08-12 19:56:03.000000000 +0200 +++ linux-2.6/include/linux/input.h 2009-08-12 19:57:53.000000000 +0200 @@ -98,6 +98,7 @@ #define EV_FF 0x15 #define EV_PWR 0x16 #define EV_FF_STATUS 0x17 +#define EV_IR 0x18 #define EV_MAX 0x1f #define EV_CNT (EV_MAX+1) -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] Add EV_IR bit 2009-08-12 22:33 [PATCH] Add EV_IR bit David Härdeman @ 2009-08-13 6:49 ` Dmitry Torokhov 2009-08-13 8:32 ` David Härdeman 0 siblings, 1 reply; 5+ messages in thread From: Dmitry Torokhov @ 2009-08-13 6:49 UTC (permalink / raw) To: David Härdeman; +Cc: linux-input On Thu, Aug 13, 2009 at 12:33:27AM +0200, David Härdeman wrote: > This patch adds an EV_IR bit to allow input drivers to let userspace > know that the hardware is an infrared remote control transceiver. > > No dev->irbit is defined yet but I plan to add that later (which would > be used to indicate whether the hardware supports RX, TX, etc). > > If the patch is accepted I'll send followup patches adding the > appropriate input_set_capability call to the relevant drivers. > Do you expect devices to actually _send_ EV_IR events? -- Dmitry -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] Add EV_IR bit 2009-08-13 6:49 ` Dmitry Torokhov @ 2009-08-13 8:32 ` David Härdeman 2009-08-13 15:57 ` Dmitry Torokhov 0 siblings, 1 reply; 5+ messages in thread From: David Härdeman @ 2009-08-13 8:32 UTC (permalink / raw) To: Dmitry Torokhov; +Cc: linux-input On Thu, August 13, 2009 08:49, Dmitry Torokhov wrote: > On Thu, Aug 13, 2009 at 12:33:27AM +0200, David Härdeman wrote: >> This patch adds an EV_IR bit to allow input drivers to let userspace >> know that the hardware is an infrared remote control transceiver. >> >> No dev->irbit is defined yet but I plan to add that later (which would >> be used to indicate whether the hardware supports RX, TX, etc). >> >> If the patch is accepted I'll send followup patches adding the >> appropriate input_set_capability call to the relevant drivers. >> > > Do you expect devices to actually _send_ EV_IR events? Yes, and I have some patches going in that direction (loosely based on Jon Smirl's in-kernel IR driver patchset). The basic idea so far would be to add: IR_PROTOCOL IR_TOGGLE IR_DEVICE IR_SUBDEVICE IR_COMMAND Which should be sufficient at this point for the major IR protocols. Then input drivers can report to userspace the actual command they received (and hacks like using MSC_SCAN / MSC_RAW / the sysfs attribute you commented in my winbond-cir driver would not be necessary). It also allows userspace to know that a button was pressed even where there is no keymap for that particular remote which should help a lot in building custom keymaps. Further along we might also add IR_RAW to signal to willing user-space apps the raw IR timings for more esoteric remotes (I guess it would need to be enabled/disabled by an ioctl). -- David Härdeman -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] Add EV_IR bit 2009-08-13 8:32 ` David Härdeman @ 2009-08-13 15:57 ` Dmitry Torokhov 2009-08-13 20:21 ` David Härdeman 0 siblings, 1 reply; 5+ messages in thread From: Dmitry Torokhov @ 2009-08-13 15:57 UTC (permalink / raw) To: David Härdeman; +Cc: linux-input On Thu, Aug 13, 2009 at 10:32:16AM +0200, David Härdeman wrote: > On Thu, August 13, 2009 08:49, Dmitry Torokhov wrote: > > On Thu, Aug 13, 2009 at 12:33:27AM +0200, David Härdeman wrote: > >> This patch adds an EV_IR bit to allow input drivers to let userspace > >> know that the hardware is an infrared remote control transceiver. > >> > >> No dev->irbit is defined yet but I plan to add that later (which would > >> be used to indicate whether the hardware supports RX, TX, etc). > >> > >> If the patch is accepted I'll send followup patches adding the > >> appropriate input_set_capability call to the relevant drivers. > >> > > > > Do you expect devices to actually _send_ EV_IR events? > > Yes, and I have some patches going in that direction (loosely based on Jon > Smirl's in-kernel IR driver patchset). > > The basic idea so far would be to add: > > IR_PROTOCOL > IR_TOGGLE > IR_DEVICE > IR_SUBDEVICE > IR_COMMAND > > Which should be sufficient at this point for the major IR protocols. Then > input drivers can report to userspace the actual command they received > (and hacks like using MSC_SCAN / MSC_RAW / the sysfs attribute you > commented in my winbond-cir driver would not be necessary). > > It also allows userspace to know that a button was pressed even where > there is no keymap for that particular remote which should help a lot in > building custom keymaps. > > Further along we might also add IR_RAW to signal to willing user-space > apps the raw IR timings for more esoteric remotes (I guess it would need > to be enabled/disabled by an ioctl). > Would you mind posting the complete patchset? -- Dmitry -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] Add EV_IR bit 2009-08-13 15:57 ` Dmitry Torokhov @ 2009-08-13 20:21 ` David Härdeman 0 siblings, 0 replies; 5+ messages in thread From: David Härdeman @ 2009-08-13 20:21 UTC (permalink / raw) To: Dmitry Torokhov; +Cc: linux-input On Thu, Aug 13, 2009 at 08:57:45AM -0700, Dmitry Torokhov wrote: >On Thu, Aug 13, 2009 at 10:32:16AM +0200, David Härdeman wrote: >> On Thu, August 13, 2009 08:49, Dmitry Torokhov wrote: >>> Do you expect devices to actually _send_ EV_IR events? >> >> Yes, and I have some patches going in that direction (loosely based on Jon >> Smirl's in-kernel IR driver patchset). ... > >Would you mind posting the complete patchset? Here's an example of what I've been considering, it should hopefully convey the general idea. (Not actually a tested patch yet). Index: linux-2.6/drivers/input/input.c =================================================================== --- linux-2.6.orig/drivers/input/input.c 2009-08-13 18:18:28.000000000 +0200 +++ linux-2.6/drivers/input/input.c 2009-08-13 21:43:29.000000000 +0200 @@ -274,6 +274,10 @@ case EV_PWR: disposition = INPUT_PASS_TO_ALL; break; + + case EV_IR: + disposition = INPUT_PASS_TO_ALL; + break; } if (disposition != INPUT_IGNORE_EVENT && type != EV_SYN) @@ -1375,6 +1379,10 @@ /* do nothing */ break; + case EV_IR: + /* do nothing */ + break; + default: printk(KERN_ERR "input_set_capability: unknown type %u (code %u)\n", Index: linux-2.6/include/linux/input.h =================================================================== --- linux-2.6.orig/include/linux/input.h 2009-08-13 18:18:28.000000000 +0200 +++ linux-2.6/include/linux/input.h 2009-08-13 21:48:49.000000000 +0200 @@ -98,6 +98,7 @@ #define EV_FF 0x15 #define EV_PWR 0x16 #define EV_FF_STATUS 0x17 +#define EV_IR 0x18 #define EV_MAX 0x1f #define EV_CNT (EV_MAX+1) @@ -985,6 +986,32 @@ #define FF_MAX 0x7f #define FF_CNT (FF_MAX+1) +/* + * IR events + */ +#define IR_RAW 0x00 +#define IR_PROTOCOL 0x01 +#define IR_TOGGLE 0x02 +#define IR_COMMAND 0x03 +#define IR_DEVICE 0x04 +#define IR_SUBDEVICE 0x05 +#define IR_CUSTOMER 0x06 + +/* + * IR Protocol values + */ +#define IR_PROTOCOL_RC5 0x00 +#define IR_PROTOCOL_RC5X 0x01 +#define IR_PROTOCOL_RC6_0 0x02 +#define IR_PROTOCOL_RC6_6A 0x03 +#define IR_PROTOCOL_SONY_12 0x04 +#define IR_PROTOCOL_SONY_15 0x05 +#define IR_PROTOCOL_SONY_20 0x06 +#define IR_PROTOCOL_NEC 0x07 /* NEC1, NEC2, NEC1X, NEC2X */ +#define IR_PROTOCOL_NEC_REP 0x08 + +/* The value of TOGGLE, COMMAND, etc, is protocol-specific */ + #ifdef __KERNEL__ /* @@ -1323,6 +1350,11 @@ input_event(dev, EV_FF_STATUS, code, value); } +static inline void input_report_ir(struct input_dev *dev, unsigned int code, int value) +{ + input_event(dev, EV_IR, code, value); +} + static inline void input_report_switch(struct input_dev *dev, unsigned int code, int value) { input_event(dev, EV_SW, code, !!value); Index: linux-2.6/include/linux/input-ir.h =================================================================== --- /dev/null 1970-01-01 00:00:00.000000000 +0000 +++ linux-2.6/include/linux/input-ir.h 2009-08-13 22:06:17.000000000 +0200 @@ -0,0 +1,104 @@ +#ifndef _INPUT_IR_H +#define _INPUT_IR_H + +#include <linux/input.h> + +static inline void input_report_ir_raw(struct input_dev *dev, int value) +{ + input_report_ir(dev, IR_RAW, value); +} + +static inline void input_report_ir_rc5(struct input_dev *dev, + unsigned int device, + unsigned int command, + unsigned int toggle) +{ + input_report_ir(dev, IR_PROTOCOL, IR_PROTOCOL_RC5); + input_report_ir(dev, IR_DEVICE, device & 0x1F); + input_report_ir(dev, IR_COMMAND, command & 0x7F); + input_report_ir(dev, IR_TOGGLE, !!toggle); +} + +static inline void input_report_ir_rc5x(struct input_dev *dev, + unsigned int device, + unsigned int subdevice, + unsigned int command, + unsigned int toggle) +{ + input_report_ir(dev, IR_PROTOCOL, IR_PROTOCOL_RC5X); + input_report_ir(dev, IR_DEVICE, device & 0x1F); + input_report_ir(dev, IR_SUBDEVICE, device & 0x7F); + input_report_ir(dev, IR_COMMAND, command & 0x3F); + input_report_ir(dev, IR_TOGGLE, !!toggle); +} + +static inline void input_report_ir_rc6_0(struct input_dev *dev, + unsigned int device, + unsigned int command, + unsigned int toggle) +{ + input_report_ir(dev, IR_PROTOCOL, IR_PROTOCOL_RC6_0); + input_report_ir(dev, IR_DEVICE, device & 0xFF); + input_report_ir(dev, IR_COMMAND, command & 0xFF); + input_report_ir(dev, IR_TOGGLE, !!toggle); +} + +static inline void input_report_ir_rc6_6a(struct input_dev *dev, + unsigned int customer, + unsigned int device, + unsigned int command, + unsigned int toggle) +{ + input_report_ir(dev, IR_PROTOCOL, IR_PROTOCOL_RC6_0); + input_report_ir(dev, IR_CUSTOMER, customer & 0xFFFF); + input_report_ir(dev, IR_DEVICE, device & 0x7F); + input_report_ir(dev, IR_COMMAND, command & 0xFF); + input_report_ir(dev, IR_TOGGLE, !!toggle); +} + +static inline void input_report_ir_sony_12(struct input_dev *dev, + unsigned int device, + unsigned int command) +{ + input_report_ir(dev, IR_PROTOCOL, IR_PROTOCOL_SONY_12); + input_report_ir(dev, IR_DEVICE, device & 0x1F); + input_report_ir(dev, IR_COMMAND, command & 0x7F); +} + +static inline void input_report_ir_sony_15(struct input_dev *dev, + unsigned int device, + unsigned int command) +{ + input_report_ir(dev, IR_PROTOCOL, IR_PROTOCOL_SONY_15); + input_report_ir(dev, IR_DEVICE, device & 0xFF); + input_report_ir(dev, IR_COMMAND, command & 0x7F); +} + +static inline void input_report_ir_sony_20(struct input_dev *dev, + unsigned int device, + unsigned int subdevice, + unsigned int command) +{ + input_report_ir(dev, IR_PROTOCOL, IR_PROTOCOL_SONY_20); + input_report_ir(dev, IR_DEVICE, device & 0x1F); + input_report_ir(dev, IR_SUBDEVICE, subdevice & 0xFF); + input_report_ir(dev, IR_COMMAND, command & 0x7F); +} + +static inline void input_report_ir_nec(struct input_dev *dev, + unsigned int device, + unsigned int subdevice, + unsigned int command) +{ + input_report_ir(dev, IR_PROTOCOL, IR_PROTOCOL_NEC); + input_report_ir(dev, IR_DEVICE, device & 0xFF); + input_report_ir(dev, IR_SUBDEVICE, subdevice & 0xFF); + input_report_ir(dev, IR_COMMAND, command & 0xFF); +} + +static inline void input_report_ir_nec_rep(struct input_dev *dev) +{ + input_report_ir(dev, IR_PROTOCOL, IR_PROTOCOL_NEC_REP); +} + +#endif /* _INPUT_IR_H */ Index: linux-2.6/drivers/media/dvb/ttpci/budget-ci.c =================================================================== --- linux-2.6.orig/drivers/media/dvb/ttpci/budget-ci.c 2009-08-13 22:11:11.000000000 +0200 +++ linux-2.6/drivers/media/dvb/ttpci/budget-ci.c 2009-08-13 22:15:52.000000000 +0200 @@ -34,6 +34,7 @@ #include <linux/slab.h> #include <linux/interrupt.h> #include <linux/input.h> +#include <linux/input-ir.h> #include <linux/spinlock.h> #include <media/ir-common.h> @@ -173,6 +174,12 @@ budget_ci->ir.rc5_device != (command & 0x1f)) return; + /* Report the raw RC5 event to userspace */ + input_report_ir_rc5(budget_ci->dev, + command & 0x1f, + budget_ci->ir.ir_key, + command & 0x20 ? 1 : 0); + /* Is this a repeated key sequence? (same device, command, toggle) */ raw = budget_ci->ir.ir_key | (command << 8); if (budget_ci->ir.last_raw != raw || !timer_pending(&budget_ci->ir.timer_keyup)) { -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2009-08-13 20:21 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2009-08-12 22:33 [PATCH] Add EV_IR bit David Härdeman 2009-08-13 6:49 ` Dmitry Torokhov 2009-08-13 8:32 ` David Härdeman 2009-08-13 15:57 ` Dmitry Torokhov 2009-08-13 20:21 ` David Härdeman
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.