diff options
author | Axel Lin <axel.lin@ingics.com> | 2015-05-16 16:08:10 +0200 |
---|---|---|
committer | Lee Jones <lee.jones@linaro.org> | 2015-06-23 16:47:34 +0200 |
commit | cdaefccefa988495e732d85d81914fbba99f4ca6 (patch) | |
tree | 8aea93f7163c652a3b485bc91e14bd7fd1a6be93 /drivers/video | |
parent | backlight: lp855x: Don't clear level on suspend/blank (diff) | |
download | linux-cdaefccefa988495e732d85d81914fbba99f4ca6.tar.xz linux-cdaefccefa988495e732d85d81914fbba99f4ca6.zip |
backlight: pwm_bl: Simplify usage of devm_gpiod_get_optional
Since 39b2bbe3d715 (gpio: add flags argument to gpiod_get*() functions),
the gpiod_get* functions take an additional parameter that allows to
specify direction and initial value for output.
Simplify the usage of devm_gpiod_get_optional accordingly.
Signed-off-by: Axel Lin <axel.lin@ingics.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
Diffstat (limited to 'drivers/video')
-rw-r--r-- | drivers/video/backlight/pwm_bl.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/video/backlight/pwm_bl.c b/drivers/video/backlight/pwm_bl.c index 3a145a643e0d..57cb9ec8be43 100644 --- a/drivers/video/backlight/pwm_bl.c +++ b/drivers/video/backlight/pwm_bl.c @@ -241,7 +241,8 @@ static int pwm_backlight_probe(struct platform_device *pdev) pb->dev = &pdev->dev; pb->enabled = false; - pb->enable_gpio = devm_gpiod_get_optional(&pdev->dev, "enable"); + pb->enable_gpio = devm_gpiod_get_optional(&pdev->dev, "enable", + GPIOD_OUT_HIGH); if (IS_ERR(pb->enable_gpio)) { ret = PTR_ERR(pb->enable_gpio); goto err_alloc; @@ -263,9 +264,6 @@ static int pwm_backlight_probe(struct platform_device *pdev) pb->enable_gpio = gpio_to_desc(data->enable_gpio); } - if (pb->enable_gpio) - gpiod_direction_output(pb->enable_gpio, 1); - pb->power_supply = devm_regulator_get(&pdev->dev, "power"); if (IS_ERR(pb->power_supply)) { ret = PTR_ERR(pb->power_supply); |