Re: [PATCH v1] Revert "drm/virtio: Add pixel blend mode property to cursor plane"

[Dmitry Osipenko <dmitry.osipenko@collabora.com>]

On 10/8/26 12:40, Marek Szyprowski wrote:
> On 08.10.2026 11:35, Dmitry Osipenko wrote:
>> On 10/8/26 01:22, Dmitry Osipenko wrote:
>>> The cursor blending mode was added twice because the same change was
>>> applied by accident from different patches. Revert the second patch.
>>>
>>> Reported-by: Marek Szyprowski <m.szyprowski@samsung.com>
>>> Link: https://lore.kernel.org/dri-devel/ce84d3d2-e78b-4939-aecf-9ef7267bd42a@samsung.com/
>>> Fixes: 6947b78df4d2 ("drm/virtio: Add pixel blend mode property to cursor plane")
>>> Signed-off-by: Dmitry Osipenko <dmitry.osipenko@collabora.com>
>>> ---
>>>  drivers/gpu/drm/virtio/virtgpu_plane.c | 13 -------------
>>>  1 file changed, 13 deletions(-)
>>>
>>> diff --git a/drivers/gpu/drm/virtio/virtgpu_plane.c b/drivers/gpu/drm/virtio/virtgpu_plane.c
>>> index b422eba42a5f..640815af4098 100644
>>> --- a/drivers/gpu/drm/virtio/virtgpu_plane.c
>>> +++ b/drivers/gpu/drm/virtio/virtgpu_plane.c
>>> @@ -589,7 +589,6 @@ struct drm_plane *virtio_gpu_plane_init(struct virtio_gpu_device *vgdev,
>>>  	struct drm_plane *plane;
>>>  	const uint32_t *formats;
>>>  	int nformats;
>>> -	int ret;
>>>  
>>>  	if (type == DRM_PLANE_TYPE_CURSOR) {
>>>  		formats = virtio_gpu_cursor_formats;
>>> @@ -615,17 +614,5 @@ struct drm_plane *virtio_gpu_plane_init(struct virtio_gpu_device *vgdev,
>>>  		drm_plane_create_blend_mode_property(plane,
>>>  						     BIT(DRM_MODE_BLEND_PREMULTI));
>>>  
>>> -	if (type == DRM_PLANE_TYPE_CURSOR) {
>>> -		/*
>>> -		 * The cursor plane exposes a format with an alpha channel,
>>> -		 * which requires a blend mode property. The host blends
>>> -		 * premultiplied alpha, matching the property's default.
>>> -		 */
>>> -		ret = drm_plane_create_blend_mode_property(plane,
>>> -							   BIT(DRM_MODE_BLEND_PREMULTI));
>>> -		if (ret)
>>> -			return ERR_PTR(ret);
>>> -	}
>>> -
>>>  	return plane;
>>>  }
>> Marek, could you please give your r-b?
> 
> Reviewed-by: Marek Szyprowski <m.szyprowski@samsung.com>

Thanks, applied to misc-fixes

-- 
Best regards,
Dmitry