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

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

[Philip Avinash]

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

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

[Philip Avinash]

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

Signed-off-by: Philip Avinash <avinashphilip@ti.com>
Cc: Grant Likely <grant.likely@secretlab.ca>
Cc: Rob Herring <rob.herring@calxeda.com>
Cc: Rob Landley <rob@landley.net>
---
 .../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

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

[Peter Korsgaard]

>>>>> "Philip" == Philip Avinash <avinashphilip@ti.com> 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 <jacmet@sunsite.dk>

 Philip> Signed-off-by: Philip Avinash <avinashphilip@ti.com>
 Philip> Cc: Grant Likely <grant.likely@secretlab.ca>
 Philip> Cc: Rob Herring <rob.herring@calxeda.com>
 Philip> Cc: Rob Landley <rob@landley.net>
 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

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

[Philip Avinash]

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

Signed-off-by: Philip Avinash <avinashphilip-l0cyMroinI0@public.gmane.org>
Cc: Grant Likely <grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org>
Cc: Rob Herring <rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org>
Cc: Rob Landley <rob-VoJi6FS/r0vR7s880joybQ@public.gmane.org>
---
 .../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

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

[Peter Korsgaard]

>>>>> "Philip" == Philip Avinash <avinashphilip@ti.com> 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 <jacmet@sunsite.dk>

 Philip> Signed-off-by: Philip Avinash <avinashphilip@ti.com>
 Philip> Cc: Grant Likely <grant.likely@secretlab.ca>
 Philip> Cc: Rob Herring <rob.herring@calxeda.com>
 Philip> Cc: Rob Landley <rob@landley.net>
 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

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

[Philip, Avinash]

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
> 
> 

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

[Thierry Reding]

[-- Attachment #1: Type: text/plain, Size: 439 bytes --]

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

[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]