catie-aq / zephyr_zest-sensor-imu

Zest_Sensor_IMU shield for Zephyr OS
0 stars 0 forks source link

Add BNO055 sensor to the shield #3

Open Sokario opened 2 months ago

Sokario commented 2 months ago

I added BNO055 sensor to the DT. However I was wondering how to made optional I2C address available without the need to change app.overlay

@bperraudin I propose the following:

To do so I defined to BNO055 in the DT each one on a different I2C address here and here. Each one got the property use_xtal = <1> Aliases are created here to allow the use of :

static const struct device bno = DEVICE_DT_GET(DT_ALIAS(bno055)); or static const struct device bno = DEVICE_DT_GET(DT_ALIAS(bno055_alt));

If the user remove the Xtal, i.e do not want to use the shield for the designed use case. He would have to change use_xtal property in the app.overlay.

@bperraudin What do you think of this proposition ?

bperraudin commented 2 days ago

@Sokario I think we should "just" implement the default configuration.

It would be the responsibility of the end user to change the address if necessary.