Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965740AbZLHSPV (ORCPT ); Tue, 8 Dec 2009 13:15:21 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S937178AbZLHSPR (ORCPT ); Tue, 8 Dec 2009 13:15:17 -0500 Received: from mail-qy0-f192.google.com ([209.85.221.192]:55954 "EHLO mail-qy0-f192.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S937077AbZLHSPO convert rfc822-to-8bit (ORCPT ); Tue, 8 Dec 2009 13:15:14 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; b=UmKE7D0IghjiWGOO1Ih7oxCfY2mEYNgh4JXjst41Dzr1VaejzAYM6xqDeX20VeyN9o dZJZDrU93/oH+pwycPK8ntjQHTx4KLlLgWKIj+7xkdwP/I4YvflRhhR9RxSA/kLvEVog q6QbDiJ5B/bLyutDnIr2wk/X2vcjm59WmNC7w= MIME-Version: 1.0 In-Reply-To: <4B1E7E56.80701@redhat.com> References: <20091208042210.GA11147@core.coreip.homeip.net> <1260275743.3094.6.camel@palomino.walls.org> <4B1E54FF.8060404@redhat.com> <9e4733910912080547j75c2c885o29664470ff5e2c6a@mail.gmail.com> <4B1E5BDF.7010202@redhat.com> <9e4733910912080619t36089c9bg5e54114844b9694a@mail.gmail.com> <4B1E640B.6030705@redhat.com> <9e4733910912080756j7e1fac32qc552c6514a307b7d@mail.gmail.com> <4B1E7E56.80701@redhat.com> Date: Tue, 8 Dec 2009 13:15:20 -0500 Message-ID: <9e4733910912081015he8b9b63o27ee802dea7adcfc@mail.gmail.com> Subject: Re: [RFC] Should we create a raw input interface for IR's ? - Was: Re: [PATCH 1/3 v2] lirc core device driver infrastructure From: Jon Smirl To: Mauro Carvalho Chehab Cc: Andy Walls , Dmitry Torokhov , Jarod Wilson , Krzysztof Halasa , Christoph Bartelmus , j@jannau.net, jarod@redhat.com, linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, superm1@ubuntu.com Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 9047 Lines: 191 On Tue, Dec 8, 2009 at 11:27 AM, Mauro Carvalho Chehab wrote: > Jon Smirl wrote: >> On Tue, Dec 8, 2009 at 9:34 AM, Mauro Carvalho Chehab >> wrote: >>> Jon Smirl wrote: >>>> On Tue, Dec 8, 2009 at 8:59 AM, Mauro Carvalho Chehab >>>> wrote: >>>>> Jon Smirl wrote: >>>>>> On Tue, Dec 8, 2009 at 8:30 AM, Mauro Carvalho Chehab >>>>>> wrote: >>>>>>> Andy Walls wrote: >>>>>>>> On Mon, 2009-12-07 at 20:22 -0800, Dmitry Torokhov wrote: >>>>>>>>> On Mon, Dec 07, 2009 at 09:42:22PM -0500, Andy Walls wrote: >>>>>>>>>> So I'll whip up an RC-6 Mode 6A decoder for cx23885-input.c before the >>>>>>>>>> end of the month. >>>>>>>>>> >>>>>>>>>> I can setup the CX2388[58] hardware to look for both RC-5 and RC-6 with >>>>>>>>>> a common set of parameters, so I may be able to set up the decoders to >>>>>>>>>> handle decoding from two different remote types at once. ?The HVR boards >>>>>>>>>> can ship with either type of remote AFAIK. >>>>>>>>>> >>>>>>>>>> I wonder if I can flip the keytables on the fly or if I have to create >>>>>>>>>> two different input devices? >>>>>>>>>> >>>>>>>>> Can you distinguish between the 2 remotes (not receivers)? >>>>>>>> Yes. ?RC-6 and RC-5 are different enough to distinguish between the two. >>>>>>>> (Honestly I could pile on more protocols that have similar pulse time >>>>>>>> periods, but that's complexity for no good reason and I don't know of a >>>>>>>> vendor that bundles 3 types of remotes per TV card.) >>>>>>> You'll be distinguishing the protocol, not the remote. If I understood >>>>>>> Dmitry's question, he is asking if you can distinguish between two different >>>>>>> remotes that may, for example, be using both RC-5 or both RC-6 or one RC-5 >>>>>>> and another RC-6. >>>>>> RC-5 and RC-6 both contain an address field. ?My opinion is that >>>>>> different addresses represent different devices and in general they >>>>>> should appear on an input devices per address. >>>>> The same IR can produce two different addresses. The IR bundled with my satellite >>>>> STB produces two different codes, depending if you previously pressed or >>>>> key (in fact, I think it can even produce different protocols for TV, as it can >>>>> be configured to work with different TV sets). >>>> You have a multi-function remote. >>> Yes. >>> >>>> That's why those keys don't send codes. When writing code you should >>>> think of this remote as being two indpendent virtual remotes, not a >>>> single one. >>> Not really. I may think on it as a single device and use the two groups >>> of functions to control two aspects at the same application. >>> >>> For example, I may map the group on kaffeine for DVB reception and the >>> group for DVD (well, probably, in this case, I'll use an IR with >>> and keys, instead ;) ). >>> >>>> By using maps containing the two different addresses for and >>>> you can split these commands onto two different evdev devices. >>> True. I can do it, but I can opt to have both mapped as one evdev device as well. >>> This will basically depend on how I want to mount my environment. >>> >>>> This model is complicated by the fact that some remotes that look like >>>> multi-function remotes aren't really multifunction. The remote bundled >>>> with the MS MCE receiver is one. That remote is a single function >>>> device even though it has function buttons for TV, Music, Pictures, >>>> etc. >>> It is very common to have such remotes bundled with multimedia devices. >>> >>> An unsolved question on my mind is how should we map such IR's? Should we >>> provide a way for them to emulate a multifunction IR (for example, after pressing >>> TV key, subsequent keystrokes would be directed to the TV evdev device?), or >>> should we let this up to some userspace app to handle this case? >> >> Splitting them into multiple devices requires remembering state and >> scripting so it needs to be done in user space. > > It shouldn't be hard to do it in kernelspace, since you'll need to have > one evdev interface associated with the IR anyway, but this will add > some extra complexity at the scancode->keycode conversion, but I'm wandering > if we should do it or not. > > Maybe the better is to not do it in kernelspace, to avoid adding there an > extra complexity that can easily be done in userspace. > >> If the user wants to >> control a radio app and a home automation app they need to choose. >> Keep the bundled remote and do some non-trivial scripting or buy a >> universal remote. > > Ok, but using the shipped IR even without a separate address group for > different applications, and having it controlling radio app and tv app > (not simultaneously) should not be hard, but I LIRC already covers such > usecase, so maybe we don't need to worry about it. > >> Universal remotes make it much easier to achieve "just works". > > True. > >> The IR core can contain default universal profiles for various classes >> of devices. Say Morotola_DVR and SciAtlanta_DVR. The core would check >> if the receiver is cable of receiving these profiles before loading >> them. There would be ten of these default universal profiles at most >> and you can unload them from RAM if they aren't needed. >> >> Now Myth can have a menu with three remote choices: >> ?Universal Morotola_DVR >> ?Universal SciAtlanta_DVR >> ?Bundled >> >> The Bundled choice came from the map built into the IR receiver's device driver. >> The other two choices were loaded by the IR core after ensuring that >> the hardware could receive from a universal remote. >> >> The core would also load a couple of default radio profiles >> ?Univeral SonyDR112_RADIO >> ?Univeral OnkyoTX8255_RADIO >> Same for automation and mouse/keyboard emulation. > > Agreed. > >> Myth looks in sysfs and builds a menu containing DVR devices and all >> bundled entries. First app to open the "Bundled" device gets to keep >> it. > > Myth (or other userspace apps) don't need to to that, since we've standardized > the keycode actions (see the IR chapter of the media DocBook). It just > needs to support the keycodes already defined, for the common case. > >> These apps could take "just works" even farther. When they start up >> they could listen on all three evdev devices: ?Morotola_DVR, >> SciAtlanta_DVR, ? Bundled. And then if you find Myth responding to >> unwanted commands you could disable the unwanted profiles by >> deselecting them in the Myth UI. > > I don't like the idea of automatically loading 3 different keycodes at the > same time. You may have overlaps between different keycode tables. The > better is to have some userspace GUI that will allow the user to select > what keycode table(s) he want to be available, if he decides to not use the > bundled IR. Of course there is going to be overlap of the keycodes, but not the scancodes. There should be almost 100% overlap. The three maps are there to support a non-technical user, a sophisticated user will disable two of them. This works because the non-technical user is only going to use one of the three IR device profiles. The other two may be loaded, but the user isn't sending any IR signals that match their maps. Where this breaks down is if they are using SciAtlanta_DVR to control MythTV and they also happen to have a physical Motorola DVR in the same room. The Linux box is going to pick up the commands meant for the Motorola DVR and both boxes will respond.. In that cause they will need to figure figure out how to disable the Motorola DVR profile. But is a non-technical person likely to have two DVRs in the same room? > > The same applies to applications: if you have 3 keymaps loaded, is because you > want do do different things with the 3 keymaps (like using keymap 1 for kaffeine, > keymap 2 for mplayer, keymap 3 for mythtv). > > So, IR-aware applications should have a setup interface to specify what IR keycodes > are relevant to that particular application, and how to associate an evdev interface > to an specific group of functions (for applications that supports several different > types of media, like MythTV and Kaffeine, where you'll end by having a "TV" keymap/evdev, > a "Radio" keymap/evdev, a "CD/DVD" Keymap/evdev, etc). > > Btw, if we're doing that, IMO, we should have an string sysfs alias attribute, > to allow associating the userspace application to an specific keymap alias > (like "radio", "tv", etc.). > >> All of this may seem complicated to build, but the purpose is to >> create an environment where a non-technical user can get an IR remote >> working without needing detailed knowledge about how IR protocols >> work. > > IMO, this is an important requisite to fulfill. > > Cheers, > Mauro. > -- Jon Smirl jonsmirl@gmail.com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/