panel: Expand the backlight support
authorSimon Glass <sjg@chromium.org>
Mon, 1 Oct 2018 18:22:41 +0000 (12:22 -0600)
committerSimon Glass <sjg@chromium.org>
Tue, 9 Oct 2018 10:40:27 +0000 (04:40 -0600)
At present the panel can be turned on but not off, and the brightness
cannot be controlled at run-time. Add a new API function to both the panel
and backlight uclasses to handle this. Enhance the PWM backlight driver
to deal with custom levels properly and allow the backlight to be turned
on and off.

Update the test to cover thes new features.

Signed-off-by: Simon Glass <sjg@chromium.org>
drivers/video/backlight-uclass.c
drivers/video/panel-uclass.c
drivers/video/pwm_backlight.c
drivers/video/simple_panel.c
include/backlight.h
include/panel.h
test/dm/panel.c

index 92715e2f1303198d04c4c46f8355f698ff1108f9..0aadf8a1f9749112765899be217de1753a9234af 100644 (file)
@@ -18,6 +18,16 @@ int backlight_enable(struct udevice *dev)
        return ops->enable(dev);
 }
 
+int backlight_set_brightness(struct udevice *dev, int percent)
+{
+       const struct backlight_ops *ops = backlight_get_ops(dev);
+
+       if (!ops->set_brightness)
+               return -ENOSYS;
+
+       return ops->set_brightness(dev, percent);
+}
+
 UCLASS_DRIVER(backlight) = {
        .id             = UCLASS_PANEL_BACKLIGHT,
        .name           = "backlight",
index aec44a8bf7a2d333e6fe5d56b4953d046e986a58..246d1b2836d7071f3714d6711dbb265441632ccd 100644 (file)
@@ -18,6 +18,24 @@ int panel_enable_backlight(struct udevice *dev)
        return ops->enable_backlight(dev);
 }
 
+/**
+ * panel_set_backlight - Set brightness for the panel backlight
+ *
+ * @dev:       Panel device containing the backlight to update
+ * @percent:   Brightness value (0=off, 1=min brightness,
+ *             100=full brightness)
+ * @return 0 if OK, -ve on error
+ */
+int panel_set_backlight(struct udevice *dev, int percent)
+{
+       struct panel_ops *ops = panel_get_ops(dev);
+
+       if (!ops->set_backlight)
+               return -ENOSYS;
+
+       return ops->set_backlight(dev, percent);
+}
+
 int panel_get_display_timing(struct udevice *dev,
                             struct display_timing *timings)
 {
index 53953179bfbdf93aa4d87ab415661eb4620669a5..c13a90770909651f12d1c7bbbd5446f144b2d040 100644 (file)
@@ -4,6 +4,8 @@
  * Written by Simon Glass <sjg@chromium.org>
  */
 
+#define LOG_CATEGORY UCLASS_PANEL_BACKLIGHT
+
 #include <common.h>
 #include <dm.h>
 #include <backlight.h>
 #include <asm/gpio.h>
 #include <power/regulator.h>
 
+/**
+ * Private information for the PWM backlight
+ *
+ * If @num_levels is 0 then the levels are simple values with the backlight
+ * value going between the minimum (default 0) and the maximum (default 255).
+ * Otherwise the levels are an index into @levels (0..n-1).
+ *
+ * @reg: Regulator to enable to turn the backlight on (NULL if none)
+ * @enable, GPIO to set to enable the backlight (can be missing)
+ * @pwm: PWM to use to change the backlight brightness
+ * @channel: PWM channel to use
+ * @period_ns: Period of the backlight in nanoseconds
+ * @levels: Levels for the backlight, or NULL if not using indexed levels
+ * @num_levels: Number of levels
+ * @cur_level: Current level for the backlight (index or value)
+ * @default_level: Default level for the backlight (index or value)
+ * @min_level: Minimum level of the backlight (full off)
+ * @min_level: Maximum level of the backlight (full on)
+ * @enabled: true if backlight is enabled
+ */
 struct pwm_backlight_priv {
        struct udevice *reg;
        struct gpio_desc enable;
        struct udevice *pwm;
        uint channel;
        uint period_ns;
+       u32 *levels;
+       int num_levels;
        uint default_level;
+       int cur_level;
        uint min_level;
        uint max_level;
+       bool enabled;
 };
 
-static int pwm_backlight_enable(struct udevice *dev)
+static int set_pwm(struct pwm_backlight_priv *priv)
 {
-       struct pwm_backlight_priv *priv = dev_get_priv(dev);
-       struct dm_regulator_uclass_platdata *plat;
        uint duty_cycle;
        int ret;
 
-       if (priv->reg) {
-               plat = dev_get_uclass_platdata(priv->reg);
-               debug("%s: Enable '%s', regulator '%s'/'%s'\n", __func__,
-                     dev->name, priv->reg->name, plat->name);
-               ret = regulator_set_enable(priv->reg, true);
-               if (ret) {
-                       debug("%s: Cannot enable regulator for PWM '%s'\n",
-                             __func__, dev->name);
-                       return ret;
-               }
-               mdelay(120);
-       }
-
-       duty_cycle = priv->period_ns * (priv->default_level - priv->min_level) /
+       duty_cycle = priv->period_ns * (priv->cur_level - priv->min_level) /
                (priv->max_level - priv->min_level + 1);
        ret = pwm_set_config(priv->pwm, priv->channel, priv->period_ns,
                             duty_cycle);
+
+       return log_ret(ret);
+}
+
+static int enable_sequence(struct udevice *dev, int seq)
+{
+       struct pwm_backlight_priv *priv = dev_get_priv(dev);
+       int ret;
+
+       switch (seq) {
+       case 0:
+               if (priv->reg) {
+                       __maybe_unused struct dm_regulator_uclass_platdata
+                               *plat;
+
+                       plat = dev_get_uclass_platdata(priv->reg);
+                       log_debug("Enable '%s', regulator '%s'/'%s'\n",
+                                 dev->name, priv->reg->name, plat->name);
+                       ret = regulator_set_enable(priv->reg, true);
+                       if (ret) {
+                               log_debug("Cannot enable regulator for PWM '%s'\n",
+                                         __func__, dev->name);
+                               return log_ret(ret);
+                       }
+                       mdelay(120);
+               }
+               break;
+       case 1:
+               mdelay(10);
+               dm_gpio_set_value(&priv->enable, 1);
+               break;
+       }
+
+       return 0;
+}
+
+static int pwm_backlight_enable(struct udevice *dev)
+{
+       struct pwm_backlight_priv *priv = dev_get_priv(dev);
+       int ret;
+
+       ret = enable_sequence(dev, 0);
+       if (ret)
+               return log_ret(ret);
+       ret = set_pwm(priv);
        if (ret)
-               return ret;
+               return log_ret(ret);
        ret = pwm_set_enable(priv->pwm, priv->channel, true);
        if (ret)
-               return ret;
-       mdelay(10);
-       dm_gpio_set_value(&priv->enable, 1);
+               return log_ret(ret);
+       ret = enable_sequence(dev, 1);
+       if (ret)
+               return log_ret(ret);
+       priv->enabled = true;
+
+       return 0;
+}
+
+static int pwm_backlight_set_brightness(struct udevice *dev, int percent)
+{
+       struct pwm_backlight_priv *priv = dev_get_priv(dev);
+       bool disable = false;
+       int level;
+       int ret;
+
+       if (!priv->enabled) {
+               ret = enable_sequence(dev, 0);
+               if (ret)
+                       return log_ret(ret);
+       }
+       if (percent == BACKLIGHT_OFF) {
+               disable = true;
+               percent = 0;
+       }
+       if (percent == BACKLIGHT_DEFAULT) {
+               level = priv->default_level;
+       } else {
+               if (priv->levels) {
+                       level = priv->levels[percent * (priv->num_levels - 1)
+                               / 100];
+               } else {
+                       level = priv->min_level +
+                               (priv->max_level - priv->min_level) *
+                               percent / 100;
+               }
+       }
+       priv->cur_level = level;
+
+       ret = set_pwm(priv);
+       if (ret)
+               return log_ret(ret);
+       if (!priv->enabled) {
+               ret = enable_sequence(dev, 1);
+               if (ret)
+                       return log_ret(ret);
+               priv->enabled = true;
+       }
+       if (disable) {
+               dm_gpio_set_value(&priv->enable, 0);
+               if (priv->reg) {
+                       ret = regulator_set_enable(priv->reg, false);
+                       if (ret)
+                               return log_ret(ret);
+               }
+               priv->enabled = false;
+       }
 
        return 0;
 }
@@ -64,31 +174,32 @@ static int pwm_backlight_ofdata_to_platdata(struct udevice *dev)
        int index, ret, count, len;
        const u32 *cell;
 
-       debug("%s: start\n", __func__);
+       log_debug("start\n");
        ret = uclass_get_device_by_phandle(UCLASS_REGULATOR, dev,
                                           "power-supply", &priv->reg);
        if (ret)
-               debug("%s: Cannot get power supply: ret=%d\n", __func__, ret);
+               log_debug("Cannot get power supply: ret=%d\n", ret);
        ret = gpio_request_by_name(dev, "enable-gpios", 0, &priv->enable,
                                   GPIOD_IS_OUT);
        if (ret) {
-               debug("%s: Warning: cannot get enable GPIO: ret=%d\n",
-                     __func__, ret);
+               log_debug("Warning: cannot get enable GPIO: ret=%d\n", ret);
                if (ret != -ENOENT)
-                       return ret;
+                       return log_ret(ret);
        }
        ret = dev_read_phandle_with_args(dev, "pwms", "#pwm-cells", 0, 0,
                                         &args);
        if (ret) {
-               debug("%s: Cannot get PWM phandle: ret=%d\n", __func__, ret);
-               return ret;
+               log_debug("Cannot get PWM phandle: ret=%d\n", ret);
+               return log_ret(ret);
        }
 
        ret = uclass_get_device_by_ofnode(UCLASS_PWM, args.node, &priv->pwm);
        if (ret) {
-               debug("%s: Cannot get PWM: ret=%d\n", __func__, ret);
-               return ret;
+               log_debug("Cannot get PWM: ret=%d\n", ret);
+               return log_ret(ret);
        }
+       if (args.args_count < 2)
+               return log_msg_ret("Not enough arguments to pwm\n", -EINVAL);
        priv->channel = args.args[0];
        priv->period_ns = args.args[1];
 
@@ -96,13 +207,20 @@ static int pwm_backlight_ofdata_to_platdata(struct udevice *dev)
        cell = dev_read_prop(dev, "brightness-levels", &len);
        count = len / sizeof(u32);
        if (cell && count > index) {
-               priv->default_level = fdt32_to_cpu(cell[index]);
-               priv->max_level = fdt32_to_cpu(cell[count - 1]);
+               priv->levels = malloc(len);
+               if (!priv->levels)
+                       return log_ret(-ENOMEM);
+               dev_read_u32_array(dev, "brightness-levels", priv->levels,
+                                  count);
+               priv->num_levels = count;
+               priv->default_level = priv->levels[index];
+               priv->max_level = priv->levels[count - 1];
        } else {
                priv->default_level = index;
                priv->max_level = 255;
        }
-       debug("%s: done\n", __func__);
+       priv->cur_level = priv->default_level;
+       log_debug("done\n");
 
 
        return 0;
@@ -114,7 +232,8 @@ static int pwm_backlight_probe(struct udevice *dev)
 }
 
 static const struct backlight_ops pwm_backlight_ops = {
-       .enable = pwm_backlight_enable,
+       .enable         = pwm_backlight_enable,
+       .set_brightness = pwm_backlight_set_brightness,
 };
 
 static const struct udevice_id pwm_backlight_ids[] = {
index 6c604f9bedb6a20f566963cdcde02582236cd51c..7a968e740c5653b7fb08f1dcb5410bca38963373 100644 (file)
@@ -32,6 +32,21 @@ static int simple_panel_enable_backlight(struct udevice *dev)
        return 0;
 }
 
+static int simple_panel_set_backlight(struct udevice *dev, int percent)
+{
+       struct simple_panel_priv *priv = dev_get_priv(dev);
+       int ret;
+
+       debug("%s: start, backlight = '%s'\n", __func__, priv->backlight->name);
+       dm_gpio_set_value(&priv->enable, 1);
+       ret = backlight_set_brightness(priv->backlight, percent);
+       debug("%s: done, ret = %d\n", __func__, ret);
+       if (ret)
+               return ret;
+
+       return 0;
+}
+
 static int simple_panel_ofdata_to_platdata(struct udevice *dev)
 {
        struct simple_panel_priv *priv = dev_get_priv(dev);
@@ -51,7 +66,7 @@ static int simple_panel_ofdata_to_platdata(struct udevice *dev)
                                           "backlight", &priv->backlight);
        if (ret) {
                debug("%s: Cannot get backlight: ret=%d\n", __func__, ret);
-               return ret;
+               return log_ret(ret);
        }
        ret = gpio_request_by_name(dev, "enable-gpios", 0, &priv->enable,
                                   GPIOD_IS_OUT);
@@ -59,7 +74,7 @@ static int simple_panel_ofdata_to_platdata(struct udevice *dev)
                debug("%s: Warning: cannot get enable GPIO: ret=%d\n",
                      __func__, ret);
                if (ret != -ENOENT)
-                       return ret;
+                       return log_ret(ret);
        }
 
        return 0;
@@ -82,6 +97,7 @@ static int simple_panel_probe(struct udevice *dev)
 
 static const struct panel_ops simple_panel_ops = {
        .enable_backlight       = simple_panel_enable_backlight,
+       .set_backlight          = simple_panel_set_backlight,
 };
 
 static const struct udevice_id simple_panel_ids[] = {
index a304c36e019817296481baa66b1241605e136102..ac59eb293b45de51686fa8c033709077bdf0465b 100644 (file)
@@ -7,6 +7,13 @@
 #ifndef _BACKLIGHT_H
 #define _BACKLIGHT_H
 
+enum {
+       BACKLIGHT_MAX           = 100,
+       BACKLIGHT_MIN           = 0,
+       BACKLIGHT_OFF           = -1,
+       BACKLIGHT_DEFAULT       = -2,
+};
+
 struct backlight_ops {
        /**
         * enable() - Enable a backlight
@@ -15,6 +22,15 @@ struct backlight_ops {
         * @return 0 if OK, -ve on error
         */
        int (*enable)(struct udevice *dev);
+
+       /**
+        * set_brightness - Set brightness
+        *
+        * @dev:        Backlight device to update
+        * @percent:    Brightness value (0 to 100, or BACKLIGHT_... value)
+        * @return 0 if OK, -ve on error
+        */
+       int (*set_brightness)(struct udevice *dev, int percent);
 };
 
 #define backlight_get_ops(dev) ((struct backlight_ops *)(dev)->driver->ops)
@@ -27,4 +43,13 @@ struct backlight_ops {
  */
 int backlight_enable(struct udevice *dev);
 
+/**
+ * backlight_set_brightness - Set brightness
+ *
+ * @dev:       Backlight device to update
+ * @percent:   Brightness value (0 to 100, or BACKLIGHT_... value)
+ * @return 0 if OK, -ve on error
+ */
+int backlight_set_brightness(struct udevice *dev, int percent);
+
 #endif
index 6237d32657d9d1abf37f1e38052ffd6fad222f91..cd596d48c027e8561b9ac3408cbcb1d4cd29d027 100644 (file)
@@ -15,6 +15,16 @@ struct panel_ops {
         * @return 0 if OK, -ve on error
         */
        int (*enable_backlight)(struct udevice *dev);
+
+       /**
+        * set_backlight - Set panel backlight brightness
+        *
+        * @dev:        Panel device containing the backlight to update
+        * @percent:    Brightness value (0 to 100, or BACKLIGHT_... value)
+        * @return 0 if OK, -ve on error
+        */
+       int (*set_backlight)(struct udevice *dev, int percent);
+
        /**
         * get_timings() - Get display timings from panel.
         *
@@ -29,13 +39,23 @@ struct panel_ops {
 #define panel_get_ops(dev)     ((struct panel_ops *)(dev)->driver->ops)
 
 /**
- * panel_enable_backlight() - Enable the panel backlight
+ * panel_enable_backlight() - Enable/disable the panel backlight
  *
  * @dev:       Panel device containing the backlight to enable
+ * @enable:    true to enable the backlight, false to dis
  * @return 0 if OK, -ve on error
  */
 int panel_enable_backlight(struct udevice *dev);
 
+/**
+ * panel_set_backlight - Set brightness for the panel backlight
+ *
+ * @dev:       Panel device containing the backlight to update
+ * @percent:   Brightness value (0 to 100, or BACKLIGHT_... value)
+ * @return 0 if OK, -ve on error
+ */
+int panel_set_backlight(struct udevice *dev, int percent);
+
 /**
  * panel_get_display_timing() - Get display timings from panel.
  *
index ca032409f8c911f6bee472a68baa36e9f0407a4e..7e4ebd6d81e6f6163878f07919cd92873d5b2fa8 100644 (file)
@@ -45,6 +45,35 @@ static int dm_test_panel(struct unit_test_state *uts)
        ut_asserteq(1, sandbox_gpio_get_value(gpio, 1));
        ut_asserteq(true, regulator_get_enable(reg));
 
+       ut_assertok(panel_set_backlight(dev, 40));
+       ut_assertok(sandbox_pwm_get_config(pwm, 0, &period_ns, &duty_ns,
+                                          &enable, &polarity));
+       ut_asserteq(64 * 1000 / 256, duty_ns);
+
+       ut_assertok(panel_set_backlight(dev, BACKLIGHT_MAX));
+       ut_assertok(sandbox_pwm_get_config(pwm, 0, &period_ns, &duty_ns,
+                                          &enable, &polarity));
+       ut_asserteq(255 * 1000 / 256, duty_ns);
+
+       ut_assertok(panel_set_backlight(dev, BACKLIGHT_MIN));
+       ut_assertok(sandbox_pwm_get_config(pwm, 0, &period_ns, &duty_ns,
+                                          &enable, &polarity));
+       ut_asserteq(0 * 1000 / 256, duty_ns);
+       ut_asserteq(1, sandbox_gpio_get_value(gpio, 1));
+
+       ut_assertok(panel_set_backlight(dev, BACKLIGHT_DEFAULT));
+       ut_assertok(sandbox_pwm_get_config(pwm, 0, &period_ns, &duty_ns,
+                                          &enable, &polarity));
+       ut_asserteq(true, enable);
+       ut_asserteq(170 * 1000 / 256, duty_ns);
+
+       ut_assertok(panel_set_backlight(dev, BACKLIGHT_OFF));
+       ut_assertok(sandbox_pwm_get_config(pwm, 0, &period_ns, &duty_ns,
+                                          &enable, &polarity));
+       ut_asserteq(0 * 1000 / 256, duty_ns);
+       ut_asserteq(0, sandbox_gpio_get_value(gpio, 1));
+       ut_asserteq(false, regulator_get_enable(reg));
+
        return 0;
 }
 DM_TEST(dm_test_panel, DM_TESTF_SCAN_PDATA | DM_TESTF_SCAN_FDT);