From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shawn Guo Subject: Re: [PATCH 2/2] ARM: dts: hummingboard: Setup pwm lines Date: Mon, 30 Mar 2015 11:54:03 +0800 Message-ID: <20150330035402.GM28410@dragon> References: <1425379336.3146.38.camel@pengutronix.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1425379336.3146.38.camel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Philipp Zabel Cc: Russell King , Sascha Hauer , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Mark Rutland , Pawel Moll , Ian Campbell , Rabeeh Khoury , Rob Herring , Kumar Gala List-Id: devicetree@vger.kernel.org On Tue, Mar 03, 2015 at 11:42:16AM +0100, Philipp Zabel wrote: > > +&pwm3 { > > + status = "disabled"; > > +}; > > + > > +&pwm4 { > > + status = "disabled"; > > +}; > > I wanted to point out these two are superfluous since pwm3/4 are > disabled in the dtsi by default, but I just learned they aren't. > > Should we add status = "disabled" to unusable (because not externally > connected) PWMs, and if so, shouldn't that be done in the dtsi already? Thanks Philipp for your patch "ARM: dts: imx6qdl: disable PWMs by default". Therefore I dropped pwm3/4 and applied the patch. Shawn -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html