r the migration of all private objs implementation by
introducing a reset callback in drm_private_state_funcs, and by calling
it if the passed state is NULL.
The latter will be removed eventually, once every driver has been
converted.
Signed-off-by: Maxime Ripard
---
drivers/gpu/drm/drm_atomic.c
On Wed, Oct 08, 2025 at 05:06:43PM +0300, Ville Syrjälä wrote:
> On Wed, Oct 08, 2025 at 02:04:03PM +0200, Maxime Ripard wrote:
> > The DP MST implementation relies on a drm_private_obj, that is
> > initialized by allocating and initializing a state, and then passing it
> > to
Now that we have all the drm_private_obj users relying on the reset
implementation, we can move that call from drm_private_obj_init, where
it was initially supposed to happen, to drm_mode_config_reset, which is
the location reset is called for every other object in KMS.
Signed-off-by: Maxime
nstance to the new pattern.
Signed-off-by: Maxime Ripard
---
Cc: Dave Stevenson
Cc: "Maíra Canal"
Cc: kernel-l...@raspberrypi.com
---
drivers/gpu/drm/vc4/vc4_kms.c | 75 +++
1 file changed, 54 insertions(+), 21 deletions(-)
diff --git a
nstance to the new pattern.
Signed-off-by: Maxime Ripard
---
Cc: Harry Wentland
Cc: Leo Li
Cc: Rodrigo Siqueira
Cc: Alex Deucher
Cc: "Christian König"
Cc: amd-...@lists.freedesktop.org
---
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 54 +--
1 file ch
grate this
instance to the new pattern.
Signed-off-by: Maxime Ripard
---
drivers/gpu/drm/display/drm_dp_mst_topology.c | 39 ++-
1 file changed, 26 insertions(+), 13 deletions(-)
diff --git a/drivers/gpu/drm/display/drm_dp_mst_topology.c
b/drivers/gpu/drm/display/drm_dp_mst_
s
instance to the new pattern.
Signed-off-by: Maxime Ripard
---
Cc: Tomi Valkeinen
---
drivers/gpu/drm/omapdrm/omap_drv.c | 24 ++--
1 file changed, 18 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/omapdrm/omap_drv.c
b/drivers/gpu/drm/omapdrm/omap_d
On Thu, Sep 25, 2025 at 05:55:06PM +0300, Dmitry Baryshkov wrote:
> > > As we will be getting more and more features, some of the InfoFrames
> > > or data packets will be 'good to have, but not required'.
> >
> > And drivers would be free to ignore those.
> >
> > > > So, no, sorry. That's still a
On Tue, Sep 30, 2025 at 10:02:28AM +0300, Dmitry Baryshkov wrote:
> On Mon, Sep 29, 2025 at 03:00:04PM +0200, Maxime Ripard wrote:
> > On Thu, Sep 25, 2025 at 05:16:07PM +0300, Dmitry Baryshkov wrote:
> > > On Thu, Sep 25, 2025 at 03:13:47PM +0200, Maxime Ripard wrote:
> >
On Thu, Oct 02, 2025 at 12:45:41PM -0700, John Stultz wrote:
> On Thu, Oct 2, 2025 at 12:47 AM Maxime Ripard wrote:
> > On Thu, Sep 11, 2025 at 03:49:43PM +0200, Jens Wiklander wrote:
> > > Export the dma-buf heap functions to allow them to be used by the OP-TEE
> > >
Hi,
On Sun, Sep 28, 2025 at 11:24:47AM +0300, Dmitry Baryshkov wrote:
> Make hdmi_write_hdmi_infoframe() and hdmi_clear_infoframe() callbacks
> return -EOPNOTSUPP for unsupported InfoFrames and make sure that
> atomic_check() callback doesn't allow unsupported InfoFrames to be
> enabled.
>
> Sign
mitry Baryshkov
>
> [ ... ]
Reviewed-by: Maxime Ripard
Thanks!
Maxime
swapped.
As such, the existing state points to the new state, and we can use
drm_atomic_get_new_crtc_state() instead.
Reviewed-by: Dmitry Baryshkov
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
drivers/gpu/drm/drm_framebuffer.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
On Tue, 02 Sep 2025 10:32:39 +0200, Maxime Ripard wrote:
> The no_vblank field in drm_crtc_state is defined as a bit-field with a
> single bit.
>
> This will create a syntax issue with the macros we'll introduce next,
> and most other booleans but the *_changed ones in drm_crt
The drm_atomic_get_existing_crtc_state() function is deprecated and
isn't used anymore, so let's remove it.
Reviewed-by: Dmitry Baryshkov
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
include/drm/drm_atomic.h | 18 --
1 file changed, 18 deletions(-)
di
() instead.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Sui Jingfeng
Cc: dri-devel@lists.freedesktop.org
---
drivers/gpu/drm/loongson/lsdc_plane.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/loongson/lsdc_plane.c
b/drivers/gpu/drm/loongson
On Thu, Sep 11, 2025 at 03:49:44PM +0200, Jens Wiklander wrote:
> +static const char *heap_id_2_name(enum tee_dma_heap_id id)
> +{
> + switch (id) {
> + case TEE_DMA_HEAP_SECURE_VIDEO_PLAY:
> + return "protected,secure-video";
> + case TEE_DMA_HEAP_TRUSTED_UI:
> +
On Thu, Sep 04, 2025 at 02:04:05PM +0200, Thierry Reding wrote:
> On Wed, Sep 03, 2025 at 11:48:38AM -0700, John Stultz wrote:
> > On Wed, Sep 3, 2025 at 8:38 AM Thierry Reding
> > wrote:
> > >
> > > On Tue, Sep 02, 2025 at 03:37:45PM -0700, John Stultz wrote:
> > > > On Tue, Sep 2, 2025 at 8:46
Hi,
On Thu, Sep 11, 2025 at 03:49:43PM +0200, Jens Wiklander wrote:
> Export the dma-buf heap functions to allow them to be used by the OP-TEE
> driver. The OP-TEE driver wants to register and manage specific secure
> DMA heaps with it.
>
> Reviewed-by: Sumit Garg
> Reviewed-by: T.J. Mercier
>
t state when we free the global state.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
include/drm/drm_atomic.h | 18 +-
1 file changed, 17 insertions(+), 1 deletion(-)
diff --git a/include/drm/drm_atomic.h b/include/drm/drm_atomic.h
index
f53a32688b33fb0f6295
().
Reviewed-by: Louis Chauvet
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Louis Chauvet
Cc: Haneen Mohammed
Cc: Simona Vetter
Cc: Melissa Wen
Cc: dri-devel@lists.freedesktop.org
---
drivers/gpu/drm/vkms/vkms_crtc.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions
shkov
Suggested-by: Ville Syrjälä
Reviewed-by: Paul Cercueil
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Paul Cercueil
Cc: linux-m...@vger.kernel.org
---
drivers/gpu/drm/ingenic/ingenic-drm-drv.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --
drm_atomic_get_new_crtc_state() instead.
Reviewed-by: Paul Cercueil
Tested-by: Paul Cercueil
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Paul Cercueil
Cc: linux-m...@vger.kernel.org
---
drivers/gpu/drm/ingenic/ingenic-ipu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
t state when we free the global state.
Reviewed-by: Luca Ceresoli
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Maarten Lankhorst
To: Maxime Ripard
To: Thomas Zimmermann
To: David Airlie
To: Simona Vetter
Cc: dri-devel@lists.freedesktop.org
---
include/d
The drm_atomic_get_existing_plane_state() function is deprecated and
isn't used anymore, so let's remove it.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Maarten Lankhorst
To: Maxime Ripard
To: Thomas Zimmermann
To: David Airlie
To: Simona Vetter
Cc:
Syrjälä
Acked-by: Jyri Sarha
Signed-off-by: Maxime Ripard
---
To: Jyri Sarha
To: Tomi Valkeinen
Cc: dri-devel@lists.freedesktop.org
---
drivers/gpu/drm/tilcdc/tilcdc_crtc.c | 9 +
1 file changed, 1 insertion(+), 8 deletions(-)
diff --git a/drivers/gpu/drm/tilcdc/tilcdc_crtc.c
b/drivers
ccessors.
This was tested on tidss.
Let me know what you think,
Maxime
Signed-off-by: Maxime Ripard
---
Changes in v5:
- Added some more documentation on state teardown
- Link to v4:
https://lore.kernel.org/r/20250917-drm-no-more-existing-state-v4-0-5d4b9889c...@kernel.org
Changes in v4:
- Fix
On Fri, Sep 26, 2025 at 05:59:43PM +0200, Luca Ceresoli wrote:
> drm_encoder_cleanup() modifies the encoder chain by removing bridges via
> drm_bridge_detach(). Protect this whole operation by taking the mutex, so
> any users iterating over the chain will not access it during the change.
>
> Signe
On Wed, Sep 10, 2025 at 06:16:25PM +0300, Dmitry Baryshkov wrote:
> On Wed, Sep 10, 2025 at 01:03:47PM +0200, Maxime Ripard wrote:
> > On Tue, Sep 09, 2025 at 05:51:59PM +0300, Dmitry Baryshkov wrote:
> > > Currently DRM framework expects that the HDMI connector driver supports
&
On Wed, 24 Sep 2025 17:11:01 +0200, Luca Ceresoli wrote:
> Get a reference for the last_bridge when it is obtained and release it
> using a cleanup action.
>
> Signed-off-by: Luca Ceresoli
Reviewed-by: Maxime Ripard
Thanks!
Maxime
On Wed, Sep 24, 2025 at 09:26:17PM -0500, Randolph Sapp wrote:
> On Wed Sep 17, 2025 at 10:24 AM CDT, Kevin Hilman wrote:
> > Michael Walle writes:
> >
> >> The TISCI firmware will return 0 if the clock or consumer is not
> >> enabled although there is a stored value in the firmware. IOW a call to
On Tue, Sep 23, 2025 at 01:32:23PM +0300, Laurent Pinchart wrote:
> On Tue, Sep 23, 2025 at 01:28:57PM +0300, Dmitry Baryshkov wrote:
> > On Tue, Sep 23, 2025 at 11:38:17AM +0200, Maxime Ripard wrote:
> > > On Mon, Sep 15, 2025 at 09:38:44PM +0300, Dmitry Baryshkov wrote:
>
On Mon, Sep 15, 2025 at 09:40:57PM +0300, Dmitry Baryshkov wrote:
> On Tue, Sep 02, 2025 at 10:32:38AM +0200, Maxime Ripard wrote:
> > In order to enable drivers to fill their initial state from the hardware
> > state, we need to provide an alternative atomic_reset helper.
>
On Wed, Sep 10, 2025 at 05:16:49PM +0200, Jocelyn Falempe wrote:
> On 10/09/2025 10:33, Maxime Ripard wrote:
> > Hi,
> >
> > On Mon, Sep 08, 2025 at 11:00:30AM +0200, Jocelyn Falempe wrote:
> > > Add kunit tests for drm_panic.
> > > They check that drawing t
On Mon, Sep 15, 2025 at 09:38:44PM +0300, Dmitry Baryshkov wrote:
> On Mon, Sep 15, 2025 at 10:42:22AM +0200, Maxime Ripard wrote:
> > Hi Tohmas,
> >
> > On Tue, Sep 02, 2025 at 03:44:54PM +0200, Thomas Zimmermann wrote:
> > > > +/**
> > > > + * drm_at
On Mon, Sep 15, 2025 at 11:11:39AM +0200, Thomas Zimmermann wrote:
> Hi
>
> Am 15.09.25 um 10:42 schrieb Maxime Ripard:
> > Hi Tohmas,
> >
> > On Tue, Sep 02, 2025 at 03:44:54PM +0200, Thomas Zimmermann wrote:
> > > > +/**
> > > > + * drm_at
Hi Thomas,
On Tue, Sep 02, 2025 at 03:08:12PM +0200, Thomas Zimmermann wrote:
> > + * or a newer one if a commit has been made. The new state is the
> > + * state we just allocated and we might eventually commit to the
> > + * hardware. The existing state points to the state we'll even
On Wed, Sep 17, 2025 at 08:24:47AM -0700, Kevin Hilman wrote:
> Michael Walle writes:
>
> > The TISCI firmware will return 0 if the clock or consumer is not
> > enabled although there is a stored value in the firmware. IOW a call to
> > set rate will work but at get rate will always return 0 if t
() instead.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Chun-Kuang Hu
To: Philipp Zabel
To: Matthias Brugger
To: AngeloGioacchino Del Regno
Cc: dri-devel@lists.freedesktop.org
Cc: linux-media...@lists.infradead.org
Cc: linux-ker...@vger.kernel.org
Cc: linux-arm-ker
On Wed, Sep 10, 2025 at 06:47:52PM +0200, Luca Ceresoli wrote:
> On Wed, 10 Sep 2025 12:59:12 +0200
> Maxime Ripard wrote:
>
> > On Mon, Sep 08, 2025 at 03:49:06PM +0200, Luca Ceresoli wrote:
> > > Hi Maxime,
> > >
> > > On Wed, 27 Aug 2025
drm_atomic_get_new_crtc_state() instead.
Reviewed-by: Paul Cercueil
Tested-by: Paul Cercueil
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Paul Cercueil
Cc: linux-m...@vger.kernel.org
---
drivers/gpu/drm/ingenic/ingenic-ipu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
On Mon, Sep 08, 2025 at 03:49:01PM +0200, Luca Ceresoli wrote:
> Hello Maxime,
>
> On Wed, 20 Aug 2025 13:13:02 +0200
> Luca Ceresoli wrote:
>
> > > > + /*
> > > > +* sn65dsi83_atomic_disable() should release some resources,
> > > > but it
> > > > +* cannot if we call drm_
Hi,
On Mon, Sep 08, 2025 at 11:00:31AM +0200, Jocelyn Falempe wrote:
> This adds a new drm_panic/draw_test file in debugfs.
> This file allows to test the panic screen rendering at different
> resolution and pixel format.
> It's useful only for kernel developers that want to create or
> customize
atomic_check), the existing state is the new state
and we can thus replace the hand-crafted logic by a call to
drm_atomic_get_new_private_obj_state().
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
drivers/gpu/drm/drm_atomic.c | 8
1 file changed, 4 insertions(+), 4 deletions
On Tue, Sep 09, 2025 at 05:52:07PM +0300, Dmitry Baryshkov wrote:
> Check that connector initialized by drmm_connector_hdmi_init() supports
> AVI InfoFrames and warn if it doesn't support Vendor-Specific
> InfofRames (HDMI InfoFrames are more or less required).
>
> Suggest
() instead.
Reviewed-by: Hans de Goede
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Hans de Goede
Cc: dri-devel@lists.freedesktop.org
---
drivers/gpu/drm/vboxvideo/vbox_mode.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/vboxvideo
() instead.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Paul Kocialkowski
---
drivers/gpu/drm/logicvc/logicvc_layer.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/logicvc/logicvc_layer.c
b/drivers/gpu/drm/logicvc/logicvc_layer.c
index
On Wed, Sep 10, 2025 at 06:34:40PM +0200, Luca Ceresoli wrote:
> Hi Maxime,
>
> On Wed, 10 Sep 2025 09:52:21 +0200
> Maxime Ripard wrote:
>
> > On Mon, Sep 08, 2025 at 03:49:01PM +0200, Luca Ceresoli wrote:
> > > Hello Maxime,
> > >
> > > On Wed
() instead.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Liu Ying
To: Shawn Guo
To: Sascha Hauer
Cc: Pengutronix Kernel Team
Cc: Fabio Estevam
Cc: dri-devel@lists.freedesktop.org
Cc: i...@lists.linux.dev
Cc: linux-arm-ker...@lists.infradead.org
---
drivers/gpu/drm/imx/dc/dc
The drm_atomic_get_existing_crtc_state() function is deprecated and
isn't used anymore, so let's remove it.
Reviewed-by: Dmitry Baryshkov
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
include/drm/drm_atomic.h | 18 --
1 file changed, 18 deletions(-)
di
t state when we free the global state.
Reviewed-by: Luca Ceresoli
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
include/drm/drm_atomic.h | 17 -
1 file changed, 16 insertions(+), 1 deletion(-)
diff --git a/include/drm/drm_atomic.h b/include/dr
t state when we free the global state.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
include/drm/drm_atomic.h | 16 +++-
1 file changed, 15 insertions(+), 1 deletion(-)
diff --git a/include/drm/drm_atomic.h b/include/drm/drm_atomic.h
index
cba63857e920515b48b7
().
Reviewed-by: Louis Chauvet
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Louis Chauvet
Cc: Haneen Mohammed
Cc: Simona Vetter
Cc: Melissa Wen
Cc: dri-devel@lists.freedesktop.org
---
drivers/gpu/drm/vkms/vkms_crtc.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions
Hi,
Here's this week drm-misc-fixes PR.
Maxime
drm-misc-fixes-2025-09-18:
One fix for a documentation warning, a null pointer dereference fix for
anx7625, and a mutex unlock fix for cdns-mhdp8546
The following changes since commit 87b90cee22d8658a69c0fbd43633839b75f8f05f:
MAINTAINERS: drm-mis
Syrjälä
Signed-off-by: Maxime Ripard
---
To: Jyri Sarha
To: Tomi Valkeinen
Cc: dri-devel@lists.freedesktop.org
---
drivers/gpu/drm/tilcdc/tilcdc_crtc.c | 9 +
1 file changed, 1 insertion(+), 8 deletions(-)
diff --git a/drivers/gpu/drm/tilcdc/tilcdc_crtc.c
b/drivers/gpu/drm/tilcdc/tilcd
On Wed, Sep 03, 2025 at 02:08:28PM +0530, Swamil Jain wrote:
> Hi Tomi, Maxime,
>
> On 8/27/25 15:19, Tomi Valkeinen wrote:
> > Hi,
> >
> > On 27/08/2025 12:27, Maxime Ripard wrote:
> > > On Wed, Aug 27, 2025 at 11:49:22AM +0300, Tomi Valkeinen wrote:
>
atomic_check), the existing state is the new state
and drm_atomic_get_existing_crtc_state() can thus be replaced by
drm_atomic_get_new_crtc_state().
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Maarten Lankhorst
To: Maxime Ripard
To: Thomas Zimmermann
---
drivers/gpu/drm/drm_atomic.c
mitry Baryshkov
>
> [ ... ]
Acked-by: Maxime Ripard
Thanks!
Maxime
() instead.
Reviewed-by: Dmitry Baryshkov
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
drivers/gpu/drm/drm_atomic.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomic.c
index
te as the new state, and
plane->state is always going to be the old state. Any plane isn't
guaranteed to be part of the state though, so we can't rely on
drm_atomic_get_old_plane_state() and we still need to use plane->state.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
() instead.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Russell King
---
drivers/gpu/drm/armada/armada_plane.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/armada/armada_plane.c
b/drivers/gpu/drm/armada/armada_plane.c
index
() instead.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Liu Ying
To: Shawn Guo
To: Sascha Hauer
Cc: Pengutronix Kernel Team
Cc: Fabio Estevam
Cc: dri-devel@lists.freedesktop.org
Cc: i...@lists.linux.dev
Cc: linux-arm-ker...@lists.infradead.org
---
drivers/gpu/drm/imx/dc/dc
On Tue, Sep 09, 2025 at 01:45:25PM +0200, Danilo Krummrich wrote:
> On 9/9/25 1:43 PM, Maxime Ripard wrote:
> > We should probably amend the nouveau entry to point to the drm-misc repo
> > if it's the de-facto tree these days?
> This already happened with a preceding patch b
t state when we free the global state.
Reviewed-by: Luca Ceresoli
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
include/drm/drm_atomic.h | 16 +++-
1 file changed, 15 insertions(+), 1 deletion(-)
diff --git a/include/drm/drm_atomic.h b/include/dr
Hi,
On Thu, Sep 11, 2025 at 04:07:33PM +0300, Marius Vlad wrote:
> diff --git a/drivers/video/hdmi.c b/drivers/video/hdmi.c
> index 45b42f14a750..74fe925c69a2 100644
> --- a/drivers/video/hdmi.c
> +++ b/drivers/video/hdmi.c
> @@ -1031,8 +1031,8 @@ static const char *hdmi_colorspace_get_name(enum
atomic_check), the existing state is the new state and
drm_atomic_get_existing_plane_state() can thus be replaced by
drm_atomic_get_new_plane_state().
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
drivers/gpu/drm/drm_atomic.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
On Thu, Sep 11, 2025 at 04:37:15PM +0530, Swamil Jain wrote:
> From: Jayesh Choudhary
>
> Since OLDI consumes DSS VP clock directly as serial clock, mode_valid()
> check cannot be performed in tidss driver which should be checked
> in OLDI driver.
>
> Fixes: 7246e0929945 ("drm/tidss: Add OLDI br
The drm_atomic_get_existing_connector_state() function is deprecated and
isn't used anymore, so let's remove it.
Reviewed-by: Dmitry Baryshkov
Reviewed-by: Luca Ceresoli
Tested-by: Luca Ceresoli
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
include/drm/drm_ato
() instead.
Reviewed-by: Laurentiu Palcu
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Laurentiu Palcu
To: Shawn Guo
To: Sascha Hauer
Cc: Lucas Stach
Cc: Pengutronix Kernel Team
Cc: Fabio Estevam
Cc: dri-devel@lists.freedesktop.org
Cc: i...@lists.linux.dev
Cc: linux-arm-ker
() instead.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Thierry Reding
To: Mikko Perttunen
To: Jonathan Hunter
Cc: dri-devel@lists.freedesktop.org
Cc: linux-te...@vger.kernel.org
---
drivers/gpu/drm/tegra/dc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a
The drm_atomic_get_existing_plane_state() function is deprecated and
isn't used anymore, so let's remove it.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Maarten Lankhorst
To: Maxime Ripard
To: Thomas Zimmermann
To: David Airlie
To: Simona Vetter
Cc:
On Sat, Sep 13, 2025 at 04:57:26PM +, Rahul Rameshbabu wrote:
> On Wed, 27 Aug, 2025 08:57:58 +0200 "Maxime Ripard"
> wrote:
> > Hi Rahul,
> >
> > On Wed, Aug 20, 2025 at 04:46:52AM +, Rahul Rameshbabu wrote:
> >> On Tue, 19 Aug, 2025
() instead.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Sui Jingfeng
Cc: dri-devel@lists.freedesktop.org
---
drivers/gpu/drm/loongson/lsdc_plane.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/loongson/lsdc_plane.c
b/drivers/gpu/drm/loongson
() instead.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Inki Dae
To: Seung-Woo Kim
To: Kyungmin Park
To: Krzysztof Kozlowski
Cc: Alim Akhtar
Cc: dri-devel@lists.freedesktop.org
Cc: linux-arm-ker...@lists.infradead.org
Cc: linux-samsung-...@vger.kernel.org
---
drivers/gpu/drm
atomic_check), the existing state is the new state
and drm_atomic_get_existing_crtc_state() can thus be replaced by
drm_atomic_get_new_crtc_state().
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Maarten Lankhorst
To: Maxime Ripard
To: Thomas Zimmermann
---
drivers/gpu/drm/drm_atomic.c
atomic_check), the existing state is the new state
and we can thus replace the hand-crafted logic by a call to
drm_atomic_get_new_private_obj_state().
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
drivers/gpu/drm/drm_atomic.c | 8
1 file changed, 4 insertions(+), 4 deletions
shkov
Suggested-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Paul Cercueil
Cc: linux-m...@vger.kernel.org
---
drivers/gpu/drm/ingenic/ingenic-drm-drv.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/ingenic/ingenic-drm-drv.c
b/driver
t state when we free the global state.
Reviewed-by: Luca Ceresoli
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
include/drm/drm_atomic.h | 16 +++-
1 file changed, 15 insertions(+), 1 deletion(-)
diff --git a/include/drm/drm_atomic.h b/include/dr
.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Maxime Ripard
To: Dave Stevenson
Cc: "Maíra Canal"
Cc: Raspberry Pi Kernel Maintenance
---
drivers/gpu/drm/vc4/vc4_plane.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/g
The drm_atomic_get_existing_crtc_state() function is deprecated and
isn't used anymore, so let's remove it.
Reviewed-by: Dmitry Baryshkov
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
include/drm/drm_atomic.h | 18 --
1 file changed, 18 deletions(-)
di
t state when we free the global state.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
include/drm/drm_atomic.h | 16 +++-
1 file changed, 15 insertions(+), 1 deletion(-)
diff --git a/include/drm/drm_atomic.h b/include/drm/drm_atomic.h
index
cba63857e920515b48b7
() instead.
Reviewed-by: Ville Syrjälä
Acked-by: Jyri Sarha
Signed-off-by: Maxime Ripard
---
To: Jyri Sarha
To: Tomi Valkeinen
Cc: dri-devel@lists.freedesktop.org
---
drivers/gpu/drm/tilcdc/tilcdc_plane.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/tilcdc
swapped.
As such, the existing state points to the new state, and we can use
drm_atomic_get_new_crtc_state() instead.
Reviewed-by: Dmitry Baryshkov
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
drivers/gpu/drm/drm_framebuffer.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
.
Reviewed-by: Dmitry Baryshkov
Acked-by: Dmitry Baryshkov
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Rob Clark
To: Dmitry Baryshkov
Cc: Abhinav Kumar
Cc: Jessica Zhang
Cc: Sean Paul
Cc: Marijn Suijten
Cc: linux-arm-...@vger.kernel.org
Cc: dri-devel
() instead.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Sandy Huang
To: "Heiko Stübner"
To: Andy Yan
Cc: dri-devel@lists.freedesktop.org
Cc: linux-arm-ker...@lists.infradead.org
Cc: linux-rockc...@lists.infradead.org
---
drivers/gpu/drm/rockchip/rockchip_drm_
() instead.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Maxime Ripard
To: Chen-Yu Tsai
To: Jernej Skrabec
To: Samuel Holland
Cc: Jernej Skrabec
Cc: dri-devel@lists.freedesktop.org
Cc: linux-arm-ker...@lists.infradead.org
Cc: linux-su...@lists.linux.dev
---
drivers/gpu/drm/sun4i
.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Tomi Valkeinen
Cc: dri-devel@lists.freedesktop.org
---
drivers/gpu/drm/omapdrm/omap_plane.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/omapdrm/omap_plane.c
b/drivers/gpu/drm/omapdrm
() instead.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Chun-Kuang Hu
To: Philipp Zabel
To: Matthias Brugger
To: AngeloGioacchino Del Regno
Cc: dri-devel@lists.freedesktop.org
Cc: linux-media...@lists.infradead.org
Cc: linux-ker...@vger.kernel.org
Cc: linux-arm-ker
() instead.
Reviewed-by: Paul Cercueil
Tested-by: Paul Cercueil
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Paul Cercueil
Cc: linux-m...@vger.kernel.org
---
drivers/gpu/drm/ingenic/ingenic-drm-drv.c | 3 +--
drivers/gpu/drm/ingenic/ingenic-ipu.c | 2 +-
2 files changed, 2
() instead.
Reviewed-by: Ville Syrjälä
Reviewed-by: Manikandan Muralidharan
Signed-off-by: Maxime Ripard
---
To: Manikandan Muralidharan
To: Dharma Balasubiramani
To: Nicolas Ferre
To: Alexandre Belloni
To: Claudiu Beznea
Cc: dri-devel@lists.freedesktop.org
Cc: linux-arm-ker
.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Anitha Chrisanthus
To: Edmund Dea
---
drivers/gpu/drm/kmb/kmb_plane.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/kmb/kmb_plane.c b/drivers/gpu/drm/kmb/kmb_plane.c
index
() instead.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Philipp Zabel
To: Shawn Guo
To: Sascha Hauer
Cc: Pengutronix Kernel Team
Cc: Fabio Estevam
Cc: dri-devel@lists.freedesktop.org
Cc: i...@lists.linux.dev
Cc: linux-arm-ker...@lists.infradead.org
---
drivers/gpu/drm/imx
drm_atomic_get_new_crtc_state() instead.
Reviewed-by: Paul Cercueil
Tested-by: Paul Cercueil
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Paul Cercueil
Cc: linux-m...@vger.kernel.org
---
drivers/gpu/drm/ingenic/ingenic-ipu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
The drm_atomic_state pointer passed to atomic_check is always valid, so
checking if it's NULL or not is pointless. Remove the check.
Suggested-by: Ville Syrjälä
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Russell King
---
drivers/gpu/drm/armada/armada_plane.
t state when we free the global state.
Reviewed-by: Luca Ceresoli
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
include/drm/drm_atomic.h | 17 -
1 file changed, 16 insertions(+), 1 deletion(-)
diff --git a/include/drm/drm_atomic.h b/include/dr
() instead.
Reviewed-by: Liviu Dudau
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Liviu Dudau
---
drivers/gpu/drm/arm/malidp_planes.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/arm/malidp_planes.c
b/drivers/gpu/drm/arm/malidp_planes.c
index
t state when we free the global state.
Reviewed-by: Luca Ceresoli
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
To: Maarten Lankhorst
To: Maxime Ripard
To: Thomas Zimmermann
To: David Airlie
To: Simona Vetter
Cc: dri-devel@lists.freedesktop.org
---
include/drm/drm_atomic.h | 16
Syrjälä
Acked-by: Jyri Sarha
Signed-off-by: Maxime Ripard
---
To: Jyri Sarha
To: Tomi Valkeinen
Cc: dri-devel@lists.freedesktop.org
---
drivers/gpu/drm/tilcdc/tilcdc_crtc.c | 9 +
1 file changed, 1 insertion(+), 8 deletions(-)
diff --git a/drivers/gpu/drm/tilcdc/tilcdc_crtc.c
b/drivers
atomic_check), the existing state is the new state and
drm_atomic_get_existing_plane_state() can thus be replaced by
drm_atomic_get_new_plane_state().
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
---
drivers/gpu/drm/drm_atomic.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
te as the new state, and
plane->state is always going to be the old state. Any plane isn't
guaranteed to be part of the state though, so we can't rely on
drm_atomic_get_old_plane_state() and we still need to use plane->state.
Reviewed-by: Ville Syrjälä
Signed-off-by: Maxime Ripard
1 - 100 of 3903 matches
Mail list logo
