Commit d90b3b6e authored by Sam Ravnborg's avatar Sam Ravnborg

drm/panel: olimex-lcd-olinuxino: use drm_panel backlight support

Use the backlight support in drm_panel to simplify the driver
Signed-off-by: default avatarSam Ravnborg <sam@ravnborg.org>
Reviewed-by: default avatarLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Stefan Mavrodiev <stefan@olimex.com>
Cc: Thierry Reding <thierry.reding@gmail.com>
Cc: Sam Ravnborg <sam@ravnborg.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20191207140353.23967-14-sam@ravnborg.org
parent 17d5183e
...@@ -6,7 +6,6 @@ ...@@ -6,7 +6,6 @@
* Author: Stefan Mavrodiev <stefan@olimex.com> * Author: Stefan Mavrodiev <stefan@olimex.com>
*/ */
#include <linux/backlight.h>
#include <linux/crc32.h> #include <linux/crc32.h>
#include <linux/gpio/consumer.h> #include <linux/gpio/consumer.h>
#include <linux/i2c.h> #include <linux/i2c.h>
...@@ -68,7 +67,6 @@ struct lcd_olinuxino { ...@@ -68,7 +67,6 @@ struct lcd_olinuxino {
bool prepared; bool prepared;
bool enabled; bool enabled;
struct backlight_device *backlight;
struct regulator *supply; struct regulator *supply;
struct gpio_desc *enable_gpio; struct gpio_desc *enable_gpio;
...@@ -87,8 +85,6 @@ static int lcd_olinuxino_disable(struct drm_panel *panel) ...@@ -87,8 +85,6 @@ static int lcd_olinuxino_disable(struct drm_panel *panel)
if (!lcd->enabled) if (!lcd->enabled)
return 0; return 0;
backlight_disable(lcd->backlight);
lcd->enabled = false; lcd->enabled = false;
return 0; return 0;
...@@ -134,8 +130,6 @@ static int lcd_olinuxino_enable(struct drm_panel *panel) ...@@ -134,8 +130,6 @@ static int lcd_olinuxino_enable(struct drm_panel *panel)
if (lcd->enabled) if (lcd->enabled)
return 0; return 0;
backlight_enable(lcd->backlight);
lcd->enabled = true; lcd->enabled = true;
return 0; return 0;
...@@ -283,13 +277,13 @@ static int lcd_olinuxino_probe(struct i2c_client *client, ...@@ -283,13 +277,13 @@ static int lcd_olinuxino_probe(struct i2c_client *client,
if (IS_ERR(lcd->enable_gpio)) if (IS_ERR(lcd->enable_gpio))
return PTR_ERR(lcd->enable_gpio); return PTR_ERR(lcd->enable_gpio);
lcd->backlight = devm_of_find_backlight(dev);
if (IS_ERR(lcd->backlight))
return PTR_ERR(lcd->backlight);
drm_panel_init(&lcd->panel, dev, &lcd_olinuxino_funcs, drm_panel_init(&lcd->panel, dev, &lcd_olinuxino_funcs,
DRM_MODE_CONNECTOR_DPI); DRM_MODE_CONNECTOR_DPI);
ret = drm_panel_of_backlight(&lcd->panel);
if (ret)
return ret;
return drm_panel_add(&lcd->panel); return drm_panel_add(&lcd->panel);
} }
...@@ -299,8 +293,8 @@ static int lcd_olinuxino_remove(struct i2c_client *client) ...@@ -299,8 +293,8 @@ static int lcd_olinuxino_remove(struct i2c_client *client)
drm_panel_remove(&panel->panel); drm_panel_remove(&panel->panel);
lcd_olinuxino_disable(&panel->panel); drm_panel_disable(&panel->panel);
lcd_olinuxino_unprepare(&panel->panel); drm_panel_unprepare(&panel->panel);
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment