From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dmitry Torokhov Subject: Re: [PATCH] generic driver for rotary encoders on GPIOs Date: Tue, 3 Mar 2009 00:52:00 -0800 Message-ID: <20090303085200.GA13793@dtor-d630.eng.vmware.com> References: <20090227064314.GB32470@buzzloop.caiaq.de> <1235757652-16514-1-git-send-email-daniel@caiaq.de> <20090227183449.GD8855@buzzloop.caiaq.de> <20090302144308.GB11072@buzzloop.caiaq.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from el-out-1112.google.com ([209.85.162.183]:46918 "EHLO el-out-1112.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752988AbZCCIwO (ORCPT ); Tue, 3 Mar 2009 03:52:14 -0500 Received: by el-out-1112.google.com with SMTP id b25so2044326elf.1 for ; Tue, 03 Mar 2009 00:52:11 -0800 (PST) Content-Disposition: inline In-Reply-To: <20090302144308.GB11072@buzzloop.caiaq.de> Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Daniel Mack Cc: linux-input@vger.kernel.org Hi Daniel, On Mon, Mar 02, 2009 at 03:43:08PM +0100, Daniel Mack wrote: > On Fri, Feb 27, 2009 at 03:15:23PM -0500, hartleys wrote: > > Not sure if it matters but you can add my Tested-by if you like. > > > > Tested-by: H Hartley Sweeten > > > > >> I have a minor patch to it that allows the platform init to > > >> configure the absolute/relative axis to report the encoder on. > > >> If you would like to see it I can put together a patch. > > > > > > Yes, and probably just post it as follow-up as soon as this one > > > has been commited. I'm new to this list, so I don't know who > > > can/will/wants to do this? > > > > Dmitry Torokhov is listed as the maintainer of the input drivers. I > > think he needs to Ack the driver in order for it to be accepted and then > > committed to the kernel. > > Dmitry, are you willing to take this patch? Just wanted to make sure > it's not getting lost :) > I am willing, however I would like you to change it so you call input_unregister_device() instead of input_free_device() in cases when input device was successfully registered with the input core. When input_register_device() fails (or if it wasn't invoked) then you should call input_free_device(). Thanks! -- Dmitry