Hi Greg On Thu, Jun 16, 2022 at 12:26:18PM +0200, Greg Kroah-Hartman wrote: > On Thu, Jun 16, 2022 at 12:00:06PM +0200, Jacopo Mondi wrote: > > From: Jacopo Mondi <jacopo@xxxxxxxxxx> > > > > Add support for CEVA BNO08x Sensor Hub. > > > > The BNO08X family (BNO080/85/86) is a System in Package (SiP) that > > integrates a triaxial accelerometer, triaxial gyroscope, magnetometer > > and a 32-bit ARM Cortex-M0+ microcontroller running CEVA's SH-2 > > firmware. > > > > Datasheet: > > https://www.ceva-dsp.com/wp-content/uploads/2019/10/BNO080_085-Datasheet.pdf > > > > Signed-off-by: Jacopo Mondi <jacopo@xxxxxxxxxx> > > --- > > MAINTAINERS | 7 + > > drivers/staging/iio/Kconfig | 1 + > > drivers/staging/iio/Makefile | 1 + > > drivers/staging/iio/imu/bno08x/Kconfig | 11 + > > drivers/staging/iio/imu/bno08x/Makefile | 3 + > > drivers/staging/iio/imu/bno08x/bno08x.c | 618 ++++++++++++++++++++++++ > > Why are you adding a new driver to staging/iio/ and not to the real part > of the kernel? What needs to be done to it to get it out of here? Mostly because I don't have access to the hw development platform anymore, and the one I had was not reliable enough for any further development anyway. I went for staging as I cannot guarantee I can test any requested change on the series. I'm afraid this is a "take it or leave it" series, and as the driver does anyway work (even if for a single feature) I thought it was worth sending it upstream anyway. > > I'd prefer that we not add any new iio drivers if at all possible, > please fix them up properly and merge them to the correct location. > > thanks, > > greg k-h