Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756461AbaDVOSF (ORCPT ); Tue, 22 Apr 2014 10:18:05 -0400 Received: from moutng.kundenserver.de ([212.227.126.130]:60231 "EHLO moutng.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753950AbaDVORz (ORCPT ); Tue, 22 Apr 2014 10:17:55 -0400 From: Arnd Bergmann To: Javier Martinez Canillas Subject: Re: [RFC PATCH 0/5] add gpio_chip_ops to hold GPIO operations Date: Tue, 22 Apr 2014 16:17:42 +0200 User-Agent: KMail/1.12.2 (Linux/3.8.0-22-generic; KDE/4.3.2; x86_64; ; ) Cc: Linus Walleij , Javier Martinez Canillas , Alexandre Courbot , Mika Westerberg , Andy Shevchenko , Santosh Shilimkar , Kevin Hilman , "linux-gpio@vger.kernel.org" , "Linux-OMAP" , "linux-kernel@vger.kernel.org" References: <1396981215-24888-1-git-send-email-javier.martinez@collabora.co.uk> In-Reply-To: MIME-Version: 1.0 Content-Type: Text/Plain; charset="utf-8" Content-Transfer-Encoding: 7bit Message-Id: <201404221617.43272.arnd@arndb.de> X-Provags-ID: V02:K0:Y8/mRxZsSx08kuBQRlwn8mLWdSOucIPgnfgMxgHybrv 3K/9TAQDy4cctcfEKXseIUwlxsvKvDaPMjc7huGBNuXoQ0DiFh FM9EOKshjxu42Jw2YZKIC3RZCiMAEEN0yhrs7U/HUAldvUhaiq EiGOU3T/mtfIj2IHaB1IHmIDYWwEIa8KE7CZ02+ioUIkjB07sK i17p660c28QrYHgiHxcDgNpqwzxdCZcZOnSlc5LzaevRyvfph3 7lYrGuRwqb0zkN4V0Shl3xGrp8VecsLaNxHIo85/qcRfreiVRA h5HCRxIm5F7BXvM9aBQGfTBEcjEp6b7XkDIc8E5UBGNeTYz8rD ltW7nOwFG1Eo0OKeWwx0= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tuesday 22 April 2014, Javier Martinez Canillas wrote: > Hello Linus, > > On Tue, Apr 22, 2014 at 1:36 PM, Linus Walleij wrote: > > On Tue, Apr 8, 2014 at 8:20 PM, Javier Martinez Canillas > > wrote: > > > >> So this is an RFC patch-set to add a virtual table to be used by > >> GPIO chip controllers and consist of the following patches: > > > > Overall I like this. Agreed, it's a very good cleanup. > > However I don't want to see any transitional phase. I prefer a BIG > > fat patch converting everyone and its dog to the new vtable and > > removing the old function pointers. This can be based on the HEAD > > of my GPIO devel branch. > > > > Ok, I was adding a commit per GPIO driver but the patch-set would have > been very big (~200 patches). > > > It may be a good idea to use coccinelle for this refactoring in order > > not to miss any users. > > > > Agreed, I was manually searching for users by using grep but I agree > that is much safer to use coccinelle for this. I don't have previous > experience writing coccinelle semantics patches though so it may take > more time than I thought but it is the perfect excuse to finally learn > how to do it :-) I'm not a big fan of doing this all at once, but it's not my call here. Just one recommendation: if you can't do an obvious coccinelle patch to do everything at once, use extra patches in the beginning to clean up the code enough to make it work, then have the large patch fully automated. Arnd -- 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/