Re: [PATCH v3] iio: imu: st_lsm6dsx: drop huge include in sensor-hub driver

[Jonathan Cameron <jic23@kernel.org>]

On Mon,  6 Apr 2020 21:00:25 +0200
Lorenzo Bianconi <lorenzo@kernel.org> 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@gmail.com>
> Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org>
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,