Received: by 2002:a05:6a10:8c0a:0:0:0:0 with SMTP id go10csp1418896pxb; Mon, 22 Feb 2021 01:04:19 -0800 (PST) X-Google-Smtp-Source: ABdhPJwcKAugQfsl46llMC8eL5TBXu0OrdshH7d2M+vpXxjnmHp4Gsr4X/kUw9QTianMEG4l6ctG X-Received: by 2002:aa7:c144:: with SMTP id r4mr9348208edp.251.1613984659386; Mon, 22 Feb 2021 01:04:19 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1613984659; cv=none; d=google.com; s=arc-20160816; b=ZG3HsI5h0B2XHhDKqRir1llN3KalhSmDvn9QaH+pDr+OzVBj/K3lqim6kklY/y4yjR Ew5ma7pd9zrocCV+63yOLr6a5xBkqQdXQJOp1SPjYrKXQpB30v78M57ixTCD6NSRmMDq Ua6TbMxC/RoNyh1/Br1Pz4mVMlgAx6/HacDl0P225VWALKe0Oq/BNqfkQn95Cczw9F56 pOI8roLLPi/i2RYTeKfIGV7u2CsuHNaFKqz51cfhGaz1VXgxnknCmFnHjg/uJ47Zj3KF LCnR5UjRcds7Ru6aNO84J2WzoSDmj98+h2kq/CNDgShduwfTBuoTK/pHuYRC+66b9if7 valg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject; bh=/30maB2VO1Igd9tUIh7UCbskd0OeAd6t/cNmS09O+iY=; b=dK4Q1nsFrfnSoBPObTGZIRaiyUC6/3cNrJzcSkKmFmUEOVqS5eX/Ruw3+yb0fCyGH6 HM6mlrlZathKNc5b805LKsrS+oxVsdQFc/N8cAGVttzfMm4cQfhRci9CfcTTqkBRXh3k HTXE3M9SEOCNFSCjjao0PiDM8sZF/pAqR2XxvWzRv4qNJkQlJxL8yB2efmcvgchq0JWz m19Oa5B691mqr70nUiqggYyzYLdLLFdxMdnqibRGfMohC3fhvoHQu3AbOOBpFYRphWbL k2wADdYqd3o4m3DT9/O1dOGDYnmGevLQtKPTCdObh1Jb6vMfqhEPQ9loYp8YJbmiJ93P esuw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id k7si10911610ejs.165.2021.02.22.01.03.55; Mon, 22 Feb 2021 01:04:19 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230042AbhBVJCE (ORCPT + 99 others); Mon, 22 Feb 2021 04:02:04 -0500 Received: from mx2.suse.de ([195.135.220.15]:55682 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229934AbhBVJBw (ORCPT ); Mon, 22 Feb 2021 04:01:52 -0500 X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay2.suse.de (unknown [195.135.221.27]) by mx2.suse.de (Postfix) with ESMTP id 20F10AFD5; Mon, 22 Feb 2021 09:01:05 +0000 (UTC) Subject: Re: [PATCH v3 04/11] drm/atomic: Pass the full state to planes atomic_check To: Maxime Ripard , Maarten Lankhorst , Daniel Vetter , David Airlie Cc: dri-devel@lists.freedesktop.org, Laurent Pinchart , Harry Wentland , Leo Li , Alex Deucher , =?UTF-8?Q?Christian_K=c3=b6nig?= , Daniel Vetter , "James (Qian) Wang" , Liviu Dudau , Mihail Atanassov , Brian Starkey , Russell King , Dave Airlie , Sam Ravnborg , Boris Brezillon , Nicolas Ferre , Alexandre Belloni , Ludovic Desroches , Maxime Ripard , Inki Dae , Joonyoung Shim , Seung-Woo Kim , Kyungmin Park , Krzysztof Kozlowski , Stefan Agner , Alison Wang , Xinliang Liu , Tian Tao , John Stultz , Xinwei Kong , Chen Feng , Laurentiu Palcu , Lucas Stach , Philipp Zabel , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Paul Cercueil , Anitha Chrisanthus , Edmund Dea , Chun-Kuang Hu , Matthias Brugger , Neil Armstrong , Kevin Hilman , Jerome Brunet , Martin Blumenstingl , Rob Clark , Sean Paul , Marek Vasut , Ben Skeggs , Tomi Valkeinen , Gerd Hoffmann , Kieran Bingham , Sandy Huang , =?UTF-8?Q?Heiko_St=c3=bcbner?= , Benjamin Gaignard , Vincent Abriou , Yannick Fertre , Philippe Cornu , Maxime Coquelin , Alexandre Torgue , Chen-Yu Tsai , Jernej Skrabec , Thierry Reding , Jonathan Hunter , Jyri Sarha , Hans de Goede , Eric Anholt , Rodrigo Siqueira , Melissa Wen , Haneen Mohammed , VMware Graphics , Roland Scheidegger , Zack Rusin , Hyun Kwon , Michal Simek , amd-gfx@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-mips@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-amlogic@lists.infradead.org, linux-arm-msm@vger.kernel.org, freedreno@lists.freedesktop.org, nouveau@lists.freedesktop.org, virtualization@lists.linux-foundation.org, spice-devel@lists.freedesktop.org, linux-renesas-soc@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com, linux-tegra@vger.kernel.org References: <20210219120032.260676-1-maxime@cerno.tech> <20210219120032.260676-4-maxime@cerno.tech> From: Thomas Zimmermann Message-ID: <2c4a7545-acb6-4d20-dee2-f29483679c6c@suse.de> Date: Mon, 22 Feb 2021 10:00:57 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.7.1 MIME-Version: 1.0 In-Reply-To: <20210219120032.260676-4-maxime@cerno.tech> Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="GgHTCSYfhQtxhqzWLFKSFn9iB3UTsE2Ut" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --GgHTCSYfhQtxhqzWLFKSFn9iB3UTsE2Ut Content-Type: multipart/mixed; boundary="DpGLs9b7KGSubA3i3QzqbDsxuJZFsfytr"; protected-headers="v1" From: Thomas Zimmermann To: Maxime Ripard , Maarten Lankhorst , Daniel Vetter , David Airlie Cc: dri-devel@lists.freedesktop.org, Laurent Pinchart , Harry Wentland , Leo Li , Alex Deucher , =?UTF-8?Q?Christian_K=c3=b6nig?= , Daniel Vetter , "James (Qian) Wang" , Liviu Dudau , Mihail Atanassov , Brian Starkey , Russell King , Dave Airlie , Sam Ravnborg , Boris Brezillon , Nicolas Ferre , Alexandre Belloni , Ludovic Desroches , Maxime Ripard , Inki Dae , Joonyoung Shim , Seung-Woo Kim , Kyungmin Park , Krzysztof Kozlowski , Stefan Agner , Alison Wang , Xinliang Liu , Tian Tao , John Stultz , Xinwei Kong , Chen Feng , Laurentiu Palcu , Lucas Stach , Philipp Zabel , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Paul Cercueil , Anitha Chrisanthus , Edmund Dea , Chun-Kuang Hu , Matthias Brugger , Neil Armstrong , Kevin Hilman , Jerome Brunet , Martin Blumenstingl , Rob Clark , Sean Paul , Marek Vasut , Ben Skeggs , Tomi Valkeinen , Gerd Hoffmann , Kieran Bingham , Sandy Huang , =?UTF-8?Q?Heiko_St=c3=bcbner?= , Benjamin Gaignard , Vincent Abriou , Yannick Fertre , Philippe Cornu , Maxime Coquelin , Alexandre Torgue , Chen-Yu Tsai , Jernej Skrabec , Thierry Reding , Jonathan Hunter , Jyri Sarha , Hans de Goede , Eric Anholt , Rodrigo Siqueira , Melissa Wen , Haneen Mohammed , VMware Graphics , Roland Scheidegger , Zack Rusin , Hyun Kwon , Michal Simek , amd-gfx@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-mips@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-amlogic@lists.infradead.org, linux-arm-msm@vger.kernel.org, freedreno@lists.freedesktop.org, nouveau@lists.freedesktop.org, virtualization@lists.linux-foundation.org, spice-devel@lists.freedesktop.org, linux-renesas-soc@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com, linux-tegra@vger.kernel.org Message-ID: <2c4a7545-acb6-4d20-dee2-f29483679c6c@suse.de> Subject: Re: [PATCH v3 04/11] drm/atomic: Pass the full state to planes atomic_check References: <20210219120032.260676-1-maxime@cerno.tech> <20210219120032.260676-4-maxime@cerno.tech> In-Reply-To: <20210219120032.260676-4-maxime@cerno.tech> --DpGLs9b7KGSubA3i3QzqbDsxuJZFsfytr Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: quoted-printable Am 19.02.21 um 13:00 schrieb Maxime Ripard: > The current atomic helpers have either their object state being passed = as > an argument or the full atomic state. >=20 > The former is the pattern that was done at first, before switching to t= he > latter for new hooks or when it was needed. >=20 > Let's convert all the remaining helpers to provide a consistent > interface, starting with the planes atomic_check. >=20 > The conversion was done using the coccinelle script below plus some > manual changes for vmwgfx, built tested on all the drivers. >=20 > @@ > identifier plane, plane_state; > symbol state; > @@ >=20 > struct drm_plane_helper_funcs { > ... > int (*atomic_check)(struct drm_plane *plane, > - struct drm_plane_state *plane_state); > + struct drm_atomic_state *state); > ... > } >=20 > @ plane_atomic_func @ > identifier helpers; > identifier func; > @@ >=20 > static const struct drm_plane_helper_funcs helpers =3D { > ..., > .atomic_check =3D func, > ..., > }; >=20 > @@ > struct drm_plane_helper_funcs *FUNCS; > identifier f; > identifier dev; > identifier plane, plane_state, state; > @@ >=20 > f(struct drm_device *dev, struct drm_atomic_state *state) > { > <+... > - FUNCS->atomic_check(plane, plane_state) > + FUNCS->atomic_check(plane, state) > ...+> > } >=20 > @ ignores_new_state @ > identifier plane_atomic_func.func; > identifier plane, new_plane_state; > @@ >=20 > func(struct drm_plane *plane, struct drm_plane_state *new_plane_state= ) > { > ... when !=3D new_plane_state > } >=20 > @ adds_new_state depends on plane_atomic_func && !ignores_new_state @ > identifier plane_atomic_func.func; > identifier plane, new_plane_state; > @@ >=20 > func(struct drm_plane *plane, struct drm_plane_state *new_plane_state= ) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, plane); > ... > } >=20 > @ depends on plane_atomic_func @ > identifier plane_atomic_func.func; > identifier plane, new_plane_state; > @@ >=20 > func(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state > + struct drm_atomic_state *state > ) > { ... } >=20 > @ include depends on adds_new_state @ > @@ >=20 > #include >=20 > @ no_include depends on !include && adds_new_state @ > @@ >=20 > + #include > #include >=20 > Reviewed-by: Laurent Pinchart > Signed-off-by: Maxime Ripard Acked-by: Thomas Zimmermann >=20 > --- >=20 > Changes from v1: > - Rewording and removal of a coccinelle rule suggested by Laurent > --- > drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 4 +++- > drivers/gpu/drm/arm/display/komeda/komeda_plane.c | 4 +++- > drivers/gpu/drm/arm/hdlcd_crtc.c | 4 +++- > drivers/gpu/drm/arm/malidp_planes.c | 4 +++- > drivers/gpu/drm/armada/armada_plane.c | 4 +++- > drivers/gpu/drm/armada/armada_plane.h | 2 +- > drivers/gpu/drm/ast/ast_mode.c | 8 ++++++-- > drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 3 ++- > drivers/gpu/drm/drm_atomic_helper.c | 2 +- > drivers/gpu/drm/drm_simple_kms_helper.c | 4 +++- > drivers/gpu/drm/exynos/exynos_drm_plane.c | 4 +++- > drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_plane.c | 5 ++++- > drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c | 4 +++- > drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c | 4 +++- > drivers/gpu/drm/imx/dcss/dcss-plane.c | 4 +++- > drivers/gpu/drm/imx/ipuv3-plane.c | 4 +++- > drivers/gpu/drm/ingenic/ingenic-drm-drv.c | 4 +++- > drivers/gpu/drm/ingenic/ingenic-ipu.c | 4 +++- > drivers/gpu/drm/kmb/kmb_plane.c | 4 +++- > drivers/gpu/drm/mediatek/mtk_drm_plane.c | 4 +++- > drivers/gpu/drm/meson/meson_overlay.c | 4 +++- > drivers/gpu/drm/meson/meson_plane.c | 4 +++- > drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c | 5 ++++- > drivers/gpu/drm/msm/disp/mdp4/mdp4_plane.c | 2 +- > drivers/gpu/drm/msm/disp/mdp5/mdp5_plane.c | 4 +++- > drivers/gpu/drm/mxsfb/mxsfb_kms.c | 4 +++- > drivers/gpu/drm/nouveau/dispnv50/wndw.c | 5 ++++- > drivers/gpu/drm/omapdrm/omap_plane.c | 4 +++- > drivers/gpu/drm/qxl/qxl_display.c | 4 +++- > drivers/gpu/drm/rcar-du/rcar_du_plane.c | 4 +++- > drivers/gpu/drm/rcar-du/rcar_du_vsp.c | 5 ++++- > drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 4 +++- > drivers/gpu/drm/sti/sti_cursor.c | 4 +++- > drivers/gpu/drm/sti/sti_gdp.c | 4 +++- > drivers/gpu/drm/sti/sti_hqvdp.c | 4 +++- > drivers/gpu/drm/stm/ltdc.c | 4 +++- > drivers/gpu/drm/sun4i/sun8i_ui_layer.c | 4 +++- > drivers/gpu/drm/sun4i/sun8i_vi_layer.c | 4 +++- > drivers/gpu/drm/tegra/dc.c | 8 ++++++-- > drivers/gpu/drm/tegra/hub.c | 4 +++- > drivers/gpu/drm/tidss/tidss_plane.c | 4 +++- > drivers/gpu/drm/tilcdc/tilcdc_plane.c | 4 +++- > drivers/gpu/drm/vboxvideo/vbox_mode.c | 8 ++++++-- > drivers/gpu/drm/vc4/vc4_plane.c | 4 +++- > drivers/gpu/drm/virtio/virtgpu_plane.c | 4 +++- > drivers/gpu/drm/vkms/vkms_plane.c | 4 +++- > drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 8 ++++++-- > drivers/gpu/drm/vmwgfx/vmwgfx_kms.h | 4 ++-- > drivers/gpu/drm/xlnx/zynqmp_disp.c | 4 +++- > drivers/gpu/drm/zte/zx_plane.c | 8 ++++++-- > include/drm/drm_modeset_helper_vtables.h | 7 +++---- > 51 files changed, 164 insertions(+), 60 deletions(-) >=20 > diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/driver= s/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c > index 906fa4ae25c9..1cdff048b0c0 100644 > --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c > +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c > @@ -6432,8 +6432,10 @@ static int dm_plane_helper_check_state(struct dr= m_plane_state *state, > } > =20 > static int dm_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct amdgpu_device *adev =3D drm_to_adev(plane->dev); > struct dc *dc =3D adev->dm.dc; > struct dm_plane_state *dm_plane_state; > diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_plane.c b/driver= s/gpu/drm/arm/display/komeda/komeda_plane.c > index 00fd83cbe565..96a6fe95a4e7 100644 > --- a/drivers/gpu/drm/arm/display/komeda/komeda_plane.c > +++ b/drivers/gpu/drm/arm/display/komeda/komeda_plane.c > @@ -69,8 +69,10 @@ komeda_plane_init_data_flow(struct drm_plane_state *= st, > */ > static int > komeda_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct komeda_plane *kplane =3D to_kplane(plane); > struct komeda_plane_state *kplane_st =3D to_kplane_st(new_plane_stat= e); > struct komeda_layer *layer =3D kplane->layer; > diff --git a/drivers/gpu/drm/arm/hdlcd_crtc.c b/drivers/gpu/drm/arm/hdl= cd_crtc.c > index d5a79a4aa996..9da9d0581ce9 100644 > --- a/drivers/gpu/drm/arm/hdlcd_crtc.c > +++ b/drivers/gpu/drm/arm/hdlcd_crtc.c > @@ -229,8 +229,10 @@ static const struct drm_crtc_helper_funcs hdlcd_cr= tc_helper_funcs =3D { > }; > =20 > static int hdlcd_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > int i; > struct drm_crtc *crtc; > struct drm_crtc_state *crtc_state; > diff --git a/drivers/gpu/drm/arm/malidp_planes.c b/drivers/gpu/drm/arm/= malidp_planes.c > index e64367f55c70..c94c4a96db68 100644 > --- a/drivers/gpu/drm/arm/malidp_planes.c > +++ b/drivers/gpu/drm/arm/malidp_planes.c > @@ -502,8 +502,10 @@ static void malidp_de_prefetch_settings(struct mal= idp_plane *mp, > } > =20 > static int malidp_de_plane_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct malidp_plane *mp =3D to_malidp_plane(plane); > struct malidp_plane_state *ms =3D to_malidp_plane_state(new_plane_st= ate); > bool rotated =3D new_plane_state->rotation & MALIDP_ROTATED_MASK; > diff --git a/drivers/gpu/drm/armada/armada_plane.c b/drivers/gpu/drm/ar= mada/armada_plane.c > index 27f83b07c8eb..b1266c588102 100644 > --- a/drivers/gpu/drm/armada/armada_plane.c > +++ b/drivers/gpu/drm/armada/armada_plane.c > @@ -106,8 +106,10 @@ void armada_drm_plane_cleanup_fb(struct drm_plane = *plane, > } > =20 > int armada_drm_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct armada_plane_state *st =3D to_armada_plane_state(new_plane_st= ate); > struct drm_crtc *crtc =3D new_plane_state->crtc; > struct drm_crtc_state *crtc_state; > diff --git a/drivers/gpu/drm/armada/armada_plane.h b/drivers/gpu/drm/ar= mada/armada_plane.h > index 2707ec781941..51dab8d8da22 100644 > --- a/drivers/gpu/drm/armada/armada_plane.h > +++ b/drivers/gpu/drm/armada/armada_plane.h > @@ -26,7 +26,7 @@ int armada_drm_plane_prepare_fb(struct drm_plane *pla= ne, > void armada_drm_plane_cleanup_fb(struct drm_plane *plane, > struct drm_plane_state *old_state); > int armada_drm_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *state); > + struct drm_atomic_state *state); > void armada_plane_reset(struct drm_plane *plane); > struct drm_plane_state *armada_plane_duplicate_state(struct drm_plane= *plane); > void armada_plane_destroy_state(struct drm_plane *plane, > diff --git a/drivers/gpu/drm/ast/ast_mode.c b/drivers/gpu/drm/ast/ast_m= ode.c > index 2665d3d570f9..cb8650142f13 100644 > --- a/drivers/gpu/drm/ast/ast_mode.c > +++ b/drivers/gpu/drm/ast/ast_mode.c > @@ -536,8 +536,10 @@ static const uint32_t ast_primary_plane_formats[] = =3D { > }; > =20 > static int ast_primary_plane_helper_atomic_check(struct drm_plane *pl= ane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct drm_crtc_state *crtc_state; > struct ast_crtc_state *ast_crtc_state; > int ret; > @@ -756,8 +758,10 @@ static const uint32_t ast_cursor_plane_formats[] =3D= { > }; > =20 > static int ast_cursor_plane_helper_atomic_check(struct drm_plane *pla= ne, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct drm_framebuffer *fb =3D new_plane_state->fb; > struct drm_crtc_state *crtc_state; > int ret; > diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c b/drivers/= gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c > index c62e930bccad..445105e75a97 100644 > --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c > +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c > @@ -593,8 +593,9 @@ atmel_hlcdc_plane_update_disc_area(struct atmel_hlc= dc_plane *plane, > } > =20 > static int atmel_hlcdc_plane_atomic_check(struct drm_plane *p, > - struct drm_plane_state *s) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *s =3D drm_atomic_get_new_plane_state(state, p= ); > struct atmel_hlcdc_plane *plane =3D drm_plane_to_atmel_hlcdc_plane(p= ); > struct atmel_hlcdc_plane_state *hstate =3D > drm_plane_state_to_atmel_hlcdc_plane_state(s); > diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_= atomic_helper.c > index bd0bb74437c7..c06bd5f43e61 100644 > --- a/drivers/gpu/drm/drm_atomic_helper.c > +++ b/drivers/gpu/drm/drm_atomic_helper.c > @@ -902,7 +902,7 @@ drm_atomic_helper_check_planes(struct drm_device *d= ev, > if (!funcs || !funcs->atomic_check) > continue; > =20 > - ret =3D funcs->atomic_check(plane, new_plane_state); > + ret =3D funcs->atomic_check(plane, state); > if (ret) { > DRM_DEBUG_ATOMIC("[PLANE:%d:%s] atomic driver check failed\n", > plane->base.id, plane->name); > diff --git a/drivers/gpu/drm/drm_simple_kms_helper.c b/drivers/gpu/drm/= drm_simple_kms_helper.c > index 89675d4b7b6c..a43e34e04d6b 100644 > --- a/drivers/gpu/drm/drm_simple_kms_helper.c > +++ b/drivers/gpu/drm/drm_simple_kms_helper.c > @@ -177,8 +177,10 @@ static const struct drm_crtc_funcs drm_simple_kms_= crtc_funcs =3D { > }; > =20 > static int drm_simple_kms_plane_atomic_check(struct drm_plane *plane,= > - struct drm_plane_state *plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *plane_state =3D drm_atomic_get_new_plane_stat= e(state, > + plane); > struct drm_simple_display_pipe *pipe; > struct drm_crtc_state *crtc_state; > int ret; > diff --git a/drivers/gpu/drm/exynos/exynos_drm_plane.c b/drivers/gpu/dr= m/exynos/exynos_drm_plane.c > index 009a62978fbc..2c4ceb768a08 100644 > --- a/drivers/gpu/drm/exynos/exynos_drm_plane.c > +++ b/drivers/gpu/drm/exynos/exynos_drm_plane.c > @@ -228,8 +228,10 @@ exynos_drm_plane_check_size(const struct exynos_dr= m_plane_config *config, > } > =20 > static int exynos_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct exynos_drm_plane *exynos_plane =3D to_exynos_plane(plane); > struct exynos_drm_plane_state *exynos_state =3D > to_exynos_plane_state(new_plane_state); > diff --git a/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_plane.c b/drivers/gpu/= drm/fsl-dcu/fsl_dcu_drm_plane.c > index fc3ec6b4c732..7d2aa2cbcff6 100644 > --- a/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_plane.c > +++ b/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_plane.c > @@ -7,6 +7,7 @@ > =20 > #include > =20 > +#include > #include > #include > #include > @@ -33,8 +34,10 @@ static int fsl_dcu_drm_plane_index(struct drm_plane = *plane) > } > =20 > static int fsl_dcu_drm_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct drm_framebuffer *fb =3D new_plane_state->fb; > =20 > if (!new_plane_state->fb || !new_plane_state->crtc) > diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c b/drivers/g= pu/drm/hisilicon/hibmc/hibmc_drm_de.c > index d276f37d9d80..ddcf121af542 100644 > --- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c > +++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c > @@ -53,8 +53,10 @@ static const struct hibmc_dislay_pll_config hibmc_pl= l_table[] =3D { > }; > =20 > static int hibmc_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct drm_framebuffer *fb =3D new_plane_state->fb; > struct drm_crtc *crtc =3D new_plane_state->crtc; > struct drm_crtc_state *crtc_state; > diff --git a/drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c b/drivers/= gpu/drm/hisilicon/kirin/kirin_drm_ade.c > index cba99b8d9e59..fab083380ef7 100644 > --- a/drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c > +++ b/drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c > @@ -757,8 +757,10 @@ static void ade_disable_channel(struct kirin_plane= *kplane) > } > =20 > static int ade_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct drm_framebuffer *fb =3D new_plane_state->fb; > struct drm_crtc *crtc =3D new_plane_state->crtc; > struct drm_crtc_state *crtc_state; > diff --git a/drivers/gpu/drm/imx/dcss/dcss-plane.c b/drivers/gpu/drm/im= x/dcss/dcss-plane.c > index c76fce2e8cf6..0e68d295bd60 100644 > --- a/drivers/gpu/drm/imx/dcss/dcss-plane.c > +++ b/drivers/gpu/drm/imx/dcss/dcss-plane.c > @@ -137,8 +137,10 @@ static bool dcss_plane_is_source_size_allowed(u16 = src_w, u16 src_h, u32 pix_fmt) > } > =20 > static int dcss_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct dcss_plane *dcss_plane =3D to_dcss_plane(plane); > struct dcss_dev *dcss =3D plane->dev->dev_private; > struct drm_framebuffer *fb =3D new_plane_state->fb; > diff --git a/drivers/gpu/drm/imx/ipuv3-plane.c b/drivers/gpu/drm/imx/ip= uv3-plane.c > index 1873a155bb26..28571091ff1c 100644 > --- a/drivers/gpu/drm/imx/ipuv3-plane.c > +++ b/drivers/gpu/drm/imx/ipuv3-plane.c > @@ -337,8 +337,10 @@ static const struct drm_plane_funcs ipu_plane_func= s =3D { > }; > =20 > static int ipu_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_state =3D drm_atomic_get_new_plane_state(= state, > + plane); > struct drm_plane_state *old_state =3D plane->state; > struct drm_crtc_state *crtc_state; > struct device *dev =3D plane->dev->dev; > diff --git a/drivers/gpu/drm/ingenic/ingenic-drm-drv.c b/drivers/gpu/dr= m/ingenic/ingenic-drm-drv.c > index f589923b4a5d..8232104e598f 100644 > --- a/drivers/gpu/drm/ingenic/ingenic-drm-drv.c > +++ b/drivers/gpu/drm/ingenic/ingenic-drm-drv.c > @@ -359,8 +359,10 @@ static void ingenic_drm_crtc_atomic_flush(struct d= rm_crtc *crtc, > } > =20 > static int ingenic_drm_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct ingenic_drm *priv =3D drm_device_get_priv(plane->dev); > struct drm_crtc_state *crtc_state; > struct drm_crtc *crtc =3D new_plane_state->crtc ?: plane->state->crt= c; > diff --git a/drivers/gpu/drm/ingenic/ingenic-ipu.c b/drivers/gpu/drm/in= genic/ingenic-ipu.c > index 623f42d44b07..7394b0af1259 100644 > --- a/drivers/gpu/drm/ingenic/ingenic-ipu.c > +++ b/drivers/gpu/drm/ingenic/ingenic-ipu.c > @@ -514,8 +514,10 @@ static void ingenic_ipu_plane_atomic_update(struct= drm_plane *plane, > } > =20 > static int ingenic_ipu_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > unsigned int num_w, denom_w, num_h, denom_h, xres, yres, max_w, max_= h; > struct ingenic_ipu *ipu =3D plane_to_ingenic_ipu(plane); > struct drm_crtc *crtc =3D new_plane_state->crtc ?: plane->state->crt= c; > diff --git a/drivers/gpu/drm/kmb/kmb_plane.c b/drivers/gpu/drm/kmb/kmb_= plane.c > index 51ceaae9e7e8..909045b67b28 100644 > --- a/drivers/gpu/drm/kmb/kmb_plane.c > +++ b/drivers/gpu/drm/kmb/kmb_plane.c > @@ -77,8 +77,10 @@ static unsigned int check_pixel_format(struct drm_pl= ane *plane, u32 format) > } > =20 > static int kmb_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct drm_framebuffer *fb; > int ret; > struct drm_crtc_state *crtc_state; > diff --git a/drivers/gpu/drm/mediatek/mtk_drm_plane.c b/drivers/gpu/drm= /mediatek/mtk_drm_plane.c > index cdd2f8cfb4ab..31f1cc2085c7 100644 > --- a/drivers/gpu/drm/mediatek/mtk_drm_plane.c > +++ b/drivers/gpu/drm/mediatek/mtk_drm_plane.c > @@ -141,8 +141,10 @@ static const struct drm_plane_funcs mtk_plane_func= s =3D { > }; > =20 > static int mtk_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct drm_framebuffer *fb =3D new_plane_state->fb; > struct drm_crtc_state *crtc_state; > int ret; > diff --git a/drivers/gpu/drm/meson/meson_overlay.c b/drivers/gpu/drm/me= son/meson_overlay.c > index a419a8c514e1..629ca303af25 100644 > --- a/drivers/gpu/drm/meson/meson_overlay.c > +++ b/drivers/gpu/drm/meson/meson_overlay.c > @@ -165,8 +165,10 @@ struct meson_overlay { > #define FRAC_16_16(mult, div) (((mult) << 16) / (div)) > =20 > static int meson_overlay_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct drm_crtc_state *crtc_state; > =20 > if (!new_plane_state->crtc) > diff --git a/drivers/gpu/drm/meson/meson_plane.c b/drivers/gpu/drm/meso= n/meson_plane.c > index 2c1256caf48a..f8355cb2a2e1 100644 > --- a/drivers/gpu/drm/meson/meson_plane.c > +++ b/drivers/gpu/drm/meson/meson_plane.c > @@ -71,8 +71,10 @@ struct meson_plane { > #define FRAC_16_16(mult, div) (((mult) << 16) / (div)) > =20 > static int meson_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct drm_crtc_state *crtc_state; > =20 > if (!new_plane_state->crtc) > diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c b/drivers/gpu/dr= m/msm/disp/dpu1/dpu_plane.c > index 9bce72627ff0..9945d11bb6ed 100644 > --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c > +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c > @@ -10,6 +10,7 @@ > #include > #include > =20 > +#include > #include > #include > #include > @@ -950,8 +951,10 @@ static bool dpu_plane_validate_src(struct drm_rect= *src, > } > =20 > static int dpu_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > int ret =3D 0, min_scale; > struct dpu_plane *pdpu =3D to_dpu_plane(plane); > struct dpu_plane_state *pstate =3D to_dpu_plane_state(new_plane_stat= e); > diff --git a/drivers/gpu/drm/msm/disp/mdp4/mdp4_plane.c b/drivers/gpu/d= rm/msm/disp/mdp4/mdp4_plane.c > index da3cc1d8c331..f95b14ebfa8a 100644 > --- a/drivers/gpu/drm/msm/disp/mdp4/mdp4_plane.c > +++ b/drivers/gpu/drm/msm/disp/mdp4/mdp4_plane.c > @@ -106,7 +106,7 @@ static void mdp4_plane_cleanup_fb(struct drm_plane = *plane, > =20 > =20 > static int mdp4_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *state) > + struct drm_atomic_state *state) > { > return 0; > } > diff --git a/drivers/gpu/drm/msm/disp/mdp5/mdp5_plane.c b/drivers/gpu/d= rm/msm/disp/mdp5/mdp5_plane.c > index f5434a5254e0..45f7780da901 100644 > --- a/drivers/gpu/drm/msm/disp/mdp5/mdp5_plane.c > +++ b/drivers/gpu/drm/msm/disp/mdp5/mdp5_plane.c > @@ -404,8 +404,10 @@ static int mdp5_plane_atomic_check_with_state(stru= ct drm_crtc_state *crtc_state, > } > =20 > static int mdp5_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct drm_crtc *crtc; > struct drm_crtc_state *crtc_state; > =20 > diff --git a/drivers/gpu/drm/mxsfb/mxsfb_kms.c b/drivers/gpu/drm/mxsfb/= mxsfb_kms.c > index 3e1bb0aefb87..85b547dcf86d 100644 > --- a/drivers/gpu/drm/mxsfb/mxsfb_kms.c > +++ b/drivers/gpu/drm/mxsfb/mxsfb_kms.c > @@ -402,8 +402,10 @@ static const struct drm_encoder_funcs mxsfb_encode= r_funcs =3D { > */ > =20 > static int mxsfb_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *plane_state =3D drm_atomic_get_new_plane_stat= e(state, > + plane); > struct mxsfb_drm_private *mxsfb =3D to_mxsfb_drm_private(plane->dev)= ; > struct drm_crtc_state *crtc_state; > =20 > diff --git a/drivers/gpu/drm/nouveau/dispnv50/wndw.c b/drivers/gpu/drm/= nouveau/dispnv50/wndw.c > index f83bfc0794ab..1382ccdfe7e3 100644 > --- a/drivers/gpu/drm/nouveau/dispnv50/wndw.c > +++ b/drivers/gpu/drm/nouveau/dispnv50/wndw.c > @@ -30,6 +30,7 @@ > #include > #include > =20 > +#include > #include > #include > =20 > @@ -435,8 +436,10 @@ nv50_wndw_atomic_check_lut(struct nv50_wndw *wndw,= > =20 > static int > nv50_wndw_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct nouveau_drm *drm =3D nouveau_drm(plane->dev); > struct nv50_wndw *wndw =3D nv50_wndw(plane); > struct nv50_wndw_atom *armw =3D nv50_wndw_atom(wndw->plane.state); > diff --git a/drivers/gpu/drm/omapdrm/omap_plane.c b/drivers/gpu/drm/oma= pdrm/omap_plane.c > index 53ad0744e048..6bd384c79324 100644 > --- a/drivers/gpu/drm/omapdrm/omap_plane.c > +++ b/drivers/gpu/drm/omapdrm/omap_plane.c > @@ -99,8 +99,10 @@ static void omap_plane_atomic_disable(struct drm_pla= ne *plane, > } > =20 > static int omap_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct drm_crtc_state *crtc_state; > =20 > if (!new_plane_state->fb) > diff --git a/drivers/gpu/drm/qxl/qxl_display.c b/drivers/gpu/drm/qxl/qx= l_display.c > index 9b49b6c0af08..3304fdb020b2 100644 > --- a/drivers/gpu/drm/qxl/qxl_display.c > +++ b/drivers/gpu/drm/qxl/qxl_display.c > @@ -463,8 +463,10 @@ static const struct drm_crtc_helper_funcs qxl_crtc= _helper_funcs =3D { > }; > =20 > static int qxl_primary_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct qxl_device *qdev =3D to_qxl(plane->dev); > struct qxl_bo *bo; > =20 > diff --git a/drivers/gpu/drm/rcar-du/rcar_du_plane.c b/drivers/gpu/drm/= rcar-du/rcar_du_plane.c > index b887ab8fc577..7afe7442214b 100644 > --- a/drivers/gpu/drm/rcar-du/rcar_du_plane.c > +++ b/drivers/gpu/drm/rcar-du/rcar_du_plane.c > @@ -607,8 +607,10 @@ int __rcar_du_plane_atomic_check(struct drm_plane = *plane, > } > =20 > static int rcar_du_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct rcar_du_plane_state *rstate =3D to_rcar_plane_state(new_plane= _state); > =20 > return __rcar_du_plane_atomic_check(plane, new_plane_state, > diff --git a/drivers/gpu/drm/rcar-du/rcar_du_vsp.c b/drivers/gpu/drm/rc= ar-du/rcar_du_vsp.c > index f6cc1a76708a..ab82fda22b70 100644 > --- a/drivers/gpu/drm/rcar-du/rcar_du_vsp.c > +++ b/drivers/gpu/drm/rcar-du/rcar_du_vsp.c > @@ -7,6 +7,7 @@ > * Contact: Laurent Pinchart (laurent.pinchart@ideasonboard.com) > */ > =20 > +#include > #include > #include > #include > @@ -265,8 +266,10 @@ static void rcar_du_vsp_plane_cleanup_fb(struct dr= m_plane *plane, > } > =20 > static int rcar_du_vsp_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct rcar_du_vsp_plane_state *rstate =3D to_rcar_vsp_plane_state(n= ew_plane_state); > =20 > return __rcar_du_plane_atomic_check(plane, new_plane_state, > diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c b/drivers/gpu/= drm/rockchip/rockchip_drm_vop.c > index 8a507917a0dc..d96d42dfc6c7 100644 > --- a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c > +++ b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c > @@ -778,8 +778,10 @@ static bool rockchip_mod_supported(struct drm_plan= e *plane, > } > =20 > static int vop_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct drm_crtc *crtc =3D new_plane_state->crtc; > struct drm_crtc_state *crtc_state; > struct drm_framebuffer *fb =3D new_plane_state->fb; > diff --git a/drivers/gpu/drm/sti/sti_cursor.c b/drivers/gpu/drm/sti/sti= _cursor.c > index c04c868e337f..ceb30d545ab9 100644 > --- a/drivers/gpu/drm/sti/sti_cursor.c > +++ b/drivers/gpu/drm/sti/sti_cursor.c > @@ -181,8 +181,10 @@ static void sti_cursor_init(struct sti_cursor *cur= sor) > } > =20 > static int sti_cursor_atomic_check(struct drm_plane *drm_plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + drm_plane); > struct sti_plane *plane =3D to_sti_plane(drm_plane); > struct sti_cursor *cursor =3D to_sti_cursor(plane); > struct drm_crtc *crtc =3D new_plane_state->crtc; > diff --git a/drivers/gpu/drm/sti/sti_gdp.c b/drivers/gpu/drm/sti/sti_gd= p.c > index 4fe91ab4f191..63823827417c 100644 > --- a/drivers/gpu/drm/sti/sti_gdp.c > +++ b/drivers/gpu/drm/sti/sti_gdp.c > @@ -615,8 +615,10 @@ static int sti_gdp_get_dst(struct device *dev, int= dst, int src) > } > =20 > static int sti_gdp_atomic_check(struct drm_plane *drm_plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + drm_plane); > struct sti_plane *plane =3D to_sti_plane(drm_plane); > struct sti_gdp *gdp =3D to_sti_gdp(plane); > struct drm_crtc *crtc =3D new_plane_state->crtc; > diff --git a/drivers/gpu/drm/sti/sti_hqvdp.c b/drivers/gpu/drm/sti/sti_= hqvdp.c > index 5d492ac1f2dd..b3b10ee96bb2 100644 > --- a/drivers/gpu/drm/sti/sti_hqvdp.c > +++ b/drivers/gpu/drm/sti/sti_hqvdp.c > @@ -1017,8 +1017,10 @@ static void sti_hqvdp_start_xp70(struct sti_hqvd= p *hqvdp) > } > =20 > static int sti_hqvdp_atomic_check(struct drm_plane *drm_plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + drm_plane); > struct sti_plane *plane =3D to_sti_plane(drm_plane); > struct sti_hqvdp *hqvdp =3D to_sti_hqvdp(plane); > struct drm_crtc *crtc =3D new_plane_state->crtc; > diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c > index dbd3994a18e1..7367a1e73d73 100644 > --- a/drivers/gpu/drm/stm/ltdc.c > +++ b/drivers/gpu/drm/stm/ltdc.c > @@ -749,8 +749,10 @@ static const struct drm_crtc_funcs ltdc_crtc_funcs= =3D { > */ > =20 > static int ltdc_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct drm_framebuffer *fb =3D new_plane_state->fb; > u32 src_w, src_h; > =20 > diff --git a/drivers/gpu/drm/sun4i/sun8i_ui_layer.c b/drivers/gpu/drm/s= un4i/sun8i_ui_layer.c > index 19ed531c006b..b5fb50e72868 100644 > --- a/drivers/gpu/drm/sun4i/sun8i_ui_layer.c > +++ b/drivers/gpu/drm/sun4i/sun8i_ui_layer.c > @@ -257,8 +257,10 @@ static int sun8i_ui_layer_update_buffer(struct sun= 8i_mixer *mixer, int channel, > } > =20 > static int sun8i_ui_layer_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct sun8i_ui_layer *layer =3D plane_to_sun8i_ui_layer(plane); > struct drm_crtc *crtc =3D new_plane_state->crtc; > struct drm_crtc_state *crtc_state; > diff --git a/drivers/gpu/drm/sun4i/sun8i_vi_layer.c b/drivers/gpu/drm/s= un4i/sun8i_vi_layer.c > index 6074863f41c3..85c2927755a4 100644 > --- a/drivers/gpu/drm/sun4i/sun8i_vi_layer.c > +++ b/drivers/gpu/drm/sun4i/sun8i_vi_layer.c > @@ -361,8 +361,10 @@ static int sun8i_vi_layer_update_buffer(struct sun= 8i_mixer *mixer, int channel, > } > =20 > static int sun8i_vi_layer_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct sun8i_vi_layer *layer =3D plane_to_sun8i_vi_layer(plane); > struct drm_crtc *crtc =3D new_plane_state->crtc; > struct drm_crtc_state *crtc_state; > diff --git a/drivers/gpu/drm/tegra/dc.c b/drivers/gpu/drm/tegra/dc.c > index 2d91956bc762..bd2ee94f6f52 100644 > --- a/drivers/gpu/drm/tegra/dc.c > +++ b/drivers/gpu/drm/tegra/dc.c > @@ -604,8 +604,10 @@ static const u64 tegra124_modifiers[] =3D { > }; > =20 > static int tegra_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct tegra_plane_state *plane_state =3D to_tegra_plane_state(new_p= lane_state); > unsigned int supported_rotation =3D DRM_MODE_ROTATE_0 | > DRM_MODE_REFLECT_X | > @@ -831,8 +833,10 @@ static const u32 tegra_cursor_plane_formats[] =3D = { > }; > =20 > static int tegra_cursor_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct tegra_plane *tegra =3D to_tegra_plane(plane); > int err; > =20 > diff --git a/drivers/gpu/drm/tegra/hub.c b/drivers/gpu/drm/tegra/hub.c > index 8a2d359c4ff6..e5b22508e09a 100644 > --- a/drivers/gpu/drm/tegra/hub.c > +++ b/drivers/gpu/drm/tegra/hub.c > @@ -336,8 +336,10 @@ static void tegra_dc_remove_shared_plane(struct te= gra_dc *dc, > } > =20 > static int tegra_shared_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct tegra_plane_state *plane_state =3D to_tegra_plane_state(new_p= lane_state); > struct tegra_shared_plane *tegra =3D to_tegra_shared_plane(plane); > struct tegra_bo_tiling *tiling =3D &plane_state->tiling; > diff --git a/drivers/gpu/drm/tidss/tidss_plane.c b/drivers/gpu/drm/tids= s/tidss_plane.c > index 6dab9ad89644..980b65725d66 100644 > --- a/drivers/gpu/drm/tidss/tidss_plane.c > +++ b/drivers/gpu/drm/tidss/tidss_plane.c > @@ -20,8 +20,10 @@ > /* drm_plane_helper_funcs */ > =20 > static int tidss_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct drm_device *ddev =3D plane->dev; > struct tidss_device *tidss =3D to_tidss(ddev); > struct tidss_plane *tplane =3D to_tidss_plane(plane); > diff --git a/drivers/gpu/drm/tilcdc/tilcdc_plane.c b/drivers/gpu/drm/ti= lcdc/tilcdc_plane.c > index 389c80a5873c..f43670aff5ad 100644 > --- a/drivers/gpu/drm/tilcdc/tilcdc_plane.c > +++ b/drivers/gpu/drm/tilcdc/tilcdc_plane.c > @@ -21,8 +21,10 @@ static const struct drm_plane_funcs tilcdc_plane_fun= cs =3D { > }; > =20 > static int tilcdc_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_state =3D drm_atomic_get_new_plane_state(= state, > + plane); > struct drm_crtc_state *crtc_state; > struct drm_plane_state *old_state =3D plane->state; > unsigned int pitch; > diff --git a/drivers/gpu/drm/vboxvideo/vbox_mode.c b/drivers/gpu/drm/vb= oxvideo/vbox_mode.c > index 6e4ad966be71..7140086d8308 100644 > --- a/drivers/gpu/drm/vboxvideo/vbox_mode.c > +++ b/drivers/gpu/drm/vboxvideo/vbox_mode.c > @@ -253,8 +253,10 @@ static const struct drm_crtc_funcs vbox_crtc_funcs= =3D { > }; > =20 > static int vbox_primary_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_state =3D drm_atomic_get_new_plane_state(= state, > + plane); > struct drm_crtc_state *crtc_state =3D NULL; > =20 > if (new_state->crtc) { > @@ -326,8 +328,10 @@ static void vbox_primary_atomic_disable(struct drm= _plane *plane, > } > =20 > static int vbox_cursor_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_state =3D drm_atomic_get_new_plane_state(= state, > + plane); > struct drm_crtc_state *crtc_state =3D NULL; > u32 width =3D new_state->crtc_w; > u32 height =3D new_state->crtc_h; > diff --git a/drivers/gpu/drm/vc4/vc4_plane.c b/drivers/gpu/drm/vc4/vc4_= plane.c > index e4b2f537967f..b46d79efa4b0 100644 > --- a/drivers/gpu/drm/vc4/vc4_plane.c > +++ b/drivers/gpu/drm/vc4/vc4_plane.c > @@ -1040,8 +1040,10 @@ static int vc4_plane_mode_set(struct drm_plane *= plane, > * in the CRTC's flush. > */ > static int vc4_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct vc4_plane_state *vc4_state =3D to_vc4_plane_state(new_plane_s= tate); > int ret; > =20 > diff --git a/drivers/gpu/drm/virtio/virtgpu_plane.c b/drivers/gpu/drm/v= irtio/virtgpu_plane.c > index 9b2ec4db1265..0f4fdd8c28f9 100644 > --- a/drivers/gpu/drm/virtio/virtgpu_plane.c > +++ b/drivers/gpu/drm/virtio/virtgpu_plane.c > @@ -83,8 +83,10 @@ static const struct drm_plane_funcs virtio_gpu_plane= _funcs =3D { > }; > =20 > static int virtio_gpu_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > bool is_cursor =3D plane->type =3D=3D DRM_PLANE_TYPE_CURSOR; > struct drm_crtc_state *crtc_state; > int ret; > diff --git a/drivers/gpu/drm/vkms/vkms_plane.c b/drivers/gpu/drm/vkms/v= kms_plane.c > index eef120a573a8..751695a76e26 100644 > --- a/drivers/gpu/drm/vkms/vkms_plane.c > +++ b/drivers/gpu/drm/vkms/vkms_plane.c > @@ -114,8 +114,10 @@ static void vkms_plane_atomic_update(struct drm_pl= ane *plane, > } > =20 > static int vkms_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct drm_crtc_state *crtc_state; > bool can_position =3D false; > int ret; > diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c b/drivers/gpu/drm/vmwg= fx/vmwgfx_kms.c > index 1b0c2e44b586..5be1e4bd8a5f 100644 > --- a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c > +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c > @@ -437,8 +437,10 @@ vmw_du_cursor_plane_atomic_update(struct drm_plane= *plane, > * Returns 0 on success > */ > int vmw_du_primary_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_state =3D drm_atomic_get_new_plane_state(= state, > + plane); > struct drm_crtc_state *crtc_state =3D NULL; > struct drm_framebuffer *new_fb =3D new_state->fb; > int ret; > @@ -476,8 +478,10 @@ int vmw_du_primary_plane_atomic_check(struct drm_p= lane *plane, > * Returns 0 on success > */ > int vmw_du_cursor_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_state =3D drm_atomic_get_new_plane_state(= state, > + plane); > int ret =3D 0; > struct drm_crtc_state *crtc_state =3D NULL; > struct vmw_surface *surface =3D NULL; > diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.h b/drivers/gpu/drm/vmwg= fx/vmwgfx_kms.h > index 6267ccf54944..c5183e7eea04 100644 > --- a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.h > +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.h > @@ -456,9 +456,9 @@ void vmw_du_cursor_plane_destroy(struct drm_plane *= plane); > =20 > /* Atomic Helpers */ > int vmw_du_primary_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *state); > + struct drm_atomic_state *state); > int vmw_du_cursor_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *state); > + struct drm_atomic_state *state); > void vmw_du_cursor_plane_atomic_update(struct drm_plane *plane, > struct drm_plane_state *old_state); > int vmw_du_cursor_plane_prepare_fb(struct drm_plane *plane, > diff --git a/drivers/gpu/drm/xlnx/zynqmp_disp.c b/drivers/gpu/drm/xlnx/= zynqmp_disp.c > index ee7793d6a26e..b0a3ba528718 100644 > --- a/drivers/gpu/drm/xlnx/zynqmp_disp.c > +++ b/drivers/gpu/drm/xlnx/zynqmp_disp.c > @@ -1143,8 +1143,10 @@ static inline struct zynqmp_disp_layer *plane_to= _layer(struct drm_plane *plane) > =20 > static int > zynqmp_disp_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *new_plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *new_plane_state =3D drm_atomic_get_new_plane_= state(state, > + plane); > struct drm_crtc_state *crtc_state; > =20 > if (!new_plane_state->crtc) > diff --git a/drivers/gpu/drm/zte/zx_plane.c b/drivers/gpu/drm/zte/zx_pl= ane.c > index 78d787afe594..20ac29212991 100644 > --- a/drivers/gpu/drm/zte/zx_plane.c > +++ b/drivers/gpu/drm/zte/zx_plane.c > @@ -46,8 +46,10 @@ static const uint32_t vl_formats[] =3D { > #define FRAC_16_16(mult, div) (((mult) << 16) / (div)) > =20 > static int zx_vl_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *plane_state =3D drm_atomic_get_new_plane_stat= e(state, > + plane); > struct drm_framebuffer *fb =3D plane_state->fb; > struct drm_crtc *crtc =3D plane_state->crtc; > struct drm_crtc_state *crtc_state; > @@ -275,8 +277,10 @@ static const struct drm_plane_helper_funcs zx_vl_p= lane_helper_funcs =3D { > }; > =20 > static int zx_gl_plane_atomic_check(struct drm_plane *plane, > - struct drm_plane_state *plane_state) > + struct drm_atomic_state *state) > { > + struct drm_plane_state *plane_state =3D drm_atomic_get_new_plane_stat= e(state, > + plane); > struct drm_framebuffer *fb =3D plane_state->fb; > struct drm_crtc *crtc =3D plane_state->crtc; > struct drm_crtc_state *crtc_state; > diff --git a/include/drm/drm_modeset_helper_vtables.h b/include/drm/drm= _modeset_helper_vtables.h > index 2b4d085da450..484f91d772ea 100644 > --- a/include/drm/drm_modeset_helper_vtables.h > +++ b/include/drm/drm_modeset_helper_vtables.h > @@ -1233,9 +1233,8 @@ struct drm_plane_helper_funcs { > * NOTE: > * > * This function is called in the check phase of an atomic update. T= he > - * driver is not allowed to change anything outside of the free-stand= ing > - * state objects passed-in or assembled in the overall &drm_atomic_st= ate > - * update tracking structure. > + * driver is not allowed to change anything outside of the > + * &drm_atomic_state update tracking structure. > * > * RETURNS: > * > @@ -1245,7 +1244,7 @@ struct drm_plane_helper_funcs { > * deadlock. > */ > int (*atomic_check)(struct drm_plane *plane, > - struct drm_plane_state *state); > + struct drm_atomic_state *state); > =20 > /** > * @atomic_update: >=20 --=20 Thomas Zimmermann Graphics Driver Developer SUSE Software Solutions Germany GmbH Maxfeldstr. 5, 90409 N=C3=BCrnberg, Germany (HRB 36809, AG N=C3=BCrnberg) Gesch=C3=A4ftsf=C3=BChrer: Felix Imend=C3=B6rffer --DpGLs9b7KGSubA3i3QzqbDsxuJZFsfytr-- --GgHTCSYfhQtxhqzWLFKSFn9iB3UTsE2Ut Content-Type: application/pgp-signature; name="OpenPGP_signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="OpenPGP_signature" -----BEGIN PGP SIGNATURE----- wsF5BAABCAAjFiEExndm/fpuMUdwYFFolh/E3EQov+AFAmAzcskFAwAAAAAACgkQlh/E3EQov+DD nBAAhd0HAToVk3DoyU4B5UQoRg4drX8W6euGDPcSccEHJeUI2Z8rOf5His1L5AKYRUyaaJxgTxNs Hk3TCog/drJkOLBbIb/WCNBFoAsNIxu2s/VW+OIwF51fuRvAtD+VCSAdsLIn6eFbTRj0tR9yYYZE QBZi+KKiaPGBuJ+tboPL/idrNP/rcvPnNNDSzXBNckGpKfbJLcrJ+OgtZ9USWsVO3KmXntYdhiyZ Aarmg9t3bCCaokoLw4Yer1JgBGF58g6RTEKvRNb3X5XNKIVS1cvy7+q7EL9nqTJTS582Q8ijJoKY zBQRKOZoga+xKHIvOwi56qkd1pz2i7HtKZRQJh+lSQrtervp6Zdef/GO1F2B5Huh8jU0W7+a2ng9 HrBYCGyfZqfolTMGYTNYvl94jvJsY35dTjYZbYjtrAUNo/KrB3VsNsRyKXauaUcy9w9eTiidgzyU f8mQnvauoTJnv/yd2WNDNx3+ozoS5xJ9t57ILc3kT7S3jyHZ9iDvE94yCzFS2vMaGKwgehyD7t+A X6INvw8gvBKyOcHVOgpkXK+X4wUX6MVtVVWgk4V0LMKCjDOIDq3LvlUdO89L+lT2pZG3zQDlu/ll cT6UHXuklj+tpBdFI+xGw4DT/2yUs+CftVweD0usQ1T8UThTaBOSri0ojJAwsWWbqcsJGCCOymB/ qyc= =+l66 -----END PGP SIGNATURE----- --GgHTCSYfhQtxhqzWLFKSFn9iB3UTsE2Ut--