On Mon, Nov 18, 2013 at 06:54:57PM +0000, Tim Kryger wrote: > Add the binding description for the kona-pwm block found on Broadcom's > mobile SoCs. > > Signed-off-by: Tim Kryger <tim.kryger@xxxxxxxxxx> > Reviewed-by: Alex Elder <elder@xxxxxxxxxx> > Reviewed-by: Markus Mayer <markus.mayer@xxxxxxxxxx> > --- > .../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..5c3ea1b > --- /dev/null > +++ b/Documentation/devicetree/bindings/pwm/bcm-kona-pwm.txt > @@ -0,0 +1,24 @@ > +Broadcom's PWM Controller Device Tree bindings > + > +Broadcom's Kona PWM Controller has 6 channels > + > +Required Properties : > +- compatible: should be "brcm,kona-pwm" > +- reg: physical base address and length of the controller's registers > +- clocks: clock specifier for the kona pwm external clock Minor nit: phandle + clock-specifier pair > +- #pwm-cells: should be 2. The first cell specifies the per-chip index > + of the PWM to use and the second cell is the period in nanoseconds. > + > +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 = <2>; > +}; Otherwise this looks fine. Mark. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html