Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756328Ab3GQWWu (ORCPT ); Wed, 17 Jul 2013 18:22:50 -0400 Received: from mail-qc0-f178.google.com ([209.85.216.178]:41383 "EHLO mail-qc0-f178.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754703Ab3GQWWt (ORCPT ); Wed, 17 Jul 2013 18:22:49 -0400 Date: Wed, 17 Jul 2013 18:22:46 -0400 (EDT) From: Nicolas Pitre To: Samuel Ortiz cc: Russell King - ARM Linux , Pawel Moll , Jon Medhurst , Lorenzo Pieralisi , Sudeep KarkadaNagesha , "devicetree-discuss@lists.ozlabs.org" , "linux-kernel@vger.kernel.org" , Amit Kucheria , Olof Johansson , Achin Gupta , "linux-arm-kernel@lists.infradead.org" Subject: Re: [RFC PATCH v5 0/1] drivers: mfd: Versatile Express SPC support In-Reply-To: <20130717212343.GC19864@zurbaran> Message-ID: References: <1373990743-23106-1-git-send-email-lorenzo.pieralisi@arm.com> <1374052705.3146.86.camel@hornet> <1374067786.3146.123.camel@hornet> <1374070811.3146.124.camel@hornet> <20130717170038.GP24642@n2100.arm.linux.org.uk> <20130717212343.GC19864@zurbaran> User-Agent: Alpine 2.03 (LFD 1266 2009-07-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2183 Lines: 50 On Wed, 17 Jul 2013, Samuel Ortiz wrote: > On Wed, Jul 17, 2013 at 02:29:02PM -0400, Nicolas Pitre wrote: > > At this point I don't really care about the name. I just want the damn > > thing merged upstream. But after several iterations to either fit one > > or another maintainers taste, each rework ends up in that maintainer > > saying: "Now that you've reworked the code, I still don't like it since > > this no longer fits in my subsystem tree." > FWIW, we asked Pawel to rework the sysreg and config parts of the > vexpress driver, make it an actual MFD driver, and spread the remaining > bits of the code into their respective subsystems. I don't think > this is an eccentric requirement. I agree. However the code that Lorenzo just posted can't be deprived of any more sysreg/config parts. They are simply nonexistent. Even the larger code you reviewed before is completely useless without _additional_ drivers to go on top which are indeed waiting after this code to be merged before they are submitted to their respective subsystems. And those additional drivers are way more interesting than this dumb register access arbitrator. Because this is fundamentally the only thing it does. > > In fact what we'd need at this point is > > drivers/code_that_no_subsystem_maintainers_wants/. > Which is what some people think drivers/mfd/ is... Does mfd still stand for "Multi Function Device"? > I don't mind merging Lorenzo's SPC driver as it is if he can explain to > me how it will eventually evolve into an actual MFD driver. If that's > not the case, I don't see how I could justify merging it through the > MFD tree. Maybe the misunderstanding is about what actually is a MFD driver. Given your persisting reluctance, I may only conclude that this is indeed not a MFD driver after all. So I'll follow existing precedents in the kernel and move Lorenzo's code to drivers/platform/vexpress/. Nicolas -- 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/