On Thu, 15 Jul 2021 16:17:41 +0200 Andrea Merello <andrea.merello@xxxxxxxxx> wrote: > Introduce new documentation file for the BNO055 serdev driver that will dt bindings are for the device not the driver (so don't mention driver in the binding or the patch description). > be included in next patches of this same series > > Signed-off-by: Andrea Merello <andrea.merello@xxxxxx> > Cc: Andrea Merello <andrea.merello@xxxxxxxxx> > Cc: Rob Herring <robh+dt@xxxxxxxxxx> > Cc: Matt Ranostay <matt.ranostay@xxxxxxxxxxxx> > Cc: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > Cc: Vlad Dogaru <vlad.dogaru@xxxxxxxxx> > Cc: linux-kernel@xxxxxxxxxxxxxxx > Cc: linux-iio@xxxxxxxxxxxxxxx > --- > .../bindings/iio/imu/bosch,bno055-serial.yaml | 40 +++++++++++++++++++ > 1 file changed, 40 insertions(+) > create mode 100644 Documentation/devicetree/bindings/iio/imu/bosch,bno055-serial.yaml > > diff --git a/Documentation/devicetree/bindings/iio/imu/bosch,bno055-serial.yaml b/Documentation/devicetree/bindings/iio/imu/bosch,bno055-serial.yaml > new file mode 100644 > index 000000000000..743c784ebc94 > --- /dev/null > +++ b/Documentation/devicetree/bindings/iio/imu/bosch,bno055-serial.yaml Better to have just one doc covering this interface and i2c if that gets added. > @@ -0,0 +1,40 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/iio/imu/bosch,bno055-serial.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Serial-attached Bosch BNO055 > + > +maintainers: > + - Jonathan Cameron <jic23@xxxxxxxxxx> That's just mean! I have plenty of these to look after already! Joking aside, you'd be a better maintainer for this than me as more likely to pay attention. > + > +description: | > + Inertial Measurement Unit with Accelerometer, Gyroscope, Magnetometer and > + internal MCU for sensor fusion > + https://www.bosch-sensortec.com/products/smart-sensors/bno055/ > + > +properties: > + compatible: > + enum: > + - bosch,bno055-serial > + > + reset-gpios: > + maxItems: 1 > + > + clocks: > + maxItems: 1 > + > +required: > + - compatible > + > +additionalProperties: false > + > +examples: > + - | > + #include <dt-bindings/gpio/gpio.h> > + bno055 { name needs to be the one for the device type found in the device tree spec or if it's not there, something in same 'spirit'. Probably imu here > + compatible = "bosch,bno055-serial"; Don't need the -serial. It will bind based on the bus this is under. Speaking of which, it's normal to provide that bus info as part of the example. See for example chemical/sensiron,scd30.yaml > + reset-gpios = <&gpio0 54 GPIO_ACTIVE_LOW>; > + clocks = <&imu_clk>; > + };
