From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from imap1.codethink.co.uk ([176.9.8.82]:34262 "EHLO imap1.codethink.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753917AbeDAUtn (ORCPT ); Sun, 1 Apr 2018 16:49:43 -0400 Message-ID: <1522615780.2654.76.camel@codethink.co.uk> Subject: Re: [PATCH 4.4 089/134] ARM: dts: omap3-n900: Fix the audio CODECs reset pin From: Ben Hutchings To: Greg Kroah-Hartman , linux-kernel@vger.kernel.org Cc: stable@vger.kernel.org, "Andrew F. Davis" , Tony Lindgren , Sasha Levin Date: Sun, 01 Apr 2018 21:49:40 +0100 In-Reply-To: <20180319171902.160796295@linuxfoundation.org> References: <20180319171849.024066323@linuxfoundation.org> <20180319171902.160796295@linuxfoundation.org> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: stable-owner@vger.kernel.org List-ID: On Mon, 2018-03-19 at 19:06 +0100, Greg Kroah-Hartman wrote: > 4.4-stable review patch.  If anyone has any objections, please let me know. > > ------------------ > > From: "Andrew F. Davis" > > > [ Upstream commit 7be4b5dc7ffa9499ac6ef33a5ffa9ff43f9b7057 ] > > The correct DT property for specifying a GPIO used for reset > is "reset-gpios", fix this here. This also depends on a driver change. Please revert it for all stable branches. Ben. > Fixes: 14e3e295b2b9 ("ARM: dts: omap3-n900: Add TLV320AIC3X support") > > Signed-off-by: Andrew F. Davis > Signed-off-by: Tony Lindgren > Signed-off-by: Sasha Levin > Signed-off-by: Greg Kroah-Hartman > --- >  arch/arm/boot/dts/omap3-n900.dts |    4 ++-- >  1 file changed, 2 insertions(+), 2 deletions(-) > > --- a/arch/arm/boot/dts/omap3-n900.dts > +++ b/arch/arm/boot/dts/omap3-n900.dts > @@ -488,7 +488,7 @@ >   tlv320aic3x: tlv320aic3x@18 { >   compatible = "ti,tlv320aic3x"; >   reg = <0x18>; > - gpio-reset = <&gpio2 28 GPIO_ACTIVE_HIGH>; /* 60 */ > + reset-gpios = <&gpio2 28 GPIO_ACTIVE_LOW>; /* 60 */ >   ai3x-gpio-func = < >   0 /* AIC3X_GPIO1_FUNC_DISABLED */ >   5 /* AIC3X_GPIO2_FUNC_DIGITAL_MIC_INPUT */ > @@ -505,7 +505,7 @@ >   tlv320aic3x_aux: tlv320aic3x@19 { >   compatible = "ti,tlv320aic3x"; >   reg = <0x19>; > - gpio-reset = <&gpio2 28 GPIO_ACTIVE_HIGH>; /* 60 */ > + reset-gpios = <&gpio2 28 GPIO_ACTIVE_LOW>; /* 60 */ >   >   AVDD-supply = <&vmmc2>; >   DRVDD-supply = <&vmmc2>; -- Ben Hutchings Software Developer, Codethink Ltd.