* [PATCH v2 11/37] drm/ingenic: ipu: Switch to drm_atomic_get_new_crtc_state()
2025-09-02 9:34 [PATCH v2 00/37] drm/atomic: Get rid of existing states (not really) Maxime Ripard
@ 2025-09-02 9:35 ` Maxime Ripard
2025-09-03 19:03 ` Paul Cercueil
2025-09-02 9:35 ` [PATCH v2 19/37] drm/ingenic: " Maxime Ripard
` (2 subsequent siblings)
3 siblings, 1 reply; 7+ messages in thread
From: Maxime Ripard @ 2025-09-02 9:35 UTC (permalink / raw)
To: Maarten Lankhorst, Thomas Zimmermann, David Airlie, Simona Vetter
Cc: dri-devel, Maxime Ripard, Paul Cercueil, linux-mips
The ingenic IPU atomic_set_property implementation uses the deprecated
drm_atomic_get_existing_crtc_state() helper.
This hook is called during the state building process, before
atomic_check, and thus before the states are swapped. The existing state
thus points to the new state, and we can use
drm_atomic_get_new_crtc_state() instead.
Signed-off-by: Maxime Ripard <mripard@kernel.org>
---
To: Paul Cercueil <paul@crapouillou.net>
Cc: linux-mips@vger.kernel.org
---
drivers/gpu/drm/ingenic/ingenic-ipu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/ingenic/ingenic-ipu.c b/drivers/gpu/drm/ingenic/ingenic-ipu.c
index 26ebf424d63ec21ccee80221745c3e8bcc6b3d7f..2574a4b4d40a2c27cb212114117829d9f6ab3ddb 100644
--- a/drivers/gpu/drm/ingenic/ingenic-ipu.c
+++ b/drivers/gpu/drm/ingenic/ingenic-ipu.c
@@ -703,11 +703,11 @@ ingenic_ipu_plane_atomic_set_property(struct drm_plane *plane,
mode_changed = val != ipu->sharpness;
ipu->sharpness = val;
if (state->crtc) {
- crtc_state = drm_atomic_get_existing_crtc_state(state->state, state->crtc);
+ crtc_state = drm_atomic_get_new_crtc_state(state->state, state->crtc);
if (WARN_ON(!crtc_state))
return -EINVAL;
crtc_state->mode_changed |= mode_changed;
}
--
2.50.1
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH v2 11/37] drm/ingenic: ipu: Switch to drm_atomic_get_new_crtc_state()
2025-09-02 9:35 ` [PATCH v2 11/37] drm/ingenic: ipu: Switch to drm_atomic_get_new_crtc_state() Maxime Ripard
@ 2025-09-03 19:03 ` Paul Cercueil
0 siblings, 0 replies; 7+ messages in thread
From: Paul Cercueil @ 2025-09-03 19:03 UTC (permalink / raw)
To: Maxime Ripard, Maarten Lankhorst, Thomas Zimmermann, David Airlie,
Simona Vetter
Cc: dri-devel, linux-mips
Le mardi 02 septembre 2025 à 11:35 +0200, Maxime Ripard a écrit :
> The ingenic IPU atomic_set_property implementation uses the
> deprecated
> drm_atomic_get_existing_crtc_state() helper.
>
> This hook is called during the state building process, before
> atomic_check, and thus before the states are swapped. The existing
> state
> thus points to the new state, and we can use
> drm_atomic_get_new_crtc_state() instead.
>
> Signed-off-by: Maxime Ripard <mripard@kernel.org>
Tested it and my screen still works.
Reviewed-by: Paul Cercueil <paul@crapouillou.net>
and/or:
Tested-by: Paul Cercueil <paul@crapouillou.net>
Cheers,
-Paul
>
> ---
> To: Paul Cercueil <paul@crapouillou.net>
> Cc: linux-mips@vger.kernel.org
> ---
> drivers/gpu/drm/ingenic/ingenic-ipu.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/ingenic/ingenic-ipu.c
> b/drivers/gpu/drm/ingenic/ingenic-ipu.c
> index
> 26ebf424d63ec21ccee80221745c3e8bcc6b3d7f..2574a4b4d40a2c27cb212114117
> 829d9f6ab3ddb 100644
> --- a/drivers/gpu/drm/ingenic/ingenic-ipu.c
> +++ b/drivers/gpu/drm/ingenic/ingenic-ipu.c
> @@ -703,11 +703,11 @@ ingenic_ipu_plane_atomic_set_property(struct
> drm_plane *plane,
>
> mode_changed = val != ipu->sharpness;
> ipu->sharpness = val;
>
> if (state->crtc) {
> - crtc_state =
> drm_atomic_get_existing_crtc_state(state->state, state->crtc);
> + crtc_state = drm_atomic_get_new_crtc_state(state-
> >state, state->crtc);
> if (WARN_ON(!crtc_state))
> return -EINVAL;
>
> crtc_state->mode_changed |= mode_changed;
> }
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v2 19/37] drm/ingenic: Switch to drm_atomic_get_new_crtc_state()
2025-09-02 9:34 [PATCH v2 00/37] drm/atomic: Get rid of existing states (not really) Maxime Ripard
2025-09-02 9:35 ` [PATCH v2 11/37] drm/ingenic: ipu: Switch to drm_atomic_get_new_crtc_state() Maxime Ripard
@ 2025-09-02 9:35 ` Maxime Ripard
2025-09-03 19:04 ` Paul Cercueil
2025-09-02 14:29 ` [PATCH v2 00/37] drm/atomic: Get rid of existing states (not really) Ville Syrjälä
2025-09-03 19:09 ` Ville Syrjälä
3 siblings, 1 reply; 7+ messages in thread
From: Maxime Ripard @ 2025-09-02 9:35 UTC (permalink / raw)
To: Maarten Lankhorst, Thomas Zimmermann, David Airlie, Simona Vetter
Cc: dri-devel, Maxime Ripard, Paul Cercueil, linux-mips
The ingenic atomic_check implementation uses the deprecated
drm_atomic_get_existing_crtc_state() helper.
This hook is called as part of the global atomic_check, thus before the
states are swapped. The existing state thus points to the new state, and
we can use drm_atomic_get_new_crtc_state() instead.
Signed-off-by: Maxime Ripard <mripard@kernel.org>
---
To: Paul Cercueil <paul@crapouillou.net>
Cc: linux-mips@vger.kernel.org
---
drivers/gpu/drm/ingenic/ingenic-drm-drv.c | 3 +--
drivers/gpu/drm/ingenic/ingenic-ipu.c | 2 +-
2 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/ingenic/ingenic-drm-drv.c b/drivers/gpu/drm/ingenic/ingenic-drm-drv.c
index 9db1ceaed5188a4ef0897280dc72108eb3815b5f..05faed933e5619c796f2a4fa1906e0eaa029ac68 100644
--- a/drivers/gpu/drm/ingenic/ingenic-drm-drv.c
+++ b/drivers/gpu/drm/ingenic/ingenic-drm-drv.c
@@ -469,12 +469,11 @@ static int ingenic_drm_plane_atomic_check(struct drm_plane *plane,
return 0;
if (priv->soc_info->plane_f0_not_working && plane == &priv->f0)
return -EINVAL;
- crtc_state = drm_atomic_get_existing_crtc_state(state,
- crtc);
+ crtc_state = drm_atomic_get_new_crtc_state(state, crtc);
if (WARN_ON(!crtc_state))
return -EINVAL;
priv_state = ingenic_drm_get_priv_state(priv, state);
if (IS_ERR(priv_state))
diff --git a/drivers/gpu/drm/ingenic/ingenic-ipu.c b/drivers/gpu/drm/ingenic/ingenic-ipu.c
index 2574a4b4d40a2c27cb212114117829d9f6ab3ddb..32638a713241abbd4eaed09f0aaec2b790650cc9 100644
--- a/drivers/gpu/drm/ingenic/ingenic-ipu.c
+++ b/drivers/gpu/drm/ingenic/ingenic-ipu.c
@@ -578,11 +578,11 @@ static int ingenic_ipu_plane_atomic_check(struct drm_plane *plane,
struct ingenic_ipu_private_state *ipu_state;
if (!crtc)
return 0;
- crtc_state = drm_atomic_get_existing_crtc_state(state, crtc);
+ crtc_state = drm_atomic_get_new_crtc_state(state, crtc);
if (WARN_ON(!crtc_state))
return -EINVAL;
ipu_state = ingenic_ipu_get_priv_state(ipu, state);
if (IS_ERR(ipu_state))
--
2.50.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2 19/37] drm/ingenic: Switch to drm_atomic_get_new_crtc_state()
2025-09-02 9:35 ` [PATCH v2 19/37] drm/ingenic: " Maxime Ripard
@ 2025-09-03 19:04 ` Paul Cercueil
0 siblings, 0 replies; 7+ messages in thread
From: Paul Cercueil @ 2025-09-03 19:04 UTC (permalink / raw)
To: Maxime Ripard, Maarten Lankhorst, Thomas Zimmermann, David Airlie,
Simona Vetter
Cc: dri-devel, linux-mips
Hi Maxime,
Le mardi 02 septembre 2025 à 11:35 +0200, Maxime Ripard a écrit :
> The ingenic atomic_check implementation uses the deprecated
> drm_atomic_get_existing_crtc_state() helper.
>
> This hook is called as part of the global atomic_check, thus before
> the
> states are swapped. The existing state thus points to the new state,
> and
> we can use drm_atomic_get_new_crtc_state() instead.
>
> Signed-off-by: Maxime Ripard <mripard@kernel.org>
Reviewed-by: Paul Cercueil <paul@crapouillou.net>
and/or:
Tested-by: Paul Cercueil <paul@crapouillou.net>
Cheers,
-Paul
>
> ---
> To: Paul Cercueil <paul@crapouillou.net>
> Cc: linux-mips@vger.kernel.org
> ---
> drivers/gpu/drm/ingenic/ingenic-drm-drv.c | 3 +--
> drivers/gpu/drm/ingenic/ingenic-ipu.c | 2 +-
> 2 files changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/ingenic/ingenic-drm-drv.c
> b/drivers/gpu/drm/ingenic/ingenic-drm-drv.c
> index
> 9db1ceaed5188a4ef0897280dc72108eb3815b5f..05faed933e5619c796f2a4fa190
> 6e0eaa029ac68 100644
> --- a/drivers/gpu/drm/ingenic/ingenic-drm-drv.c
> +++ b/drivers/gpu/drm/ingenic/ingenic-drm-drv.c
> @@ -469,12 +469,11 @@ static int
> ingenic_drm_plane_atomic_check(struct drm_plane *plane,
> return 0;
>
> if (priv->soc_info->plane_f0_not_working && plane == &priv-
> >f0)
> return -EINVAL;
>
> - crtc_state = drm_atomic_get_existing_crtc_state(state,
> - crtc);
> + crtc_state = drm_atomic_get_new_crtc_state(state, crtc);
> if (WARN_ON(!crtc_state))
> return -EINVAL;
>
> priv_state = ingenic_drm_get_priv_state(priv, state);
> if (IS_ERR(priv_state))
> diff --git a/drivers/gpu/drm/ingenic/ingenic-ipu.c
> b/drivers/gpu/drm/ingenic/ingenic-ipu.c
> index
> 2574a4b4d40a2c27cb212114117829d9f6ab3ddb..32638a713241abbd4eaed09f0aa
> ec2b790650cc9 100644
> --- a/drivers/gpu/drm/ingenic/ingenic-ipu.c
> +++ b/drivers/gpu/drm/ingenic/ingenic-ipu.c
> @@ -578,11 +578,11 @@ static int
> ingenic_ipu_plane_atomic_check(struct drm_plane *plane,
> struct ingenic_ipu_private_state *ipu_state;
>
> if (!crtc)
> return 0;
>
> - crtc_state = drm_atomic_get_existing_crtc_state(state,
> crtc);
> + crtc_state = drm_atomic_get_new_crtc_state(state, crtc);
> if (WARN_ON(!crtc_state))
> return -EINVAL;
>
> ipu_state = ingenic_ipu_get_priv_state(ipu, state);
> if (IS_ERR(ipu_state))
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 00/37] drm/atomic: Get rid of existing states (not really)
2025-09-02 9:34 [PATCH v2 00/37] drm/atomic: Get rid of existing states (not really) Maxime Ripard
2025-09-02 9:35 ` [PATCH v2 11/37] drm/ingenic: ipu: Switch to drm_atomic_get_new_crtc_state() Maxime Ripard
2025-09-02 9:35 ` [PATCH v2 19/37] drm/ingenic: " Maxime Ripard
@ 2025-09-02 14:29 ` Ville Syrjälä
2025-09-03 19:09 ` Ville Syrjälä
3 siblings, 0 replies; 7+ messages in thread
From: Ville Syrjälä @ 2025-09-02 14:29 UTC (permalink / raw)
To: Maxime Ripard
Cc: Maarten Lankhorst, Thomas Zimmermann, David Airlie, Simona Vetter,
dri-devel, Dmitry Baryshkov, Louis Chauvet, Haneen Mohammed,
Melissa Wen, Jyri Sarha, Tomi Valkeinen, Paul Cercueil,
linux-mips, Liviu Dudau, Russell King, Manikandan Muralidharan,
Dharma Balasubiramani, Nicolas Ferre, Alexandre Belloni,
Claudiu Beznea, linux-arm-kernel, Inki Dae, Seung-Woo Kim,
Kyungmin Park, Krzysztof Kozlowski, Alim Akhtar,
linux-samsung-soc, Liu Ying, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, imx, Laurentiu Palcu,
Lucas Stach, Philipp Zabel, Anitha Chrisanthus, Edmund Dea,
Paul Kocialkowski, Sui Jingfeng, Chun-Kuang Hu, Matthias Brugger,
AngeloGioacchino Del Regno, linux-mediatek, linux-kernel,
Rob Clark, Dmitry Baryshkov, Abhinav Kumar, Jessica Zhang <
On Tue, Sep 02, 2025 at 11:34:59AM +0200, Maxime Ripard wrote:
> Hi,
>
> Here's a series to get rid of the drm_atomic_helper_get_existing_*_state
> accessors.
>
> The initial intent was to remove the __drm_*_state->state pointer to
> only rely on old and new states, but we still need it now to know which
> of the two we need to free: if a state has not been committed (either
> dropped or checked only), then we need to free the new one, if it has
> been committed we need to free the old state.
>
> Thus, the state pointer is kept (and documented) only to point to the
> state we should free eventually.
>
> All users have been converted to the relevant old or new state
> accessors.
>
> This was tested on tidss.
>
> Let me know what you think,
> Maxime
>
> Signed-off-by: Maxime Ripard <mripard@kernel.org>
Other than the pre-existing ingenic private state issue that
Dmitry spotted I didn't see anything obviously wrong.
So apart from that the series is
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> ---
> Changes in v2:
> - Dropped the first and second patches
> - Reworked the recipient list to be nicer with SMTPs
> - Link to v1: https://lore.kernel.org/r/20250825-drm-no-more-existing-state-v1-0-f08ccd9f85c9@kernel.org
>
> ---
> Maxime Ripard (37):
> drm/atomic: Convert drm_atomic_get_connector_state() to use new connector state
> drm/atomic: Remove unused drm_atomic_get_existing_connector_state()
> drm/atomic: Document __drm_connectors_state state pointer
> drm/atomic: Convert __drm_atomic_get_current_plane_state() to modern accessor
> drm/atomic: Convert drm_atomic_get_plane_state() to use new plane state
> drm/vkms: Convert vkms_crtc_atomic_check() to use new plane state
> drm/tilcdc: crtc: Use drm_atomic_helper_check_crtc_primary_plane()
> drm/atomic: Remove unused drm_atomic_get_existing_plane_state()
> drm/atomic: Document __drm_planes_state state pointer
> drm/atomic: Convert drm_atomic_get_crtc_state() to use new connector state
> drm/ingenic: ipu: Switch to drm_atomic_get_new_crtc_state()
> drm/arm/malidp: Switch to drm_atomic_get_new_crtc_state()
> drm/armada: Switch to drm_atomic_get_new_crtc_state()
> drm/atmel-hlcdc: Switch to drm_atomic_get_new_crtc_state()
> drm/exynos: Switch to drm_atomic_get_new_crtc_state()
> drm/imx-dc: Switch to drm_atomic_get_new_crtc_state()
> drm/imx-dcss: Switch to drm_atomic_get_new_crtc_state()
> drm/imx-ipuv3: Switch to drm_atomic_get_new_crtc_state()
> drm/ingenic: Switch to drm_atomic_get_new_crtc_state()
> drm/kmb: Switch to drm_atomic_get_new_crtc_state()
> drm/logicvc: Switch to drm_atomic_get_new_crtc_state()
> drm/loongson: Switch to drm_atomic_get_new_crtc_state()
> drm/mediatek: Switch to drm_atomic_get_new_crtc_state()
> drm/msm/mdp5: Switch to drm_atomic_get_new_crtc_state()
> drm/omap: Switch to drm_atomic_get_new_crtc_state()
> drm/rockchip: Switch to drm_atomic_get_new_crtc_state()
> drm/sun4i: Switch to drm_atomic_get_new_crtc_state()
> drm/tegra: Switch to drm_atomic_get_new_crtc_state()
> drm/tilcdc: Switch to drm_atomic_get_new_crtc_state()
> drm/vboxvideo: Switch to drm_atomic_get_new_crtc_state()
> drm/vc4: Switch to drm_atomic_get_new_crtc_state()
> drm/atomic: Switch to drm_atomic_get_new_crtc_state()
> drm/framebuffer: Switch to drm_atomic_get_new_crtc_state()
> drm/atomic: Remove unused drm_atomic_get_existing_crtc_state()
> drm/atomic: Document __drm_crtcs_state state pointer
> drm/atomic: Convert drm_atomic_get_private_obj_state() to use new plane state
> drm/atomic: Document __drm_private_objs_state state pointer
>
> drivers/gpu/drm/arm/malidp_planes.c | 2 +-
> drivers/gpu/drm/armada/armada_plane.c | 3 +-
> drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 2 +-
> drivers/gpu/drm/drm_atomic.c | 21 ++--
> drivers/gpu/drm/drm_framebuffer.c | 2 +-
> drivers/gpu/drm/exynos/exynos_drm_plane.c | 2 +-
> drivers/gpu/drm/imx/dc/dc-plane.c | 2 +-
> drivers/gpu/drm/imx/dcss/dcss-plane.c | 4 +-
> drivers/gpu/drm/imx/ipuv3/ipuv3-plane.c | 3 +-
> drivers/gpu/drm/ingenic/ingenic-drm-drv.c | 3 +-
> drivers/gpu/drm/ingenic/ingenic-ipu.c | 4 +-
> drivers/gpu/drm/kmb/kmb_plane.c | 3 +-
> drivers/gpu/drm/logicvc/logicvc_layer.c | 4 +-
> drivers/gpu/drm/loongson/lsdc_plane.c | 2 +-
> drivers/gpu/drm/mediatek/mtk_plane.c | 3 +-
> drivers/gpu/drm/msm/disp/mdp5/mdp5_plane.c | 7 +-
> drivers/gpu/drm/omapdrm/omap_plane.c | 2 +-
> drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 6 +-
> drivers/gpu/drm/rockchip/rockchip_drm_vop2.c | 2 +-
> drivers/gpu/drm/sun4i/sun8i_ui_layer.c | 3 +-
> drivers/gpu/drm/sun4i/sun8i_vi_layer.c | 3 +-
> drivers/gpu/drm/tegra/dc.c | 2 +-
> drivers/gpu/drm/tilcdc/tilcdc_crtc.c | 9 +-
> drivers/gpu/drm/tilcdc/tilcdc_plane.c | 3 +-
> drivers/gpu/drm/vboxvideo/vbox_mode.c | 8 +-
> drivers/gpu/drm/vc4/vc4_plane.c | 6 +-
> drivers/gpu/drm/vkms/vkms_crtc.c | 4 +-
> include/drm/drm_atomic.h | 144 ++++++++++++------------
> 28 files changed, 124 insertions(+), 135 deletions(-)
> ---
> base-commit: 7fa4d8dc380fbd81a9d702a855c50690c9c6442c
> change-id: 20250825-drm-no-more-existing-state-9b3252c1a33b
>
> Best regards,
> --
> Maxime Ripard <mripard@kernel.org>
--
Ville Syrjälä
Intel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 00/37] drm/atomic: Get rid of existing states (not really)
2025-09-02 9:34 [PATCH v2 00/37] drm/atomic: Get rid of existing states (not really) Maxime Ripard
` (2 preceding siblings ...)
2025-09-02 14:29 ` [PATCH v2 00/37] drm/atomic: Get rid of existing states (not really) Ville Syrjälä
@ 2025-09-03 19:09 ` Ville Syrjälä
3 siblings, 0 replies; 7+ messages in thread
From: Ville Syrjälä @ 2025-09-03 19:09 UTC (permalink / raw)
To: Maxime Ripard
Cc: Maarten Lankhorst, Thomas Zimmermann, David Airlie, Simona Vetter,
dri-devel, Dmitry Baryshkov, Louis Chauvet, Haneen Mohammed,
Melissa Wen, Jyri Sarha, Tomi Valkeinen, Paul Cercueil,
linux-mips, Liviu Dudau, Russell King, Manikandan Muralidharan,
Dharma Balasubiramani, Nicolas Ferre, Alexandre Belloni,
Claudiu Beznea, linux-arm-kernel, Inki Dae, Seung-Woo Kim,
Kyungmin Park, Krzysztof Kozlowski, Alim Akhtar,
linux-samsung-soc, Liu Ying, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, imx, Laurentiu Palcu,
Lucas Stach, Philipp Zabel, Anitha Chrisanthus, Edmund Dea,
Paul Kocialkowski, Sui Jingfeng, Chun-Kuang Hu, Matthias Brugger,
AngeloGioacchino Del Regno, linux-mediatek, linux-kernel,
Rob Clark, Dmitry Baryshkov, Abhinav Kumar, Jessica Zhang,
Sean Paul, Marijn Suijten, linux-arm-msm, freedreno, Sandy Huang,
Heiko Stübner, Andy Yan, linux-rockchip, Chen-Yu Tsai,
Jernej Skrabec, Samuel Holland, linux-sunxi, Thierry Reding,
Mikko Perttunen, Jonathan Hunter, linux-tegra, Hans de Goede,
Dave Stevenson, Maíra Canal, Raspberry Pi Kernel Maintenance
On Tue, Sep 02, 2025 at 11:34:59AM +0200, Maxime Ripard wrote:
> Hi,
>
> Here's a series to get rid of the drm_atomic_helper_get_existing_*_state
> accessors.
>
> The initial intent was to remove the __drm_*_state->state pointer to
> only rely on old and new states, but we still need it now to know which
> of the two we need to free: if a state has not been committed (either
> dropped or checked only), then we need to free the new one, if it has
> been committed we need to free the old state.
>
> Thus, the state pointer is kept (and documented) only to point to the
> state we should free eventually.
>
> All users have been converted to the relevant old or new state
> accessors.
>
> This was tested on tidss.
>
> Let me know what you think,
> Maxime
>
> Signed-off-by: Maxime Ripard <mripard@kernel.org>
<replying again to see if my smtp issues are now solved...>
Other than the pre-existing ingenic private state issue that
Dmitry spotted I didn't see anything obviously wrong.
So apart from that the series is
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> ---
> Changes in v2:
> - Dropped the first and second patches
> - Reworked the recipient list to be nicer with SMTPs
> - Link to v1: https://lore.kernel.org/r/20250825-drm-no-more-existing-state-v1-0-f08ccd9f85c9@kernel.org
>
> ---
> Maxime Ripard (37):
> drm/atomic: Convert drm_atomic_get_connector_state() to use new connector state
> drm/atomic: Remove unused drm_atomic_get_existing_connector_state()
> drm/atomic: Document __drm_connectors_state state pointer
> drm/atomic: Convert __drm_atomic_get_current_plane_state() to modern accessor
> drm/atomic: Convert drm_atomic_get_plane_state() to use new plane state
> drm/vkms: Convert vkms_crtc_atomic_check() to use new plane state
> drm/tilcdc: crtc: Use drm_atomic_helper_check_crtc_primary_plane()
> drm/atomic: Remove unused drm_atomic_get_existing_plane_state()
> drm/atomic: Document __drm_planes_state state pointer
> drm/atomic: Convert drm_atomic_get_crtc_state() to use new connector state
> drm/ingenic: ipu: Switch to drm_atomic_get_new_crtc_state()
> drm/arm/malidp: Switch to drm_atomic_get_new_crtc_state()
> drm/armada: Switch to drm_atomic_get_new_crtc_state()
> drm/atmel-hlcdc: Switch to drm_atomic_get_new_crtc_state()
> drm/exynos: Switch to drm_atomic_get_new_crtc_state()
> drm/imx-dc: Switch to drm_atomic_get_new_crtc_state()
> drm/imx-dcss: Switch to drm_atomic_get_new_crtc_state()
> drm/imx-ipuv3: Switch to drm_atomic_get_new_crtc_state()
> drm/ingenic: Switch to drm_atomic_get_new_crtc_state()
> drm/kmb: Switch to drm_atomic_get_new_crtc_state()
> drm/logicvc: Switch to drm_atomic_get_new_crtc_state()
> drm/loongson: Switch to drm_atomic_get_new_crtc_state()
> drm/mediatek: Switch to drm_atomic_get_new_crtc_state()
> drm/msm/mdp5: Switch to drm_atomic_get_new_crtc_state()
> drm/omap: Switch to drm_atomic_get_new_crtc_state()
> drm/rockchip: Switch to drm_atomic_get_new_crtc_state()
> drm/sun4i: Switch to drm_atomic_get_new_crtc_state()
> drm/tegra: Switch to drm_atomic_get_new_crtc_state()
> drm/tilcdc: Switch to drm_atomic_get_new_crtc_state()
> drm/vboxvideo: Switch to drm_atomic_get_new_crtc_state()
> drm/vc4: Switch to drm_atomic_get_new_crtc_state()
> drm/atomic: Switch to drm_atomic_get_new_crtc_state()
> drm/framebuffer: Switch to drm_atomic_get_new_crtc_state()
> drm/atomic: Remove unused drm_atomic_get_existing_crtc_state()
> drm/atomic: Document __drm_crtcs_state state pointer
> drm/atomic: Convert drm_atomic_get_private_obj_state() to use new plane state
> drm/atomic: Document __drm_private_objs_state state pointer
>
> drivers/gpu/drm/arm/malidp_planes.c | 2 +-
> drivers/gpu/drm/armada/armada_plane.c | 3 +-
> drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 2 +-
> drivers/gpu/drm/drm_atomic.c | 21 ++--
> drivers/gpu/drm/drm_framebuffer.c | 2 +-
> drivers/gpu/drm/exynos/exynos_drm_plane.c | 2 +-
> drivers/gpu/drm/imx/dc/dc-plane.c | 2 +-
> drivers/gpu/drm/imx/dcss/dcss-plane.c | 4 +-
> drivers/gpu/drm/imx/ipuv3/ipuv3-plane.c | 3 +-
> drivers/gpu/drm/ingenic/ingenic-drm-drv.c | 3 +-
> drivers/gpu/drm/ingenic/ingenic-ipu.c | 4 +-
> drivers/gpu/drm/kmb/kmb_plane.c | 3 +-
> drivers/gpu/drm/logicvc/logicvc_layer.c | 4 +-
> drivers/gpu/drm/loongson/lsdc_plane.c | 2 +-
> drivers/gpu/drm/mediatek/mtk_plane.c | 3 +-
> drivers/gpu/drm/msm/disp/mdp5/mdp5_plane.c | 7 +-
> drivers/gpu/drm/omapdrm/omap_plane.c | 2 +-
> drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 6 +-
> drivers/gpu/drm/rockchip/rockchip_drm_vop2.c | 2 +-
> drivers/gpu/drm/sun4i/sun8i_ui_layer.c | 3 +-
> drivers/gpu/drm/sun4i/sun8i_vi_layer.c | 3 +-
> drivers/gpu/drm/tegra/dc.c | 2 +-
> drivers/gpu/drm/tilcdc/tilcdc_crtc.c | 9 +-
> drivers/gpu/drm/tilcdc/tilcdc_plane.c | 3 +-
> drivers/gpu/drm/vboxvideo/vbox_mode.c | 8 +-
> drivers/gpu/drm/vc4/vc4_plane.c | 6 +-
> drivers/gpu/drm/vkms/vkms_crtc.c | 4 +-
> include/drm/drm_atomic.h | 144 ++++++++++++------------
> 28 files changed, 124 insertions(+), 135 deletions(-)
> ---
> base-commit: 7fa4d8dc380fbd81a9d702a855c50690c9c6442c
> change-id: 20250825-drm-no-more-existing-state-9b3252c1a33b
>
> Best regards,
> --
> Maxime Ripard <mripard@kernel.org>
--
Ville Syrjälä
Intel
^ permalink raw reply [flat|nested] 7+ messages in thread