Re: [PATCH 2/2] staging: iio: imu: Add CEVA BNO08x driver

Greg Kroah-Hartman
2022-06-16T10:26:27Z

On Thu, Jun 16, 2022 at 12:00:06PM +0200, Jacopo Mondi wrote:
> From: Jacopo Mondi <jacopo@jmondi.org>
>
> 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@jmondi.org>
> ---
> 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?

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