On Mon, 6 Apr 2020 21:00:25 +0200
Lorenzo Bianconi <lorenzo@xxxxxxxxxx> wrote:
> st_lsm6dsx is a standalone driver for STM IMU sensors and does not rely
> on st_sensor common framework, so it does not include st_sensor common
> definitions.
> In st_lsm6dsx_shub driver st_sensors.h is used just to introduce the
> default wai address for LIS3MDL sensor.
> Drop this largely unconnected include file and introduce the default wai
> address for LIS3MDL in st_lsm6dsx_ext_dev_settings register map
>
> Reviewed-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx>
> Signed-off-by: Lorenzo Bianconi <lorenzo@xxxxxxxxxx>
Applied to the togreg branch of iio.git and pushed out as testing for
the autobuilders to play with it.
Thanks,
Jonathan
> ---
> Changes since v2:
> - rewrite commit message
>
> Changes since v1:
> - improve commit message
> ---
> drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_shub.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_shub.c b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_shub.c
> index 280925dd8edb..947ca3a7dcaf 100644
> --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_shub.c
> +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_shub.c
> @@ -28,7 +28,6 @@
> #include <linux/iio/sysfs.h>
> #include <linux/bitfield.h>
>
> -#include <linux/iio/common/st_sensors.h>
> #include "st_lsm6dsx.h"
>
> #define ST_LSM6DSX_SLV_ADDR(n, base) ((base) + (n) * 3)
> @@ -93,7 +92,7 @@ static const struct st_lsm6dsx_ext_dev_settings st_lsm6dsx_ext_dev_table[] = {
> {
> .i2c_addr = { 0x1e },
> .wai = {
> - .addr = ST_SENSORS_DEFAULT_WAI_ADDRESS,
> + .addr = 0x0f,
> .val = 0x3d,
> },
> .id = ST_LSM6DSX_ID_MAGN,
