From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH V3] pwm: lpc32xx - Add a driver for the motor PWM Date: Wed, 10 Sep 2014 11:21:29 +0200 Message-ID: <5900146.ZQPMvSzLeR@wuerfel> References: <1410277361-26848-1-git-send-email-alban.bedel@avionic-design.de> <20140909160548.GB3896@leverpostej> <20140910104220.44558cf7@avionic-0020> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Return-path: In-Reply-To: <20140910104220.44558cf7@avionic-0020> Sender: linux-pwm-owner@vger.kernel.org To: Alban Bedel Cc: Mark Rutland , Thierry Reding , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-pwm@vger.kernel.org" , "grant.likely@linaro.org" , Kumar Gala , Ian Campbell , Pawel Moll , Rob Herring , Roland Stigge List-Id: devicetree@vger.kernel.org On Wednesday 10 September 2014 10:42:20 Alban Bedel wrote: > > > Please describe clocks in the binding. If the clock inputs are named, > > please use clock-names. > > No clock is defined in the current LPC32xx DTS, what should I do in > this case? This is a bit tricky. I would recommend describing the clock inputs in the binding anyway, as "optional" properties, in case we ever get a full DT-aware clocksource driver for lpc32xx. I've just mentioned the topic to Roland yesterday, I would really like to see such a clocksource driver done for other reasons (multiplatform support), but introducing one with full DT support is hard to do without breaking existing dtb files. Arnd