2013-03-25 07:04:36

by Philip, Avinash

[permalink] [raw]
Subject: [PATCH 0/2] Update device tree binding document for pwm-tiehrpwm & pwm-tiecap

Update device tree document of pwm-tiehrpwm & pwm-tiecap in order to reflect the
usage of similar modules in both da850 and am33xx platforms.

Philip Avinash (2):
pwm: pwm-tiecap: Update device-tree binding document
pwm: pwm-tiehrpwm: Update device-tree binding document

.../devicetree/bindings/pwm/pwm-tiecap.txt | 12 ++++++++++--
.../devicetree/bindings/pwm/pwm-tiehrpwm.txt | 12 ++++++++++--
2 files changed, 20 insertions(+), 4 deletions(-)

--
1.7.9.5


2013-03-25 07:04:47

by Philip, Avinash

[permalink] [raw]
Subject: [PATCH 1/2] pwm: pwm-tiecap: Update device-tree binding document

Update binding document of pwm-tiecap to reflect the usage of similar
modules in da850 and am3xx SOCs.

Signed-off-by: Philip Avinash <[email protected]>
Cc: Grant Likely <[email protected]>
Cc: Rob Herring <[email protected]>
Cc: Rob Landley <[email protected]>
---
.../devicetree/bindings/pwm/pwm-tiecap.txt | 12 ++++++++++--
1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/pwm/pwm-tiecap.txt b/Documentation/devicetree/bindings/pwm/pwm-tiecap.txt
index 131e8c1..43c4e81 100644
--- a/Documentation/devicetree/bindings/pwm/pwm-tiecap.txt
+++ b/Documentation/devicetree/bindings/pwm/pwm-tiecap.txt
@@ -1,7 +1,9 @@
TI SOC ECAP based APWM controller

Required properties:
-- compatible: Must be "ti,am33xx-ecap"
+- compatible: Must be "ti,<soc>-ecap".
+ for am33xx - compatible = "ti,am33xx-ecap";
+ for da850 - compatible = "ti,da850-ecap", "ti,am33xx-ecap";
- #pwm-cells: Should be 3. Number of cells being used to specify PWM property.
First cell specifies the per-chip index of the PWM to use, the second
cell is the period in nanoseconds and bit 0 in the third cell is used to
@@ -15,9 +17,15 @@ Optional properties:

Example:

-ecap0: ecap@0 {
+ecap0: ecap@0 { /* ECAP on am33xx */
compatible = "ti,am33xx-ecap";
#pwm-cells = <3>;
reg = <0x48300100 0x80>;
ti,hwmods = "ecap0";
};
+
+ecap0: ecap@0 { /* ECAP on da850 */
+ compatible = "ti,da850-ecap", "ti,am33xx-ecap";
+ #pwm-cells = <3>;
+ reg = <0x306000 0x80>;
+};
--
1.7.9.5

2013-03-25 07:04:58

by Philip, Avinash

[permalink] [raw]
Subject: [PATCH 2/2] pwm: pwm-tiehrpwm: Update device-tree binding document

Update binding document of pwm-tiehrpwm to reflect the usage of similar
modules in da850 and am3xx SOCs.

Signed-off-by: Philip Avinash <[email protected]>
Cc: Grant Likely <[email protected]>
Cc: Rob Herring <[email protected]>
Cc: Rob Landley <[email protected]>
---
.../devicetree/bindings/pwm/pwm-tiehrpwm.txt | 12 ++++++++++--
1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/pwm/pwm-tiehrpwm.txt b/Documentation/devicetree/bindings/pwm/pwm-tiehrpwm.txt
index 4fc7079..9a71369 100644
--- a/Documentation/devicetree/bindings/pwm/pwm-tiehrpwm.txt
+++ b/Documentation/devicetree/bindings/pwm/pwm-tiehrpwm.txt
@@ -1,7 +1,9 @@
TI SOC EHRPWM based PWM controller

Required properties:
-- compatible : Must be "ti,am33xx-ehrpwm"
+- compatible: Must be "ti,<soc>-ehrpwm".
+ for am33xx - compatible = "ti,am33xx-ehrpwm";
+ for da850 - compatible = "ti,da850-ehrpwm", "ti,am33xx-ehrpwm";
- #pwm-cells: Should be 3. Number of cells being used to specify PWM property.
First cell specifies the per-chip index of the PWM to use, the second
cell is the period in nanoseconds and bit 0 in the third cell is used to
@@ -15,9 +17,15 @@ Optional properties:

Example:

-ehrpwm0: ehrpwm@0 {
+ehrpwm0: ehrpwm@0 { /* EHRPWM on am33xx */
compatible = "ti,am33xx-ehrpwm";
#pwm-cells = <3>;
reg = <0x48300200 0x100>;
ti,hwmods = "ehrpwm0";
};
+
+ehrpwm0: ehrpwm@0 { /* EHRPWM on da850 */
+ compatible = "ti,da850-ehrpwm", "ti,am33xx-ehrpwm";
+ #pwm-cells = <3>;
+ reg = <0x300000 0x2000>;
+};
--
1.7.9.5

2013-03-25 07:09:20

by Peter Korsgaard

[permalink] [raw]
Subject: Re: [PATCH 1/2] pwm: pwm-tiecap: Update device-tree binding document

>>>>> "Philip" == Philip Avinash <[email protected]> writes:

Philip> Update binding document of pwm-tiecap to reflect the usage of similar
Philip> modules in da850 and am3xx SOCs.

Acked-by: Peter Korsgaard <[email protected]>

Philip> Signed-off-by: Philip Avinash <[email protected]>
Philip> Cc: Grant Likely <[email protected]>
Philip> Cc: Rob Herring <[email protected]>
Philip> Cc: Rob Landley <[email protected]>
Philip> ---
Philip> .../devicetree/bindings/pwm/pwm-tiecap.txt | 12 ++++++++++--
Philip> 1 file changed, 10 insertions(+), 2 deletions(-)

Philip> diff --git a/Documentation/devicetree/bindings/pwm/pwm-tiecap.txt b/Documentation/devicetree/bindings/pwm/pwm-tiecap.txt
Philip> index 131e8c1..43c4e81 100644
Philip> --- a/Documentation/devicetree/bindings/pwm/pwm-tiecap.txt
Philip> +++ b/Documentation/devicetree/bindings/pwm/pwm-tiecap.txt
Philip> @@ -1,7 +1,9 @@
Philip> TI SOC ECAP based APWM controller

Philip> Required properties:
Philip> -- compatible: Must be "ti,am33xx-ecap"
Philip> +- compatible: Must be "ti,<soc>-ecap".
Philip> + for am33xx - compatible = "ti,am33xx-ecap";
Philip> + for da850 - compatible = "ti,da850-ecap", "ti,am33xx-ecap";
Philip> - #pwm-cells: Should be 3. Number of cells being used to specify PWM property.
Philip> First cell specifies the per-chip index of the PWM to use, the second
Philip> cell is the period in nanoseconds and bit 0 in the third cell is used to
Philip> @@ -15,9 +17,15 @@ Optional properties:

Philip> Example:

Philip> -ecap0: ecap@0 {
Philip> +ecap0: ecap@0 { /* ECAP on am33xx */
Philip> compatible = "ti,am33xx-ecap";
Philip> #pwm-cells = <3>;
Philip> reg = <0x48300100 0x80>;
Philip> ti,hwmods = "ecap0";
Philip> };
Philip> +
Philip> +ecap0: ecap@0 { /* ECAP on da850 */
Philip> + compatible = "ti,da850-ecap", "ti,am33xx-ecap";
Philip> + #pwm-cells = <3>;
Philip> + reg = <0x306000 0x80>;
Philip> +};
Philip> --
Philip> 1.7.9.5



--
Bye, Peter Korsgaard

2013-03-25 07:09:28

by Peter Korsgaard

[permalink] [raw]
Subject: Re: [PATCH 2/2] pwm: pwm-tiehrpwm: Update device-tree binding document

>>>>> "Philip" == Philip Avinash <[email protected]> writes:

Philip> Update binding document of pwm-tiehrpwm to reflect the usage of similar
Philip> modules in da850 and am3xx SOCs.

Acked-by: Peter Korsgaard <[email protected]>

Philip> Signed-off-by: Philip Avinash <[email protected]>
Philip> Cc: Grant Likely <[email protected]>
Philip> Cc: Rob Herring <[email protected]>
Philip> Cc: Rob Landley <[email protected]>
Philip> ---
Philip> .../devicetree/bindings/pwm/pwm-tiehrpwm.txt | 12 ++++++++++--
Philip> 1 file changed, 10 insertions(+), 2 deletions(-)

Philip> diff --git a/Documentation/devicetree/bindings/pwm/pwm-tiehrpwm.txt b/Documentation/devicetree/bindings/pwm/pwm-tiehrpwm.txt
Philip> index 4fc7079..9a71369 100644
Philip> --- a/Documentation/devicetree/bindings/pwm/pwm-tiehrpwm.txt
Philip> +++ b/Documentation/devicetree/bindings/pwm/pwm-tiehrpwm.txt
Philip> @@ -1,7 +1,9 @@
Philip> TI SOC EHRPWM based PWM controller

Philip> Required properties:
Philip> -- compatible : Must be "ti,am33xx-ehrpwm"
Philip> +- compatible: Must be "ti,<soc>-ehrpwm".
Philip> + for am33xx - compatible = "ti,am33xx-ehrpwm";
Philip> + for da850 - compatible = "ti,da850-ehrpwm", "ti,am33xx-ehrpwm";
Philip> - #pwm-cells: Should be 3. Number of cells being used to specify PWM property.
Philip> First cell specifies the per-chip index of the PWM to use, the second
Philip> cell is the period in nanoseconds and bit 0 in the third cell is used to
Philip> @@ -15,9 +17,15 @@ Optional properties:

Philip> Example:

Philip> -ehrpwm0: ehrpwm@0 {
Philip> +ehrpwm0: ehrpwm@0 { /* EHRPWM on am33xx */
Philip> compatible = "ti,am33xx-ehrpwm";
Philip> #pwm-cells = <3>;
Philip> reg = <0x48300200 0x100>;
Philip> ti,hwmods = "ehrpwm0";
Philip> };
Philip> +
Philip> +ehrpwm0: ehrpwm@0 { /* EHRPWM on da850 */
Philip> + compatible = "ti,da850-ehrpwm", "ti,am33xx-ehrpwm";
Philip> + #pwm-cells = <3>;
Philip> + reg = <0x300000 0x2000>;
Philip> +};
Philip> --
Philip> 1.7.9.5



--
Bye, Peter Korsgaard

2013-04-08 05:58:09

by Philip, Avinash

[permalink] [raw]
Subject: RE: [PATCH 0/2] Update device tree binding document for pwm-tiehrpwm & pwm-tiecap

Thierry,

On Mon, Mar 25, 2013 at 12:34:51, Philip, Avinash wrote:
> Update device tree document of pwm-tiehrpwm & pwm-tiecap in order to reflect the
> usage of similar modules in both da850 and am33xx platforms.

Can you accept both documentation update patches with Peter Korsgaard's Ack.

Thanks
Avinash
>
> Philip Avinash (2):
> pwm: pwm-tiecap: Update device-tree binding document
> pwm: pwm-tiehrpwm: Update device-tree binding document
>
> .../devicetree/bindings/pwm/pwm-tiecap.txt | 12 ++++++++++--
> .../devicetree/bindings/pwm/pwm-tiehrpwm.txt | 12 ++++++++++--
> 2 files changed, 20 insertions(+), 4 deletions(-)
>
> --
> 1.7.9.5
>
>

2013-04-08 08:23:40

by Thierry Reding

[permalink] [raw]
Subject: Re: [PATCH 0/2] Update device tree binding document for pwm-tiehrpwm & pwm-tiecap

On Mon, Apr 08, 2013 at 05:56:39AM +0000, Philip, Avinash wrote:
> Thierry,
>
> On Mon, Mar 25, 2013 at 12:34:51, Philip, Avinash wrote:
> > Update device tree document of pwm-tiehrpwm & pwm-tiecap in order to reflect the
> > usage of similar modules in both da850 and am33xx platforms.
>
> Can you accept both documentation update patches with Peter Korsgaard's Ack.

Thanks for the reminder. Both patches applied.

Thierry


Attachments:
(No filename) (427.00 B)
(No filename) (836.00 B)
Download all attachments