diff options
| author |   Axel Lin <axel.lin@ingics.com> | 2015-05-16 22:08:10 +0800 |
|---|---|---|
| committer |   Lee Jones <lee.jones@linaro.org> | 2015-06-23 15:47:34 +0100 |
| commit | cdaefccefa988495e732d85d81914fbba99f4ca6 (patch) | |
| tree | 8aea93f7163c652a3b485bc91e14bd7fd1a6be93 | |
| parent | backlight: lp855x: Don't clear level on suspend/blank (diff) | |
| download | linux-dev-cdaefccefa988495e732d85d81914fbba99f4ca6.tar.xz linux-dev-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>
| -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); |