Re: [PATCH 1/2] dt-bindings: power: domain-idle-state: Allow idle-state-name

[Krzysztof Kozlowski]

On Sat, Nov 30, 2024 at 05:39:36PM +0100, Konrad Dybcio wrote:
> From: Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxxxx>
> 
> Allow specifying a name for idle states, similar to CPU idle states
> in cpu/idle-states.yaml
> 
> Signed-off-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxxxx>
> ---
>  Documentation/devicetree/bindings/power/domain-idle-state.yaml | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/power/domain-idle-state.yaml b/Documentation/devicetree/bindings/power/domain-idle-state.yaml
> index ec1f6f669e50b2e7b0756cb6a737c685ad81045b..4dd4f59bbbec3058cf20f064aeec4c9602a99eb7 100644
> --- a/Documentation/devicetree/bindings/power/domain-idle-state.yaml
> +++ b/Documentation/devicetree/bindings/power/domain-idle-state.yaml
> @@ -54,6 +54,11 @@ patternProperties:
>            (i.e. idle states node with entry-method property is set to "psci")
>            must specify this property.
>  
> +      idle-state-name:
> +        $ref: /schemas/types.yaml#/definitions/string
> +        description:
> +          A string used as a descriptive name for the idle state.

For user-visible strings we have 'label' property, but I see cpu idle
states already use this one, so I guess it is fine.

Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>

Best regards,
Krzysztof