From mboxrd@z Thu Jan 1 00:00:00 1970 From: Grant Likely Subject: Re: [PATCH v2 3/3] leds: leds-pwm: Add device tree bindings Date: Mon, 10 Dec 2012 22:37:22 +0000 Message-ID: <20121210223722.461773E08FB@localhost> References: <1352731270-27534-1-git-send-email-peter.ujfalusi@ti.com> <1352731270-27534-4-git-send-email-peter.ujfalusi@ti.com> <20121206100052.EC8293E0E22@localhost> <50C0913B.9070802@ti.com> <20121207133453.800C43E0B87@localhost> <50C1F783.6090501@ti.com> Return-path: In-Reply-To: <50C1F783.6090501@ti.com> Sender: linux-kernel-owner@vger.kernel.org To: Peter Ujfalusi Cc: Bryan Wu , Richard Purdie , linux-leds@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree-discuss@lists.ozlabs.org, Thierry Reding , Linus Walleij List-Id: devicetree@vger.kernel.org On Fri, 7 Dec 2012 15:04:51 +0100, Peter Ujfalusi wrote: > On 12/07/2012 02:34 PM, Grant Likely wrote: > > It really needs to look like this: > > > > pwmleds { > > compatible = "pwm-leds"; > > kpad { > > label = "omap4::keypad"; > > max-brightness = <127>; > > pwms = <&twl_pwm 0 7812500>; > > }; > > charging { > > label = "omap4:green:chrg"; > > max-brightness = <255>; > > pwms = <&twl_pwmled 0 7812500>; > > }; > > }; > > OK. > So What we need to do is export the of_pwm_request() from drivers/pwm/core.c > Client drivers us of_pwm_request(node, NULL); to get the pwm when booted with DT. > Works fine on my setup this way and the DTS section looks just as you have > described. > Now I need to clean them up and resubmit. Cool, thanks. g.