[PATCH v2 1/3] dt-bindings: add Marvell PMU documentation

[PATCH v2 1/3] dt-bindings: add Marvell PMU documentation

[Russell King]

Add the required DT binding documentation for the Marvell PMU driver.

Acked-by: Rob Herring <robh@kernel.org>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
---
Who takes these patches?  This never got merged when the PMU driver
itself was merged.

 Documentation/devicetree/bindings/soc/dove/pmu.txt | 56 ++++++++++++++++++++++
 1 file changed, 56 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/soc/dove/pmu.txt

diff --git a/Documentation/devicetree/bindings/soc/dove/pmu.txt b/Documentation/devicetree/bindings/soc/dove/pmu.txt
new file mode 100644
index 000000000000..edd40b796b74
--- /dev/null
+++ b/Documentation/devicetree/bindings/soc/dove/pmu.txt
@@ -0,0 +1,56 @@
+Device Tree bindings for Marvell PMU
+
+Required properties:
+ - compatible: value should be "marvell,dove-pmu".
+    May also include "simple-bus" if there are child devices, in which
+    case the ranges node is required.
+ - reg: two base addresses and sizes of the PM controller and PMU.
+ - interrupts: single interrupt number for the PMU interrupt
+ - interrupt-controller: must be specified as the PMU itself is an
+    interrupt controller.
+ - #interrupt-cells: must be 1.
+ - #reset-cells: must be 1.
+ - domains: sub-node containing domain descriptions
+
+Optional properties:
+ - ranges: defines the address mapping for child devices, as per the
+   standard property of this name.  Required when compatible includes
+   "simple-bus".
+
+Power domain descriptions are listed as child nodes of the "domains"
+sub-node.  Each domain has the following properties:
+
+Required properties:
+ - #power-domain-cells: must be 0.
+
+Optional properties:
+ - marvell,pmu_pwr_mask: specifies the mask value for PMU power register
+ - marvell,pmu_iso_mask: specifies the mask value for PMU isolation register
+ - resets: points to the reset manager (PMU node) and reset index.
+
+Example:
+
+	pmu: power-management at d0000 {
+		compatible = "marvell,dove-pmu";
+		reg = <0xd0000 0x8000>, <0xd8000 0x8000>;
+		interrupts = <33>;
+		interrupt-controller;
+		#interrupt-cells = <1>;
+		#reset-cells = <1>;
+
+		domains {
+			vpu_domain: vpu-domain {
+				#power-domain-cells = <0>;
+				marvell,pmu_pwr_mask = <0x00000008>;
+				marvell,pmu_iso_mask = <0x00000001>;
+				resets = <&pmu 16>;
+			};
+
+			gpu_domain: gpu-domain {
+				#power-domain-cells = <0>;
+				marvell,pmu_pwr_mask = <0x00000004>;
+				marvell,pmu_iso_mask = <0x00000002>;
+				resets = <&pmu 18>;
+			};
+		};
+	};
-- 
2.1.0

[PATCH v2 1/3] dt-bindings: add Marvell PMU documentation

[Arnd Bergmann]

On Sunday 06 December 2015 23:52:21 Russell King wrote:
> Add the required DT binding documentation for the Marvell PMU driver.
> 
> Acked-by: Rob Herring <robh@kernel.org>
> Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
> ---
> Who takes these patches?  This never got merged when the PMU driver
> itself was merged.

I'd say it should go along with the dts changes for mvebu, and we'll
merge it through arm-soc.

	Arnd

[PATCH v2 1/3] dt-bindings: add Marvell PMU documentation

[Russell King - ARM Linux]

On Mon, Dec 07, 2015 at 10:09:48AM +0100, Arnd Bergmann wrote:
> On Sunday 06 December 2015 23:52:21 Russell King wrote:
> > Add the required DT binding documentation for the Marvell PMU driver.
> > 
> > Acked-by: Rob Herring <robh@kernel.org>
> > Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
> > ---
> > Who takes these patches?  This never got merged when the PMU driver
> > itself was merged.
> 
> I'd say it should go along with the dts changes for mvebu, and we'll
> merge it through arm-soc.

Well, it didn't, the dts changes are already in as well.  Someone now
needs to pick it up independently.

I wonder how many more of the undocumented bindings are actually caused
by people not realising that the binding docs patches have been ignored.
(Yes, I've found cases in the past where bindings are undocumented, and
no, I don't keep a list and I don't remember which they were.)

-- 
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.

[PATCH v2 1/3] dt-bindings: add Marvell PMU documentation

[Gregory CLEMENT]

Hi Russell,
 
 On lun., d?c. 07 2015, Russell King - ARM Linux <linux@arm.linux.org.uk> wrote:

> On Mon, Dec 07, 2015 at 10:09:48AM +0100, Arnd Bergmann wrote:
>> On Sunday 06 December 2015 23:52:21 Russell King wrote:
>> > Add the required DT binding documentation for the Marvell PMU driver.
>> > 
>> > Acked-by: Rob Herring <robh@kernel.org>
>> > Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
>> > ---
>> > Who takes these patches?  This never got merged when the PMU driver
>> > itself was merged.
>> 
>> I'd say it should go along with the dts changes for mvebu, and we'll
>> merge it through arm-soc.
>
> Well, it didn't, the dts changes are already in as well.  Someone now
> needs to pick it up independently.

OK I will.

The last time Andrew took your series to port it onto 4.2-rc and I
thought he took everything but actually he only focused on the commits
with conflict. Then I merged the series without checking it was
complete. Sorry for having missed it.

Gregory

>
> I wonder how many more of the undocumented bindings are actually caused
> by people not realising that the binding docs patches have been ignored.
> (Yes, I've found cases in the past where bindings are undocumented, and
> no, I don't keep a list and I don't remember which they were.)
>
> -- 
> FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
> according to speedtest.net.

-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

[PATCH v2 1/3] dt-bindings: add Marvell PMU documentation

[Gregory CLEMENT]

Hi Russell,
 
 On lun., d?c. 07 2015, Russell King <rmk+kernel@arm.linux.org.uk> wrote:

> Add the required DT binding documentation for the Marvell PMU driver.
>
> Acked-by: Rob Herring <robh@kernel.org>
> Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>

Applied on mvebu/dt

Thanks,

Gregory

> ---
> Who takes these patches?  This never got merged when the PMU driver
> itself was merged.
>
>  Documentation/devicetree/bindings/soc/dove/pmu.txt | 56 ++++++++++++++++++++++
>  1 file changed, 56 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/soc/dove/pmu.txt
>
> diff --git a/Documentation/devicetree/bindings/soc/dove/pmu.txt b/Documentation/devicetree/bindings/soc/dove/pmu.txt
> new file mode 100644
> index 000000000000..edd40b796b74
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/soc/dove/pmu.txt
> @@ -0,0 +1,56 @@
> +Device Tree bindings for Marvell PMU
> +
> +Required properties:
> + - compatible: value should be "marvell,dove-pmu".
> +    May also include "simple-bus" if there are child devices, in which
> +    case the ranges node is required.
> + - reg: two base addresses and sizes of the PM controller and PMU.
> + - interrupts: single interrupt number for the PMU interrupt
> + - interrupt-controller: must be specified as the PMU itself is an
> +    interrupt controller.
> + - #interrupt-cells: must be 1.
> + - #reset-cells: must be 1.
> + - domains: sub-node containing domain descriptions
> +
> +Optional properties:
> + - ranges: defines the address mapping for child devices, as per the
> +   standard property of this name.  Required when compatible includes
> +   "simple-bus".
> +
> +Power domain descriptions are listed as child nodes of the "domains"
> +sub-node.  Each domain has the following properties:
> +
> +Required properties:
> + - #power-domain-cells: must be 0.
> +
> +Optional properties:
> + - marvell,pmu_pwr_mask: specifies the mask value for PMU power register
> + - marvell,pmu_iso_mask: specifies the mask value for PMU isolation register
> + - resets: points to the reset manager (PMU node) and reset index.
> +
> +Example:
> +
> +	pmu: power-management at d0000 {
> +		compatible = "marvell,dove-pmu";
> +		reg = <0xd0000 0x8000>, <0xd8000 0x8000>;
> +		interrupts = <33>;
> +		interrupt-controller;
> +		#interrupt-cells = <1>;
> +		#reset-cells = <1>;
> +
> +		domains {
> +			vpu_domain: vpu-domain {
> +				#power-domain-cells = <0>;
> +				marvell,pmu_pwr_mask = <0x00000008>;
> +				marvell,pmu_iso_mask = <0x00000001>;
> +				resets = <&pmu 16>;
> +			};
> +
> +			gpu_domain: gpu-domain {
> +				#power-domain-cells = <0>;
> +				marvell,pmu_pwr_mask = <0x00000004>;
> +				marvell,pmu_iso_mask = <0x00000002>;
> +				resets = <&pmu 18>;
> +			};
> +		};
> +	};
> -- 
> 2.1.0
>

-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com