Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753183AbZJVKLL (ORCPT ); Thu, 22 Oct 2009 06:11:11 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751675AbZJVKLK (ORCPT ); Thu, 22 Oct 2009 06:11:10 -0400 Received: from buzzloop.caiaq.de ([212.112.241.133]:35953 "EHLO buzzloop.caiaq.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751881AbZJVKLI (ORCPT ); Thu, 22 Oct 2009 06:11:08 -0400 Date: Thu, 22 Oct 2009 12:11:03 +0200 From: Daniel Mack To: Constantin Baranov Cc: Dmitry Torokhov , Richard Purdie , linux-kernel@vger.kernel.org, linux-input@vger.kernel.org Subject: Re: [PATCH] leds-alix2: add support for button connected to J15 Message-ID: <20091022101103.GR14091@buzzloop.caiaq.de> References: <20091018072738.GF3935@core.coreip.homeip.net> <20091019073728.GC28832@buzzloop.caiaq.de> <20091020013852.GC24370@core.coreip.homeip.net> <20091020101357.GR28832@buzzloop.caiaq.de> <20091021043553.GA378@core.coreip.homeip.net> <20091021193347.GL14091@buzzloop.caiaq.de> <20091022012121.5b35ff22.const@mimas.ru> <20091021204158.GN14091@buzzloop.caiaq.de> <20091022023928.2a8afea8.const@mimas.ru> <20091022030909.742b3ba1.const@mimas.ru> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20091022030909.742b3ba1.const@mimas.ru> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2077 Lines: 36 On Thu, Oct 22, 2009 at 03:09:09AM +0500, Constantin Baranov wrote: > On Thu, 22 Oct 2009 02:39:28 +0500 Constantin Baranov wrote: > > On Wed, 21 Oct 2009 22:41:58 +0200 Daniel Mack wrote: > > > On Thu, Oct 22, 2009 at 01:21:21AM +0500, Constantin Baranov wrote: > > > > I wonder why do you integrate the input driver into the leds driver. They > > > > should be separated. Or at least the complex driver should be moved to the > > > > "X86 Platform Specific Device Drivers" and renamed to some like "PC Engines > > > > ALIX Extras". I personally would prefer the separation way. > > > > > > I would as well, yes. And I was of course considerating this when I > > > hacked these lines. However, the major part of the LED driver and what > > > it currently shares with the button implementation is the BIOS detection > > > code which is so ugly that I didn't want to duplicate it ;) That would, > > > however, be the only option if you wanted to split the drivers up. Or > > > do you have any better idea? > > > > We may introduce new ALIX2 extended platform (module in arch/x86/kernel). > > In the init function it shall perform detection. The module shall export > > the is_alix2() function which shall be called from init functions of drivers. > > Also the cs5535_gpio driver could be reused for gpio access (the previous > > patch for leds-alix2 is copy&paste from cs5535_gpio). > > Moreover, the cs5535_gpio driver itself could be reworked and ported to > the gpiolib. This goes far from initial task (button support) indeed :) Indeed, agreed to all points, that would be the sane way to do it. However, I won't have time to care for that soon. So the best thing to do for now is drop the patch and leave it in the mail archives. And wait untils eventually cares to do ir right :) Daniel -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/