From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dmitry Torokhov Subject: Re: [PATCH v4 3/3] Input: gpio_keys.c: Enable use with non-local GPIO chips. Date: Wed, 22 Jun 2011 00:00:52 -0700 Message-ID: <20110622070052.GA16561@core.coreip.homeip.net> References: <20110618145154.GA18190@core.coreip.homeip.net> <20110618151645.GG8195@ponder.secretlab.ca> <20110620094815.341d1cff@archvile> <20110620084511.GB23113@core.coreip.homeip.net> <20110621114631.GC25032@sirena.org.uk> <20110621172744.GB26592@opensource.wolfsonmicro.com> <20110621204804.GC3731@core.coreip.homeip.net> <20110621230242.GA8791@opensource.wolfsonmicro.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-pz0-f46.google.com ([209.85.210.46]:33414 "EHLO mail-pz0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750806Ab1FVHBC (ORCPT ); Wed, 22 Jun 2011 03:01:02 -0400 Received: by pzk9 with SMTP id 9so331870pzk.19 for ; Wed, 22 Jun 2011 00:01:01 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20110621230242.GA8791@opensource.wolfsonmicro.com> Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Mark Brown Cc: Grant Likely , linux-input@vger.kernel.org, David Jander , David Jander On Wed, Jun 22, 2011 at 12:02:42AM +0100, Mark Brown wrote: > On Tue, Jun 21, 2011 at 01:48:05PM -0700, Dmitry Torokhov wrote: > > On Tue, Jun 21, 2011 at 06:27:45PM +0100, Mark Brown wrote: > > > > Right, but this is something that it's not reasonable to implement in > > > board code - if nothing else implementing it in board code would mean > > > we'd got lots of repitition of common patterns. > > > I agree here. I just disagree that we should be implementing this in > > driver core by having special -EAGAIN handling. Having a common > > library-like code (probably tied to device-tree) that handles device > > dependencies would be great. > > Ah, that's more OK then. I'm not entirely sure about the -EAGAIN > proposal but it does seem to have some advantages in terms of > deployment. > > > Ah, OK, so we basically in agreement here with the exception that I do > > not want the band-aid to hit mainline since it takes the heat off people > > who need inter-device dependency to actually work. > > > Can the initcall stuff be kept out of mainline? I'd expect > > The init order stuff is in mainline already, you're far too late to the > party here. For some drivers it might be already in mainline, it does not matter that we should continue adding more. > > > there exist board-specific trees where such patches could be kept? Or > > maybe interested parties could create board-crap tree to store patches > > like this one? > > Keeping things in board trees is exactly the sort of thing we want to > avoid people doing. That just means people do all sorts of stuff that > wouldn't be acceptable upstream, either out of ignorance or through > knowing that only their systems have to work with what they're doing, > and just don't bother working upstream at all half the time making life > miserable for pretty much everyone. So you are saying that we should accept such crap directly into mainline? Again, it looks like we agree that shuffling initcalls is not proper solution for this problem nor it is maintainable, so it is exactly the kind of patches that should be kept in the board trees and out of mainline. -- Dmitry