Re: [PATCH 1/2] iio: pressure: st_pressure: fix drdy configuration for LPS22HB and LPS25H

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 11 Aug 2017 21:44:13 +0200
Lorenzo Bianconi <lorenzo.bianconi83@xxxxxxxxx> wrote:

> Remove int2 configuration parameter for LPS22HB and LPS25H since
> these devices export just int1 as data-ready line
> 
> Fixes: 931878405b86 (iio:pressure: Add support for LPS25H pressure sensor)
> Fixes: e039e2f5b4da (iio:st_pressure:initial lps22hb sensor support)
> Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@xxxxxx>
While this is clearly a fix, as I understand it you would also need to have
some crazy configuration to make this a problem so I am going to take it
for the next merge window rather than quicker.

Applied to the togreg branch of iio.git and pushed out as testing for
the autobuilders to play with it.

Thanks,

Jonathan
> ---
>  drivers/iio/pressure/st_pressure_core.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/iio/pressure/st_pressure_core.c b/drivers/iio/pressure/st_pressure_core.c
> index aa61ec15c139..c84eea73b3ae 100644
> --- a/drivers/iio/pressure/st_pressure_core.c
> +++ b/drivers/iio/pressure/st_pressure_core.c
> @@ -390,7 +390,7 @@ static const struct st_sensor_settings st_press_sensors_settings[] = {
>  		.drdy_irq = {
>  			.addr = 0x23,
>  			.mask_int1 = 0x01,
> -			.mask_int2 = 0x10,
> +			.mask_int2 = 0x00,
>  			.addr_ihl = 0x22,
>  			.mask_ihl = 0x80,
>  			.addr_od = 0x22,
> @@ -449,7 +449,7 @@ static const struct st_sensor_settings st_press_sensors_settings[] = {
>  		.drdy_irq = {
>  			.addr = 0x12,
>  			.mask_int1 = 0x04,
> -			.mask_int2 = 0x08,
> +			.mask_int2 = 0x00,
>  			.addr_ihl = 0x12,
>  			.mask_ihl = 0x80,
>  			.addr_od = 0x12,

--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux