Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1162955AbbKTOjc (ORCPT ); Fri, 20 Nov 2015 09:39:32 -0500 Received: from mail.kernel.org ([198.145.29.136]:51433 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1161912AbbKTOja (ORCPT ); Fri, 20 Nov 2015 09:39:30 -0500 Date: Fri, 20 Nov 2015 08:39:25 -0600 From: Rob Herring To: Charles Keepax Cc: cw00.choi@samsung.com, myungjoo.ham@samsung.com, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, patches@opensource.wolfsonmicro.com Subject: Re: [PATCH] extcon: arizona: Update naming for second jack detection DT binding Message-ID: <20151120143925.GA11632@rob-hp-laptop> References: <1448009639-5331-1-git-send-email-ckeepax@opensource.wolfsonmicro.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1448009639-5331-1-git-send-email-ckeepax@opensource.wolfsonmicro.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2563 Lines: 58 On Fri, Nov 20, 2015 at 08:53:59AM +0000, Charles Keepax wrote: > Update the name for the second jack detection pin binding to be a little > less confusing. This breaks compatibility with old dtbs. Either the driver needs to support both names or you just have to live with the old name. > > Signed-off-by: Charles Keepax > --- > .../devicetree/bindings/extcon/extcon-arizona.txt | 4 ++-- > drivers/extcon/extcon-arizona.c | 4 ++-- > 2 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/Documentation/devicetree/bindings/extcon/extcon-arizona.txt b/Documentation/devicetree/bindings/extcon/extcon-arizona.txt > index 03fa7b7..da9813b 100644 > --- a/Documentation/devicetree/bindings/extcon/extcon-arizona.txt > +++ b/Documentation/devicetree/bindings/extcon/extcon-arizona.txt > @@ -14,9 +14,9 @@ Optional properties: > If this node is not mentioned or if the value is unknown, then > headphone detection mode is set to HPDETL. > > - - wlf,use-jd-gpio : Use GPIO input along with JD1 for dual pin jack > + - wlf,use-jd2 : Use the additional JD input along with JD1 for dual pin jack > detection. > - - wlf,use-jd-gpio-nopull : Internal pull on GPIO is disabled when used for > + - wlf,use-jd2-nopull : Internal pull on JD2 is disabled when used for > jack detection. > - wlf,jd-invert : Invert the polarity of the jack detection switch > > diff --git a/drivers/extcon/extcon-arizona.c b/drivers/extcon/extcon-arizona.c > index 7c9598d..c377030 100644 > --- a/drivers/extcon/extcon-arizona.c > +++ b/drivers/extcon/extcon-arizona.c > @@ -1245,9 +1245,9 @@ static int arizona_extcon_device_get_pdata(struct arizona *arizona) > device_property_read_u32(arizona->dev, "wlf,gpsw", &pdata->gpsw); > > pdata->jd_gpio5 = device_property_read_bool(arizona->dev, > - "wlf,use-jd-gpio"); > + "wlf,use-jd2"); > pdata->jd_gpio5_nopull = device_property_read_bool(arizona->dev, > - "wlf,use-jd-gpio-nopull"); > + "wlf,use-jd2-nopull"); > > return 0; > } > -- > 1.7.2.5 > > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- 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/