Message ID | 20160816093508.28307-18-lee.jones@linaro.org |
---|---|
State | Accepted |
Headers | show |
On Tue, Aug 16, 2016 at 10:35:08AM +0100, Lee Jones wrote: > We're renaming the 'st,pwm-num-chan' binding to 'st,pwm-num-devs' to > be more inline with the naming conventions of the subsystem. Where > we used to treat each line as a channel, the PWM convention is to > describe them as devices. This is out of date now. Otherwise, the binding looks fine. > The second documentation adaption entails adding support for PWM > capture devices. A new clock is required as well as an IRQ line. > We're also adding a new property similar to the one described > above, but for capture channels. Typically, there will be less > capture channels than PWM-out, since all channels have the latter > capability, but only some have capture support. > > Signed-off-by: Lee Jones <lee.jones@linaro.org> > --- > Documentation/devicetree/bindings/pwm/pwm-st.txt | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-pwm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Thu, 18 Aug 2016, Rob Herring wrote: > On Tue, Aug 16, 2016 at 10:35:08AM +0100, Lee Jones wrote: > > We're renaming the 'st,pwm-num-chan' binding to 'st,pwm-num-devs' to > > be more inline with the naming conventions of the subsystem. Where > > we used to treat each line as a channel, the PWM convention is to > > describe them as devices. > > This is out of date now. Otherwise, the binding looks fine. Yes, you are right. Perhaps if this is okay with Thierry, he can remove these few lines. If not, I don't mind re-sending on request. I'll leave it until I hear otherwise. > > The second documentation adaption entails adding support for PWM > > capture devices. A new clock is required as well as an IRQ line. > > We're also adding a new property similar to the one described > > above, but for capture channels. Typically, there will be less > > capture channels than PWM-out, since all channels have the latter > > capability, but only some have capture support. > > > > Signed-off-by: Lee Jones <lee.jones@linaro.org> > > --- > > Documentation/devicetree/bindings/pwm/pwm-st.txt | 8 +++++--- > > 1 file changed, 5 insertions(+), 3 deletions(-)
On Fri, Aug 19, 2016 at 3:35 AM, Lee Jones <lee.jones@linaro.org> wrote: > On Thu, 18 Aug 2016, Rob Herring wrote: > >> On Tue, Aug 16, 2016 at 10:35:08AM +0100, Lee Jones wrote: >> > We're renaming the 'st,pwm-num-chan' binding to 'st,pwm-num-devs' to >> > be more inline with the naming conventions of the subsystem. Where >> > we used to treat each line as a channel, the PWM convention is to >> > describe them as devices. >> >> This is out of date now. Otherwise, the binding looks fine. > > Yes, you are right. > > Perhaps if this is okay with Thierry, he can remove these few lines. > If not, I don't mind re-sending on request. > > I'll leave it until I hear otherwise. Fine with me. Acked-by: Rob Herring <robh@kernel.org> -- To unsubscribe from this list: send the line "unsubscribe linux-pwm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Tue, Aug 16, 2016 at 10:35:08AM +0100, Lee Jones wrote: > We're renaming the 'st,pwm-num-chan' binding to 'st,pwm-num-devs' to > be more inline with the naming conventions of the subsystem. Where > we used to treat each line as a channel, the PWM convention is to > describe them as devices. > > The second documentation adaption entails adding support for PWM > capture devices. A new clock is required as well as an IRQ line. > We're also adding a new property similar to the one described > above, but for capture channels. Typically, there will be less > capture channels than PWM-out, since all channels have the latter > capability, but only some have capture support. > > Signed-off-by: Lee Jones <lee.jones@linaro.org> > --- > Documentation/devicetree/bindings/pwm/pwm-st.txt | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) Applied with a reworded commit message as discussed elsewhere in this thread. I've also applied this in front of all other patches because I think of the DT bindings as specifications that drivers implement. Thierry
diff --git a/Documentation/devicetree/bindings/pwm/pwm-st.txt b/Documentation/devicetree/bindings/pwm/pwm-st.txt index 84d2fb8..19fce77 100644 --- a/Documentation/devicetree/bindings/pwm/pwm-st.txt +++ b/Documentation/devicetree/bindings/pwm/pwm-st.txt @@ -13,13 +13,14 @@ Required parameters: - pinctrl-0: List of phandles pointing to pin configuration nodes for PWM module. For Pinctrl properties, please refer to [1]. -- clock-names: Set to "pwm". +- clock-names: Valid entries are "pwm" and/or "capture". - clocks: phandle of the clock used by the PWM module. For Clk properties, please refer to [2]. +- interrupts: IRQ for the Capture device Optional properties: -- st,pwm-num-chan: Number of available channels. If not passed, the driver - will consider single channel by default. +- st,pwm-num-chan: Number of available PWM channels. Default is 0. +- st,capture-num-chan: Number of available Capture channels. Default is 0. [1] Documentation/devicetree/bindings/pinctrl/pinctrl-bindings.txt [2] Documentation/devicetree/bindings/clock/clock-bindings.txt @@ -38,4 +39,5 @@ pwm1: pwm@fe510000 { clocks = <&clk_sysin>; clock-names = "pwm"; st,pwm-num-chan = <4>; + st,capture-num-chan = <2>; };
We're renaming the 'st,pwm-num-chan' binding to 'st,pwm-num-devs' to be more inline with the naming conventions of the subsystem. Where we used to treat each line as a channel, the PWM convention is to describe them as devices. The second documentation adaption entails adding support for PWM capture devices. A new clock is required as well as an IRQ line. We're also adding a new property similar to the one described above, but for capture channels. Typically, there will be less capture channels than PWM-out, since all channels have the latter capability, but only some have capture support. Signed-off-by: Lee Jones <lee.jones@linaro.org> --- Documentation/devicetree/bindings/pwm/pwm-st.txt | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-)