On 14/04/17 07:21, Matt Ranostay wrote:
> AS3935_WRITE_DATA macro bit is incorrect and the actual write
> sequence is two leading zeros.
>
> Cc: George McCollister <george.mccollister@xxxxxxxxx>
> Signed-off-by: Matt Ranostay <matt.ranostay@xxxxxxxxxxxx>
Applied to the fixes-togreg branch of iio.git and marked
for stable.
Thanks,
Jonathan
> ---
> drivers/iio/proximity/as3935.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/iio/proximity/as3935.c b/drivers/iio/proximity/as3935.c
> index 5656deb17261..020459513384 100644
> --- a/drivers/iio/proximity/as3935.c
> +++ b/drivers/iio/proximity/as3935.c
> @@ -50,7 +50,6 @@
> #define AS3935_TUNE_CAP 0x08
> #define AS3935_CALIBRATE 0x3D
>
> -#define AS3935_WRITE_DATA BIT(15)
> #define AS3935_READ_DATA BIT(14)
> #define AS3935_ADDRESS(x) ((x) << 8)
>
> @@ -105,7 +104,7 @@ static int as3935_write(struct as3935_state *st,
> {
> u8 *buf = st->buf;
>
> - buf[0] = (AS3935_WRITE_DATA | AS3935_ADDRESS(reg)) >> 8;
> + buf[0] = AS3935_ADDRESS(reg) >> 8;
> buf[1] = val;
>
> return spi_write(st->spi, buf, 2);
>
--
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
