On 13/06/2023 12:19, Matti Vaittinen wrote: > The ROHM BU27010 is a sensor with 6 photodiodes (red, green, blue, clear, > IR and flickering detection) with five configurable channels. Red, green > and flickering detection being always available and two out of the rest > three (blue, clear, IR) can be selected to be simultaneously measured. > Typical application is adjusting LCD/OLED backlight of TVs, mobile phones > and tablet PCs. Please use subject prefixes matching the subsystem. You can get them for example with `git log --oneline -- DIRECTORY_OR_FILE` on the directory your patch is touching. e.g.: "dt-bindings: iio:" > > Add binding document for ROHM BU27010. > > Signed-off-by: Matti Vaittinen <mazziesaccount@xxxxxxxxx> > --- > .../bindings/iio/light/rohm,bu27010.yaml | 49 +++++++++++++++++++ > 1 file changed, 49 insertions(+) > create mode 100644 Documentation/devicetree/bindings/iio/light/rohm,bu27010.yaml > > diff --git a/Documentation/devicetree/bindings/iio/light/rohm,bu27010.yaml b/Documentation/devicetree/bindings/iio/light/rohm,bu27010.yaml > new file mode 100644 > index 000000000000..2bde9d2f1def > --- /dev/null > +++ b/Documentation/devicetree/bindings/iio/light/rohm,bu27010.yaml > @@ -0,0 +1,49 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/iio/light/rohm,bu27010.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: ROHM BU27010 color sensor > + > +maintainers: > + - Matti Vaittinen <mazziesaccount@xxxxxxxxx> > + > +description: | > + The ROHM BU27010 is a sensor with 6 photodiodes (red, green, blue, clear, > + IR and flickering detection) with five configurable channels. Red, green > + and flickering detection being always available and two out of the rest > + three (blue, clear, IR) can be selected to be simultaneously measured. > + Typical application is adjusting LCD/OLED backlight of TVs, mobile phones > + and tablet PCs. > + > +properties: > + compatible: > + const: rohm,bu27010 > + > + reg: > + maxItems: 1 > + > + interrupts: > + maxItems: 1 > + > + vdd-supply: true Isn't vdd-supply required for the hardware to work? How does it get the power otherwise? > + > +required: > + - compatible > + - reg > + > +additionalProperties: false > + > +examples: > + - | > + i2c { > + #address-cells = <1>; > + #size-cells = <0>; > + > + light-sensor@38 { > + compatible = "rohm,bu27010"; > + reg = <0x38>; > + }; > + }; > + Trailing blank line. Best regards, Krzysztof
