diff options
author | Sascha Hauer <s.hauer@pengutronix.de> | 2016-08-29 14:47:02 +0200 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2016-08-29 17:36:36 +0200 |
commit | ac20c4f03b70bc5ea76681690da767fe367c34ae (patch) | |
tree | 76d1f5f355bcc7668800932c3b9fb8ee2d4f3f3f /drivers/video | |
parent | d3700c0e818e20e0654338f7e25f147072bff991 (diff) | |
download | barebox-ac20c4f03b70bc5ea76681690da767fe367c34ae.tar.gz barebox-ac20c4f03b70bc5ea76681690da767fe367c34ae.tar.xz |
video: backlight: fix the value of 'brightness_max'
Fixes: 87c6a88 video/backlight-pwm: fix the value of 'brightness_max'.
brightness_max should really contain the number of brightness steps, so
the number of elements in the brightness array.
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'drivers/video')
-rw-r--r-- | drivers/video/backlight-pwm.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/video/backlight-pwm.c b/drivers/video/backlight-pwm.c index 7f1715ddde..2915dba174 100644 --- a/drivers/video/backlight-pwm.c +++ b/drivers/video/backlight-pwm.c @@ -156,6 +156,8 @@ static int pwm_backlight_parse_dt(struct device_d *dev, if (ret < 0) return ret; + pwm_backlight->backlight.brightness_max = length - 1; + for (i = 0; i < length; i++) if (pwm_backlight->levels[i] > pwm_backlight->scale) pwm_backlight->scale = pwm_backlight->levels[i]; @@ -165,8 +167,8 @@ static int pwm_backlight_parse_dt(struct device_d *dev, } else { /* We implicitly assume here a linear levels array { 0, 1, 2, ... 100 } */ pwm_backlight->scale = 100; + pwm_backlight->backlight.brightness_max = pwm_backlight->scale; } - pwm_backlight->backlight.brightness_max = pwm_backlight->scale; pwm_backlight->enable_gpio = of_get_named_gpio_flags(node, "enable-gpios", 0, &flags); |