Hi, Rob Best Regards! Anson Huang
> -----Original Message----- > From: Anson Huang > Sent: 2019年3月26日 8:56 > To: 'Rob Herring' <[email protected]> > Cc: [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; Leonard Crestez <[email protected]>; > Robin Gong <[email protected]>; [email protected]; linux- > [email protected]; [email protected]; linux-arm- > [email protected]; [email protected]; u.kleine- > [email protected]; dl-linux-imx <[email protected]> > Subject: RE: [PATCH V9 1/5] dt-bindings: pwm: Add i.MX TPM PWM binding > > Hi, Rob > > Best Regards! > Anson Huang > > > -----Original Message----- > > From: Rob Herring [mailto:[email protected]] > > Sent: 2019年3月26日 4:42 > > To: Anson Huang <[email protected]> > > Cc: [email protected]; [email protected]; > > [email protected]; [email protected]; [email protected]; > > [email protected]; [email protected]; [email protected]; > > [email protected]; Leonard Crestez <[email protected]>; > > Robin Gong <[email protected]>; [email protected]; linux- > > [email protected]; [email protected]; linux-arm- > > [email protected]; [email protected]; u.kleine- > > [email protected]; dl-linux-imx <[email protected]> > > Subject: Re: [PATCH V9 1/5] dt-bindings: pwm: Add i.MX TPM PWM binding > > > > On Fri, Mar 22, 2019 at 01:48:05AM +0000, Anson Huang wrote: > > > Add i.MX TPM(Low Power Timer/Pulse Width Modulation Module) PWM > > binding. > > > > > > Signed-off-by: Anson Huang <[email protected]> > > > --- > > > No changes. > > > > v9? I don't recall seeing any previous versions. > > I checked my mailbox, you are in the "to" list in every version. The first > version's subject is: > [PATCH 1/5] dt-bindings: pwm: Add i.MX TPM PWM binding... > > > > > > --- > > > .../devicetree/bindings/pwm/imx-tpm-pwm.txt | 22 > > ++++++++++++++++++++++ > > > 1 file changed, 22 insertions(+) > > > create mode 100644 > > > Documentation/devicetree/bindings/pwm/imx-tpm-pwm.txt > > > > > > diff --git a/Documentation/devicetree/bindings/pwm/imx-tpm-pwm.txt > > > b/Documentation/devicetree/bindings/pwm/imx-tpm-pwm.txt > > > new file mode 100644 > > > index 0000000..94f1ad5 > > > --- /dev/null > > > +++ b/Documentation/devicetree/bindings/pwm/imx-tpm-pwm.txt > > > @@ -0,0 +1,22 @@ > > > +Freescale i.MX TPM PWM controller > > > + > > > +Required properties: > > > +- compatible : Should be "fsl,imx-tpm". > > > > Needs to be SoC specific. > > OK, will change it to "fsl,imx7ulp-tpm" in next verson. As " fsl,imx7ulp-tpm " is already used for timer, TPM can work as a timer or PWM, so I have to use " fsl,imx7ulp-pwm" in the V10 patch I just sent. Anson. > > > > > > +- reg: Physical base address and length of the controller's registers. > > > +- #pwm-cells: Should be 3. See pwm.txt in this directory for a > > > +description > > of the cells format. > > > +- clocks : The clock provided by the SoC to drive the PWM. > > > +- interrupts: The interrupt for the PWM controller. > > > + > > > +Note: The TPM counter and period counter are shared between > > > +multiple channels, so all channels should use same period setting. > > > + > > > +Example: > > > + > > > +pwm0: tpm@40250000 { > > > > pwm@... > > I forgot to update the example in DT binding, sorry for that, will update it > according to below dts node: > > pwm0: pwm@40250000 { > > Anson.
