Re: [PATCH 01/19] staging: iio: isl29028: remove nested if statements

2016-12-04 Thread Jonathan Cameron
On 04/12/16 02:19, Brian Masney wrote: > There are two callers to the function isl29028_set_als_ir_mode() and > both instances use a nested if statement to only change the chip state > if it is not in the proper mode. This patch moves this check into the > isl29028_set_als_ir_mode() function to

Re: [PATCH 01/19] staging: iio: isl29028: remove nested if statements

2016-12-04 Thread Jonathan Cameron
On 04/12/16 02:19, Brian Masney wrote: > There are two callers to the function isl29028_set_als_ir_mode() and > both instances use a nested if statement to only change the chip state > if it is not in the proper mode. This patch moves this check into the > isl29028_set_als_ir_mode() function to

[PATCH 01/19] staging: iio: isl29028: remove nested if statements

2016-12-03 Thread Brian Masney
There are two callers to the function isl29028_set_als_ir_mode() and both instances use a nested if statement to only change the chip state if it is not in the proper mode. This patch moves this check into the isl29028_set_als_ir_mode() function to remove the nested if statements. Signed-off-by:

[PATCH 01/19] staging: iio: isl29028: remove nested if statements

2016-12-03 Thread Brian Masney
There are two callers to the function isl29028_set_als_ir_mode() and both instances use a nested if statement to only change the chip state if it is not in the proper mode. This patch moves this check into the isl29028_set_als_ir_mode() function to remove the nested if statements. Signed-off-by: