From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jon Smirl Subject: Re: [PATCH 00/15] ir-core: Several improvements to allow adding LIRC and decoder plugins Date: Thu, 1 Apr 2010 21:44:12 -0400 Message-ID: References: <20100401145632.5631756f@pedra> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from qw-out-2122.google.com ([74.125.92.26]:51499 "EHLO qw-out-2122.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754480Ab0DBBvU convert rfc822-to-8bit (ORCPT ); Thu, 1 Apr 2010 21:51:20 -0400 In-Reply-To: <20100401145632.5631756f@pedra> Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Mauro Carvalho Chehab Cc: linux-input@vger.kernel.org, Linux Media Mailing List , Jarod Wilson On Thu, Apr 1, 2010 at 1:56 PM, Mauro Carvalho Chehab wrote: > This series of 15 patches improves support for IR, as discussed at th= e > "What are the goals for the architecture of an in-kernel IR system?" > thread. > > It basically adds a raw decoder layer at ir-core, allowing decoders t= o plug > into IR core, and preparing for the addition of a lirc_dev driver tha= t will > allow raw IR codes to be sent to userspace. > > There's no lirc patch in this series. I have also a few other patches= from > David H=E4rdeman that I'm about to test/review probably later today, = but > as I prefer to first merge what I have at V4L/DVB tree, before applyi= ng > them. Has anyone ported the MSMCE driver onto these patches yet? That would be a good check to make sure that rc-core has the necessary API. Cooler if it works both through LIRC and with an internal protocol decoder. The MSMCE driver in my old patches was very simplified, it removed about half of the code from the LIRC version. > > There are two patches on this series that deserve a better analysis, = IMO: > > - =A0V4L/DVB: ir-core: rename sysfs remote controller class from ir t= o rc > > As discussed, "IR" is not a good name, as this infrastructure could l= ater > be used by other types of Remote Controllers, as it has nothing that > is specific to IR inside the code, except for the name. So, I'm propo= sing > to replace the sysfs notes do "rc", instead of "ir". The sooner we do > such changes, the better, as userspace apps using it are still under > development. So, an API change is still possible, without causing > much hurt. > > Also, as some RC devices allow RC code transmission, we probably need= to add > a TX node somewhere, associated with the same RX part (as some device= s > don't allow simultaneous usage of TX and RX). > > So, we have a few alternatives for the RC device sysfs node: > a) /sys/class/rc/rc0 > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 |--> rx > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 ---> tx > b) /sys/class/rc/rcrcv0 > =A0 /sys/class/rc/rctx0 > > c) /sys/class/rc/rc0 > =A0and have there the RX and TX nodes/attributes mixed. IMO, (b) is a= bad idea, > so, I am between (a) and (c). > > - =A0V4L/DVB: input: Add support for EVIO[CS]GKEYCODEBIG > > Adds two new ioctls in order to handle with big keycode tables. As al= ready > said, we'll need another ioctl, in order to get the maximum keycode s= upported > by a given device. I didn't wrote the patch for the new ioctl yet. > This patch will probably have a small conflict with upstream input, b= ut I > prefer to keep it on my tree and fix the upstream conflicts when subm= iting > it, as the rest of the new IR code is also on my tree, and this patch= is > needed to procced with the IR code development. > > Mauro Carvalho Chehab (15): > =A0V4L/DVB: ir-core: be less pedantic with RC protocol name > =A0V4L/DVB: saa7134: use a full scancode table for M135A > =A0V4L/DVB: saa7134: add code to allow changing IR protocol > =A0V4L/DVB: ir-core: Add logic to decode IR protocols at the IR core > =A0V4L/DVB: ir-core: add two functions to report keyup/keydown events > =A0V4L/DVB: ir-core/saa7134: Move ir keyup/keydown code to the ir-cor= e > =A0V4L/DVB: saa7134: don't wait too much to generate an IR event on r= aw_decode > =A0V4L/DVB: ir-core: dynamically load the compiled IR protocols > =A0V4L/DVB: ir-core: prepare to add more operations for ir decoders > =A0V4L/DVB: ir-nec-decoder: Add sysfs node to enable/disable per irrc= v > =A0V4L/DVB: saa7134: clear warning noise > =A0V4L/DVB: ir-core: rename sysfs remote controller class from ir to = rc > =A0V4L/DVB: ir-core: Add callbacks for input/evdev open/close on IR c= ore > =A0V4L/DVB: cx88: Only start IR if the input device is opened > =A0V4L/DVB: input: Add support for EVIO[CS]GKEYCODEBIG > > =A0drivers/input/evdev.c =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 = | =A0 39 +++ > =A0drivers/input/input.c =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 = | =A0260 ++++++++++++++++++-- > =A0drivers/media/IR/Kconfig =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0| = =A0 =A09 + > =A0drivers/media/IR/Makefile =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 | =A0= =A03 +- > =A0drivers/media/IR/ir-keymaps.c =A0 =A0 =A0 =A0 =A0 =A0 =A0 | =A0 98= ++++---- > =A0drivers/media/IR/ir-keytable.c =A0 =A0 =A0 =A0 =A0 =A0 =A0| =A0 75= ++++++- > =A0drivers/media/IR/ir-nec-decoder.c =A0 =A0 =A0 =A0 =A0 | =A0351 +++= ++++++++++++++++++++++++ > =A0drivers/media/IR/ir-raw-event.c =A0 =A0 =A0 =A0 =A0 =A0 | =A0231 += +++++++++++++++++ > =A0drivers/media/IR/ir-sysfs.c =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 | =A0 = 29 ++- > =A0drivers/media/video/cx88/cx88-input.c =A0 =A0 =A0 | =A0 69 +++++- > =A0drivers/media/video/cx88/cx88-video.c =A0 =A0 =A0 | =A0 =A06 +- > =A0drivers/media/video/cx88/cx88.h =A0 =A0 =A0 =A0 =A0 =A0 | =A0 =A06= +- > =A0drivers/media/video/saa7134/saa7134-core.c =A0| =A0 =A02 +- > =A0drivers/media/video/saa7134/saa7134-input.c | =A0207 +++++++++++++= ++-- > =A0drivers/media/video/saa7134/saa7134.h =A0 =A0 =A0 | =A0 =A04 +- > =A0include/linux/input.h =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 = | =A0 40 +++- > =A0include/media/ir-common.h =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 | =A0= =A09 +- > =A0include/media/ir-core.h =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 | = =A0 59 +++++- > =A018 files changed, 1368 insertions(+), 129 deletions(-) > =A0create mode 100644 drivers/media/IR/ir-nec-decoder.c > =A0create mode 100644 drivers/media/IR/ir-raw-event.c > > -- > 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 =A0http://vger.kernel.org/majordomo-info.html > --=20 Jon Smirl jonsmirl@gmail.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