Patchwork [PATCHv4,3/3] pwm: documentation: Add 'big-endian' property for FTM PWM.

login
register
mail settings
Submitter Xiubo Li
Date Aug. 19, 2014, 4:38 a.m.
Message ID <1408423083-19827-4-git-send-email-Li.Xiubo@freescale.com>
Download mbox | patch
Permalink /patch/381224/
State Accepted
Headers show

Comments

Xiubo Li - Aug. 19, 2014, 4:38 a.m.
For the same FTM PWM device, which on different SoCs will in
differrent endian mode, using the same device driver need
one boolean properties to distingush this.

For now the FTM PWM is only applied to the ARM SoCs which are all
in LE mode.

The FTM PWM device can be integrated as little-endian or big-endian.
This endianness differs per SoC:
SoC     | FTM-PWM endianness
--------+-------------------
Vybrid  | LE
LS1     | BE
LS2     | LE

Signed-off-by: Xiubo Li <Li.Xiubo@freescale.com>
---
 .../devicetree/bindings/pwm/pwm-fsl-ftm.txt        | 23 +++++++++++++++++++++-
 1 file changed, 22 insertions(+), 1 deletion(-)

Patch

diff --git a/Documentation/devicetree/bindings/pwm/pwm-fsl-ftm.txt b/Documentation/devicetree/bindings/pwm/pwm-fsl-ftm.txt
index 0bda229..49eedf4 100644
--- a/Documentation/devicetree/bindings/pwm/pwm-fsl-ftm.txt
+++ b/Documentation/devicetree/bindings/pwm/pwm-fsl-ftm.txt
@@ -1,5 +1,24 @@ 
 Freescale FlexTimer Module (FTM) PWM controller
 
+For the same FTM PWM device, which on different SoCs will in
+differrent endian mode, using the same device driver need
+one boolean properties to distingush this.
+
+For now the FTM PWM is only applied to the ARM SoCs which are all
+in LE mode.
+
+The FTM PWM device can be integrated as little-endian or big-endian.
+This endianness differs per SoC:
+SoC     | FTM-PWM endianness
+--------+-------------------
+Vybrid  | LE
+LS1     | BE
+LS2     | LE
+
+Please see "Documentation/devicetree/bindings/regmap/regmap.txt" for more
+detail about the endianness.
+
+
 Required properties:
 - compatible: Should be "fsl,vf610-ftm-pwm".
 - reg: Physical base address and length of the controller's registers
@@ -16,7 +35,8 @@  Required properties:
 - pinctrl-names: Must contain a "default" entry.
 - pinctrl-NNN: One property must exist for each entry in pinctrl-names.
   See pinctrl/pinctrl-bindings.txt for details of the property values.
-
+- big-endian: Boolean property, required if all the FTM_PWM registers
+  are big-endian rather than little-endian.
 
 Example:
 
@@ -32,4 +52,5 @@  pwm0: pwm@40038000 {
 			<&clks VF610_CLK_FTM0_EXT_FIX_EN>;
 		pinctrl-names = "default";
 		pinctrl-0 = <&pinctrl_pwm0_1>;
+		big-endian;
 };