Hi Rob, On lun., déc. 04 2017, Rob Herring <robh@xxxxxxxxxx> wrote: > On Fri, Dec 01, 2017 at 12:25:03PM +0100, Gregory CLEMENT wrote: >> Extend the documentation of the Armada 37xx SoC with the the North >> Bridge Power Management component. >> >> Signed-off-by: Gregory CLEMENT <gregory.clement@xxxxxxxxxxxxxxxxxx> >> --- >> .../devicetree/bindings/arm/marvell/armada-37xx.txt | 19 +++++++++++++++++++ >> 1 file changed, 19 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt >> index 51336e5fc761..7ad9830d9177 100644 >> --- a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt >> +++ b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt >> @@ -14,3 +14,22 @@ following property before the previous one: >> Example: >> >> compatible = "marvell,armada-3720-db", "marvell,armada3720", "marvell,armada3710"; >> + >> + >> +Power management >> +---------------- >> + >> +For power management (particularly DVFS and AVS), the North Bridge >> +Power Management component is needed: >> + >> +Required properties: >> +- compatible : should contain "marvell,armada-3700-nb-pm", "syscon"; >> +- reg : the register start and length for the North Bridge >> + Power Management >> + >> +Example: >> + >> +nb_pm: nb_pm@14000 { > > Don't use underscore in node or property names. "syscon" is a better > choice here. OK I will change this. Thanks, Gregory > >> + compatible = "marvell,armada-3700-nb-pm", "syscon"; >> + reg = <0x14000 0x60>; >> +} >> -- >> 2.15.0 >> -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html