[v1] dt-bindings: spi: rockchip: add power-domains property

[PATCH v1] dt-bindings: spi: rockchip: add power-domains property

Posted by Johan Jonker 3 years, 6 months ago

The Rockchip rk3399 TRM mentions that pd_sdioaudio includes
sdio, spi, i2s and spdif. Add a power-domains property to
reduce notifications with spi-rockchip.yaml.

Signed-off-by: Johan Jonker <jbx6244@gmail.com>
---
 Documentation/devicetree/bindings/spi/spi-rockchip.yaml | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/Documentation/devicetree/bindings/spi/spi-rockchip.yaml b/Documentation/devicetree/bindings/spi/spi-rockchip.yaml
index 52768894b..66e49947b 100644
--- a/Documentation/devicetree/bindings/spi/spi-rockchip.yaml
+++ b/Documentation/devicetree/bindings/spi/spi-rockchip.yaml
@@ -82,6 +82,9 @@ properties:
       where the "sleep" configuration may describe the state
       the pins should be in during system suspend.
 
+  power-domains:
+    maxItems: 1
+
 required:
   - compatible
   - reg
-- 
2.20.1

Re: [PATCH v1] dt-bindings: spi: rockchip: add power-domains property

Posted by Heiko Stübner 3 years, 6 months ago

Am Samstag, 10. September 2022, 20:56:28 CEST schrieb Johan Jonker:
> The Rockchip rk3399 TRM mentions that pd_sdioaudio includes
> sdio, spi, i2s and spdif. Add a power-domains property to
> reduce notifications with spi-rockchip.yaml.
> 
> Signed-off-by: Johan Jonker <jbx6244@gmail.com>

Acked-by: Heiko Stuebner <heiko@sntech.de>


> ---
>  Documentation/devicetree/bindings/spi/spi-rockchip.yaml | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/spi/spi-rockchip.yaml b/Documentation/devicetree/bindings/spi/spi-rockchip.yaml
> index 52768894b..66e49947b 100644
> --- a/Documentation/devicetree/bindings/spi/spi-rockchip.yaml
> +++ b/Documentation/devicetree/bindings/spi/spi-rockchip.yaml
> @@ -82,6 +82,9 @@ properties:
>        where the "sleep" configuration may describe the state
>        the pins should be in during system suspend.
>  
> +  power-domains:
> +    maxItems: 1
> +
>  required:
>    - compatible
>    - reg
>

Re: [PATCH v1] dt-bindings: spi: rockchip: add power-domains property

Posted by Krzysztof Kozlowski 3 years, 6 months ago

On 10/09/2022 20:56, Johan Jonker wrote:
> The Rockchip rk3399 TRM mentions that pd_sdioaudio includes
> sdio, spi, i2s and spdif. Add a power-domains property to
> reduce notifications with spi-rockchip.yaml.

Drop the last sentence - there are no notifications. Just write bindings
to match reality (hardware).

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

Best regards,
Krzysztof