leds: leds-ns2: set devm_gpio_request_one() flags param correctly
authorJingoo Han <jg1.han@samsung.com>
Fri, 8 Mar 2013 02:38:26 +0000 (18:38 -0800)
committerBryan Wu <cooloney@gmail.com>
Mon, 1 Apr 2013 18:04:49 +0000 (11:04 -0700)
commit9d04cbaadf563db3ba04426c317d3e84bc27bac1
tree80c4d0c778ce746e145307ddb1b6b66bcf35a0ac
parent84f6942cde279e812c59eed456d1f43a39ca6c40
leds: leds-ns2: set devm_gpio_request_one() flags param correctly

The devm_gpio_request_one() flags parameter was set to:

  GPIOF_DIR_OUT | gpio_get_value(template->cmd)

GPIOF_DIR_OUT and GPIOF_DIR_IN are defined as below:

  GPIOF_DIR_OUT   (0 << 0)
  GPIOF_DIR_IN    (1 << 0)

So, when 'gpio_get_value(template->cmd)' is 1, the gpio pin can
be set as input, instead of output.

To prevent this problem, GPIOF_OUT_INIT flags should be used when
using devm_gpio_request_one().

Same goes for 'gpio_get_value(template->slow)' case.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Signed-off-by: Bryan Wu <cooloney@gmail.com>
drivers/leds/leds-ns2.c