drm/connector: Add a helper to attach the colorspace property
authorMaxime Ripard <maxime@cerno.tech>
Fri, 9 Apr 2021 15:07:32 +0000 (17:07 +0200)
committerpopcornmix <popcornmix@gmail.com>
Mon, 12 Apr 2021 17:08:50 +0000 (18:08 +0100)
The intel driver uses the same logic to attach the Colorspace property
in multiple places and we'll need it in vc4 too. Let's move that common
code in a helper.

Signed-off-by: Maxime Ripard <maxime@cerno.tech>
drivers/gpu/drm/drm_connector.c
drivers/gpu/drm/i915/display/intel_connector.c
include/drm/drm_connector.h

index d631f0a..cdb2dd2 100644 (file)
@@ -2165,6 +2165,26 @@ int drm_connector_attach_hdr_output_metadata_property(struct drm_connector *conn
 EXPORT_SYMBOL(drm_connector_attach_hdr_output_metadata_property);
 
 /**
+ * drm_connector_attach_colorspace_property - attach "Colorspace" property
+ * @connector: connector to attach the property on.
+ *
+ * This is used to allow the userspace to signal the output colorspace
+ * to the driver.
+ *
+ * Returns:
+ * Zero on success, negative errno on failure.
+ */
+int drm_connector_attach_colorspace_property(struct drm_connector *connector)
+{
+       struct drm_property *prop = connector->colorspace_property;
+
+       drm_object_attach_property(&connector->base, prop, DRM_MODE_COLORIMETRY_DEFAULT);
+
+       return 0;
+}
+EXPORT_SYMBOL(drm_connector_attach_colorspace_property);
+
+/**
  * drm_connector_atomic_hdr_metadata_equal - checks if the hdr metadata changed
  * @old_state: old connector state to compare
  * @new_state: new connector state to compare
index 406e967..d1f8c7f 100644 (file)
@@ -297,6 +297,5 @@ intel_attach_colorspace_property(struct drm_connector *connector)
                return;
        }
 
-       drm_object_attach_property(&connector->base,
-                                  connector->colorspace_property, 0);
+       drm_connector_attach_colorspace_property(connector);
 }
index ca94c7a..42cbacd 100644 (file)
@@ -1622,6 +1622,7 @@ int drm_connector_attach_scaling_mode_property(struct drm_connector *connector,
                                               u32 scaling_mode_mask);
 int drm_connector_attach_vrr_capable_property(
                struct drm_connector *connector);
+int drm_connector_attach_colorspace_property(struct drm_connector *connector);
 int drm_connector_attach_hdr_output_metadata_property(struct drm_connector *connector);
 bool drm_connector_atomic_hdr_metadata_equal(struct drm_connector_state *old_state,
                                             struct drm_connector_state *new_state);