From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yendapally Reddy Dhananjaya Reddy Subject: Re: [PATCH 3/3] ARM: dts: NSP: Add PWM Support to DT Date: Wed, 2 Mar 2016 20:22:01 +0530 Message-ID: Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: Sender: linux-pwm-owner@vger.kernel.org To: Arnd Bergmann Cc: linux-arm-kernel@lists.infradead.org, Thierry Reding , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King , Ray Jui , Scott Branden , Jon Mason , linux-pwm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, BCM Kernel Feedback List-Id: devicetree@vger.kernel.org On Wed, Mar 2, 2016 at 1:55 AM, Arnd Bergmann wrote: > On Tuesday 01 March 2016 12:42:49 Yendapally Reddy Dhananjaya Reddy wrote: >> }; >> >> + pwm: pwm@31000 { >> + compatible = "brcm,iproc-pwm"; >> + reg = <0x31000 0x28>; >> + clocks = <&osc>; >> + #pwm-cells = >; >> + status = "disabled"; >> + }; >> > > Should this also list "brcm,kona-pwm" as compatible? From patch > 1, it sounds like that was the original one. Arnd, The NSP is a iProc based SoC and the pwm controller register offsets are slightly differ with the kona pwm controller. The compatible will be either "brcm,kona-pwm" or "brcm,iproc-pwm". > > Arnd Regards Dhananjay