diff mbox series

[1/4] dt-bindings: iio: light: stk33xx: add regulator for vdd supply

Message ID 20240414175716.958831-1-aren@peacevolution.org (mailing list archive)
State New, archived
Headers show
Series iio: light: stk3310: support powering off during suspend | expand

Commit Message

Aren April 14, 2024, 5:57 p.m. UTC
Signed-off-by: Aren Moynihan <aren@peacevolution.org>
---
 Documentation/devicetree/bindings/iio/light/stk33xx.yaml | 1 +
 1 file changed, 1 insertion(+)

Comments

Krzysztof Kozlowski April 14, 2024, 6:39 p.m. UTC | #1
On 14/04/2024 19:57, Aren Moynihan wrote:
> Signed-off-by: Aren Moynihan <aren@peacevolution.org>
> ---

Why? Please provide proper commit msg which will explain why you are
doing it (e.g. say about hardware).

Anyway empty commit msgs cannot be accepted.



Best regards,
Krzysztof
Jernej Škrabec April 15, 2024, 8:46 p.m. UTC | #2
Dne nedelja, 14. april 2024 ob 19:57:13 GMT +2 je Aren Moynihan napisal(a):
> Signed-off-by: Aren Moynihan <aren@peacevolution.org>

Commit message cannot be empty.

Best regards,
Jernej

> ---
>  Documentation/devicetree/bindings/iio/light/stk33xx.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/iio/light/stk33xx.yaml b/Documentation/devicetree/bindings/iio/light/stk33xx.yaml
> index f6e22dc9814a..db35e239d4a8 100644
> --- a/Documentation/devicetree/bindings/iio/light/stk33xx.yaml
> +++ b/Documentation/devicetree/bindings/iio/light/stk33xx.yaml
> @@ -29,6 +29,7 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> +  vdd-supply: true
>    proximity-near-level: true
>  
>  required:
>
Jonathan Cameron April 20, 2024, 1:05 p.m. UTC | #3
On Sun, 14 Apr 2024 13:57:13 -0400
Aren Moynihan <aren@peacevolution.org> wrote:

> Signed-off-by: Aren Moynihan <aren@peacevolution.org>
> ---
>  Documentation/devicetree/bindings/iio/light/stk33xx.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/iio/light/stk33xx.yaml b/Documentation/devicetree/bindings/iio/light/stk33xx.yaml
> index f6e22dc9814a..db35e239d4a8 100644
> --- a/Documentation/devicetree/bindings/iio/light/stk33xx.yaml
> +++ b/Documentation/devicetree/bindings/iio/light/stk33xx.yaml
> @@ -29,6 +29,7 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> +  vdd-supply: true

As per review of patch 2, make it required and add one to the example.
That doesn't mean it will be in every dts file, just that we expect
it to exist given chips tend to work poorly without power.

>    proximity-near-level: true
>  
>  required:
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/iio/light/stk33xx.yaml b/Documentation/devicetree/bindings/iio/light/stk33xx.yaml
index f6e22dc9814a..db35e239d4a8 100644
--- a/Documentation/devicetree/bindings/iio/light/stk33xx.yaml
+++ b/Documentation/devicetree/bindings/iio/light/stk33xx.yaml
@@ -29,6 +29,7 @@  properties:
   interrupts:
     maxItems: 1
 
+  vdd-supply: true
   proximity-near-level: true
 
 required: