Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753325AbaARC4L (ORCPT ); Fri, 17 Jan 2014 21:56:11 -0500 Received: from mail-ie0-f171.google.com ([209.85.223.171]:33538 "EHLO mail-ie0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753087AbaARC4I (ORCPT ); Fri, 17 Jan 2014 21:56:08 -0500 Date: Fri, 17 Jan 2014 21:56:00 -0500 From: Matt Porter To: Sherman Yin Cc: Linus Walleij , "arm@kernel.org" , Mark Brown , Rob Herring , Heiko =?iso-8859-1?Q?St=FCbner?= , Pawel Moll , Mark Rutland , Stephen Warren , Ian Campbell , Rob Landley , Christian Daudt , Russell King , Grant Likely , "devicetree@vger.kernel.org" , "linux-doc@vger.kernel.org" , "linux-kernel@vger.kernel.org" , bcm-kernel-feedback-list , "linux-arm-kernel@lists.infradead.org" Subject: Re: [PATCH v4 3/4] ARM: pinctrl: Add Broadcom Capri pinctrl driver Message-ID: <20140118025600.GS31049@beef> References: <1386787041-6035-1-git-send-email-syin@broadcom.com> <1387592016-13740-1-git-send-email-syin@broadcom.com> <1387592016-13740-4-git-send-email-syin@broadcom.com> <52D98B99.5050604@broadcom.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <52D98B99.5050604@broadcom.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jan 17, 2014 at 11:59:21AM -0800, Sherman Yin wrote: > On 14-01-16 05:19 AM, Linus Walleij wrote: > >On Sat, Dec 21, 2013 at 3:13 AM, Sherman Yin wrote: > > > >'> Adds pinctrl driver for Broadcom Capri (BCM281xx) SoCs. > >> > >>Signed-off-by: Sherman Yin > >>Reviewed-by: Christian Daudt > >>Reviewed-by: Matt Porter > >>--- > >>v4: - PINCTRL selected in Kconfig, PINCTRL_CAPRI selected in bcm_defconfig > >> - make use of regmap > >> - change CAPRI_PIN_UPDATE from macro to inline function. > >> - Handle pull-up strength arg in Ohm instead of enum > > > >Patch applied. It is really good now! It's late before the merge > >window, but you've done a tremendous work on this driver and > >I don't want to delay its deployment further. > > Great, thanks for the support and reviews! Very nice! Now after having completely missing something fundamental on my reviews, I feel compelled to bring it up at the 11^H^H12th hour. That is, this is the *only* BCM281xx driver to be named Capri, both in the filename and driver code, but also in the binding compatible. We didn't do that on anything else that's gone upstream to date. This really introduces an unfortunate inconsistency as it obscures which SoC family this binding and driver belong with. I wonder if Linus would accept a rename at this point (too late for 3.14 presumably, but for 3.15) of s/capri/bcm281xx throughout, bcm11351 for the compatible string, as we have for the machine compatible, and also BCM281XX for the Kconfig option. If not, I'll survive, but it pains me to see one thing completely different out of this entire family. If nothing else, it would be great to address the compatible string before this hits the 3.14 release. Linus? -Matt -- 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/