Re: [PATCH v3 1/2] dt-bindings: pfuze100: add optional disable switch-regulators binding
On Mon, Jul 23, 2018 at 09:47:46AM +0200, Marco Felsch wrote: > This binding is used to keep the backward compatibility with the current > dtb's [1]. The binding informs the driver that the unused switch regulators > can be disabled. > If it is not specified, the driver doesn't disable the switch regulators. > > [1] https://patchwork.kernel.org/patch/10490381/ > > Signed-off-by: Marco Felsch > > --- > Changes in v3: > - rename dt binding, add -sw suffix > - fix description > > Changes in V2: > - add more information about the binding > - rename binding and add vendor prefix > > Documentation/devicetree/bindings/regulator/pfuze100.txt | 9 + > 1 file changed, 9 insertions(+) Reviewed-by: Rob Herring
Re: [PATCH v3 1/2] dt-bindings: pfuze100: add optional disable switch-regulators binding
On Mon, Jul 23, 2018 at 09:47:46AM +0200, Marco Felsch wrote: > This binding is used to keep the backward compatibility with the current > dtb's [1]. The binding informs the driver that the unused switch regulators > can be disabled. > If it is not specified, the driver doesn't disable the switch regulators. > > [1] https://patchwork.kernel.org/patch/10490381/ > > Signed-off-by: Marco Felsch > > --- > Changes in v3: > - rename dt binding, add -sw suffix > - fix description > > Changes in V2: > - add more information about the binding > - rename binding and add vendor prefix > > Documentation/devicetree/bindings/regulator/pfuze100.txt | 9 + > 1 file changed, 9 insertions(+) Reviewed-by: Rob Herring
Re: [PATCH v3 1/2] dt-bindings: pfuze100: add optional disable switch-regulators binding
On Mon, Jul 23, 2018 at 4:47 AM, Marco Felsch wrote: > This binding is used to keep the backward compatibility with the current > dtb's [1]. The binding informs the driver that the unused switch regulators > can be disabled. > If it is not specified, the driver doesn't disable the switch regulators. > > [1] https://patchwork.kernel.org/patch/10490381/ > > Signed-off-by: Marco Felsch Reviewed-by: Fabio Estevam
Re: [PATCH v3 1/2] dt-bindings: pfuze100: add optional disable switch-regulators binding
On Mon, Jul 23, 2018 at 4:47 AM, Marco Felsch wrote: > This binding is used to keep the backward compatibility with the current > dtb's [1]. The binding informs the driver that the unused switch regulators > can be disabled. > If it is not specified, the driver doesn't disable the switch regulators. > > [1] https://patchwork.kernel.org/patch/10490381/ > > Signed-off-by: Marco Felsch Reviewed-by: Fabio Estevam
[PATCH v3 1/2] dt-bindings: pfuze100: add optional disable switch-regulators binding
This binding is used to keep the backward compatibility with the current dtb's [1]. The binding informs the driver that the unused switch regulators can be disabled. If it is not specified, the driver doesn't disable the switch regulators. [1] https://patchwork.kernel.org/patch/10490381/ Signed-off-by: Marco Felsch --- Changes in v3: - rename dt binding, add -sw suffix - fix description Changes in V2: - add more information about the binding - rename binding and add vendor prefix Documentation/devicetree/bindings/regulator/pfuze100.txt | 9 + 1 file changed, 9 insertions(+) diff --git a/Documentation/devicetree/bindings/regulator/pfuze100.txt b/Documentation/devicetree/bindings/regulator/pfuze100.txt index 672c939045ff..c7610718adff 100644 --- a/Documentation/devicetree/bindings/regulator/pfuze100.txt +++ b/Documentation/devicetree/bindings/regulator/pfuze100.txt @@ -4,6 +4,15 @@ Required properties: - compatible: "fsl,pfuze100", "fsl,pfuze200", "fsl,pfuze3000", "fsl,pfuze3001" - reg: I2C slave address +Optional properties: +- fsl,pfuze-support-disable-sw: Boolean, if present disable all unused switch + regulators to save power consumption. Attention, ensure that all important + regulators (e.g. DDR ref, DDR supply) has set the "regulator-always-on" + property. If not present, the switched regualtors are always on and can't be + disabled. This binding is a workaround to keep backward compatibility with + old dtb's which rely on the fact that the switched regulators are always on + and don't mark them explicit as "regulator-always-on". + Required child node: - regulators: This is the list of child nodes that specify the regulator initialization data for defined regulators. Please refer to below doc -- 2.18.0
[PATCH v3 1/2] dt-bindings: pfuze100: add optional disable switch-regulators binding
This binding is used to keep the backward compatibility with the current dtb's [1]. The binding informs the driver that the unused switch regulators can be disabled. If it is not specified, the driver doesn't disable the switch regulators. [1] https://patchwork.kernel.org/patch/10490381/ Signed-off-by: Marco Felsch --- Changes in v3: - rename dt binding, add -sw suffix - fix description Changes in V2: - add more information about the binding - rename binding and add vendor prefix Documentation/devicetree/bindings/regulator/pfuze100.txt | 9 + 1 file changed, 9 insertions(+) diff --git a/Documentation/devicetree/bindings/regulator/pfuze100.txt b/Documentation/devicetree/bindings/regulator/pfuze100.txt index 672c939045ff..c7610718adff 100644 --- a/Documentation/devicetree/bindings/regulator/pfuze100.txt +++ b/Documentation/devicetree/bindings/regulator/pfuze100.txt @@ -4,6 +4,15 @@ Required properties: - compatible: "fsl,pfuze100", "fsl,pfuze200", "fsl,pfuze3000", "fsl,pfuze3001" - reg: I2C slave address +Optional properties: +- fsl,pfuze-support-disable-sw: Boolean, if present disable all unused switch + regulators to save power consumption. Attention, ensure that all important + regulators (e.g. DDR ref, DDR supply) has set the "regulator-always-on" + property. If not present, the switched regualtors are always on and can't be + disabled. This binding is a workaround to keep backward compatibility with + old dtb's which rely on the fact that the switched regulators are always on + and don't mark them explicit as "regulator-always-on". + Required child node: - regulators: This is the list of child nodes that specify the regulator initialization data for defined regulators. Please refer to below doc -- 2.18.0
