| Message ID | 20230808063223.80431-1-krzysztof.kozlowski@linaro.org (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
| Series | dt-bindings: iio: rohm,bu27010: add missing vdd-supply to example | expand |
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>
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>
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
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>; }; };
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(+)