[PATCH 07/12] drm/cirrus: use universal plane interfaces for primary plane
Varad Gautam
varadgautam at gmail.com
Fri Sep 8 13:35:12 UTC 2017
From: Varad Gautam <varad.gautam at collabora.com>
cirrus exposes one legacy primary plane tied to the crtc. convert this to
use the universal planes interface in preparation for atomic.
v2: avoid early plane cleanup to fix faulty error handling. (krisman)
Signed-off-by: Varad Gautam <varad.gautam at collabora.com>
Reviewed-by: Gabriel Krisman Bertazi <krisman at collabora.co.uk>
---
drivers/gpu/drm/cirrus/cirrus_mode.c | 45 +++++++++++++++++++++++++++++++++++-
1 file changed, 44 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/cirrus/cirrus_mode.c b/drivers/gpu/drm/cirrus/cirrus_mode.c
index 6a8d09c9a36b..d12399986c81 100644
--- a/drivers/gpu/drm/cirrus/cirrus_mode.c
+++ b/drivers/gpu/drm/cirrus/cirrus_mode.c
@@ -348,11 +348,29 @@ static const struct drm_crtc_helper_funcs cirrus_helper_funcs = {
.commit = cirrus_crtc_commit,
};
+static const uint32_t cirrus_plane_formats[] = {
+ DRM_FORMAT_XRGB8888,
+ DRM_FORMAT_ARGB8888,
+ DRM_FORMAT_RGB888,
+ DRM_FORMAT_RGB565,
+};
+
+static const struct drm_plane_funcs cirrus_plane_funcs = {
+ .update_plane = drm_primary_helper_update,
+ .disable_plane = drm_primary_helper_disable,
+ .destroy = drm_primary_helper_destroy,
+};
+
+static const struct drm_plane_helper_funcs cirrus_plane_helper_funcs = {
+};
+
/* CRTC setup */
static void cirrus_crtc_init(struct drm_device *dev)
{
struct cirrus_device *cdev = dev->dev_private;
struct cirrus_crtc *cirrus_crtc;
+ struct drm_plane *primary;
+ int ret;
cirrus_crtc = kzalloc(sizeof(struct cirrus_crtc) +
(CIRRUSFB_CONN_LIMIT * sizeof(struct drm_connector *)),
@@ -361,12 +379,37 @@ static void cirrus_crtc_init(struct drm_device *dev)
if (cirrus_crtc == NULL)
return;
- drm_crtc_init(dev, &cirrus_crtc->base, &cirrus_crtc_funcs);
+ primary = kzalloc(sizeof(*primary), GFP_KERNEL);
+ if (primary == NULL)
+ goto cleanup_crtc;
+ drm_plane_helper_add(primary, &cirrus_plane_helper_funcs);
+ ret = drm_universal_plane_init(dev, primary, 1,
+ &cirrus_plane_funcs,
+ cirrus_plane_formats,
+ ARRAY_SIZE(cirrus_plane_formats),
+ NULL, DRM_PLANE_TYPE_PRIMARY, NULL);
+ if (ret) {
+ kfree(primary);
+ goto cleanup_crtc;
+ }
+
+ ret = drm_crtc_init_with_planes(dev, &cirrus_crtc->base, primary, NULL,
+ &cirrus_crtc_funcs, NULL);
+ if (ret)
+ goto cleanup;
drm_mode_crtc_set_gamma_size(&cirrus_crtc->base, CIRRUS_LUT_SIZE);
cdev->mode_info.crtc = cirrus_crtc;
drm_crtc_helper_add(&cirrus_crtc->base, &cirrus_helper_funcs);
+ return;
+
+cleanup:
+ drm_plane_cleanup(primary);
+ kfree(primary);
+cleanup_crtc:
+ kfree(cirrus_crtc);
+ return;
}
static void cirrus_encoder_mode_set(struct drm_encoder *encoder,
--
2.13.1
More information about the dri-devel
mailing list