From: Jarod Wilson <jarod@wilsonet.com>
To: Devin Heitmueller <dheitmueller@kernellabs.com>
Cc: "Mauro Carvalho Chehab" <mchehab@infradead.org>,
"Ville Syrjälä" <syrjala@sci.fi>,
"Linux Media Mailing List" <linux-media@vger.kernel.org>,
"Linux Input" <linux-input@vger.kernel.org>
Subject: Re: [RFC] Infrared Keycode standardization
Date: Fri, 28 Aug 2009 00:00:54 -0400 [thread overview]
Message-ID: <200908280000.54986.jarod@wilsonet.com> (raw)
In-Reply-To: <829197380908271506i251b47caoe8c08d483e78e938@mail.gmail.com>
On Thursday 27 August 2009 18:06:51 Devin Heitmueller wrote:
> On Thu, Aug 27, 2009 at 5:58 PM, Mauro Carvalho
> Chehab<mchehab@infradead.org> wrote:
> > Em Thu, 27 Aug 2009 21:36:36 +0300
> > Ville Syrjälä <syrjala@sci.fi> escreveu:
> >
> >
> >> I welcome this effort. It would be nice to have some kind of consistent
> >> behaviour between devices. But just limiting the effort to IR devices
> >> doesn't make sense. It shouldn't matter how the device is connected.
> >
> > Agreed.
> >
> >>
> >> FASTWORWARD,REWIND,FORWARD and BACK aren't very clear. To me it would
> >> make most sense if FASTFORWARD and REWIND were paired and FORWARD and
> >> BACK were paired. I actually have those two a bit confused in
> >> ati_remote2 too where I used FASTFORWARD and BACK. I suppose it should
> >> be REWIND instead.
> >
> > Makes sense. I updated it at the wiki. I also tried to group the keycodes by
> > function there.
> >
> >> Also I should probably use ZOOM for the maximize/restore button (it's
> >> FRONT now), and maybe SETUP instead of ENTER for another. It has a
> >> picture of a checkbox, Windows software apparently shows a setup menu
> >> when it's pressed.
> >>
> >> There are also a couple of buttons where no keycode really seems to
> >> match. One is the mouse button drag. I suppose I could implement the
> >> drag lock feature in the driver but I'm not sure if that's a good idea.
> >> It would make that button special and unmappable. Currently I have that
> >> mapped to EDIT IIRC.
> >
> > I'm not sure what we should do with those buttons.
> >
> > Probably, the most complete IR spec is the RC5 codes:
> > http://c6000.spectrumdigital.com/davincievm/revf/files/msp430/rc5_codes.pdf
> > (not sure if this table is complete or accurate, but on a search I did
> > today, this is the one that gave me a better documentation)
> >
> > I suspect that, after solving the most used cases, we'll need to take a better look on it,
> > identifying the missing cases of the real implementations and add them to input.h.
> >
> >> The other oddball button has a picture of a stopwatch (I think, it's
> >> not very clear). Currently it uses COFFEE, but maybe TIMER or something
> >> like that should be added. The Windows software's manual just say it
> >> toggles TV-on-demand, but I have no idea what that actually is.
> >
> > Hmm... Maybe TV-on-demand is another name for pay-per-view?
> >
> >
> >
> > Cheers,
> > Mauro
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-media" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
> >
>
> Since we're on the topic of IR support, there are probably a couple of
> other things we may want to be thinking about if we plan on
> refactoring the API at all:
>
> 1. The fact that for RC5 remote controls, the tables in ir-keymaps.c
> only have the second byte. In theory, they should have both bytes
> since the vendor byte helps prevents receiving spurious commands from
> unrelated remote controls. We should include the ability to "ignore
> the vendor byte" so we can continue to support all the remotes
> currently in the ir-keymaps.c where we don't know what the vendor byte
> should contain.
>
> 2.. The fact that the current API provides no real way to change the
> mode of operation for the IR receiver, for those receivers that
> support multiple modes (NEC/RC5/RC6). While you have the ability to
> change the mapping table from userland via the keytable program, there
> is currently no way to tell the IR receiver which mode to operate in.
>
> One would argue that the above keymaps structure should include new
> fields to indicate what type of remote it is (NEC/RC5/RC6 etc), as
> well as field to indicate that the vendor codes are absent from the
> key mapping for that remote). Given this, I can change the dib0700
> and em28xx IR receivers to automatically set the IR capture mode
> appropriate based on which remote is in the device profile.
Jon Smirl actually wrote some fully functional proof-of-concept IR
handling code about a year ago, that included auto-detection and auto
decoding of several protocols. Perhaps some of that is relevant and
reusable here? (I still have a copy of the tree here somewhere...)
I've been toying with the notion of extending the input device support
that was added to the lirc_imon driver a bit ago, and add a full key
map that delivers events (we already do this for mouse functionality),
but include the ability to also use the remote and/or receiver in a
raw IR mode with lircd. Wouldn't be terribly difficult I think to do
something similar for the standard MCE remotes and receivers... Just
a simple matter of some time and some code. Unfortunately, I'm a bit
short on the time part right now...
--
Jarod Wilson
jarod@wilsonet.com
--
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
WARNING: multiple messages have this Message-ID (diff)
From: Jarod Wilson <jarod@wilsonet.com>
To: Devin Heitmueller <dheitmueller@kernellabs.com>
Cc: "Mauro Carvalho Chehab" <mchehab@infradead.org>,
"Ville Syrjälä" <syrjala@sci.fi>,
"Linux Media Mailing List" <linux-media@vger.kernel.org>,
"Linux Input" <linux-input@vger.kernel.org>
Subject: Re: [RFC] Infrared Keycode standardization
Date: Fri, 28 Aug 2009 00:00:54 -0400 [thread overview]
Message-ID: <200908280000.54986.jarod@wilsonet.com> (raw)
In-Reply-To: <829197380908271506i251b47caoe8c08d483e78e938@mail.gmail.com>
On Thursday 27 August 2009 18:06:51 Devin Heitmueller wrote:
> On Thu, Aug 27, 2009 at 5:58 PM, Mauro Carvalho
> Chehab<mchehab@infradead.org> wrote:
> > Em Thu, 27 Aug 2009 21:36:36 +0300
> > Ville Syrjälä <syrjala@sci.fi> escreveu:
> >
> >
> >> I welcome this effort. It would be nice to have some kind of consistent
> >> behaviour between devices. But just limiting the effort to IR devices
> >> doesn't make sense. It shouldn't matter how the device is connected.
> >
> > Agreed.
> >
> >>
> >> FASTWORWARD,REWIND,FORWARD and BACK aren't very clear. To me it would
> >> make most sense if FASTFORWARD and REWIND were paired and FORWARD and
> >> BACK were paired. I actually have those two a bit confused in
> >> ati_remote2 too where I used FASTFORWARD and BACK. I suppose it should
> >> be REWIND instead.
> >
> > Makes sense. I updated it at the wiki. I also tried to group the keycodes by
> > function there.
> >
> >> Also I should probably use ZOOM for the maximize/restore button (it's
> >> FRONT now), and maybe SETUP instead of ENTER for another. It has a
> >> picture of a checkbox, Windows software apparently shows a setup menu
> >> when it's pressed.
> >>
> >> There are also a couple of buttons where no keycode really seems to
> >> match. One is the mouse button drag. I suppose I could implement the
> >> drag lock feature in the driver but I'm not sure if that's a good idea.
> >> It would make that button special and unmappable. Currently I have that
> >> mapped to EDIT IIRC.
> >
> > I'm not sure what we should do with those buttons.
> >
> > Probably, the most complete IR spec is the RC5 codes:
> > http://c6000.spectrumdigital.com/davincievm/revf/files/msp430/rc5_codes.pdf
> > (not sure if this table is complete or accurate, but on a search I did
> > today, this is the one that gave me a better documentation)
> >
> > I suspect that, after solving the most used cases, we'll need to take a better look on it,
> > identifying the missing cases of the real implementations and add them to input.h.
> >
> >> The other oddball button has a picture of a stopwatch (I think, it's
> >> not very clear). Currently it uses COFFEE, but maybe TIMER or something
> >> like that should be added. The Windows software's manual just say it
> >> toggles TV-on-demand, but I have no idea what that actually is.
> >
> > Hmm... Maybe TV-on-demand is another name for pay-per-view?
> >
> >
> >
> > Cheers,
> > Mauro
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-media" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
> >
>
> Since we're on the topic of IR support, there are probably a couple of
> other things we may want to be thinking about if we plan on
> refactoring the API at all:
>
> 1. The fact that for RC5 remote controls, the tables in ir-keymaps.c
> only have the second byte. In theory, they should have both bytes
> since the vendor byte helps prevents receiving spurious commands from
> unrelated remote controls. We should include the ability to "ignore
> the vendor byte" so we can continue to support all the remotes
> currently in the ir-keymaps.c where we don't know what the vendor byte
> should contain.
>
> 2.. The fact that the current API provides no real way to change the
> mode of operation for the IR receiver, for those receivers that
> support multiple modes (NEC/RC5/RC6). While you have the ability to
> change the mapping table from userland via the keytable program, there
> is currently no way to tell the IR receiver which mode to operate in.
>
> One would argue that the above keymaps structure should include new
> fields to indicate what type of remote it is (NEC/RC5/RC6 etc), as
> well as field to indicate that the vendor codes are absent from the
> key mapping for that remote). Given this, I can change the dib0700
> and em28xx IR receivers to automatically set the IR capture mode
> appropriate based on which remote is in the device profile.
Jon Smirl actually wrote some fully functional proof-of-concept IR
handling code about a year ago, that included auto-detection and auto
decoding of several protocols. Perhaps some of that is relevant and
reusable here? (I still have a copy of the tree here somewhere...)
I've been toying with the notion of extending the input device support
that was added to the lirc_imon driver a bit ago, and add a full key
map that delivers events (we already do this for mouse functionality),
but include the ability to also use the remote and/or receiver in a
raw IR mode with lircd. Wouldn't be terribly difficult I think to do
something similar for the standard MCE remotes and receivers... Just
a simple matter of some time and some code. Unfortunately, I'm a bit
short on the time part right now...
--
Jarod Wilson
jarod@wilsonet.com
next prev parent reply other threads:[~2009-08-28 4:01 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-08-27 7:57 [RFC] Infrared Keycode standardization Mauro Carvalho Chehab
2009-08-27 17:06 ` Peter Brouwer
2009-08-27 17:12 ` Jarod Wilson
2009-08-27 17:17 ` Devin Heitmueller
2009-08-27 18:29 ` Trent Piepho
2009-08-27 18:34 ` Devin Heitmueller
2009-08-27 18:34 ` Devin Heitmueller
2009-08-27 19:28 ` Mauro Carvalho Chehab
2009-08-27 19:32 ` Mauro Carvalho Chehab
2009-08-27 20:17 ` Dmitry Torokhov
2009-08-27 21:43 ` semiRocket
2009-08-27 18:36 ` Ville Syrjälä
2009-08-27 18:36 ` Ville Syrjälä
2009-08-27 20:15 ` Dmitry Torokhov
2009-08-27 20:15 ` Dmitry Torokhov
2009-08-27 22:03 ` Mauro Carvalho Chehab
2009-08-27 22:03 ` Mauro Carvalho Chehab
2009-08-27 21:58 ` Mauro Carvalho Chehab
2009-08-27 21:58 ` Mauro Carvalho Chehab
2009-08-27 22:06 ` Devin Heitmueller
2009-08-27 22:06 ` Devin Heitmueller
2009-08-28 3:46 ` Mauro Carvalho Chehab
2009-08-28 7:14 ` Mauro Carvalho Chehab
2009-08-28 10:13 ` Peter Brouwer
2009-08-28 12:36 ` Mauro Carvalho Chehab
2009-08-28 14:22 ` Alistair Buxton
2009-08-28 10:50 ` Patrick Boettcher
2009-08-28 12:30 ` Mauro Carvalho Chehab
2009-08-29 18:45 ` Mauro Carvalho Chehab
2009-08-31 5:47 ` Mauro Carvalho Chehab
2009-09-01 1:38 ` Dmitry Torokhov
2009-08-28 4:00 ` Jarod Wilson [this message]
2009-08-28 4:00 ` Jarod Wilson
2009-08-28 14:05 ` Mauro Carvalho Chehab
2009-08-28 14:05 ` Mauro Carvalho Chehab
2009-08-28 11:41 ` Andy Walls
2009-08-28 11:41 ` Andy Walls
2009-08-28 14:02 ` Mauro Carvalho Chehab
2009-08-28 14:02 ` Mauro Carvalho Chehab
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200908280000.54986.jarod@wilsonet.com \
--to=jarod@wilsonet.com \
--cc=dheitmueller@kernellabs.com \
--cc=linux-input@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@infradead.org \
--cc=syrjala@sci.fi \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.