mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Bert Karwatzki <spasswolf@web.de>
To: Joshua Peisach <jpeisach@ubuntu.com>, linux-kernel@vger.kernel.org
Cc: linux-next@vger.kernel.org,
	Alex Deucher <alexander.deucher@amd.com>,
	 Leandro Ribeiro <leandro.ribeiro@collabora.com>,
	"Wentland, Harry" <Harry.Wentland@amd.com>,
	Leo Li	 <Sunpeng.Li@amd.com>, Alex Hung <alex.hung@amd.com>,
		amd-gfx@lists.freedesktop.org, Jesse Zhang <jesse.zhang@amd.com>,
	Amber Lin	 <Amber.Lin@amd.com>,
	Mario Limonciello <mario.limonciello@amd.com>,
	amd-gfx	 <amd-gfx-bounces@lists.freedesktop.org>,
	spasswolf@web.de
Subject: Re: [PATCH 1/8] display: fix blend_mode for non-overlay planes
Date: Fri, 21 Aug 2026 21:42:52 +0200	[thread overview]
Message-ID: <7ac562a41bc25f459c2102a01f2d07f2437984b8.camel@web.de> (raw)
In-Reply-To: <DKUUGMJ1H5N5.2FRQG3ZKH6SNB@ubuntu.com>

Am Freitag, dem 21.08.2026 um 14:47 -0400 schrieb Joshua Peisach:
> On Fri Aug 14, 2026 at 11:17 AM EDT, Bert Karwatzki wrote:
> > Previously drm_plane_create_blend_mode_property() was only called for
> > planes of type DRM_PLANE_TYPE_OVERLAY. Since commit commit 860e748bddcc
> > ("drm: ensure blend mode supported if pixel format with alpha exposed")
> > this results in warnings from validate_blend_mode_for_alpha_formats().
> > We fix this here by calling drm_plane_create_blend_mode_property() for
> > all plane types if the plane is supporting a format with
> > per_pixel_alpha.
> > 
> > Signed-off-by: Bert Karwatzki <spasswolf@web.de>
> > ---
> >  drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_crtc.c  | 5 ++++-
> >  drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.c | 3 +--
> >  2 files changed, 5 insertions(+), 3 deletions(-)
> > 
> > diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_crtc.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_crtc.c
> > index 62eac6e65334..a6a54351ea56 100644
> > --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_crtc.c
> > +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_crtc.c
> > @@ -755,13 +755,16 @@ int amdgpu_dm_crtc_init(struct amdgpu_display_manager *dm,
> >  	struct drm_plane *cursor_plane;
> >  	bool has_degamma;
> >  	int res = -ENOMEM;
> > +	struct dc_plane_cap dummy_cap = {
> > +		.per_pixel_alpha = 1,
> > +	};
> >  
> >  	cursor_plane = kzalloc_obj(*cursor_plane);
> >  	if (!cursor_plane)
> >  		goto fail;
> >  
> >  	cursor_plane->type = DRM_PLANE_TYPE_CURSOR;
> > -	res = amdgpu_dm_plane_init(dm, cursor_plane, 0, NULL);
> > +	res = amdgpu_dm_plane_init(dm, cursor_plane, 0, &dummy_cap);
> >  
> >  	acrtc = kzalloc_obj(struct amdgpu_crtc);
> >  	if (!acrtc)
> > diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.c
> > index 824ef3ce5de0..4c0c6a8a0dac 100644
> > --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.c
> > +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.c
> > @@ -2208,8 +2208,7 @@ int amdgpu_dm_plane_init(struct amdgpu_display_manager *dm,
> >  	if (res)
> >  		return res;
> >  
> > -	if (plane->type == DRM_PLANE_TYPE_OVERLAY &&
> > -	    plane_cap && plane_cap->per_pixel_alpha) {
> > +	if (plane_cap && plane_cap->per_pixel_alpha) {
> >  		unsigned int blend_caps = BIT(DRM_MODE_BLEND_PIXEL_NONE) |
> >  					  BIT(DRM_MODE_BLEND_PREMULTI) |
> >  					  BIT(DRM_MODE_BLEND_COVERAGE);
> 
> Not sure where the full patch set is, but this does make the warning
> go away.
> 
> Tested-by: Joshua Peisach <jpeisach@ubuntu.com>

This is actually the only patch, the 1/8 is an artifact from using 
git format-patch which I forgot to remove.

Bert Karwatzki

  reply	other threads:[~2026-08-21 19:43 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-07-16 15:29 warnings from validate_blend_mode_for_alpha_formats() in next-20260715 Bert Karwatzki
2026-07-17 14:55 ` Alex Deucher
2026-07-17 22:54   ` Leandro Ribeiro
2026-07-18  9:21     ` Bert Karwatzki
2026-07-18 12:22       ` Bert Karwatzki
2026-08-14 15:17         ` [PATCH 1/8] display: fix blend_mode for non-overlay planes Bert Karwatzki
2026-08-21 18:47           ` Joshua Peisach
2026-08-21 19:42             ` Bert Karwatzki [this message]
2026-08-21 20:31               ` Alex Hung
2026-08-24  9:33                 ` Bert Karwatzki
2026-08-25 21:28 ` warnings from validate_blend_mode_for_alpha_formats() in next-20260715 Mikhail Gavrilov
2026-08-26  9:22   ` Greg KH

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=7ac562a41bc25f459c2102a01f2d07f2437984b8.camel@web.de \
    --to=spasswolf@web.de \
    --cc=Amber.Lin@amd.com \
    --cc=Harry.Wentland@amd.com \
    --cc=Sunpeng.Li@amd.com \
    --cc=alex.hung@amd.com \
    --cc=alexander.deucher@amd.com \
    --cc=amd-gfx-bounces@lists.freedesktop.org \
    --cc=amd-gfx@lists.freedesktop.org \
    --cc=jesse.zhang@amd.com \
    --cc=jpeisach@ubuntu.com \
    --cc=leandro.ribeiro@collabora.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-next@vger.kernel.org \
    --cc=mario.limonciello@amd.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®