Re: [PATCH 1/2] iio: st_pressure: Fix data sign

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

 



On 13/12/16 08:51, Linus Walleij wrote:
> On Mon, Dec 12, 2016 at 5:58 PM, Marcin Niestroj
> <m.niestroj@xxxxxxxxxxxxxxxx> wrote:
> 
>> Datasheet of each device (lps331ap, lps25h, lps001wp, lps22hb) says that
>> the pressure and temperature data is a 2's complement.
>>
>> Fixes: 217494e5b780 ("iio:pressure: Add STMicroelectronics pressures driver")
>> Fixes: 2f5effcbd097 ("iio: pressure-core: st: Expand and rename LPS331AP's channel descriptor")
>> Fixes: 7885a8ce6800 ("iio: pressure: st: Add support for new LPS001WP pressure sensor")
>> Fixes: e039e2f5b4da ("iio:st_pressure:initial lps22hb sensor support")
>> Signed-off-by: Marcin Niestroj <m.niestroj@xxxxxxxxxxxxxxxx>
> 
> Looks correct. I guess noone thought of testing negative pressures :/
> 
> Reviewed-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
I've applied to the togreg branch of iio.git - initially pushed out as testing
because the very nature of a fix for multiple previous patches will make it tricky
to apply to older kernels.

If you want to do backports after it has gone into mainline, they'll be most
welcome.

Thanks,

Jonathan
> 
> Yours,
> Linus Walleij
> 

--
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