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
next prev parent reply other threads:[~2009-08-28 4:01 UTC|newest]
Thread overview: 29+ 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 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 20:15 ` Dmitry Torokhov
2009-08-27 22:03 ` Mauro Carvalho Chehab
2009-08-27 21:58 ` Mauro Carvalho Chehab
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 14:05 ` Mauro Carvalho Chehab
2009-08-28 11:41 ` Andy Walls
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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).