Re: [PATCH 5/6] dt-bindings: power: supply: summit,smb347: Add missing power-supply ref

[David Heidelberg <david@ixit.cz>]

Thank you,

Reviewed-by: David Heidelberg <david@ixit.cz>


On 04/06/2025 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>
> ---
>   .../devicetree/bindings/power/supply/summit,smb347-charger.yaml      | 5 ++---
>   1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/power/supply/summit,smb347-charger.yaml b/Documentation/devicetree/bindings/power/supply/summit,smb347-charger.yaml
> index 2d552becbfe6cb08c72f2a5584bdddc67443a2e4..65ed92bb05f30f91e24978d3261c7f29e4233e2b 100644
> --- a/Documentation/devicetree/bindings/power/supply/summit,smb347-charger.yaml
> +++ b/Documentation/devicetree/bindings/power/supply/summit,smb347-charger.yaml
> @@ -23,9 +23,7 @@ properties:
>     interrupts:
>       maxItems: 1
>   
> -  monitored-battery:
> -    description: phandle to the battery node
> -    $ref: /schemas/types.yaml#/definitions/phandle
> +  monitored-battery: true
>   
>     summit,enable-usb-charging:
>       type: boolean
> @@ -94,6 +92,7 @@ properties:
>       unevaluatedProperties: false
>   
>   allOf:
> +  - $ref: power-supply.yaml#
>     - if:
>         properties:
>           compatible:
>