Message ID | 1416621212-11701-3-git-send-email-Naidu.Tellapati@gmail.com |
---|---|
State | Superseded |
Headers | show |
On Sat, Nov 22, 2014 at 07:23:30AM +0530, naidu.tellapati@gmail.com wrote: > From: Naidu Tellapati <Naidu.Tellapati@imgtec.com> > > Add binding document for IMG Pulse Width Modulator (PWM) DAC present on the > Pistachio SOC. The PWM DAC has four channels. > > Signed-off-by: Naidu Tellapati <Naidu.Tellapati@imgtec.com> > Signed-off-by: Sai Masarapu <Sai.Masarapu@imgtec.com> > Reviewed-by: Andrew Bresticker <abrestic@chromium.org> > --- > No changes from v3. > --- > Documentation/devicetree/bindings/pwm/img-pwm.txt | 23 +++++++++++++++++++++ > 1 files changed, 23 insertions(+), 0 deletions(-) > create mode 100644 Documentation/devicetree/bindings/pwm/img-pwm.txt > > diff --git a/Documentation/devicetree/bindings/pwm/img-pwm.txt b/Documentation/devicetree/bindings/pwm/img-pwm.txt > new file mode 100644 > index 0000000..d3828dd > --- /dev/null > +++ b/Documentation/devicetree/bindings/pwm/img-pwm.txt > @@ -0,0 +1,23 @@ > +*Imagination Technologies PWM DAC driver > + > +Required properties: > + - compatible: Should be "img,pistachio-pwm" > + - reg: Should contain physical base address and length of pwm registers. > + - clocks: Must contain an entry for each entry in clock-names. > + See ../clock/clock-bindings.txt for details. > + - clock-names: input clock names. > + Required elements: "pwm" and "sys". I think it'd be useful to explain at least what the "sys" clock is. "pwm" I'd assume that it clocks the IP block, but what does "sys" do? Thierry
Hi Thierry, Many thanks for your review comments. > On Sat, Nov 22, 2014 at 07:23:30AM +0530, naidu.tellapati@gmail.com wrote: >> From: Naidu Tellapati <Naidu.Tellapati@imgtec.com> >> >> Add binding document for IMG Pulse Width Modulator (PWM) DAC present on the >> Pistachio SOC. The PWM DAC has four channels. >> >> Signed-off-by: Naidu Tellapati <Naidu.Tellapati@imgtec.com> >> Signed-off-by: Sai Masarapu <Sai.Masarapu@imgtec.com> >> Reviewed-by: Andrew Bresticker <abrestic@chromium.org> >> --- >> No changes from v3. >> --- >> Documentation/devicetree/bindings/pwm/img-pwm.txt | 23 +++++++++++++++++++++ >> 1 files changed, 23 insertions(+), 0 deletions(-) >> create mode 100644 Documentation/devicetree/bindings/pwm/img-pwm.txt >> >> diff --git a/Documentation/devicetree/bindings/pwm/img-pwm.txt b/Documentation/devicetree/bindings/pwm/img-pwm.txt >> new file mode 100644 >> index 0000000..d3828dd >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/pwm/img-pwm.txt >> @@ -0,0 +1,23 @@ >> +*Imagination Technologies PWM DAC driver >> + >> +Required properties: >> + - compatible: Should be "img,pistachio-pwm" >> + - reg: Should contain physical base address and length of pwm registers. >> + - clocks: Must contain an entry for each entry in clock-names. >> + See ../clock/clock-bindings.txt for details. >> + - clock-names: input clock names. >> + Required elements: "pwm" and "sys". > I think it'd be useful to explain at least what the "sys" clock is. > "pwm" I'd assume that it clocks the IP block, but what does "sys" do? OK. Will add the description in the next patch set. The sys clock enables access to Internal configuration registers of PWM block. > Thierry Regards, Naidu. -- 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/img-pwm.txt b/Documentation/devicetree/bindings/pwm/img-pwm.txt new file mode 100644 index 0000000..d3828dd --- /dev/null +++ b/Documentation/devicetree/bindings/pwm/img-pwm.txt @@ -0,0 +1,23 @@ +*Imagination Technologies PWM DAC driver + +Required properties: + - compatible: Should be "img,pistachio-pwm" + - reg: Should contain physical base address and length of pwm registers. + - clocks: Must contain an entry for each entry in clock-names. + See ../clock/clock-bindings.txt for details. + - clock-names: input clock names. + Required elements: "pwm" and "sys". + - #pwm-cells: Should be 2. See pwm.txt in this directory for the + description of the cells format. + - img,cr-periph: Must contain a phandle to the peripheral control + syscon node which contains PWM control registers. + +Example: + pwm: pwm@18101300 { + compatible = "img,pistachio-pwm"; + reg = <0x18101300 0x100>; + clocks = <&pwm_clk>, <&system_clk>; + clock-names = "pwm", "sys"; + #pwm-cells = <2>; + img,cr-periph = <&cr_periph>; + };