Revert "drm: Add solid fill pixel source"
authorDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Mon, 4 Dec 2023 13:13:52 +0000 (15:13 +0200)
committerDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Mon, 4 Dec 2023 19:33:08 +0000 (21:33 +0200)
This reverts commit 4b64167042927531f4cfaf035b8f88c2f7a05f06.

Although the Solid Fill planes patchset got all reviews and
acknowledgements, it doesn't fulfill requirements for the new uABI. It
has neither corresponding open-source userspace implementation nor the
IGT tests coverage. Reverting this patchset until userspace obligations
are fulfilled.

Acked-by: Simon Ser <contact@emersion.fr>
Acked-by: Maxime Ripard <mripard@kernel.org>
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20231204131455.19023-6-dmitry.baryshkov@linaro.org
drivers/gpu/drm/drm_blend.c
include/drm/drm_plane.h

index 37b31b7..665c5d9 100644 (file)
  *     "FB":
  *             Framebuffer source set by the "FB_ID" property.
  *
- *     "SOLID_FILL":
- *             Solid fill color source set by the "solid_fill" property.
- *
  * solid_fill:
  *     solid_fill is set up with drm_plane_create_solid_fill_property(). It
  *     contains pixel data that drivers can use to fill a plane.
@@ -645,7 +642,6 @@ EXPORT_SYMBOL(drm_plane_create_blend_mode_property);
 static const struct drm_prop_enum_list drm_pixel_source_enum_list[] = {
        { DRM_PLANE_PIXEL_SOURCE_NONE, "NONE" },
        { DRM_PLANE_PIXEL_SOURCE_FB, "FB" },
-       { DRM_PLANE_PIXEL_SOURCE_SOLID_FILL, "SOLID_FILL" },
 };
 
 /**
@@ -670,9 +666,6 @@ static const struct drm_prop_enum_list drm_pixel_source_enum_list[] = {
  * "FB":
  *     Framebuffer pixel source
  *
- * "SOLID_FILL":
- *     Solid fill color pixel source
- *
  * Returns:
  * Zero on success, negative errno on failure.
  */
@@ -682,8 +675,7 @@ int drm_plane_create_pixel_source_property(struct drm_plane *plane,
        struct drm_device *dev = plane->dev;
        struct drm_property *prop;
        static const unsigned int valid_source_mask = BIT(DRM_PLANE_PIXEL_SOURCE_FB) |
-                                                     BIT(DRM_PLANE_PIXEL_SOURCE_NONE) |
-                                                     BIT(DRM_PLANE_PIXEL_SOURCE_SOLID_FILL);
+                                                     BIT(DRM_PLANE_PIXEL_SOURCE_NONE);
        int i;
 
        /* FB is supported by default */
index 4b7af43..5bac644 100644 (file)
@@ -43,7 +43,6 @@ enum drm_scaling_filter {
 enum drm_plane_pixel_source {
        DRM_PLANE_PIXEL_SOURCE_NONE,
        DRM_PLANE_PIXEL_SOURCE_FB,
-       DRM_PLANE_PIXEL_SOURCE_SOLID_FILL,
        DRM_PLANE_PIXEL_SOURCE_MAX
 };