From: Ville Syrjala <ville.syrjala@linux.intel.com> To: dri-devel@lists.freedesktop.org Cc: intel-gfx@lists.freedesktop.org Subject: [PATCH 09/22] drm/imx: Use drm_mode_duplicate() Date: Fri, 18 Feb 2022 12:03:50 +0200 [thread overview] Message-ID: <20220218100403.7028-10-ville.syrjala@linux.intel.com> (raw) In-Reply-To: <20220218100403.7028-1-ville.syrjala@linux.intel.com> From: Ville Syrjälä <ville.syrjala@linux.intel.com> Replace the hand rolled drm_mode_duplicate() with the real thing. @is_dup@ @@ drm_mode_duplicate(...) { ... } @depends on !is_dup@ expression dev, oldmode; identifier newmode; @@ - newmode = drm_mode_create(dev); + newmode = drm_mode_duplicate(dev, oldmode); ... - drm_mode_copy(newmode, oldmode); Cc: Philipp Zabel <p.zabel@pengutronix.de> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> --- drivers/gpu/drm/imx/imx-ldb.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/imx/imx-ldb.c b/drivers/gpu/drm/imx/imx-ldb.c index e5078d03020d..c8cf819f39ce 100644 --- a/drivers/gpu/drm/imx/imx-ldb.c +++ b/drivers/gpu/drm/imx/imx-ldb.c @@ -150,10 +150,9 @@ static int imx_ldb_connector_get_modes(struct drm_connector *connector) if (imx_ldb_ch->mode_valid) { struct drm_display_mode *mode; - mode = drm_mode_create(connector->dev); + mode = drm_mode_duplicate(connector->dev, &imx_ldb_ch->mode); if (!mode) return -EINVAL; - drm_mode_copy(mode, &imx_ldb_ch->mode); mode->type |= DRM_MODE_TYPE_DRIVER | DRM_MODE_TYPE_PREFERRED; drm_mode_probed_add(connector, mode); num_modes++; -- 2.34.1
WARNING: multiple messages have this Message-ID (diff)
From: Ville Syrjala <ville.syrjala@linux.intel.com> To: dri-devel@lists.freedesktop.org Cc: intel-gfx@lists.freedesktop.org, Philipp Zabel <p.zabel@pengutronix.de> Subject: [Intel-gfx] [PATCH 09/22] drm/imx: Use drm_mode_duplicate() Date: Fri, 18 Feb 2022 12:03:50 +0200 [thread overview] Message-ID: <20220218100403.7028-10-ville.syrjala@linux.intel.com> (raw) In-Reply-To: <20220218100403.7028-1-ville.syrjala@linux.intel.com> From: Ville Syrjälä <ville.syrjala@linux.intel.com> Replace the hand rolled drm_mode_duplicate() with the real thing. @is_dup@ @@ drm_mode_duplicate(...) { ... } @depends on !is_dup@ expression dev, oldmode; identifier newmode; @@ - newmode = drm_mode_create(dev); + newmode = drm_mode_duplicate(dev, oldmode); ... - drm_mode_copy(newmode, oldmode); Cc: Philipp Zabel <p.zabel@pengutronix.de> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> --- drivers/gpu/drm/imx/imx-ldb.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/imx/imx-ldb.c b/drivers/gpu/drm/imx/imx-ldb.c index e5078d03020d..c8cf819f39ce 100644 --- a/drivers/gpu/drm/imx/imx-ldb.c +++ b/drivers/gpu/drm/imx/imx-ldb.c @@ -150,10 +150,9 @@ static int imx_ldb_connector_get_modes(struct drm_connector *connector) if (imx_ldb_ch->mode_valid) { struct drm_display_mode *mode; - mode = drm_mode_create(connector->dev); + mode = drm_mode_duplicate(connector->dev, &imx_ldb_ch->mode); if (!mode) return -EINVAL; - drm_mode_copy(mode, &imx_ldb_ch->mode); mode->type |= DRM_MODE_TYPE_DRIVER | DRM_MODE_TYPE_PREFERRED; drm_mode_probed_add(connector, mode); num_modes++; -- 2.34.1
next prev parent reply other threads:[~2022-02-18 10:05 UTC|newest] Thread overview: 157+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-02-18 10:03 [PATCH 00/22] drm: Review of mode copies Ville Syrjala 2022-02-18 10:03 ` Ville Syrjala 2022-02-18 10:03 ` Ville Syrjala 2022-02-18 10:03 ` [Intel-gfx] " Ville Syrjala 2022-02-18 10:03 ` Ville Syrjala 2022-02-18 10:03 ` Ville Syrjala 2022-02-18 10:03 ` [PATCH 01/22] drm: Add drm_mode_init() Ville Syrjala 2022-02-18 10:03 ` [Intel-gfx] " Ville Syrjala 2022-02-18 11:22 ` Andrzej Hajda 2022-02-18 11:22 ` [Intel-gfx] " Andrzej Hajda 2022-02-18 11:56 ` Ville Syrjälä 2022-02-18 11:56 ` [Intel-gfx] " Ville Syrjälä 2022-02-18 12:23 ` Andrzej Hajda 2022-02-18 12:23 ` [Intel-gfx] " Andrzej Hajda 2022-02-18 16:34 ` Harry Wentland 2022-02-18 16:34 ` [Intel-gfx] " Harry Wentland 2022-02-18 10:03 ` [PATCH 02/22] drm/amdgpu: Remove pointless on stack mode copies Ville Syrjala 2022-02-18 10:03 ` Ville Syrjala 2022-02-18 10:03 ` [Intel-gfx] " Ville Syrjala 2022-02-18 16:28 ` Harry Wentland 2022-02-18 16:28 ` [Intel-gfx] " Harry Wentland 2022-03-15 15:57 ` Alex Deucher 2022-03-15 15:57 ` Alex Deucher 2022-03-15 15:57 ` [Intel-gfx] " Alex Deucher 2022-02-18 10:03 ` [PATCH 03/22] drm/amdgpu: Use drm_mode_init() for on-stack modes Ville Syrjala 2022-02-18 10:03 ` Ville Syrjala 2022-02-18 10:03 ` [Intel-gfx] " Ville Syrjala 2022-02-18 16:30 ` Harry Wentland 2022-02-18 16:30 ` [Intel-gfx] " Harry Wentland 2022-02-18 10:03 ` [PATCH 04/22] drm/amdgpu: Use drm_mode_copy() Ville Syrjala 2022-02-18 10:03 ` Ville Syrjala 2022-02-18 10:03 ` [Intel-gfx] " Ville Syrjala 2022-02-18 16:32 ` Harry Wentland 2022-02-18 16:32 ` [Intel-gfx] " Harry Wentland 2022-03-15 18:50 ` Alex Deucher 2022-03-15 18:50 ` Alex Deucher 2022-03-15 18:50 ` [Intel-gfx] " Alex Deucher 2022-02-18 10:03 ` [PATCH 05/22] drm/radeon: " Ville Syrjala 2022-02-18 10:03 ` [Intel-gfx] " Ville Syrjala 2022-03-15 18:49 ` Alex Deucher 2022-03-15 18:49 ` [Intel-gfx] " Alex Deucher 2022-02-18 10:03 ` [PATCH 06/22] drm/bridge: " Ville Syrjala 2022-02-18 10:03 ` [Intel-gfx] " Ville Syrjala 2022-02-18 11:25 ` Andrzej Hajda 2022-02-18 11:25 ` [Intel-gfx] " Andrzej Hajda 2022-02-18 17:47 ` Laurent Pinchart 2022-02-18 17:47 ` [Intel-gfx] " Laurent Pinchart 2022-02-18 10:03 ` [PATCH 07/22] drm/gma500: " Ville Syrjala 2022-02-18 10:03 ` [Intel-gfx] " Ville Syrjala 2022-03-16 13:16 ` Patrik Jakobsson 2022-03-16 13:16 ` [Intel-gfx] " Patrik Jakobsson 2022-02-18 10:03 ` [PATCH 08/22] drm/hisilicon: Use drm_mode_init() for on-stack modes Ville Syrjala 2022-02-18 10:03 ` [Intel-gfx] " Ville Syrjala 2022-02-18 10:03 ` Ville Syrjala [this message] 2022-02-18 10:03 ` [Intel-gfx] [PATCH 09/22] drm/imx: Use drm_mode_duplicate() Ville Syrjala 2022-02-18 11:42 ` Philipp Zabel 2022-02-18 11:42 ` [Intel-gfx] " Philipp Zabel 2022-02-18 10:03 ` [PATCH 10/22] drm/msm: Nuke weird on stack mode copy Ville Syrjala 2022-02-18 10:03 ` [Intel-gfx] " Ville Syrjala 2022-02-18 10:03 ` Ville Syrjala 2022-03-23 10:19 ` Dmitry Baryshkov 2022-03-23 10:19 ` [Intel-gfx] " Dmitry Baryshkov 2022-03-23 10:19 ` Dmitry Baryshkov 2022-02-18 10:03 ` [PATCH 11/22] drm/msm: Use drm_mode_init() for on-stack modes Ville Syrjala 2022-02-18 10:03 ` [Intel-gfx] " Ville Syrjala 2022-02-18 10:03 ` Ville Syrjala 2022-03-23 10:11 ` Dmitry Baryshkov 2022-03-23 10:11 ` [Intel-gfx] " Dmitry Baryshkov 2022-03-23 10:11 ` Dmitry Baryshkov 2022-03-23 20:04 ` Abhinav Kumar 2022-03-23 20:04 ` [Intel-gfx] " Abhinav Kumar 2022-03-23 20:04 ` Abhinav Kumar 2022-02-18 10:03 ` [PATCH 12/22] drm/msm: Use drm_mode_copy() Ville Syrjala 2022-02-18 10:03 ` [Intel-gfx] " Ville Syrjala 2022-02-18 10:03 ` Ville Syrjala 2022-03-23 10:12 ` Dmitry Baryshkov 2022-03-23 10:12 ` [Intel-gfx] " Dmitry Baryshkov 2022-03-23 10:12 ` Dmitry Baryshkov 2022-03-23 20:09 ` Abhinav Kumar 2022-03-23 20:09 ` [Intel-gfx] " Abhinav Kumar 2022-03-23 20:09 ` Abhinav Kumar 2022-02-18 10:03 ` [PATCH 13/22] drm/mtk: Use drm_mode_init() for on-stack modes Ville Syrjala 2022-02-18 10:03 ` [Intel-gfx] " Ville Syrjala 2022-02-18 10:03 ` [PATCH 14/22] drm/rockchip: Use drm_mode_copy() Ville Syrjala 2022-02-18 10:03 ` Ville Syrjala 2022-02-18 10:03 ` Ville Syrjala 2022-02-18 10:03 ` [Intel-gfx] " Ville Syrjala 2022-02-18 10:03 ` [PATCH 15/22] drm/sti: " Ville Syrjala 2022-02-18 10:03 ` [Intel-gfx] " Ville Syrjala 2022-02-18 10:03 ` [PATCH 16/22] drm/tilcdc: " Ville Syrjala 2022-02-18 10:03 ` [Intel-gfx] " Ville Syrjala 2022-03-15 4:54 ` Tomi Valkeinen 2022-03-15 4:54 ` [Intel-gfx] " Tomi Valkeinen 2022-02-18 10:03 ` [Intel-gfx] [PATCH 17/22] drm/vc4: " Ville Syrjala 2022-02-18 10:03 ` Ville Syrjala 2022-02-21 10:13 ` (subset) [Intel-gfx] " Maxime Ripard 2022-02-21 10:13 ` [Intel-gfx] (subset) " Maxime Ripard 2022-02-18 10:03 ` [PATCH 18/22] drm/i915: Use drm_mode_init() for on-stack modes Ville Syrjala 2022-02-18 10:03 ` [Intel-gfx] " Ville Syrjala 2022-03-16 8:00 ` Jani Nikula 2022-03-21 18:57 ` Ville Syrjälä 2022-03-21 18:57 ` [cocci] " Ville Syrjälä 2022-03-21 20:48 ` Julia Lawall 2022-03-21 20:48 ` Julia Lawall 2022-02-18 10:04 ` [PATCH 19/22] drm/i915: Use drm_mode_copy() Ville Syrjala 2022-02-18 10:04 ` [Intel-gfx] " Ville Syrjala 2022-03-16 8:04 ` Jani Nikula 2022-02-18 10:04 ` [PATCH 20/22] drm/panel: Use drm_mode_duplicate() Ville Syrjala 2022-02-18 10:04 ` [Intel-gfx] " Ville Syrjala 2022-02-18 11:51 ` Sam Ravnborg 2022-02-18 11:51 ` [Intel-gfx] " Sam Ravnborg 2022-02-18 10:04 ` [PATCH 21/22] drm: Use drm_mode_init() for on-stack modes Ville Syrjala 2022-02-18 10:04 ` [Intel-gfx] " Ville Syrjala 2022-03-22 11:10 ` Andrzej Hajda 2022-03-22 11:10 ` [Intel-gfx] " Andrzej Hajda 2022-02-18 10:04 ` [PATCH 22/22] drm: Use drm_mode_copy() Ville Syrjala 2022-02-18 10:04 ` [Intel-gfx] " Ville Syrjala 2022-03-22 11:19 ` Andrzej Hajda 2022-03-22 11:19 ` [Intel-gfx] " Andrzej Hajda 2022-02-18 13:43 ` [Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm: Review of mode copies Patchwork 2022-02-18 14:16 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork 2022-02-19 5:06 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork 2022-03-14 22:11 ` [PATCH 00/22] " Ville Syrjälä 2022-03-14 22:11 ` Ville Syrjälä 2022-03-14 22:11 ` Ville Syrjälä 2022-03-14 22:11 ` [Intel-gfx] " Ville Syrjälä 2022-03-14 22:11 ` Ville Syrjälä 2022-03-14 22:11 ` Ville Syrjälä 2022-03-15 18:52 ` Alex Deucher 2022-03-15 18:52 ` Alex Deucher 2022-03-15 18:52 ` Alex Deucher 2022-03-15 18:52 ` [Intel-gfx] " Alex Deucher 2022-03-15 18:52 ` Alex Deucher 2022-03-15 18:52 ` Alex Deucher 2022-03-21 22:37 ` Ville Syrjälä 2022-03-21 22:37 ` Ville Syrjälä 2022-03-21 22:37 ` Ville Syrjälä 2022-03-21 22:37 ` Ville Syrjälä 2022-03-21 22:37 ` Ville Syrjälä 2022-03-21 22:37 ` [Intel-gfx] " Ville Syrjälä 2022-03-23 10:39 ` Dmitry Baryshkov 2022-03-23 10:39 ` [Intel-gfx] " Dmitry Baryshkov 2022-03-23 10:39 ` Dmitry Baryshkov 2022-03-23 10:39 ` Dmitry Baryshkov 2022-03-23 10:39 ` Dmitry Baryshkov 2022-03-23 10:39 ` Dmitry Baryshkov 2022-03-23 15:10 ` Ville Syrjälä 2022-03-23 15:10 ` Ville Syrjälä 2022-03-23 15:10 ` Ville Syrjälä 2022-03-23 15:10 ` Ville Syrjälä 2022-03-23 15:10 ` [Intel-gfx] " Ville Syrjälä 2022-03-23 15:10 ` Ville Syrjälä 2022-03-23 20:50 ` Abhinav Kumar 2022-03-23 20:50 ` Abhinav Kumar 2022-03-23 20:50 ` [Intel-gfx] " Abhinav Kumar 2022-03-23 20:50 ` Abhinav Kumar 2022-03-23 20:50 ` Abhinav Kumar
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=20220218100403.7028-10-ville.syrjala@linux.intel.com \ --to=ville.syrjala@linux.intel.com \ --cc=dri-devel@lists.freedesktop.org \ --cc=intel-gfx@lists.freedesktop.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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.