Re: [PATCH v5 1/5] dt-bindings: Add RK808 device tree bindings document
On Mon, Aug 25, 2014 at 02:29:22PM +0100, Chris Zhong wrote:
> Add device tree bindings documentation and a header file
> for rockchip's RK808 pmic.
>
> Signed-off-by: Chris Zhong
>
> ---
>
> Changes in v5:
> Adviced by doug
> - add some error checking in probe
> - move "rockchip,rk808.h" into the patch about dt-bindings
>
> Changes in v4:
> Adviced by doug
> - add "clock-output-names" propertiey
> - add a header file "rockchip,rk808.h"
>
> Changes in v3:
> - fix compile err
>
> Changes in v2:
> Adviced by javier.martinez
> - separated from rtc-rk808.c
>
> Documentation/devicetree/bindings/mfd/rk808.txt | 142
> +++
> include/dt-bindings/clock/rockchip,rk808.h | 11 ++
> 2 files changed, 153 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/mfd/rk808.txt
> create mode 100644 include/dt-bindings/clock/rockchip,rk808.h
>
> diff --git a/Documentation/devicetree/bindings/mfd/rk808.txt
> b/Documentation/devicetree/bindings/mfd/rk808.txt
> new file mode 100644
> index 000..e5786e9
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/mfd/rk808.txt
> @@ -0,0 +1,142 @@
> +RK808 Power Management Integrated Circuit
> +
> +Required properties:
> +- compatible: "rockchip,rk808"
> +- reg: I2C slave address
> +- interrupt-parent: The parent interrupt controller.
> +- interrupts: the interrupt outputs of the controller.
> +- pinctrl-names: Should contain only one value - "default".
> +- pinctrl-0: Should specify pin control groups used for this controller.
> +- regulators: This is the list of child nodes that specify the regulator
> + initialization data for defined regulators. Not all regulators for the
> given
> + device need to be present. The definition for each of these nodes is
> defined
> + using the standard binding for regulators found at
> + Documentation/devicetree/bindings/regulator/regulator.txt.
> +- #clock-cells: the value should be 1
It would be nice to have a description of the valid values.
They look to be 0 and 1 in include/dt-bindings/clock/rockchip,rk808.h.
Thanks,
Mark.
> +- The following are the names of the regulators that the rk808 pmic block
> + supports. Note: The 'n' below represents the number as per the datasheet:
> +
> + - DCDC_REGn
> + - valid values for n are 1 to 4.
> + - LDO_REGn
> + - valid values for n are 1 to 8.
> + - SWITCH_REGn
> + - valid values for n are 1 to 2.
> +
> +Optional properties:
> +- clock-output-names : From common clock binding to override the
> + default output clock name
> +- rockchip,system-power-controller: Telling whether or not this pmic is
> controlling
> + the system power.
> +
> +Example:
> +rk808: pmic@1b {
> + compatible = "rockchip,rk808";
> + interrupt-parent = <>;
> + interrupts = <4 IRQ_TYPE_EDGE_FALLING>;
> + pinctrl-names = "default";
> + pinctrl-0 = <_int>;
> + reg = <0x1b>;
> + #clock-cells = <1>;
> + clock-output-names = "xin32k0", "xin32k1";
> + rockchip,system-power-controller;
> +
> + regulators {
> + rk808_dcdc1_reg: DCDC_REG1 {
> + regulator-always-on;
> + regulator-boot-on;
> + regulator-min-microvolt = <120>;
> + regulator-max-microvolt = <120>;
> + regulator-name = "vdd_arm";
> + };
> +
> + rk808_dcdc2_reg: DCDC_REG2 {
> + regulator-always-on;
> + regulator-boot-on;
> + regulator-min-microvolt = <85>;
> + regulator-max-microvolt = <125>;
> + regulator-name = "vdd_gpu";
> + };
> +
> + rk808_dcdc3_reg: DCDC_REG3 {
> + regulator-always-on;
> + regulator-boot-on;
> + regulator-name = "vdd_ddr";
> + };
> +
> + rk808_dcdc4_reg: DCDC_REG4 {
> + regulator-always-on;
> + regulator-boot-on;
> + regulator-min-microvolt = <330>;
> + regulator-max-microvolt = <330>;
> + regulator-name = "vccio";
> + };
> +
> + rk808_ldo1_reg: LDO_REG1 {
> + regulator-always-on;
> + regulator-boot-on;
> + regulator-min-microvolt = <330>;
> + regulator-max-microvolt = <330>;
> + };
> +
> + rk808_ldo2_reg: LDO_REG2 {
> + regulator-always-on;
> + regulator-boot-on;
> + regulator-min-microvolt =
Re: [PATCH v5 1/5] dt-bindings: Add RK808 device tree bindings document
On Mon, Aug 25, 2014 at 02:29:22PM +0100, Chris Zhong wrote:
Add device tree bindings documentation and a header file
for rockchip's RK808 pmic.
Signed-off-by: Chris Zhong z...@rock-chips.com
---
Changes in v5:
Adviced by doug
- add some error checking in probe
- move rockchip,rk808.h into the patch about dt-bindings
Changes in v4:
Adviced by doug
- add clock-output-names propertiey
- add a header file rockchip,rk808.h
Changes in v3:
- fix compile err
Changes in v2:
Adviced by javier.martinez
- separated from rtc-rk808.c
Documentation/devicetree/bindings/mfd/rk808.txt | 142
+++
include/dt-bindings/clock/rockchip,rk808.h | 11 ++
2 files changed, 153 insertions(+)
create mode 100644 Documentation/devicetree/bindings/mfd/rk808.txt
create mode 100644 include/dt-bindings/clock/rockchip,rk808.h
diff --git a/Documentation/devicetree/bindings/mfd/rk808.txt
b/Documentation/devicetree/bindings/mfd/rk808.txt
new file mode 100644
index 000..e5786e9
--- /dev/null
+++ b/Documentation/devicetree/bindings/mfd/rk808.txt
@@ -0,0 +1,142 @@
+RK808 Power Management Integrated Circuit
+
+Required properties:
+- compatible: rockchip,rk808
+- reg: I2C slave address
+- interrupt-parent: The parent interrupt controller.
+- interrupts: the interrupt outputs of the controller.
+- pinctrl-names: Should contain only one value - default.
+- pinctrl-0: Should specify pin control groups used for this controller.
+- regulators: This is the list of child nodes that specify the regulator
+ initialization data for defined regulators. Not all regulators for the
given
+ device need to be present. The definition for each of these nodes is
defined
+ using the standard binding for regulators found at
+ Documentation/devicetree/bindings/regulator/regulator.txt.
+- #clock-cells: the value should be 1
It would be nice to have a description of the valid values.
They look to be 0 and 1 in include/dt-bindings/clock/rockchip,rk808.h.
Thanks,
Mark.
+- The following are the names of the regulators that the rk808 pmic block
+ supports. Note: The 'n' below represents the number as per the datasheet:
+
+ - DCDC_REGn
+ - valid values for n are 1 to 4.
+ - LDO_REGn
+ - valid values for n are 1 to 8.
+ - SWITCH_REGn
+ - valid values for n are 1 to 2.
+
+Optional properties:
+- clock-output-names : From common clock binding to override the
+ default output clock name
+- rockchip,system-power-controller: Telling whether or not this pmic is
controlling
+ the system power.
+
+Example:
+rk808: pmic@1b {
+ compatible = rockchip,rk808;
+ interrupt-parent = gpio0;
+ interrupts = 4 IRQ_TYPE_EDGE_FALLING;
+ pinctrl-names = default;
+ pinctrl-0 = pmic_int;
+ reg = 0x1b;
+ #clock-cells = 1;
+ clock-output-names = xin32k0, xin32k1;
+ rockchip,system-power-controller;
+
+ regulators {
+ rk808_dcdc1_reg: DCDC_REG1 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = 120;
+ regulator-max-microvolt = 120;
+ regulator-name = vdd_arm;
+ };
+
+ rk808_dcdc2_reg: DCDC_REG2 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = 85;
+ regulator-max-microvolt = 125;
+ regulator-name = vdd_gpu;
+ };
+
+ rk808_dcdc3_reg: DCDC_REG3 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-name = vdd_ddr;
+ };
+
+ rk808_dcdc4_reg: DCDC_REG4 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = 330;
+ regulator-max-microvolt = 330;
+ regulator-name = vccio;
+ };
+
+ rk808_ldo1_reg: LDO_REG1 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = 330;
+ regulator-max-microvolt = 330;
+ };
+
+ rk808_ldo2_reg: LDO_REG2 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = 330;
+ regulator-max-microvolt = 330;
+ };
+
+ rk808_ldo3_reg:
Re: [PATCH v5 1/5] dt-bindings: Add RK808 device tree bindings document
On 08/26/2014 04:14 AM, Doug Anderson wrote:
Chris,
On Mon, Aug 25, 2014 at 6:29 AM, Chris Zhong wrote:
Add device tree bindings documentation and a header file
for rockchip's RK808 pmic.
Signed-off-by: Chris Zhong
---
Changes in v5:
Adviced by doug
- add some error checking in probe
- move "rockchip,rk808.h" into the patch about dt-bindings
Changes in v4:
Adviced by doug
- add "clock-output-names" propertiey
- add a header file "rockchip,rk808.h"
Changes in v3:
- fix compile err
Changes in v2:
Adviced by javier.martinez
- separated from rtc-rk808.c
Documentation/devicetree/bindings/mfd/rk808.txt | 142 +++
include/dt-bindings/clock/rockchip,rk808.h | 11 ++
2 files changed, 153 insertions(+)
create mode 100644 Documentation/devicetree/bindings/mfd/rk808.txt
create mode 100644 include/dt-bindings/clock/rockchip,rk808.h
diff --git a/Documentation/devicetree/bindings/mfd/rk808.txt
b/Documentation/devicetree/bindings/mfd/rk808.txt
new file mode 100644
index 000..e5786e9
--- /dev/null
+++ b/Documentation/devicetree/bindings/mfd/rk808.txt
@@ -0,0 +1,142 @@
+RK808 Power Management Integrated Circuit
+
+Required properties:
+- compatible: "rockchip,rk808"
+- reg: I2C slave address
+- interrupt-parent: The parent interrupt controller.
+- interrupts: the interrupt outputs of the controller.
+- pinctrl-names: Should contain only one value - "default".
+- pinctrl-0: Should specify pin control groups used for this controller.
You could probably skip including the pinctrl stuff here. Keep it in
the example. Nothing in the driver requires this and it's really an
artifact of the board.
Done
+- regulators: This is the list of child nodes that specify the regulator
Technically "regulators" is not a property, it's a child node. See
max8998 maybe for a sample, where it says:
Regulators: All the regulators of MAX8998 to be instantiated shall be
listed in a child node named 'regulators'. Each regulator is represented
by a child node of the 'regulators' node.
regulator-name {
/* standard regulator bindings here */
};
Done
+ initialization data for defined regulators. Not all regulators for the given
+ device need to be present. The definition for each of these nodes is defined
+ using the standard binding for regulators found at
+ Documentation/devicetree/bindings/regulator/regulator.txt.
+- #clock-cells: the value should be 1
+- The following are the names of the regulators that the rk808 pmic block
+ supports. Note: The 'n' below represents the number as per the datasheet:
+
+ - DCDC_REGn
+ - valid values for n are 1 to 4.
+ - LDO_REGn
+ - valid values for n are 1 to 8.
+ - SWITCH_REGn
+ - valid values for n are 1 to 2.
+
+Optional properties:
+- clock-output-names : From common clock binding to override the
+ default output clock name
nit: no space before the ":"
Done
+- rockchip,system-power-controller: Telling whether or not this pmic is
controlling
+ the system power.
+
+Example:
+rk808: pmic@1b {
+ compatible = "rockchip,rk808";
+ interrupt-parent = <>;
+ interrupts = <4 IRQ_TYPE_EDGE_FALLING>;
+ pinctrl-names = "default";
+ pinctrl-0 = <_int>;
+ reg = <0x1b>;
+ #clock-cells = <1>;
+ clock-output-names = "xin32k0", "xin32k1";
+ rockchip,system-power-controller;
+
+ regulators {
nit: you've indented regulators one too many spots.
Done
+ rk808_dcdc1_reg: DCDC_REG1 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = <120>;
+ regulator-max-microvolt = <120>;
+ regulator-name = "vdd_arm";
+ };
+
+ rk808_dcdc2_reg: DCDC_REG2 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = <85>;
+ regulator-max-microvolt = <125>;
+ regulator-name = "vdd_gpu";
+ };
+
+ rk808_dcdc3_reg: DCDC_REG3 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-name = "vdd_ddr";
+ };
+
+ rk808_dcdc4_reg: DCDC_REG4 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = <330>;
+ regulator-max-microvolt = <330>;
+ regulator-name = "vccio";
+ };
+
+ rk808_ldo1_reg: LDO_REG1 {
+
Re: [PATCH v5 1/5] dt-bindings: Add RK808 device tree bindings document
Chris,
On Mon, Aug 25, 2014 at 6:29 AM, Chris Zhong wrote:
> Add device tree bindings documentation and a header file
> for rockchip's RK808 pmic.
>
> Signed-off-by: Chris Zhong
>
> ---
>
> Changes in v5:
> Adviced by doug
> - add some error checking in probe
> - move "rockchip,rk808.h" into the patch about dt-bindings
>
> Changes in v4:
> Adviced by doug
> - add "clock-output-names" propertiey
> - add a header file "rockchip,rk808.h"
>
> Changes in v3:
> - fix compile err
>
> Changes in v2:
> Adviced by javier.martinez
> - separated from rtc-rk808.c
>
> Documentation/devicetree/bindings/mfd/rk808.txt | 142
> +++
> include/dt-bindings/clock/rockchip,rk808.h | 11 ++
> 2 files changed, 153 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/mfd/rk808.txt
> create mode 100644 include/dt-bindings/clock/rockchip,rk808.h
>
> diff --git a/Documentation/devicetree/bindings/mfd/rk808.txt
> b/Documentation/devicetree/bindings/mfd/rk808.txt
> new file mode 100644
> index 000..e5786e9
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/mfd/rk808.txt
> @@ -0,0 +1,142 @@
> +RK808 Power Management Integrated Circuit
> +
> +Required properties:
> +- compatible: "rockchip,rk808"
> +- reg: I2C slave address
> +- interrupt-parent: The parent interrupt controller.
> +- interrupts: the interrupt outputs of the controller.
> +- pinctrl-names: Should contain only one value - "default".
> +- pinctrl-0: Should specify pin control groups used for this controller.
You could probably skip including the pinctrl stuff here. Keep it in
the example. Nothing in the driver requires this and it's really an
artifact of the board.
> +- regulators: This is the list of child nodes that specify the regulator
Technically "regulators" is not a property, it's a child node. See
max8998 maybe for a sample, where it says:
Regulators: All the regulators of MAX8998 to be instantiated shall be
listed in a child node named 'regulators'. Each regulator is represented
by a child node of the 'regulators' node.
regulator-name {
/* standard regulator bindings here */
};
> + initialization data for defined regulators. Not all regulators for the
> given
> + device need to be present. The definition for each of these nodes is
> defined
> + using the standard binding for regulators found at
> + Documentation/devicetree/bindings/regulator/regulator.txt.
> +- #clock-cells: the value should be 1
> +- The following are the names of the regulators that the rk808 pmic block
> + supports. Note: The 'n' below represents the number as per the datasheet:
> +
> + - DCDC_REGn
> + - valid values for n are 1 to 4.
> + - LDO_REGn
> + - valid values for n are 1 to 8.
> + - SWITCH_REGn
> + - valid values for n are 1 to 2.
> +
> +Optional properties:
> +- clock-output-names : From common clock binding to override the
> + default output clock name
nit: no space before the ":"
> +- rockchip,system-power-controller: Telling whether or not this pmic is
> controlling
> + the system power.
> +
> +Example:
> +rk808: pmic@1b {
> + compatible = "rockchip,rk808";
> + interrupt-parent = <>;
> + interrupts = <4 IRQ_TYPE_EDGE_FALLING>;
> + pinctrl-names = "default";
> + pinctrl-0 = <_int>;
> + reg = <0x1b>;
> + #clock-cells = <1>;
> + clock-output-names = "xin32k0", "xin32k1";
> + rockchip,system-power-controller;
> +
> + regulators {
nit: you've indented regulators one too many spots.
> + rk808_dcdc1_reg: DCDC_REG1 {
> + regulator-always-on;
> + regulator-boot-on;
> + regulator-min-microvolt = <120>;
> + regulator-max-microvolt = <120>;
> + regulator-name = "vdd_arm";
> + };
> +
> + rk808_dcdc2_reg: DCDC_REG2 {
> + regulator-always-on;
> + regulator-boot-on;
> + regulator-min-microvolt = <85>;
> + regulator-max-microvolt = <125>;
> + regulator-name = "vdd_gpu";
> + };
> +
> + rk808_dcdc3_reg: DCDC_REG3 {
> + regulator-always-on;
> + regulator-boot-on;
> + regulator-name = "vdd_ddr";
> + };
> +
> + rk808_dcdc4_reg: DCDC_REG4 {
> + regulator-always-on;
> + regulator-boot-on;
> + regulator-min-microvolt = <330>;
> + regulator-max-microvolt = <330>;
> + regulator-name = "vccio";
> +
[PATCH v5 1/5] dt-bindings: Add RK808 device tree bindings document
Add device tree bindings documentation and a header file
for rockchip's RK808 pmic.
Signed-off-by: Chris Zhong
---
Changes in v5:
Adviced by doug
- add some error checking in probe
- move "rockchip,rk808.h" into the patch about dt-bindings
Changes in v4:
Adviced by doug
- add "clock-output-names" propertiey
- add a header file "rockchip,rk808.h"
Changes in v3:
- fix compile err
Changes in v2:
Adviced by javier.martinez
- separated from rtc-rk808.c
Documentation/devicetree/bindings/mfd/rk808.txt | 142 +++
include/dt-bindings/clock/rockchip,rk808.h | 11 ++
2 files changed, 153 insertions(+)
create mode 100644 Documentation/devicetree/bindings/mfd/rk808.txt
create mode 100644 include/dt-bindings/clock/rockchip,rk808.h
diff --git a/Documentation/devicetree/bindings/mfd/rk808.txt
b/Documentation/devicetree/bindings/mfd/rk808.txt
new file mode 100644
index 000..e5786e9
--- /dev/null
+++ b/Documentation/devicetree/bindings/mfd/rk808.txt
@@ -0,0 +1,142 @@
+RK808 Power Management Integrated Circuit
+
+Required properties:
+- compatible: "rockchip,rk808"
+- reg: I2C slave address
+- interrupt-parent: The parent interrupt controller.
+- interrupts: the interrupt outputs of the controller.
+- pinctrl-names: Should contain only one value - "default".
+- pinctrl-0: Should specify pin control groups used for this controller.
+- regulators: This is the list of child nodes that specify the regulator
+ initialization data for defined regulators. Not all regulators for the given
+ device need to be present. The definition for each of these nodes is defined
+ using the standard binding for regulators found at
+ Documentation/devicetree/bindings/regulator/regulator.txt.
+- #clock-cells: the value should be 1
+- The following are the names of the regulators that the rk808 pmic block
+ supports. Note: The 'n' below represents the number as per the datasheet:
+
+ - DCDC_REGn
+ - valid values for n are 1 to 4.
+ - LDO_REGn
+ - valid values for n are 1 to 8.
+ - SWITCH_REGn
+ - valid values for n are 1 to 2.
+
+Optional properties:
+- clock-output-names : From common clock binding to override the
+ default output clock name
+- rockchip,system-power-controller: Telling whether or not this pmic is
controlling
+ the system power.
+
+Example:
+rk808: pmic@1b {
+ compatible = "rockchip,rk808";
+ interrupt-parent = <>;
+ interrupts = <4 IRQ_TYPE_EDGE_FALLING>;
+ pinctrl-names = "default";
+ pinctrl-0 = <_int>;
+ reg = <0x1b>;
+ #clock-cells = <1>;
+ clock-output-names = "xin32k0", "xin32k1";
+ rockchip,system-power-controller;
+
+ regulators {
+ rk808_dcdc1_reg: DCDC_REG1 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = <120>;
+ regulator-max-microvolt = <120>;
+ regulator-name = "vdd_arm";
+ };
+
+ rk808_dcdc2_reg: DCDC_REG2 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = <85>;
+ regulator-max-microvolt = <125>;
+ regulator-name = "vdd_gpu";
+ };
+
+ rk808_dcdc3_reg: DCDC_REG3 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-name = "vdd_ddr";
+ };
+
+ rk808_dcdc4_reg: DCDC_REG4 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = <330>;
+ regulator-max-microvolt = <330>;
+ regulator-name = "vccio";
+ };
+
+ rk808_ldo1_reg: LDO_REG1 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = <330>;
+ regulator-max-microvolt = <330>;
+ };
+
+ rk808_ldo2_reg: LDO_REG2 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = <330>;
+ regulator-max-microvolt = <330>;
+ };
+
+ rk808_ldo3_reg: LDO_REG3 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = <100>;
+
[PATCH v5 1/5] dt-bindings: Add RK808 device tree bindings document
Add device tree bindings documentation and a header file
for rockchip's RK808 pmic.
Signed-off-by: Chris Zhong z...@rock-chips.com
---
Changes in v5:
Adviced by doug
- add some error checking in probe
- move rockchip,rk808.h into the patch about dt-bindings
Changes in v4:
Adviced by doug
- add clock-output-names propertiey
- add a header file rockchip,rk808.h
Changes in v3:
- fix compile err
Changes in v2:
Adviced by javier.martinez
- separated from rtc-rk808.c
Documentation/devicetree/bindings/mfd/rk808.txt | 142 +++
include/dt-bindings/clock/rockchip,rk808.h | 11 ++
2 files changed, 153 insertions(+)
create mode 100644 Documentation/devicetree/bindings/mfd/rk808.txt
create mode 100644 include/dt-bindings/clock/rockchip,rk808.h
diff --git a/Documentation/devicetree/bindings/mfd/rk808.txt
b/Documentation/devicetree/bindings/mfd/rk808.txt
new file mode 100644
index 000..e5786e9
--- /dev/null
+++ b/Documentation/devicetree/bindings/mfd/rk808.txt
@@ -0,0 +1,142 @@
+RK808 Power Management Integrated Circuit
+
+Required properties:
+- compatible: rockchip,rk808
+- reg: I2C slave address
+- interrupt-parent: The parent interrupt controller.
+- interrupts: the interrupt outputs of the controller.
+- pinctrl-names: Should contain only one value - default.
+- pinctrl-0: Should specify pin control groups used for this controller.
+- regulators: This is the list of child nodes that specify the regulator
+ initialization data for defined regulators. Not all regulators for the given
+ device need to be present. The definition for each of these nodes is defined
+ using the standard binding for regulators found at
+ Documentation/devicetree/bindings/regulator/regulator.txt.
+- #clock-cells: the value should be 1
+- The following are the names of the regulators that the rk808 pmic block
+ supports. Note: The 'n' below represents the number as per the datasheet:
+
+ - DCDC_REGn
+ - valid values for n are 1 to 4.
+ - LDO_REGn
+ - valid values for n are 1 to 8.
+ - SWITCH_REGn
+ - valid values for n are 1 to 2.
+
+Optional properties:
+- clock-output-names : From common clock binding to override the
+ default output clock name
+- rockchip,system-power-controller: Telling whether or not this pmic is
controlling
+ the system power.
+
+Example:
+rk808: pmic@1b {
+ compatible = rockchip,rk808;
+ interrupt-parent = gpio0;
+ interrupts = 4 IRQ_TYPE_EDGE_FALLING;
+ pinctrl-names = default;
+ pinctrl-0 = pmic_int;
+ reg = 0x1b;
+ #clock-cells = 1;
+ clock-output-names = xin32k0, xin32k1;
+ rockchip,system-power-controller;
+
+ regulators {
+ rk808_dcdc1_reg: DCDC_REG1 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = 120;
+ regulator-max-microvolt = 120;
+ regulator-name = vdd_arm;
+ };
+
+ rk808_dcdc2_reg: DCDC_REG2 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = 85;
+ regulator-max-microvolt = 125;
+ regulator-name = vdd_gpu;
+ };
+
+ rk808_dcdc3_reg: DCDC_REG3 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-name = vdd_ddr;
+ };
+
+ rk808_dcdc4_reg: DCDC_REG4 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = 330;
+ regulator-max-microvolt = 330;
+ regulator-name = vccio;
+ };
+
+ rk808_ldo1_reg: LDO_REG1 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = 330;
+ regulator-max-microvolt = 330;
+ };
+
+ rk808_ldo2_reg: LDO_REG2 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = 330;
+ regulator-max-microvolt = 330;
+ };
+
+ rk808_ldo3_reg: LDO_REG3 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = 100;
+ regulator-max-microvolt =
Re: [PATCH v5 1/5] dt-bindings: Add RK808 device tree bindings document
Chris,
On Mon, Aug 25, 2014 at 6:29 AM, Chris Zhong z...@rock-chips.com wrote:
Add device tree bindings documentation and a header file
for rockchip's RK808 pmic.
Signed-off-by: Chris Zhong z...@rock-chips.com
---
Changes in v5:
Adviced by doug
- add some error checking in probe
- move rockchip,rk808.h into the patch about dt-bindings
Changes in v4:
Adviced by doug
- add clock-output-names propertiey
- add a header file rockchip,rk808.h
Changes in v3:
- fix compile err
Changes in v2:
Adviced by javier.martinez
- separated from rtc-rk808.c
Documentation/devicetree/bindings/mfd/rk808.txt | 142
+++
include/dt-bindings/clock/rockchip,rk808.h | 11 ++
2 files changed, 153 insertions(+)
create mode 100644 Documentation/devicetree/bindings/mfd/rk808.txt
create mode 100644 include/dt-bindings/clock/rockchip,rk808.h
diff --git a/Documentation/devicetree/bindings/mfd/rk808.txt
b/Documentation/devicetree/bindings/mfd/rk808.txt
new file mode 100644
index 000..e5786e9
--- /dev/null
+++ b/Documentation/devicetree/bindings/mfd/rk808.txt
@@ -0,0 +1,142 @@
+RK808 Power Management Integrated Circuit
+
+Required properties:
+- compatible: rockchip,rk808
+- reg: I2C slave address
+- interrupt-parent: The parent interrupt controller.
+- interrupts: the interrupt outputs of the controller.
+- pinctrl-names: Should contain only one value - default.
+- pinctrl-0: Should specify pin control groups used for this controller.
You could probably skip including the pinctrl stuff here. Keep it in
the example. Nothing in the driver requires this and it's really an
artifact of the board.
+- regulators: This is the list of child nodes that specify the regulator
Technically regulators is not a property, it's a child node. See
max8998 maybe for a sample, where it says:
Regulators: All the regulators of MAX8998 to be instantiated shall be
listed in a child node named 'regulators'. Each regulator is represented
by a child node of the 'regulators' node.
regulator-name {
/* standard regulator bindings here */
};
+ initialization data for defined regulators. Not all regulators for the
given
+ device need to be present. The definition for each of these nodes is
defined
+ using the standard binding for regulators found at
+ Documentation/devicetree/bindings/regulator/regulator.txt.
+- #clock-cells: the value should be 1
+- The following are the names of the regulators that the rk808 pmic block
+ supports. Note: The 'n' below represents the number as per the datasheet:
+
+ - DCDC_REGn
+ - valid values for n are 1 to 4.
+ - LDO_REGn
+ - valid values for n are 1 to 8.
+ - SWITCH_REGn
+ - valid values for n are 1 to 2.
+
+Optional properties:
+- clock-output-names : From common clock binding to override the
+ default output clock name
nit: no space before the :
+- rockchip,system-power-controller: Telling whether or not this pmic is
controlling
+ the system power.
+
+Example:
+rk808: pmic@1b {
+ compatible = rockchip,rk808;
+ interrupt-parent = gpio0;
+ interrupts = 4 IRQ_TYPE_EDGE_FALLING;
+ pinctrl-names = default;
+ pinctrl-0 = pmic_int;
+ reg = 0x1b;
+ #clock-cells = 1;
+ clock-output-names = xin32k0, xin32k1;
+ rockchip,system-power-controller;
+
+ regulators {
nit: you've indented regulators one too many spots.
+ rk808_dcdc1_reg: DCDC_REG1 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = 120;
+ regulator-max-microvolt = 120;
+ regulator-name = vdd_arm;
+ };
+
+ rk808_dcdc2_reg: DCDC_REG2 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = 85;
+ regulator-max-microvolt = 125;
+ regulator-name = vdd_gpu;
+ };
+
+ rk808_dcdc3_reg: DCDC_REG3 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-name = vdd_ddr;
+ };
+
+ rk808_dcdc4_reg: DCDC_REG4 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = 330;
+ regulator-max-microvolt = 330;
+ regulator-name = vccio;
+ };
+
+ rk808_ldo1_reg: LDO_REG1 {
+
Re: [PATCH v5 1/5] dt-bindings: Add RK808 device tree bindings document
On 08/26/2014 04:14 AM, Doug Anderson wrote:
Chris,
On Mon, Aug 25, 2014 at 6:29 AM, Chris Zhong z...@rock-chips.com wrote:
Add device tree bindings documentation and a header file
for rockchip's RK808 pmic.
Signed-off-by: Chris Zhong z...@rock-chips.com
---
Changes in v5:
Adviced by doug
- add some error checking in probe
- move rockchip,rk808.h into the patch about dt-bindings
Changes in v4:
Adviced by doug
- add clock-output-names propertiey
- add a header file rockchip,rk808.h
Changes in v3:
- fix compile err
Changes in v2:
Adviced by javier.martinez
- separated from rtc-rk808.c
Documentation/devicetree/bindings/mfd/rk808.txt | 142 +++
include/dt-bindings/clock/rockchip,rk808.h | 11 ++
2 files changed, 153 insertions(+)
create mode 100644 Documentation/devicetree/bindings/mfd/rk808.txt
create mode 100644 include/dt-bindings/clock/rockchip,rk808.h
diff --git a/Documentation/devicetree/bindings/mfd/rk808.txt
b/Documentation/devicetree/bindings/mfd/rk808.txt
new file mode 100644
index 000..e5786e9
--- /dev/null
+++ b/Documentation/devicetree/bindings/mfd/rk808.txt
@@ -0,0 +1,142 @@
+RK808 Power Management Integrated Circuit
+
+Required properties:
+- compatible: rockchip,rk808
+- reg: I2C slave address
+- interrupt-parent: The parent interrupt controller.
+- interrupts: the interrupt outputs of the controller.
+- pinctrl-names: Should contain only one value - default.
+- pinctrl-0: Should specify pin control groups used for this controller.
You could probably skip including the pinctrl stuff here. Keep it in
the example. Nothing in the driver requires this and it's really an
artifact of the board.
Done
+- regulators: This is the list of child nodes that specify the regulator
Technically regulators is not a property, it's a child node. See
max8998 maybe for a sample, where it says:
Regulators: All the regulators of MAX8998 to be instantiated shall be
listed in a child node named 'regulators'. Each regulator is represented
by a child node of the 'regulators' node.
regulator-name {
/* standard regulator bindings here */
};
Done
+ initialization data for defined regulators. Not all regulators for the given
+ device need to be present. The definition for each of these nodes is defined
+ using the standard binding for regulators found at
+ Documentation/devicetree/bindings/regulator/regulator.txt.
+- #clock-cells: the value should be 1
+- The following are the names of the regulators that the rk808 pmic block
+ supports. Note: The 'n' below represents the number as per the datasheet:
+
+ - DCDC_REGn
+ - valid values for n are 1 to 4.
+ - LDO_REGn
+ - valid values for n are 1 to 8.
+ - SWITCH_REGn
+ - valid values for n are 1 to 2.
+
+Optional properties:
+- clock-output-names : From common clock binding to override the
+ default output clock name
nit: no space before the :
Done
+- rockchip,system-power-controller: Telling whether or not this pmic is
controlling
+ the system power.
+
+Example:
+rk808: pmic@1b {
+ compatible = rockchip,rk808;
+ interrupt-parent = gpio0;
+ interrupts = 4 IRQ_TYPE_EDGE_FALLING;
+ pinctrl-names = default;
+ pinctrl-0 = pmic_int;
+ reg = 0x1b;
+ #clock-cells = 1;
+ clock-output-names = xin32k0, xin32k1;
+ rockchip,system-power-controller;
+
+ regulators {
nit: you've indented regulators one too many spots.
Done
+ rk808_dcdc1_reg: DCDC_REG1 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = 120;
+ regulator-max-microvolt = 120;
+ regulator-name = vdd_arm;
+ };
+
+ rk808_dcdc2_reg: DCDC_REG2 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = 85;
+ regulator-max-microvolt = 125;
+ regulator-name = vdd_gpu;
+ };
+
+ rk808_dcdc3_reg: DCDC_REG3 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-name = vdd_ddr;
+ };
+
+ rk808_dcdc4_reg: DCDC_REG4 {
+ regulator-always-on;
+ regulator-boot-on;
+ regulator-min-microvolt = 330;
+ regulator-max-microvolt = 330;
+ regulator-name = vccio;
+ };
+
+ rk808_ldo1_reg: LDO_REG1 {
+ regulator-always-on;
+
