Linux-Next discussions
 help / color / mirror / Atom feed
From: "Joshua Peisach" <jpeisach@ubuntu.com>
To: "Bert Karwatzki" <spasswolf@web.de>, <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>
Subject: Re: [PATCH 1/8] display: fix blend_mode for non-overlay planes
Date: Fri, 21 Aug 2026 14:47:33 -0400	[thread overview]
Message-ID: <DKUUGMJ1H5N5.2FRQG3ZKH6SNB@ubuntu.com> (raw)
In-Reply-To: <20260814151701.65222-1-spasswolf@web.de>

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>

  reply	other threads:[~2026-08-21 18:47 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 [this message]
2026-08-21 19:42             ` Bert Karwatzki
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=DKUUGMJ1H5N5.2FRQG3ZKH6SNB@ubuntu.com \
    --to=jpeisach@ubuntu.com \
    --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=leandro.ribeiro@collabora.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-next@vger.kernel.org \
    --cc=mario.limonciello@amd.com \
    --cc=spasswolf@web.de \
    /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