On Tue, Aug 21, 2012 at 5:04 PM, Kim, Milo wrote:
> (a) use LP5523_ENABLE rather than magic number 0x40
> (b) use min_t() in lp5523_mux_parse()
> (c) skip while loop and just return if invalid command
>
Thanks for this cleanup. I will merge it later.
-Bryan
> Signed-off-by: Milo(Woogyom) Kim
(a) use LP5523_ENABLE rather than magic number 0x40
(b) use min_t() in lp5523_mux_parse()
(c) skip while loop and just return if invalid command
Signed-off-by: Milo(Woogyom) Kim
---
drivers/leds/leds-lp5523.c | 15 +++
1 files changed, 7 insertions(+), 8 deletions(-)
diff --git
On Tue, Aug 21, 2012 at 5:04 PM, Kim, Milo milo@ti.com wrote:
(a) use LP5523_ENABLE rather than magic number 0x40
(b) use min_t() in lp5523_mux_parse()
(c) skip while loop and just return if invalid command
Thanks for this cleanup. I will merge it later.
-Bryan
Signed-off-by:
(a) use LP5523_ENABLE rather than magic number 0x40
(b) use min_t() in lp5523_mux_parse()
(c) skip while loop and just return if invalid command
Signed-off-by: Milo(Woogyom) Kim milo@ti.com
---
drivers/leds/leds-lp5523.c | 15 +++
1 files changed, 7 insertions(+), 8
4 matches
Mail list logo