From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maxime Ripard Subject: Re: [PATCH 2/5] ARM: dts: sun5i: Add PWM channel 0 pinmux setting for A13/A10s Date: Tue, 22 Sep 2015 16:29:44 +0200 Message-ID: <20150922142944.GD4684@lukather> References: <1442752251-20744-1-git-send-email-hdegoede@redhat.com> <1442752251-20744-3-git-send-email-hdegoede@redhat.com> Reply-To: maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="FWNnc7UvJRgHBXIN" Return-path: Content-Disposition: inline In-Reply-To: <1442752251-20744-3-git-send-email-hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , To: Hans de Goede Cc: Chen-Yu Tsai , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree , linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org List-Id: devicetree@vger.kernel.org --FWNnc7UvJRgHBXIN Content-Type: text/plain; charset=UTF-8 Content-Disposition: inline On Sun, Sep 20, 2015 at 08:30:48AM -0400, Hans de Goede wrote: > Add a pinmux setting for the first pwm channel. This is often used for > backlight dimming on tablets. > > Signed-off-by: Hans de Goede > --- > arch/arm/boot/dts/sun5i.dtsi | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/arch/arm/boot/dts/sun5i.dtsi b/arch/arm/boot/dts/sun5i.dtsi > index 5e036a4..f0714d3 100644 > --- a/arch/arm/boot/dts/sun5i.dtsi > +++ b/arch/arm/boot/dts/sun5i.dtsi > @@ -529,6 +529,13 @@ > allwinner,drive = ; > allwinner,pull = ; > }; > + > + pwm0_pins: pwm0 { Is there a reason why you call this pinset pwm0, while it looks like there's a single pwm controller? Maxime -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com --FWNnc7UvJRgHBXIN--