Re: [PATCH v4 15/26] staging: iio: tsl2583: moved code block inside else statement

2016-11-13 Thread Jonathan Cameron
On 12/11/16 18:19, Brian Masney wrote: > The check for ch1lux > ch0lux inside tsl2583_get_lux is only valid if > the ratio is not equal to zero. Move the code block inside the else > statement. This does away with the need to initialize the variables to > zero. > > Signed-off-by: Brian Masney

Re: [PATCH v4 15/26] staging: iio: tsl2583: moved code block inside else statement

2016-11-13 Thread Jonathan Cameron
On 12/11/16 18:19, Brian Masney wrote: > The check for ch1lux > ch0lux inside tsl2583_get_lux is only valid if > the ratio is not equal to zero. Move the code block inside the else > statement. This does away with the need to initialize the variables to > zero. > > Signed-off-by: Brian Masney

[PATCH v4 15/26] staging: iio: tsl2583: moved code block inside else statement

2016-11-12 Thread Brian Masney
The check for ch1lux > ch0lux inside tsl2583_get_lux is only valid if the ratio is not equal to zero. Move the code block inside the else statement. This does away with the need to initialize the variables to zero. Signed-off-by: Brian Masney ---

[PATCH v4 15/26] staging: iio: tsl2583: moved code block inside else statement

2016-11-12 Thread Brian Masney
The check for ch1lux > ch0lux inside tsl2583_get_lux is only valid if the ratio is not equal to zero. Move the code block inside the else statement. This does away with the need to initialize the variables to zero. Signed-off-by: Brian Masney --- drivers/staging/iio/light/tsl2583.c | 23