Commit 5f1dcd8b authored by Ajay Kumar's avatar Ajay Kumar Committed by Thierry Reding

drm/exynos: dp: Modify driver to support drm_panel

Add drm_panel controls to support powerup/down of the
eDP panel, if one is present at the sink side.
Signed-off-by: default avatarAjay Kumar <ajaykumar.rs@samsung.com>
Acked-by: default avatarInki Dae <inki.dae@samsung.com>
Signed-off-by: default avatarThierry Reding <treding@nvidia.com>
parent 4deabfa0
...@@ -52,6 +52,7 @@ config DRM_EXYNOS_DP ...@@ -52,6 +52,7 @@ config DRM_EXYNOS_DP
bool "EXYNOS DRM DP driver support" bool "EXYNOS DRM DP driver support"
depends on DRM_EXYNOS_FIMD && ARCH_EXYNOS && (DRM_PTN3460=n || DRM_PTN3460=y || DRM_PTN3460=DRM_EXYNOS) depends on DRM_EXYNOS_FIMD && ARCH_EXYNOS && (DRM_PTN3460=n || DRM_PTN3460=y || DRM_PTN3460=DRM_EXYNOS)
default DRM_EXYNOS default DRM_EXYNOS
select DRM_PANEL
help help
This enables support for DP device. This enables support for DP device.
......
...@@ -16,7 +16,6 @@ ...@@ -16,7 +16,6 @@
#include <linux/clk.h> #include <linux/clk.h>
#include <linux/io.h> #include <linux/io.h>
#include <linux/interrupt.h> #include <linux/interrupt.h>
#include <linux/delay.h>
#include <linux/of.h> #include <linux/of.h>
#include <linux/of_gpio.h> #include <linux/of_gpio.h>
#include <linux/gpio.h> #include <linux/gpio.h>
...@@ -28,6 +27,7 @@ ...@@ -28,6 +27,7 @@
#include <drm/drmP.h> #include <drm/drmP.h>
#include <drm/drm_crtc.h> #include <drm/drm_crtc.h>
#include <drm/drm_crtc_helper.h> #include <drm/drm_crtc_helper.h>
#include <drm/drm_panel.h>
#include <drm/bridge/ptn3460.h> #include <drm/bridge/ptn3460.h>
#include "exynos_drm_drv.h" #include "exynos_drm_drv.h"
...@@ -41,7 +41,7 @@ struct bridge_init { ...@@ -41,7 +41,7 @@ struct bridge_init {
struct device_node *node; struct device_node *node;
}; };
static int exynos_dp_init_dp(struct exynos_dp_device *dp) static void exynos_dp_init_dp(struct exynos_dp_device *dp)
{ {
exynos_dp_reset(dp); exynos_dp_reset(dp);
...@@ -58,8 +58,6 @@ static int exynos_dp_init_dp(struct exynos_dp_device *dp) ...@@ -58,8 +58,6 @@ static int exynos_dp_init_dp(struct exynos_dp_device *dp)
exynos_dp_init_hpd(dp); exynos_dp_init_hpd(dp);
exynos_dp_init_aux(dp); exynos_dp_init_aux(dp);
return 0;
} }
static int exynos_dp_detect_hpd(struct exynos_dp_device *dp) static int exynos_dp_detect_hpd(struct exynos_dp_device *dp)
...@@ -887,6 +885,12 @@ static void exynos_dp_commit(struct exynos_drm_display *display) ...@@ -887,6 +885,12 @@ static void exynos_dp_commit(struct exynos_drm_display *display)
struct exynos_dp_device *dp = display->ctx; struct exynos_dp_device *dp = display->ctx;
int ret; int ret;
/* Keep the panel disabled while we configure video */
if (dp->panel) {
if (drm_panel_disable(dp->panel))
DRM_ERROR("failed to disable the panel\n");
}
ret = exynos_dp_detect_hpd(dp); ret = exynos_dp_detect_hpd(dp);
if (ret) { if (ret) {
/* Cable has been disconnected, we're done */ /* Cable has been disconnected, we're done */
...@@ -917,6 +921,12 @@ static void exynos_dp_commit(struct exynos_drm_display *display) ...@@ -917,6 +921,12 @@ static void exynos_dp_commit(struct exynos_drm_display *display)
ret = exynos_dp_config_video(dp); ret = exynos_dp_config_video(dp);
if (ret) if (ret)
dev_err(dp->dev, "unable to config video\n"); dev_err(dp->dev, "unable to config video\n");
/* Safe to enable the panel now */
if (dp->panel) {
if (drm_panel_enable(dp->panel))
DRM_ERROR("failed to enable the panel\n");
}
} }
static enum drm_connector_status exynos_dp_detect( static enum drm_connector_status exynos_dp_detect(
...@@ -941,15 +951,18 @@ static int exynos_dp_get_modes(struct drm_connector *connector) ...@@ -941,15 +951,18 @@ static int exynos_dp_get_modes(struct drm_connector *connector)
struct exynos_dp_device *dp = ctx_from_connector(connector); struct exynos_dp_device *dp = ctx_from_connector(connector);
struct drm_display_mode *mode; struct drm_display_mode *mode;
if (dp->panel)
return drm_panel_get_modes(dp->panel);
mode = drm_mode_create(connector->dev); mode = drm_mode_create(connector->dev);
if (!mode) { if (!mode) {
DRM_ERROR("failed to create a new display mode.\n"); DRM_ERROR("failed to create a new display mode.\n");
return 0; return 0;
} }
drm_display_mode_from_videomode(&dp->panel.vm, mode); drm_display_mode_from_videomode(&dp->priv.vm, mode);
mode->width_mm = dp->panel.width_mm; mode->width_mm = dp->priv.width_mm;
mode->height_mm = dp->panel.height_mm; mode->height_mm = dp->priv.height_mm;
connector->display_info.width_mm = mode->width_mm; connector->display_info.width_mm = mode->width_mm;
connector->display_info.height_mm = mode->height_mm; connector->display_info.height_mm = mode->height_mm;
...@@ -1029,7 +1042,10 @@ static int exynos_dp_create_connector(struct exynos_drm_display *display, ...@@ -1029,7 +1042,10 @@ static int exynos_dp_create_connector(struct exynos_drm_display *display,
drm_sysfs_connector_add(connector); drm_sysfs_connector_add(connector);
drm_mode_connector_attach_encoder(connector, encoder); drm_mode_connector_attach_encoder(connector, encoder);
return 0; if (dp->panel)
ret = drm_panel_attach(dp->panel, &dp->connector);
return ret;
} }
static void exynos_dp_phy_init(struct exynos_dp_device *dp) static void exynos_dp_phy_init(struct exynos_dp_device *dp)
...@@ -1065,6 +1081,13 @@ static void exynos_dp_poweron(struct exynos_drm_display *display) ...@@ -1065,6 +1081,13 @@ static void exynos_dp_poweron(struct exynos_drm_display *display)
if (dp->dpms_mode == DRM_MODE_DPMS_ON) if (dp->dpms_mode == DRM_MODE_DPMS_ON)
return; return;
if (dp->panel) {
if (drm_panel_prepare(dp->panel)) {
DRM_ERROR("failed to setup the panel\n");
return;
}
}
clk_prepare_enable(dp->clock); clk_prepare_enable(dp->clock);
exynos_dp_phy_init(dp); exynos_dp_phy_init(dp);
exynos_dp_init_dp(dp); exynos_dp_init_dp(dp);
...@@ -1079,10 +1102,22 @@ static void exynos_dp_poweroff(struct exynos_drm_display *display) ...@@ -1079,10 +1102,22 @@ static void exynos_dp_poweroff(struct exynos_drm_display *display)
if (dp->dpms_mode != DRM_MODE_DPMS_ON) if (dp->dpms_mode != DRM_MODE_DPMS_ON)
return; return;
if (dp->panel) {
if (drm_panel_disable(dp->panel)) {
DRM_ERROR("failed to disable the panel\n");
return;
}
}
disable_irq(dp->irq); disable_irq(dp->irq);
flush_work(&dp->hotplug_work); flush_work(&dp->hotplug_work);
exynos_dp_phy_exit(dp); exynos_dp_phy_exit(dp);
clk_disable_unprepare(dp->clock); clk_disable_unprepare(dp->clock);
if (dp->panel) {
if (drm_panel_unprepare(dp->panel))
DRM_ERROR("failed to turnoff the panel\n");
}
} }
static void exynos_dp_dpms(struct exynos_drm_display *display, int mode) static void exynos_dp_dpms(struct exynos_drm_display *display, int mode)
...@@ -1215,7 +1250,7 @@ static int exynos_dp_dt_parse_panel(struct exynos_dp_device *dp) ...@@ -1215,7 +1250,7 @@ static int exynos_dp_dt_parse_panel(struct exynos_dp_device *dp)
{ {
int ret; int ret;
ret = of_get_videomode(dp->dev->of_node, &dp->panel.vm, ret = of_get_videomode(dp->dev->of_node, &dp->priv.vm,
OF_USE_NATIVE_MODE); OF_USE_NATIVE_MODE);
if (ret) { if (ret) {
DRM_ERROR("failed: of_get_videomode() : %d\n", ret); DRM_ERROR("failed: of_get_videomode() : %d\n", ret);
...@@ -1229,16 +1264,10 @@ static int exynos_dp_bind(struct device *dev, struct device *master, void *data) ...@@ -1229,16 +1264,10 @@ static int exynos_dp_bind(struct device *dev, struct device *master, void *data)
struct platform_device *pdev = to_platform_device(dev); struct platform_device *pdev = to_platform_device(dev);
struct drm_device *drm_dev = data; struct drm_device *drm_dev = data;
struct resource *res; struct resource *res;
struct exynos_dp_device *dp; struct exynos_dp_device *dp = exynos_dp_display.ctx;
unsigned int irq_flags; unsigned int irq_flags;
int ret = 0; int ret = 0;
dp = devm_kzalloc(&pdev->dev, sizeof(struct exynos_dp_device),
GFP_KERNEL);
if (!dp)
return -ENOMEM;
dp->dev = &pdev->dev; dp->dev = &pdev->dev;
dp->dpms_mode = DRM_MODE_DPMS_OFF; dp->dpms_mode = DRM_MODE_DPMS_OFF;
...@@ -1250,9 +1279,11 @@ static int exynos_dp_bind(struct device *dev, struct device *master, void *data) ...@@ -1250,9 +1279,11 @@ static int exynos_dp_bind(struct device *dev, struct device *master, void *data)
if (ret) if (ret)
return ret; return ret;
ret = exynos_dp_dt_parse_panel(dp); if (!dp->panel) {
if (ret) ret = exynos_dp_dt_parse_panel(dp);
return ret; if (ret)
return ret;
}
dp->clock = devm_clk_get(&pdev->dev, "dp"); dp->clock = devm_clk_get(&pdev->dev, "dp");
if (IS_ERR(dp->clock)) { if (IS_ERR(dp->clock)) {
...@@ -1312,7 +1343,6 @@ static int exynos_dp_bind(struct device *dev, struct device *master, void *data) ...@@ -1312,7 +1343,6 @@ static int exynos_dp_bind(struct device *dev, struct device *master, void *data)
disable_irq(dp->irq); disable_irq(dp->irq);
dp->drm_dev = drm_dev; dp->drm_dev = drm_dev;
exynos_dp_display.ctx = dp;
platform_set_drvdata(pdev, &exynos_dp_display); platform_set_drvdata(pdev, &exynos_dp_display);
...@@ -1339,6 +1369,9 @@ static const struct component_ops exynos_dp_ops = { ...@@ -1339,6 +1369,9 @@ static const struct component_ops exynos_dp_ops = {
static int exynos_dp_probe(struct platform_device *pdev) static int exynos_dp_probe(struct platform_device *pdev)
{ {
struct device *dev = &pdev->dev;
struct device_node *panel_node;
struct exynos_dp_device *dp;
int ret; int ret;
ret = exynos_drm_component_add(&pdev->dev, EXYNOS_DEVICE_TYPE_CONNECTOR, ret = exynos_drm_component_add(&pdev->dev, EXYNOS_DEVICE_TYPE_CONNECTOR,
...@@ -1346,6 +1379,21 @@ static int exynos_dp_probe(struct platform_device *pdev) ...@@ -1346,6 +1379,21 @@ static int exynos_dp_probe(struct platform_device *pdev)
if (ret) if (ret)
return ret; return ret;
dp = devm_kzalloc(&pdev->dev, sizeof(struct exynos_dp_device),
GFP_KERNEL);
if (!dp)
return -ENOMEM;
panel_node = of_parse_phandle(dev->of_node, "panel", 0);
if (panel_node) {
dp->panel = of_drm_find_panel(panel_node);
of_node_put(panel_node);
if (!dp->panel)
return -EPROBE_DEFER;
}
exynos_dp_display.ctx = dp;
ret = component_add(&pdev->dev, &exynos_dp_ops); ret = component_add(&pdev->dev, &exynos_dp_ops);
if (ret) if (ret)
exynos_drm_component_del(&pdev->dev, exynos_drm_component_del(&pdev->dev,
......
...@@ -149,6 +149,7 @@ struct exynos_dp_device { ...@@ -149,6 +149,7 @@ struct exynos_dp_device {
struct drm_device *drm_dev; struct drm_device *drm_dev;
struct drm_connector connector; struct drm_connector connector;
struct drm_encoder *encoder; struct drm_encoder *encoder;
struct drm_panel *panel;
struct clk *clock; struct clk *clock;
unsigned int irq; unsigned int irq;
void __iomem *reg_base; void __iomem *reg_base;
...@@ -162,7 +163,7 @@ struct exynos_dp_device { ...@@ -162,7 +163,7 @@ struct exynos_dp_device {
int dpms_mode; int dpms_mode;
int hpd_gpio; int hpd_gpio;
struct exynos_drm_panel_info panel; struct exynos_drm_panel_info priv;
}; };
/* exynos_dp_reg.c */ /* exynos_dp_reg.c */
......
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