From: Tomohito Esaki <etom@igel.co.jp>
To: linux-renesas-soc@vger.kernel.org
Cc: laurent.pinchart@ideasonboard.com, etom@igel.co.jp
Subject: [PATCH] drm: rcar-du: add modifiers support
Date: Thu, 9 May 2019 14:45:18 +0900 [thread overview]
Message-ID: <20190509054518.10781-1-etom@igel.co.jp> (raw)
Add support for the linear modifier. Since the rcar-du device supports
only linear modifier, this driver doesn't support other modifiers.
Signed-off-by: Tomohito Esaki <etom@igel.co.jp>
---
drivers/gpu/drm/rcar-du/rcar_du_kms.c | 11 +++++++++++
drivers/gpu/drm/rcar-du/rcar_du_plane.c | 15 ++++++++++++++-
drivers/gpu/drm/rcar-du/rcar_du_vsp.c | 15 ++++++++++++++-
3 files changed, 39 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/rcar-du/rcar_du_kms.c b/drivers/gpu/drm/rcar-du/rcar_du_kms.c
index 3b7d50a8fb9b..9c5e15a5ab1c 100644
--- a/drivers/gpu/drm/rcar-du/rcar_du_kms.c
+++ b/drivers/gpu/drm/rcar-du/rcar_du_kms.c
@@ -214,6 +214,16 @@ rcar_du_fb_create(struct drm_device *dev, struct drm_file *file_priv,
return ERR_PTR(-EINVAL);
}
+ /*
+ * Support only LINEAR modifier.
+ */
+ if ((mode_cmd->flags & DRM_MODE_FB_MODIFIERS) &&
+ mode_cmd->modifier[0] != DRM_FORMAT_MOD_LINEAR) {
+ dev_dbg(dev->dev, "unsupported fb modifier 0x%llx\n",
+ mode_cmd->modifier[0]);
+ return ERR_PTR(-EINVAL);
+ }
+
if (rcdu->info->gen < 3) {
/*
* On Gen2 the DU limits the pitch to 4095 pixels and requires
@@ -529,6 +539,7 @@ int rcar_du_modeset_init(struct rcar_du_device *rcdu)
dev->mode_config.min_width = 0;
dev->mode_config.min_height = 0;
dev->mode_config.normalize_zpos = true;
+ dev->mode_config.allow_fb_modifiers = true;
dev->mode_config.funcs = &rcar_du_mode_config_funcs;
dev->mode_config.helper_private = &rcar_du_mode_config_helper;
diff --git a/drivers/gpu/drm/rcar-du/rcar_du_plane.c b/drivers/gpu/drm/rcar-du/rcar_du_plane.c
index c6430027169f..32135ad387fa 100644
--- a/drivers/gpu/drm/rcar-du/rcar_du_plane.c
+++ b/drivers/gpu/drm/rcar-du/rcar_du_plane.c
@@ -726,6 +726,13 @@ static int rcar_du_plane_atomic_get_property(struct drm_plane *plane,
return 0;
}
+static bool rcar_du_plane_format_mod_supported(struct drm_plane *plane,
+ uint32_t format,
+ uint64_t modifier)
+{
+ return modifier == DRM_FORMAT_MOD_LINEAR;
+}
+
static const struct drm_plane_funcs rcar_du_plane_funcs = {
.update_plane = drm_atomic_helper_update_plane,
.disable_plane = drm_atomic_helper_disable_plane,
@@ -735,6 +742,7 @@ static const struct drm_plane_funcs rcar_du_plane_funcs = {
.atomic_destroy_state = rcar_du_plane_atomic_destroy_state,
.atomic_set_property = rcar_du_plane_atomic_set_property,
.atomic_get_property = rcar_du_plane_atomic_get_property,
+ .format_mod_supported = rcar_du_plane_format_mod_supported,
};
static const uint32_t formats[] = {
@@ -750,6 +758,11 @@ static const uint32_t formats[] = {
DRM_FORMAT_NV16,
};
+static const uint64_t modifiers[] = {
+ DRM_FORMAT_MOD_LINEAR,
+ DRM_FORMAT_MOD_INVALID,
+};
+
int rcar_du_planes_init(struct rcar_du_group *rgrp)
{
struct rcar_du_device *rcdu = rgrp->dev;
@@ -776,7 +789,7 @@ int rcar_du_planes_init(struct rcar_du_group *rgrp)
ret = drm_universal_plane_init(rcdu->ddev, &plane->plane, crtcs,
&rcar_du_plane_funcs, formats,
ARRAY_SIZE(formats),
- NULL, type, NULL);
+ modifiers, type, NULL);
if (ret < 0)
return ret;
diff --git a/drivers/gpu/drm/rcar-du/rcar_du_vsp.c b/drivers/gpu/drm/rcar-du/rcar_du_vsp.c
index 0878accbd134..9d1382b02717 100644
--- a/drivers/gpu/drm/rcar-du/rcar_du_vsp.c
+++ b/drivers/gpu/drm/rcar-du/rcar_du_vsp.c
@@ -139,6 +139,11 @@ static const u32 formats_kms[] = {
DRM_FORMAT_YVU444,
};
+static const uint64_t modifiers_kms[] = {
+ DRM_FORMAT_MOD_LINEAR,
+ DRM_FORMAT_MOD_INVALID,
+};
+
static const u32 formats_v4l2[] = {
V4L2_PIX_FMT_RGB332,
V4L2_PIX_FMT_ARGB444,
@@ -344,6 +349,13 @@ static void rcar_du_vsp_plane_reset(struct drm_plane *plane)
state->state.zpos = plane->type == DRM_PLANE_TYPE_PRIMARY ? 0 : 1;
}
+static bool rcar_du_vsp_plane_format_mod_supported(struct drm_plane *plane,
+ uint32_t format,
+ uint64_t modifier)
+{
+ return modifier == DRM_FORMAT_MOD_LINEAR;
+}
+
static const struct drm_plane_funcs rcar_du_vsp_plane_funcs = {
.update_plane = drm_atomic_helper_update_plane,
.disable_plane = drm_atomic_helper_disable_plane,
@@ -351,6 +363,7 @@ static const struct drm_plane_funcs rcar_du_vsp_plane_funcs = {
.destroy = drm_plane_cleanup,
.atomic_duplicate_state = rcar_du_vsp_plane_atomic_duplicate_state,
.atomic_destroy_state = rcar_du_vsp_plane_atomic_destroy_state,
+ .format_mod_supported = rcar_du_vsp_plane_format_mod_supported,
};
int rcar_du_vsp_init(struct rcar_du_vsp *vsp, struct device_node *np,
@@ -397,7 +410,7 @@ int rcar_du_vsp_init(struct rcar_du_vsp *vsp, struct device_node *np,
&rcar_du_vsp_plane_funcs,
formats_kms,
ARRAY_SIZE(formats_kms),
- NULL, type, NULL);
+ modifiers_kms, type, NULL);
if (ret < 0)
return ret;
--
2.17.1
next reply other threads:[~2019-05-09 5:47 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-05-09 5:45 Tomohito Esaki [this message]
2019-05-09 7:14 ` [PATCH] drm: rcar-du: add modifiers support Laurent Pinchart
2019-05-09 9:25 ` Esaki Tomohito
2019-05-11 18:10 ` Laurent Pinchart
2019-05-17 7:45 ` Esaki Tomohito
2021-11-18 12:31 ` Laurent Pinchart
2021-11-18 13:02 ` Daniel Stone
2021-11-18 14:05 ` Laurent Pinchart
2021-11-30 8:44 ` Esaki Tomohito
2021-11-30 13:20 ` Daniel Stone
2021-12-01 5:47 ` Esaki Tomohito
2021-11-24 2:49 ` Esaki Tomohito
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190509054518.10781-1-etom@igel.co.jp \
--to=etom@igel.co.jp \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-renesas-soc@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).