From: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
To: Daniel Vetter <daniel.vetter@ffwll.ch>,
DRI Development <dri-devel@lists.freedesktop.org>
Cc: Daniel Vetter <daniel.vetter@intel.com>,
Intel Graphics Development <intel-gfx@lists.freedesktop.org>
Subject: Re: [Intel-gfx] [PATCH 09/11] drm/omap: Follow implicit fencing in prepare_fb
Date: Mon, 24 May 2021 10:53:34 +0300 [thread overview]
Message-ID: <a8f87dea-5c1d-0424-6764-d844b8451d39@ideasonboard.com> (raw)
In-Reply-To: <20210521090959.1663703-9-daniel.vetter@ffwll.ch>
On 21/05/2021 12:09, Daniel Vetter wrote:
> I guess no one ever tried running omap together with lima or panfrost,
> not even sure that's possible. Anyway for consistency, fix this.
>
> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
> Cc: Tomi Valkeinen <tomba@kernel.org>
> ---
> drivers/gpu/drm/omapdrm/omap_plane.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/drivers/gpu/drm/omapdrm/omap_plane.c b/drivers/gpu/drm/omapdrm/omap_plane.c
> index 801da917507d..512af976b7e9 100644
> --- a/drivers/gpu/drm/omapdrm/omap_plane.c
> +++ b/drivers/gpu/drm/omapdrm/omap_plane.c
> @@ -6,6 +6,7 @@
>
> #include <drm/drm_atomic.h>
> #include <drm/drm_atomic_helper.h>
> +#include <drm/drm_gem_atomic_helper.h>
> #include <drm/drm_plane_helper.h>
>
> #include "omap_dmm_tiler.h"
> @@ -29,6 +30,8 @@ static int omap_plane_prepare_fb(struct drm_plane *plane,
> if (!new_state->fb)
> return 0;
>
> + drm_gem_plane_helper_prepare_fb(plane, new_state);
> +
> return omap_framebuffer_pin(new_state->fb);
> }
>
>
Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Tomi
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
WARNING: multiple messages have this Message-ID (diff)
From: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
To: Daniel Vetter <daniel.vetter@ffwll.ch>,
DRI Development <dri-devel@lists.freedesktop.org>
Cc: Daniel Vetter <daniel.vetter@intel.com>,
Intel Graphics Development <intel-gfx@lists.freedesktop.org>
Subject: Re: [PATCH 09/11] drm/omap: Follow implicit fencing in prepare_fb
Date: Mon, 24 May 2021 10:53:34 +0300 [thread overview]
Message-ID: <a8f87dea-5c1d-0424-6764-d844b8451d39@ideasonboard.com> (raw)
In-Reply-To: <20210521090959.1663703-9-daniel.vetter@ffwll.ch>
On 21/05/2021 12:09, Daniel Vetter wrote:
> I guess no one ever tried running omap together with lima or panfrost,
> not even sure that's possible. Anyway for consistency, fix this.
>
> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
> Cc: Tomi Valkeinen <tomba@kernel.org>
> ---
> drivers/gpu/drm/omapdrm/omap_plane.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/drivers/gpu/drm/omapdrm/omap_plane.c b/drivers/gpu/drm/omapdrm/omap_plane.c
> index 801da917507d..512af976b7e9 100644
> --- a/drivers/gpu/drm/omapdrm/omap_plane.c
> +++ b/drivers/gpu/drm/omapdrm/omap_plane.c
> @@ -6,6 +6,7 @@
>
> #include <drm/drm_atomic.h>
> #include <drm/drm_atomic_helper.h>
> +#include <drm/drm_gem_atomic_helper.h>
> #include <drm/drm_plane_helper.h>
>
> #include "omap_dmm_tiler.h"
> @@ -29,6 +30,8 @@ static int omap_plane_prepare_fb(struct drm_plane *plane,
> if (!new_state->fb)
> return 0;
>
> + drm_gem_plane_helper_prepare_fb(plane, new_state);
> +
> return omap_framebuffer_pin(new_state->fb);
> }
>
>
Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Tomi
next prev parent reply other threads:[~2021-05-24 7:53 UTC|newest]
Thread overview: 175+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-21 9:09 [Intel-gfx] [PATCH 01/11] drm/amdgpu: Comply with implicit fencing rules Daniel Vetter
2021-05-21 9:09 ` Daniel Vetter
2021-05-21 9:09 ` [Intel-gfx] [PATCH 02/11] drm/panfrost: Remove sched_lock Daniel Vetter
2021-05-21 9:09 ` Daniel Vetter
2021-05-21 9:32 ` Lucas Stach
2021-05-21 9:32 ` [Intel-gfx] " Lucas Stach
2021-05-21 14:49 ` Daniel Vetter
2021-05-21 14:49 ` Daniel Vetter
2021-05-21 9:09 ` [PATCH 03/11] drm/panfrost: Use xarray and helpers for depedency tracking Daniel Vetter
2021-05-21 9:09 ` Daniel Vetter
2021-05-21 9:09 ` [Intel-gfx] " Daniel Vetter
2021-06-02 14:06 ` Steven Price
2021-06-02 14:06 ` Steven Price
2021-06-02 14:06 ` Steven Price
2021-06-02 18:51 ` Daniel Vetter
2021-06-02 18:51 ` Daniel Vetter
2021-06-02 18:51 ` [Intel-gfx] " Daniel Vetter
2021-06-03 7:48 ` Steven Price
2021-06-03 7:48 ` [Intel-gfx] " Steven Price
2021-06-03 7:48 ` Steven Price
2021-05-21 9:09 ` [PATCH 04/11] drm/panfrost: Fix implicit sync Daniel Vetter
2021-05-21 9:09 ` Daniel Vetter
2021-05-21 9:09 ` [Intel-gfx] " Daniel Vetter
2021-05-21 12:22 ` Daniel Stone
2021-05-21 12:22 ` Daniel Stone
2021-05-21 12:22 ` [Intel-gfx] " Daniel Stone
2021-05-21 12:28 ` [Intel-gfx] [Linaro-mm-sig] " Christian König
2021-05-21 12:28 ` Christian König
2021-05-21 12:54 ` [Intel-gfx] " Daniel Stone
2021-05-21 12:54 ` Daniel Stone
2021-05-21 12:54 ` Daniel Stone
2021-05-21 13:09 ` Christian König
2021-05-21 13:09 ` Christian König
2021-05-21 13:09 ` [Intel-gfx] " Christian König
2021-05-21 13:23 ` Daniel Stone
2021-05-21 13:23 ` Daniel Stone
2021-05-21 13:23 ` Daniel Stone
2021-05-21 9:09 ` [Intel-gfx] [PATCH 05/11] drm/atomic-helper: make drm_gem_plane_helper_prepare_fb the default Daniel Vetter
2021-05-21 9:09 ` Daniel Vetter
2021-05-21 9:09 ` [Intel-gfx] [PATCH 06/11] drm/<driver>: drm_gem_plane_helper_prepare_fb is now " Daniel Vetter
2021-05-21 9:09 ` Daniel Vetter
2021-05-21 9:09 ` Daniel Vetter
2021-05-21 9:09 ` Daniel Vetter
2021-05-21 9:09 ` Daniel Vetter
2021-05-21 9:09 ` Daniel Vetter
2021-05-21 9:09 ` Daniel Vetter
2021-05-21 9:38 ` Lucas Stach
2021-05-21 9:38 ` Lucas Stach
2021-05-21 9:38 ` Lucas Stach
2021-05-21 9:38 ` Lucas Stach
2021-05-21 9:38 ` Lucas Stach
2021-05-21 9:38 ` Lucas Stach
2021-05-21 9:38 ` [Intel-gfx] " Lucas Stach
2021-05-21 12:20 ` Heiko Stübner
2021-05-21 12:20 ` Heiko Stübner
2021-05-21 12:20 ` [Intel-gfx] " Heiko Stübner
2021-05-21 12:20 ` Heiko Stübner
2021-05-21 12:20 ` Heiko Stübner
2021-05-21 12:20 ` Heiko Stübner
2021-05-21 12:20 ` Heiko Stübner
2021-05-21 12:22 ` Paul Cercueil
2021-05-21 12:22 ` [Intel-gfx] " Paul Cercueil
2021-05-21 12:22 ` Paul Cercueil
2021-05-21 12:22 ` Paul Cercueil
2021-05-21 12:22 ` Paul Cercueil
2021-05-21 12:22 ` Paul Cercueil
2021-05-21 12:22 ` Paul Cercueil
2021-05-21 15:53 ` Jernej Škrabec
2021-05-21 15:53 ` Jernej Škrabec
2021-05-21 15:53 ` [Intel-gfx] " Jernej Škrabec
2021-05-21 15:53 ` Jernej Škrabec
2021-05-21 15:53 ` Jernej Škrabec
2021-05-21 15:53 ` Jernej Škrabec
2021-05-21 15:53 ` Jernej Škrabec
2021-05-21 23:18 ` Chun-Kuang Hu
2021-05-21 23:18 ` Chun-Kuang Hu
2021-05-21 23:18 ` Chun-Kuang Hu
2021-05-21 23:18 ` Chun-Kuang Hu
2021-05-21 23:18 ` Chun-Kuang Hu
2021-05-21 23:18 ` [Intel-gfx] " Chun-Kuang Hu
2021-05-21 23:18 ` Chun-Kuang Hu
2021-05-23 12:17 ` Martin Blumenstingl
2021-05-23 12:17 ` [Intel-gfx] " Martin Blumenstingl
2021-05-23 12:17 ` Martin Blumenstingl
2021-05-23 12:17 ` Martin Blumenstingl
2021-05-23 12:17 ` Martin Blumenstingl
2021-05-23 12:17 ` Martin Blumenstingl
2021-05-23 12:17 ` Martin Blumenstingl
2021-05-24 7:54 ` [Intel-gfx] " Tomi Valkeinen
2021-05-24 7:54 ` Tomi Valkeinen
2021-05-24 7:54 ` Tomi Valkeinen
2021-05-24 7:54 ` Tomi Valkeinen
2021-05-24 7:54 ` Tomi Valkeinen
2021-05-24 7:54 ` Tomi Valkeinen
2021-05-24 7:54 ` Tomi Valkeinen
2021-05-28 9:55 ` [Intel-gfx] " Philippe CORNU
2021-05-28 9:55 ` Philippe CORNU
2021-05-28 9:55 ` Philippe CORNU
2021-05-28 9:55 ` Philippe CORNU
2021-05-28 9:55 ` Philippe CORNU
2021-05-28 9:55 ` Philippe CORNU
2021-05-28 9:55 ` Philippe CORNU
2021-05-21 9:09 ` [Intel-gfx] [PATCH 07/11] drm/armada: Remove prepare/cleanup_fb hooks Daniel Vetter
2021-05-21 9:09 ` Daniel Vetter
2021-05-21 9:09 ` [Intel-gfx] [PATCH 08/11] drm/vram-helpers: Create DRM_GEM_VRAM_PLANE_HELPER_FUNCS Daniel Vetter
2021-05-21 9:09 ` Daniel Vetter
2021-05-21 9:33 ` tiantao (H)
2021-05-21 9:33 ` [Intel-gfx] " tiantao (H)
2021-05-21 9:09 ` [Intel-gfx] [PATCH 09/11] drm/omap: Follow implicit fencing in prepare_fb Daniel Vetter
2021-05-21 9:09 ` Daniel Vetter
2021-05-24 7:53 ` Tomi Valkeinen [this message]
2021-05-24 7:53 ` Tomi Valkeinen
2021-05-21 9:09 ` [Intel-gfx] [PATCH 10/11] drm/simple-helper: drm_gem_simple_display_pipe_prepare_fb as default Daniel Vetter
2021-05-21 9:09 ` Daniel Vetter
2021-05-25 17:48 ` Noralf Trønnes
2021-05-25 17:48 ` [Intel-gfx] " Noralf Trønnes
2021-05-25 17:53 ` Daniel Vetter
2021-05-25 17:53 ` Daniel Vetter
2021-05-21 9:09 ` [Intel-gfx] [PATCH 11/11] drm/tiny: drm_gem_simple_display_pipe_prepare_fb is the default Daniel Vetter
2021-05-21 9:09 ` Daniel Vetter
2021-05-21 9:09 ` Daniel Vetter
2021-05-21 9:09 ` Daniel Vetter
2021-05-21 13:41 ` David Lechner
2021-05-21 13:41 ` David Lechner
2021-05-21 13:41 ` [Intel-gfx] " David Lechner
2021-05-21 13:41 ` David Lechner
2021-05-21 14:09 ` Noralf Trønnes
2021-05-21 14:09 ` [Intel-gfx] " Noralf Trønnes
2021-05-21 14:09 ` Noralf Trønnes
2021-05-21 14:09 ` Noralf Trønnes
2021-05-25 16:05 ` Daniel Vetter
2021-05-25 16:05 ` Daniel Vetter
2021-05-25 16:05 ` Daniel Vetter
2021-05-25 16:05 ` [Intel-gfx] " Daniel Vetter
2021-05-21 14:13 ` Oleksandr Andrushchenko
2021-05-21 14:13 ` [Intel-gfx] " Oleksandr Andrushchenko
2021-05-21 14:13 ` Oleksandr Andrushchenko
2021-05-21 14:13 ` Oleksandr Andrushchenko
2021-05-28 0:38 ` Linus Walleij
2021-05-28 0:38 ` Linus Walleij
2021-05-28 0:38 ` Linus Walleij
2021-05-28 0:38 ` [Intel-gfx] " Linus Walleij
2021-05-21 9:18 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [01/11] drm/amdgpu: Comply with implicit fencing rules Patchwork
2021-05-21 9:20 ` [Intel-gfx] ✗ Fi.CI.SPARSE: " Patchwork
2021-05-21 9:45 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2021-05-21 9:46 ` [Intel-gfx] [PATCH 01/11] " Bas Nieuwenhuizen
2021-05-21 9:46 ` Bas Nieuwenhuizen
2021-05-21 14:37 ` [Intel-gfx] " Daniel Vetter
2021-05-21 14:37 ` Daniel Vetter
2021-05-21 15:00 ` [Intel-gfx] " Bas Nieuwenhuizen
2021-05-21 15:00 ` Bas Nieuwenhuizen
2021-05-21 15:16 ` [Intel-gfx] " Daniel Vetter
2021-05-21 15:16 ` Daniel Vetter
2021-05-21 18:08 ` [Mesa-dev] " Christian König
2021-05-21 18:08 ` [Intel-gfx] " Christian König
2021-05-21 18:31 ` Daniel Vetter
2021-05-21 18:31 ` Daniel Vetter
2021-05-22 8:30 ` [Intel-gfx] " Christian König
2021-05-22 8:30 ` Christian König
2021-05-25 13:05 ` [Intel-gfx] " Daniel Vetter
2021-05-25 13:05 ` Daniel Vetter
2021-05-25 15:05 ` Christian König
2021-05-25 15:23 ` [Intel-gfx] " Daniel Vetter
2021-05-25 15:23 ` Daniel Vetter
2021-05-26 13:32 ` Christian König
2021-05-26 13:32 ` [Intel-gfx] " Christian König
2021-05-26 13:51 ` Daniel Vetter
2021-05-26 13:51 ` Daniel Vetter
2021-05-21 11:22 ` [Intel-gfx] " Christian König
2021-05-21 11:22 ` Christian König
2021-05-21 14:58 ` [Intel-gfx] [Mesa-dev] " Rob Clark
2021-05-21 14:58 ` Rob Clark
2021-05-21 14:58 ` [Intel-gfx] " Daniel Vetter
2021-05-21 14:58 ` Daniel Vetter
2021-05-23 5:00 ` [Intel-gfx] ✗ Fi.CI.IGT: failure for series starting with [01/11] " Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=a8f87dea-5c1d-0424-6764-d844b8451d39@ideasonboard.com \
--to=tomi.valkeinen@ideasonboard.com \
--cc=daniel.vetter@ffwll.ch \
--cc=daniel.vetter@intel.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=intel-gfx@lists.freedesktop.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.