From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shawn Guo Subject: Re: [PATCH v7 1/7] ARM: dts: imx: imx7d-sbd remove fixed can2-3v3 regulator Date: Wed, 30 Sep 2015 09:56:22 +0800 Message-ID: <20150930015622.GC15771@tiger> References: <1443477401-17238-1-git-send-email-aalonso@freescale.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1443477401-17238-1-git-send-email-aalonso@freescale.com> Sender: linux-gpio-owner@vger.kernel.org To: Adrian Alonso Cc: linux-arm-kernel@lists.infradead.org, shawn.guo@linaro.org, linus.walleij@linaro.org, lznuaa@gmail.com, linux-gpio@vger.kernel.org, devicetree@vger.kernel.org, kernel@pengutronix.de, robh+dt@kernel.org, Anson.Huang@freescale.com, Frank.Li@freescale.com, yibin.gong@freescale.com, nitin.garg@freescale.com List-Id: devicetree@vger.kernel.org On Mon, Sep 28, 2015 at 04:56:35PM -0500, Adrian Alonso wrote: > Remove incorrect can2-3v3 fixed regulator, imx7d-sdb doesn't > have a dedicated can2 fixed regulator instead it shares PERI_3V3 > fixed regulator (RT8070ZS) which is enabled by default (hardwired) > from pmic pfuze3000 NVCC_3V3 power rail. @Frank, can you please confirm it? > > Signed-off-by: Adrian Alonso Please stop sending patch to shawn.guo@linaro.org, and shawnguo@kernel.org is good enough. > --- > Changes for V7: New patch in series I do not understand why the patch has to be included in this series. Shawn > > arch/arm/boot/dts/imx7d-sdb.dts | 11 +---------- > 1 file changed, 1 insertion(+), 10 deletions(-) > > diff --git a/arch/arm/boot/dts/imx7d-sdb.dts b/arch/arm/boot/dts/imx7d-sdb.dts > index 8059458..83f0053 100644 > --- a/arch/arm/boot/dts/imx7d-sdb.dts > +++ b/arch/arm/boot/dts/imx7d-sdb.dts > @@ -78,16 +78,7 @@ > enable-active-high; > }; > > - reg_can2_3v3: regulator@2 { > - compatible = "regulator-fixed"; > - reg = <2>; > - regulator-name = "can2-3v3"; > - regulator-min-microvolt = <3300000>; > - regulator-max-microvolt = <3300000>; > - gpio = <&gpio1 7 GPIO_ACTIVE_LOW>; > - }; > - > - reg_vref_1v8: regulator@3 { > + reg_vref_1v8: regulator@2 { > compatible = "regulator-fixed"; > reg = <3>; > regulator-name = "vref-1v8"; > -- > 2.1.4 >