From: Kory Maincent (Dent Project) <kory.maincent@bootlin.com>

Introduce a new property to describe the power budget of the regulator.
This property will allow power management support for regulator consumers
like PSE controllers, enabling them to make decisions based on the
available power capacity.

Signed-off-by: Kory Maincent <kory.maincent@bootlin.com>
---

Changes in v2:
- new patch.
---
 Documentation/devicetree/bindings/regulator/regulator.yaml | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/Documentation/devicetree/bindings/regulator/regulator.yaml b/Documentation/devicetree/bindings/regulator/regulator.yaml
index 1ef380d1515e..52a410b51769 100644
--- a/Documentation/devicetree/bindings/regulator/regulator.yaml
+++ b/Documentation/devicetree/bindings/regulator/regulator.yaml
@@ -34,6 +34,9 @@ properties:
   regulator-input-current-limit-microamp:
     description: maximum input current regulator allows
 
+  regulator-power-budget:
+    description: power budget of the regulator in mW
+
   regulator-always-on:
     description: boolean, regulator should never be disabled
     type: boolean

-- 
2.34.1

On 21/11/2024 15:42, Kory Maincent wrote:
> From: Kory Maincent (Dent Project) <kory.maincent@bootlin.com>
> 
> Introduce a new property to describe the power budget of the regulator.
> This property will allow power management support for regulator consumers
> like PSE controllers, enabling them to make decisions based on the
> available power capacity.
> 
> Signed-off-by: Kory Maincent <kory.maincent@bootlin.com>
> ---
> 
Considering:
1. This was not tested,
2. You did not Cc DT folks
so I assume you do not expect review as indicated by RFC. It is fine,
but if that was not the intention, please kindly first test the patch
and then use get_maintainers.pl or b4.

Best regards,
Krzysztof

On Thu, Nov 21, 2024 at 03:42:43PM +0100, Kory Maincent wrote:
>    regulator-input-current-limit-microamp:
>      description: maximum input current regulator allows
>  
> +  regulator-power-budget:
> +    description: power budget of the regulator in mW
> +

Properties are supposed to include the unit in the name.

On Thu, 21 Nov 2024 14:58:06 +0000
Mark Brown <broonie@kernel.org> wrote:

> On Thu, Nov 21, 2024 at 03:42:43PM +0100, Kory Maincent wrote:
> >    regulator-input-current-limit-microamp:
> >      description: maximum input current regulator allows
> >  
> > +  regulator-power-budget:
> > +    description: power budget of the regulator in mW
> > +  
> 
> Properties are supposed to include the unit in the name.

Oh ok, thanks!

-- 
Köry Maincent, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com