From mboxrd@z Thu Jan 1 00:00:00 1970 From: Devin Heitmueller Subject: Re: [RFC v2] Another approach to IR Date: Tue, 1 Dec 2009 12:09:53 -0500 Message-ID: <829197380912010909m59cb1078q5bd2e00af0368aaf@mail.gmail.com> References: <9e4733910912010708u1064e2c6mbc08a01293c3e7fd@mail.gmail.com> <1259682428.18599.10.camel@maxim-laptop> <9e4733910912010816q32e829a2uce180bfda69ef86d@mail.gmail.com> <4B154C54.5090906@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Return-path: In-Reply-To: <4B154C54.5090906@redhat.com> Sender: linux-kernel-owner@vger.kernel.org To: Mauro Carvalho Chehab Cc: Jon Smirl , Maxim Levitsky , awalls@radix.net, dmitry.torokhov@gmail.com, j@jannau.net, jarod@redhat.com, jarod@wilsonet.com, khc@pm.waw.pl, linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, lirc-list@lists.sourceforge.net, superm1@ubuntu.com, Christoph Bartelmus List-Id: linux-input@vger.kernel.org On Tue, Dec 1, 2009 at 12:03 PM, Mauro Carvalho Chehab wrote: > Just taking an example from the dibcom0700 driver (as the same driver > supports several different RC5 and NEC codes at the same time), > the kernel table has several keycodes added there, all working > at the same time. Providing that the scancodes won't overlap, you can > map two different scancodes (from different IR's) to return the same > keycode (table is not complete - I just got a few common keycodes): Mauro, Just to be clear, the dib0700 does not actually support receiving RC5 or NEC codes at the same time. You have to tell the chip which mode to operate in, via a REQUEST_SET_RC to the firmware (see dib0700_core.c:405). The em28xx works the same way (you have to tell it what type of IR format to receive). The fact that the driver currently uses the same lookup table for both types of remote controls however, was perhaps not the best design choice. It really should be separated out, and merged with the regular ir-functions.c. I just never got around to it. Devin -- Devin J. Heitmueller - Kernel Labs http://www.kernellabs.com