| Message ID | 1394655346-30048-2-git-send-email-tim.kryger@linaro.org |
|---|---|
| State | Rejected |
| Headers | show |
On Wed, Mar 12, 2014 at 01:15:42PM -0700, Tim Kryger wrote: > Add the binding description for the Kona PWM controller found on Broadcom's > mobile SoCs. > > Signed-off-by: Tim Kryger <tim.kryger@linaro.org> > Reviewed-by: Alex Elder <elder@linaro.org> > Reviewed-by: Markus Mayer <markus.mayer@linaro.org> > --- > .../devicetree/bindings/pwm/bcm-kona-pwm.txt | 24 ++++++++++++++++++++++ > 1 file changed, 24 insertions(+) > create mode 100644 Documentation/devicetree/bindings/pwm/bcm-kona-pwm.txt > > diff --git a/Documentation/devicetree/bindings/pwm/bcm-kona-pwm.txt b/Documentation/devicetree/bindings/pwm/bcm-kona-pwm.txt > new file mode 100644 > index 0000000..c8e2d13 > --- /dev/null > +++ b/Documentation/devicetree/bindings/pwm/bcm-kona-pwm.txt > @@ -0,0 +1,24 @@ > +Broadcom Kona PWM controller device tree bindings > + > +This controller has 6 channels. > + > +Required Properties : > +- compatible: should be "brcm,kona-pwm" This is somewhat inconsistent because the example below clearly doesn't set the compatible property to "brcm,kona-pwm". Perhaps better wording would be: - compatible: should contain "brcm,kona-pwm" > +- reg: physical base address and length of the controller's registers > +- clocks: phandle + clock specifier pair for the external clock > +- #pwm-cells: should be 3. The first cell specifies the per-chip index > + of the PWM to use, the second cell is the period in nanoseconds, and > + the third cell is the flags. Please use the canonical description for this: - #pwm-cells: Should be 3. See pwm.txt in this directory for a description of the cells format. > +Refer to pwm/pwm.txt for generic pwm controller node properties. With the above, this line should go away. But still: "pwm controller" -> "PWM controller". Thierry
On Tue, Mar 18, 2014 at 2:18 PM, Thierry Reding <thierry.reding@gmail.com> wrote: > On Wed, Mar 12, 2014 at 01:15:42PM -0700, Tim Kryger wrote: >> @@ -0,0 +1,24 @@ >> +Broadcom Kona PWM controller device tree bindings >> + >> +This controller has 6 channels. >> + >> +Required Properties : >> +- compatible: should be "brcm,kona-pwm" > > This is somewhat inconsistent because the example below clearly doesn't > set the compatible property to "brcm,kona-pwm". Perhaps better wording > would be: > > - compatible: should contain "brcm,kona-pwm" Agreed, what you wrote is more accurate. I will update it. >> +- reg: physical base address and length of the controller's registers >> +- clocks: phandle + clock specifier pair for the external clock >> +- #pwm-cells: should be 3. The first cell specifies the per-chip index >> + of the PWM to use, the second cell is the period in nanoseconds, and >> + the third cell is the flags. > > Please use the canonical description for this: > > - #pwm-cells: Should be 3. See pwm.txt in this directory for a > description of the cells format. > >> +Refer to pwm/pwm.txt for generic pwm controller node properties. > > With the above, this line should go away. But still: "pwm controller" -> > "PWM controller". Okay. I will use the text you provided and drop the other line. Sorry for missing the capitalization. I thought I had fixed them all. Thanks, Tim -- 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
diff --git a/Documentation/devicetree/bindings/pwm/bcm-kona-pwm.txt b/Documentation/devicetree/bindings/pwm/bcm-kona-pwm.txt new file mode 100644 index 0000000..c8e2d13 --- /dev/null +++ b/Documentation/devicetree/bindings/pwm/bcm-kona-pwm.txt @@ -0,0 +1,24 @@ +Broadcom Kona PWM controller device tree bindings + +This controller has 6 channels. + +Required Properties : +- compatible: should be "brcm,kona-pwm" +- reg: physical base address and length of the controller's registers +- clocks: phandle + clock specifier pair for the external clock +- #pwm-cells: should be 3. The first cell specifies the per-chip index + of the PWM to use, the second cell is the period in nanoseconds, and + the third cell is the flags. + +Refer to pwm/pwm.txt for generic pwm controller node properties. + +Refer to clocks/clock-bindings.txt for generic clock consumer properties. + +Example: + +pwm: pwm@3e01a000 { + compatible = "brcm,bcm11351-pwm", "brcm,kona-pwm"; + reg = <0x3e01a000 0xc4>; + clocks = <&pwm_clk>; + #pwm-cells = <3>; +};