mbox series

[0/6] dt-bindings: power: supply: Few cleanups around monitored-battery

Message ID 20250604-dt-bindings-psy-monitored-battery-v1-0-7f755ff75218@linaro.org
Headers show
Series dt-bindings: power: supply: Few cleanups around monitored-battery | expand

Message

Krzysztof Kozlowski June 4, 2025, 8:18 a.m. UTC 
Reference the common power supply schema to bring the definition of
monitored-battery property allowing to drop redundant pieces.

Best regards,
Krzysztof

---
Krzysztof Kozlowski (6):
      dt-bindings: power: supply: bq2515x: Add missing power-supply ref
      dt-bindings: power: supply: bq256xx: Add missing power-supply ref
      dt-bindings: power: supply: qcom,pmi8998: Add missing power-supply ref
      dt-bindings: power: supply: richtek,rt5033: Add missing power-supply ref
      dt-bindings: power: supply: summit,smb347: Add missing power-supply ref
      dt-bindings: power: supply: Drop redundant monitored-battery ref

 Documentation/devicetree/bindings/power/supply/bq24190.yaml        | 1 -
 Documentation/devicetree/bindings/power/supply/bq2515x.yaml        | 7 ++++---
 Documentation/devicetree/bindings/power/supply/bq256xx.yaml        | 5 ++---
 Documentation/devicetree/bindings/power/supply/bq25980.yaml        | 4 +---
 Documentation/devicetree/bindings/power/supply/cw2015_battery.yaml | 5 +----
 .../devicetree/bindings/power/supply/qcom,pmi8998-charger.yaml     | 7 ++++---
 .../devicetree/bindings/power/supply/richtek,rt5033-charger.yaml   | 4 +++-
 .../devicetree/bindings/power/supply/stericsson,ab8500-btemp.yaml  | 4 +---
 .../bindings/power/supply/stericsson,ab8500-chargalg.yaml          | 4 +---
 .../bindings/power/supply/stericsson,ab8500-charger.yaml           | 4 +---
 .../devicetree/bindings/power/supply/stericsson,ab8500-fg.yaml     | 4 +---
 .../devicetree/bindings/power/supply/summit,smb347-charger.yaml    | 5 ++---
 .../power/supply/x-powers,axp20x-battery-power-supply.yaml         | 6 +-----
 13 files changed, 22 insertions(+), 38 deletions(-)
---
base-commit: 3be1a7a31fbda82f3604b6c31e4f390110de1b46
change-id: 20250604-dt-bindings-psy-monitored-battery-383ccb383351

Best regards,

Comments

Casey Connolly June 11, 2025, 9:01 a.m. UTC | #1 
On 6/4/25 10:18, Krzysztof Kozlowski wrote:
> Reference the common power supply schema to bring the definition of
> monitored-battery property.  Drop also the obvious description because
> it is duplicating what is in power-supply.yaml.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Reviewed-by: Casey Connolly <casey.connolly@linaro.org>> ---
>   .../devicetree/bindings/power/supply/qcom,pmi8998-charger.yaml     | 7 ++++---
>   1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/power/supply/qcom,pmi8998-charger.yaml b/Documentation/devicetree/bindings/power/supply/qcom,pmi8998-charger.yaml
> index 90c7dc7632c58dc5cbfb3abcde8e730882bfd936..70f5cd6eaeabe822babf567d92b1d7f9124f3680 100644
> --- a/Documentation/devicetree/bindings/power/supply/qcom,pmi8998-charger.yaml
> +++ b/Documentation/devicetree/bindings/power/supply/qcom,pmi8998-charger.yaml
> @@ -38,9 +38,7 @@ properties:
>         - const: usbin_i
>         - const: usbin_v
>   
> -  monitored-battery:
> -    description: phandle to the simple-battery node
> -    $ref: /schemas/types.yaml#/definitions/phandle
> +  monitored-battery: true
>   
>   required:
>     - compatible
> @@ -51,6 +49,9 @@ required:
>     - io-channel-names
>     - monitored-battery
>   
> +allOf:
> +  - $ref: power-supply.yaml#
> +
>   additionalProperties: false
>   
>   examples:
>