Re: [PATCH RFT] leds: max8997: Simplify max8997_led_set_mode implementation

2012-07-24 Thread Bryan Wu
On Tue, Jul 24, 2012 at 2:29 PM, Axel Lin wrote: > It is not necessary to call max8997_update_reg() twice just for updating > MAX8997_REG_LEN_CNTL register. With proper val and mask arguments to > max8997_update_reg() call, this can be done in one max8997_update_reg() call. > And then we can remov

[PATCH RFT] leds: max8997: Simplify max8997_led_set_mode implementation

2012-07-23 Thread Axel Lin
It is not necessary to call max8997_update_reg() twice just for updating MAX8997_REG_LEN_CNTL register. With proper val and mask arguments to max8997_update_reg() call, this can be done in one max8997_update_reg() call. And then we can remove max8997_led_clear_mode() function. Signed-off-by: Axel