Re: [PATCH 00/15] ir-core: Several improvements to allow adding LIRC and decoder plugins

[Jarod Wilson <jarod@wilsonet.com>]

On Fri, Apr 2, 2010 at 6:20 AM, David Härdeman <david@hardeman.nu> wrote:
> On Thu, Apr 01, 2010 at 09:44:12PM -0400, Jon Smirl wrote:
>> On Thu, Apr 1, 2010 at 1:56 PM, Mauro Carvalho Chehab
>> <mchehab@redhat.com> wrote:
>> > This series of 15 patches improves support for IR, as discussed at the
>> > "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 to plug
>> > into IR core, and preparing for the addition of a lirc_dev driver that 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ärdeman 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 applying
>> > 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.
>
> I still plan to make lots of changes to the rc-core API (I just have to
> convince Mauro first, but I'll get there). What I have done is to port
> your port of the msmce driver to the suggested rc-core subsystem I sent
> you in private a week or so ago, and it works fine (I've bought the
> hardware and tested it with 20 or so different protocols).
>
> The subsystem I suggested is basically what I'm using as inspiration
> while working with Mauro in improving rc-core so msmce should work well
> with the end product...but there's still some ground to cover.
>
> Porting the msmce driver to rc-core will be high on my list of
> priorities once I've done some more changes to the API.

Very cool. Though note that the latest lirc_mceusb is quite heavily
modified from what Jon had initially ported, and I still have a few
outstanding enhancements to make, such as auto-detecting xmit mask to
eliminate the crude inverted mask list and support for the mce IR
keyboard/mouse, though that'll probably be trivial once RC5 and RC6
in-kernel decoders are in place. I'd intended to start with porting
the imon driver I'm working on over to this new infra (onboard
hardware decoder, should be rather easy to port), and then hop over to
the mceusb driver, but if you beat me to it, I've got no problem with
you doing it instead. :)

>> 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.
>
> Yes, and it was a great help to me at least...thanks :)

Yeah, copy that. Good to see we've got some major momentum going now,
just need to get off my butt and do some more work on it myself...

-- 
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