Re: [PATCH 05/10] staging: iio: tsl2583: check return values from taos_chip_{on,off}

2016-10-30 Thread Jonathan Cameron
On 28/10/16 11:00, Brian Masney wrote: > The return value from taos_chip_on() and taos_chip_off() was not > checked in taos_luxtable_store() and taos_probe(). This patch adds > proper error checking to these function calls. > > Signed-off-by: Brian Masney This does raise

Re: [PATCH 05/10] staging: iio: tsl2583: check return values from taos_chip_{on,off}

2016-10-30 Thread Jonathan Cameron
On 28/10/16 11:00, Brian Masney wrote: > The return value from taos_chip_on() and taos_chip_off() was not > checked in taos_luxtable_store() and taos_probe(). This patch adds > proper error checking to these function calls. > > Signed-off-by: Brian Masney This does raise the question of whether

[PATCH 05/10] staging: iio: tsl2583: check return values from taos_chip_{on,off}

2016-10-28 Thread Brian Masney
The return value from taos_chip_on() and taos_chip_off() was not checked in taos_luxtable_store() and taos_probe(). This patch adds proper error checking to these function calls. Signed-off-by: Brian Masney --- drivers/staging/iio/light/tsl2583.c | 26

[PATCH 05/10] staging: iio: tsl2583: check return values from taos_chip_{on,off}

2016-10-28 Thread Brian Masney
The return value from taos_chip_on() and taos_chip_off() was not checked in taos_luxtable_store() and taos_probe(). This patch adds proper error checking to these function calls. Signed-off-by: Brian Masney --- drivers/staging/iio/light/tsl2583.c | 26 ++ 1 file changed,