Commit c42001e3 authored by Laurent Pinchart's avatar Laurent Pinchart Committed by Stefan Agner

drm: mxsfb: Use drm_panel_bridge

Replace the manual connector implementation based on drm_panel with the
drm_panel_bridge helper. This simplifies the mxsfb driver by removing
connector-related code, and standardizing all pipeline control
operations on bridges.

A hack is needed to get hold of the connector, as that's our only source
of bus flags and formats for now. As soon as the bridge API provides us
with that information this can be fixed.
Signed-off-by: default avatarLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: default avatarEmil Velikov <emil.l.velikov@gmail.com>
Reviewed-by: default avatarStefan Agner <stefan@agner.ch>
Signed-off-by: default avatarStefan Agner <stefan@agner.ch>
Link: https://patchwork.freedesktop.org/patch/msgid/20200727020654.8231-3-laurent.pinchart@ideasonboard.com
parent fcf863ec
...@@ -13,6 +13,7 @@ config DRM_MXSFB ...@@ -13,6 +13,7 @@ config DRM_MXSFB
select DRM_KMS_FB_HELPER select DRM_KMS_FB_HELPER
select DRM_KMS_CMA_HELPER select DRM_KMS_CMA_HELPER
select DRM_PANEL select DRM_PANEL
select DRM_PANEL_BRIDGE
help help
Choose this option if you have an i.MX23/i.MX28/i.MX6SX MXSFB Choose this option if you have an i.MX23/i.MX28/i.MX6SX MXSFB
LCD controller. LCD controller.
......
# SPDX-License-Identifier: GPL-2.0-only # SPDX-License-Identifier: GPL-2.0-only
mxsfb-y := mxsfb_drv.o mxsfb_crtc.o mxsfb_out.o mxsfb-y := mxsfb_drv.o mxsfb_crtc.o
obj-$(CONFIG_DRM_MXSFB) += mxsfb.o obj-$(CONFIG_DRM_MXSFB) += mxsfb.o
...@@ -29,7 +29,6 @@ ...@@ -29,7 +29,6 @@
#include <drm/drm_gem_framebuffer_helper.h> #include <drm/drm_gem_framebuffer_helper.h>
#include <drm/drm_irq.h> #include <drm/drm_irq.h>
#include <drm/drm_of.h> #include <drm/drm_of.h>
#include <drm/drm_panel.h>
#include <drm/drm_probe_helper.h> #include <drm/drm_probe_helper.h>
#include <drm/drm_simple_kms_helper.h> #include <drm/drm_simple_kms_helper.h>
#include <drm/drm_vblank.h> #include <drm/drm_vblank.h>
...@@ -105,29 +104,11 @@ static void mxsfb_pipe_enable(struct drm_simple_display_pipe *pipe, ...@@ -105,29 +104,11 @@ static void mxsfb_pipe_enable(struct drm_simple_display_pipe *pipe,
struct drm_crtc_state *crtc_state, struct drm_crtc_state *crtc_state,
struct drm_plane_state *plane_state) struct drm_plane_state *plane_state)
{ {
struct drm_connector *connector;
struct mxsfb_drm_private *mxsfb = drm_pipe_to_mxsfb_drm_private(pipe); struct mxsfb_drm_private *mxsfb = drm_pipe_to_mxsfb_drm_private(pipe);
struct drm_device *drm = pipe->plane.dev; struct drm_device *drm = pipe->plane.dev;
if (!mxsfb->connector) {
list_for_each_entry(connector,
&drm->mode_config.connector_list,
head)
if (connector->encoder == &mxsfb->pipe.encoder) {
mxsfb->connector = connector;
break;
}
}
if (!mxsfb->connector) {
dev_warn(drm->dev, "No connector attached, using default\n");
mxsfb->connector = &mxsfb->panel_connector;
}
pm_runtime_get_sync(drm->dev); pm_runtime_get_sync(drm->dev);
drm_panel_prepare(mxsfb->panel);
mxsfb_crtc_enable(mxsfb); mxsfb_crtc_enable(mxsfb);
drm_panel_enable(mxsfb->panel);
} }
static void mxsfb_pipe_disable(struct drm_simple_display_pipe *pipe) static void mxsfb_pipe_disable(struct drm_simple_display_pipe *pipe)
...@@ -137,9 +118,7 @@ static void mxsfb_pipe_disable(struct drm_simple_display_pipe *pipe) ...@@ -137,9 +118,7 @@ static void mxsfb_pipe_disable(struct drm_simple_display_pipe *pipe)
struct drm_crtc *crtc = &pipe->crtc; struct drm_crtc *crtc = &pipe->crtc;
struct drm_pending_vblank_event *event; struct drm_pending_vblank_event *event;
drm_panel_disable(mxsfb->panel);
mxsfb_crtc_disable(mxsfb); mxsfb_crtc_disable(mxsfb);
drm_panel_unprepare(mxsfb->panel);
pm_runtime_put_sync(drm->dev); pm_runtime_put_sync(drm->dev);
spin_lock_irq(&drm->event_lock); spin_lock_irq(&drm->event_lock);
...@@ -149,9 +128,6 @@ static void mxsfb_pipe_disable(struct drm_simple_display_pipe *pipe) ...@@ -149,9 +128,6 @@ static void mxsfb_pipe_disable(struct drm_simple_display_pipe *pipe)
drm_crtc_send_vblank_event(crtc, event); drm_crtc_send_vblank_event(crtc, event);
} }
spin_unlock_irq(&drm->event_lock); spin_unlock_irq(&drm->event_lock);
if (mxsfb->connector != &mxsfb->panel_connector)
mxsfb->connector = NULL;
} }
static void mxsfb_pipe_update(struct drm_simple_display_pipe *pipe, static void mxsfb_pipe_update(struct drm_simple_display_pipe *pipe,
...@@ -195,6 +171,49 @@ static struct drm_simple_display_pipe_funcs mxsfb_funcs = { ...@@ -195,6 +171,49 @@ static struct drm_simple_display_pipe_funcs mxsfb_funcs = {
.disable_vblank = mxsfb_pipe_disable_vblank, .disable_vblank = mxsfb_pipe_disable_vblank,
}; };
static int mxsfb_attach_bridge(struct mxsfb_drm_private *mxsfb)
{
struct drm_device *drm = mxsfb->drm;
struct drm_connector_list_iter iter;
struct drm_panel *panel;
struct drm_bridge *bridge;
int ret;
ret = drm_of_find_panel_or_bridge(drm->dev->of_node, 0, 0, &panel,
&bridge);
if (ret)
return ret;
if (panel) {
bridge = devm_drm_panel_bridge_add_typed(drm->dev, panel,
DRM_MODE_CONNECTOR_DPI);
if (IS_ERR(bridge))
return PTR_ERR(bridge);
}
if (!bridge)
return -ENODEV;
ret = drm_simple_display_pipe_attach_bridge(&mxsfb->pipe, bridge);
if (ret) {
DRM_DEV_ERROR(drm->dev,
"failed to attach bridge: %d\n", ret);
return ret;
}
mxsfb->bridge = bridge;
/*
* Get hold of the connector. This is a bit of a hack, until the bridge
* API gives us bus flags and formats.
*/
drm_connector_list_iter_begin(drm, &iter);
mxsfb->connector = drm_connector_list_iter_next(&iter);
drm_connector_list_iter_end(&iter);
return 0;
}
static int mxsfb_load(struct drm_device *drm) static int mxsfb_load(struct drm_device *drm)
{ {
struct platform_device *pdev = to_platform_device(drm->dev); struct platform_device *pdev = to_platform_device(drm->dev);
...@@ -206,6 +225,7 @@ static int mxsfb_load(struct drm_device *drm) ...@@ -206,6 +225,7 @@ static int mxsfb_load(struct drm_device *drm)
if (!mxsfb) if (!mxsfb)
return -ENOMEM; return -ENOMEM;
mxsfb->drm = drm;
drm->dev_private = mxsfb; drm->dev_private = mxsfb;
mxsfb->devdata = &mxsfb_devdata[pdev->id_entry->driver_data]; mxsfb->devdata = &mxsfb_devdata[pdev->id_entry->driver_data];
...@@ -241,42 +261,19 @@ static int mxsfb_load(struct drm_device *drm) ...@@ -241,42 +261,19 @@ static int mxsfb_load(struct drm_device *drm)
/* Modeset init */ /* Modeset init */
drm_mode_config_init(drm); drm_mode_config_init(drm);
ret = mxsfb_create_output(drm);
if (ret < 0) {
dev_err(drm->dev, "Failed to create outputs\n");
goto err_vblank;
}
ret = drm_simple_display_pipe_init(drm, &mxsfb->pipe, &mxsfb_funcs, ret = drm_simple_display_pipe_init(drm, &mxsfb->pipe, &mxsfb_funcs,
mxsfb_formats, ARRAY_SIZE(mxsfb_formats), mxsfb_formats, ARRAY_SIZE(mxsfb_formats),
mxsfb_modifiers, mxsfb->connector); mxsfb_modifiers, NULL);
if (ret < 0) { if (ret < 0) {
dev_err(drm->dev, "Cannot setup simple display pipe\n"); dev_err(drm->dev, "Cannot setup simple display pipe\n");
goto err_vblank; goto err_vblank;
} }
/* ret = mxsfb_attach_bridge(mxsfb);
* Attach panel only if there is one.
* If there is no panel attach, it must be a bridge. In this case, we
* need a reference to its connector for a proper initialization.
* We will do this check in pipe->enable(), since the connector won't
* be attached to an encoder until then.
*/
if (mxsfb->panel) {
ret = drm_panel_attach(mxsfb->panel, mxsfb->connector);
if (ret) {
dev_err(drm->dev, "Cannot connect panel: %d\n", ret);
goto err_vblank;
}
} else if (mxsfb->bridge) {
ret = drm_simple_display_pipe_attach_bridge(&mxsfb->pipe,
mxsfb->bridge);
if (ret) { if (ret) {
dev_err(drm->dev, "Cannot connect bridge: %d\n", ret); dev_err(drm->dev, "Cannot connect bridge: %d\n", ret);
goto err_vblank; goto err_vblank;
} }
}
drm->mode_config.min_width = MXSFB_MIN_XRES; drm->mode_config.min_width = MXSFB_MIN_XRES;
drm->mode_config.min_height = MXSFB_MIN_YRES; drm->mode_config.min_height = MXSFB_MIN_YRES;
...@@ -293,7 +290,7 @@ static int mxsfb_load(struct drm_device *drm) ...@@ -293,7 +290,7 @@ static int mxsfb_load(struct drm_device *drm)
if (ret < 0) { if (ret < 0) {
dev_err(drm->dev, "Failed to install IRQ handler\n"); dev_err(drm->dev, "Failed to install IRQ handler\n");
goto err_irq; goto err_vblank;
} }
drm_kms_helper_poll_init(drm); drm_kms_helper_poll_init(drm);
...@@ -304,8 +301,6 @@ static int mxsfb_load(struct drm_device *drm) ...@@ -304,8 +301,6 @@ static int mxsfb_load(struct drm_device *drm)
return 0; return 0;
err_irq:
drm_panel_detach(mxsfb->panel);
err_vblank: err_vblank:
pm_runtime_disable(drm->dev); pm_runtime_disable(drm->dev);
......
...@@ -8,6 +8,8 @@ ...@@ -8,6 +8,8 @@
#ifndef __MXSFB_DRV_H__ #ifndef __MXSFB_DRV_H__
#define __MXSFB_DRV_H__ #define __MXSFB_DRV_H__
struct drm_device;
struct mxsfb_devdata { struct mxsfb_devdata {
unsigned int transfer_count; unsigned int transfer_count;
unsigned int cur_buf; unsigned int cur_buf;
...@@ -26,10 +28,9 @@ struct mxsfb_drm_private { ...@@ -26,10 +28,9 @@ struct mxsfb_drm_private {
struct clk *clk_axi; struct clk *clk_axi;
struct clk *clk_disp_axi; struct clk *clk_disp_axi;
struct drm_device *drm;
struct drm_simple_display_pipe pipe; struct drm_simple_display_pipe pipe;
struct drm_connector panel_connector;
struct drm_connector *connector; struct drm_connector *connector;
struct drm_panel *panel;
struct drm_bridge *bridge; struct drm_bridge *bridge;
}; };
......
// SPDX-License-Identifier: GPL-2.0-or-later
/*
* Copyright (C) 2016 Marek Vasut <marex@denx.de>
*/
#include <linux/of_graph.h>
#include <drm/drm_atomic.h>
#include <drm/drm_atomic_helper.h>
#include <drm/drm_crtc.h>
#include <drm/drm_fb_cma_helper.h>
#include <drm/drm_gem_cma_helper.h>
#include <drm/drm_of.h>
#include <drm/drm_panel.h>
#include <drm/drm_plane_helper.h>
#include <drm/drm_probe_helper.h>
#include <drm/drm_simple_kms_helper.h>
#include "mxsfb_drv.h"
static struct mxsfb_drm_private *
drm_connector_to_mxsfb_drm_private(struct drm_connector *connector)
{
return container_of(connector, struct mxsfb_drm_private,
panel_connector);
}
static int mxsfb_panel_get_modes(struct drm_connector *connector)
{
struct mxsfb_drm_private *mxsfb =
drm_connector_to_mxsfb_drm_private(connector);
if (mxsfb->panel)
return drm_panel_get_modes(mxsfb->panel, connector);
return 0;
}
static const struct
drm_connector_helper_funcs mxsfb_panel_connector_helper_funcs = {
.get_modes = mxsfb_panel_get_modes,
};
static enum drm_connector_status
mxsfb_panel_connector_detect(struct drm_connector *connector, bool force)
{
struct mxsfb_drm_private *mxsfb =
drm_connector_to_mxsfb_drm_private(connector);
if (mxsfb->panel)
return connector_status_connected;
return connector_status_disconnected;
}
static void mxsfb_panel_connector_destroy(struct drm_connector *connector)
{
struct mxsfb_drm_private *mxsfb =
drm_connector_to_mxsfb_drm_private(connector);
if (mxsfb->panel)
drm_panel_detach(mxsfb->panel);
drm_connector_unregister(connector);
drm_connector_cleanup(connector);
}
static const struct drm_connector_funcs mxsfb_panel_connector_funcs = {
.detect = mxsfb_panel_connector_detect,
.fill_modes = drm_helper_probe_single_connector_modes,
.destroy = mxsfb_panel_connector_destroy,
.reset = drm_atomic_helper_connector_reset,
.atomic_duplicate_state = drm_atomic_helper_connector_duplicate_state,
.atomic_destroy_state = drm_atomic_helper_connector_destroy_state,
};
int mxsfb_create_output(struct drm_device *drm)
{
struct mxsfb_drm_private *mxsfb = drm->dev_private;
int ret;
ret = drm_of_find_panel_or_bridge(drm->dev->of_node, 0, 0,
&mxsfb->panel, &mxsfb->bridge);
if (ret)
return ret;
if (mxsfb->panel) {
mxsfb->connector = &mxsfb->panel_connector;
mxsfb->connector->dpms = DRM_MODE_DPMS_OFF;
mxsfb->connector->polled = 0;
drm_connector_helper_add(mxsfb->connector,
&mxsfb_panel_connector_helper_funcs);
ret = drm_connector_init(drm, mxsfb->connector,
&mxsfb_panel_connector_funcs,
DRM_MODE_CONNECTOR_Unknown);
}
return ret;
}
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