Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1163135AbbKTPC1 (ORCPT ); Fri, 20 Nov 2015 10:02:27 -0500 Received: from mx0b-001ae601.pphosted.com ([67.231.152.168]:40064 "EHLO mx0b-001ae601.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1760186AbbKTO4J (ORCPT ); Fri, 20 Nov 2015 09:56:09 -0500 Subject: Re: [PATCH] extcon: arizona: Update naming for second jack detection DT binding From: Richard Fitzgerald To: Rob Herring CC: Charles Keepax , , , , , In-Reply-To: <20151120143925.GA11632@rob-hp-laptop> References: <1448009639-5331-1-git-send-email-ckeepax@opensource.wolfsonmicro.com> <20151120143925.GA11632@rob-hp-laptop> Content-Type: text/plain; charset="UTF-8" Date: Fri, 20 Nov 2015 14:55:58 +0000 Message-ID: <1448031358.3536.39.camel@rf-debian.wolfsonmicro.main> MIME-Version: 1.0 X-Mailer: Evolution 2.30.3 Content-Transfer-Encoding: 7bit X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0 suspectscore=2 malwarescore=0 phishscore=0 adultscore=0 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1507310000 definitions=main-1511200265 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2827 Lines: 65 On Fri, 2015-11-20 at 08:39 -0600, Rob Herring wrote: > 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. > This is a new DT option, it's never gone into a release kernel so there can't be any dtbs using it. > > > > 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/