From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Walls Subject: Re: [PATCH 3/4] ir-core: move decoding state to ir_raw_event_ctrl Date: Thu, 17 Jun 2010 08:14:19 -0400 Message-ID: <1276776859.2461.16.camel@localhost> References: <20100424211411.11570.2189.stgit@localhost.localdomain> <4BDF2B45.9060806@redhat.com> <20100607190003.GC19390@hardeman.nu> <20100607201530.GG16638@redhat.com> <20100608175017.GC5181@hardeman.nu> <20100609132908.GM16638@redhat.com> <20100609175621.GA19620@hardeman.nu> <20100609181506.GO16638@redhat.com> <20100613202945.GA5883@hardeman.nu> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-media-owner@vger.kernel.org To: Jarod Wilson Cc: David =?ISO-8859-1?Q?H=E4rdeman?= , Jarod Wilson , Mauro Carvalho Chehab , linux-media@vger.kernel.org, linux-input@vger.kernel.org List-Id: linux-input@vger.kernel.org On Wed, 2010-06-16 at 16:41 -0400, Jarod Wilson wrote: > On Wed, Jun 16, 2010 at 4:04 PM, Jarod Wilson wrote: > ... > >> I have another suggestion, let's keep the client register/unregister > >> callbacks for decoders (but add a comment that they're only used for > >> lirc). Then teach drivers/media/IR/ir-raw-event.c to keep track of the > >> raw clients so that it can pass all pre-existing clients to newly added > >> decoders. > >> > >> I'll post two patches (compile tested only) in a few seconds to show > >> what I mean. > > > > Consider them now runtime tested as well. They appear to do the trick, > > the lirc bridge comes up just fine, even when ir-lirc-codec isn't > > loaded until after mceusb. *Much* better implementation than my ugly > > trick. I'll ack your patches and submit a series on top of them for > > lirc support, hopefully this evening (in addition to a few other fixes > > that aren't dependent on any of them). > > A fully functional tree carrying both of David's patches and the > entire stack of other patches I've submitted today, based on top of > the linuxtv staging/rc branch, can be found here: > > http://git.wilsonet.com/linux-2.6-ir-wip.git/?a=shortlog;h=refs/heads/patches > > Also includes the lirc patches that I believe are ready to be > submitted for actual consideration (note that they're dependent on > David's two patches). I'll try and play with this this weekend along with some cx23885 cleanup. Regards, Andy