[PATCH] dt-bindings: iio: rohm,bu27010: add missing vdd-supply to example

Krzysztof Kozlowski posted 1 patch 2 years, 1 month ago
Documentation/devicetree/bindings/iio/light/rohm,bu27010.yaml | 1 +
1 file changed, 1 insertion(+)
[PATCH] dt-bindings: iio: rohm,bu27010: add missing vdd-supply to example
Posted by Krzysztof Kozlowski 2 years, 1 month ago
Bindings require vdd-supply but the example DTS was missing one.  This
fixes dt_binding_check error:

  Documentation/devicetree/bindings/iio/light/rohm,bu27010.example.dtb: light-sensor@38: 'vdd-supply' is a required property

Fixes: ae8c47c51e5c ("dt-bindings: iio: ROHM BU27010 RGBC + flickering sensor")
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 Documentation/devicetree/bindings/iio/light/rohm,bu27010.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/iio/light/rohm,bu27010.yaml b/Documentation/devicetree/bindings/iio/light/rohm,bu27010.yaml
index 8376d64a641a..bed42d5d0d94 100644
--- a/Documentation/devicetree/bindings/iio/light/rohm,bu27010.yaml
+++ b/Documentation/devicetree/bindings/iio/light/rohm,bu27010.yaml
@@ -45,5 +45,6 @@ examples:
       light-sensor@38 {
         compatible = "rohm,bu27010";
         reg = <0x38>;
+        vdd-supply = <&vdd>;
       };
     };
-- 
2.34.1
Re: [PATCH] dt-bindings: iio: rohm,bu27010: add missing vdd-supply to example
Posted by Rob Herring 2 years, 1 month ago
On Tue, Aug 08, 2023 at 08:32:23AM +0200, Krzysztof Kozlowski wrote:
> Bindings require vdd-supply but the example DTS was missing one.  This
> fixes dt_binding_check error:
> 
>   Documentation/devicetree/bindings/iio/light/rohm,bu27010.example.dtb: light-sensor@38: 'vdd-supply' is a required property
> 
> Fixes: ae8c47c51e5c ("dt-bindings: iio: ROHM BU27010 RGBC + flickering sensor")
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  Documentation/devicetree/bindings/iio/light/rohm,bu27010.yaml | 1 +
>  1 file changed, 1 insertion(+)

Acked-by: Rob Herring <robh@kernel.org>
Re: [PATCH] dt-bindings: iio: rohm,bu27010: add missing vdd-supply to example
Posted by Jonathan Cameron 2 years ago
On Fri, 11 Aug 2023 09:17:55 -0600
Rob Herring <robh@kernel.org> wrote:

> On Tue, Aug 08, 2023 at 08:32:23AM +0200, Krzysztof Kozlowski wrote:
> > Bindings require vdd-supply but the example DTS was missing one.  This
> > fixes dt_binding_check error:
> > 
> >   Documentation/devicetree/bindings/iio/light/rohm,bu27010.example.dtb: light-sensor@38: 'vdd-supply' is a required property
> > 
> > Fixes: ae8c47c51e5c ("dt-bindings: iio: ROHM BU27010 RGBC + flickering sensor")
> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> > ---
> >  Documentation/devicetree/bindings/iio/light/rohm,bu27010.yaml | 1 +
> >  1 file changed, 1 insertion(+)  
> 
> Acked-by: Rob Herring <robh@kernel.org>
Applied to the fixes-togreg branch of iio.git.
Note I want to rebase that on rc1 before sending next pull request so it will be a week or two.

Thanks,

Jonathan
Re: [PATCH] dt-bindings: iio: rohm,bu27010: add missing vdd-supply to example
Posted by Matti Vaittinen 2 years, 1 month ago
On 8/8/23 09:32, Krzysztof Kozlowski wrote:
> Bindings require vdd-supply but the example DTS was missing one.  This
> fixes dt_binding_check error:
> 
>    Documentation/devicetree/bindings/iio/light/rohm,bu27010.example.dtb: light-sensor@38: 'vdd-supply' is a required property
> 
> Fixes: ae8c47c51e5c ("dt-bindings: iio: ROHM BU27010 RGBC + flickering sensor")
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Ah. Added supply to required ones when you pointed out it should be - 
and obviously didn't fix the example :/

Thanks a lot for fixing it!

Reviewed-by: Matti Vaittinen <mazziesaccount@gmail.com>

-- 
Matti Vaittinen
Linux kernel developer at ROHM Semiconductors
Oulu Finland

~~ When things go utterly wrong vim users can always type :help! ~~