This commit implements the "active color format" drm property for the AMD
GPU driver.

Signed-off-by: Werner Sembach <w...@tuxedocomputers.com>
---
 .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 29 +++++++++++++++++--
 .../display/amdgpu_dm/amdgpu_dm_mst_types.c   |  4 +++
 2 files changed, 31 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c 
b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
index d984de82ae63..098f3d53e681 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
@@ -6710,6 +6710,24 @@ static int convert_dc_color_depth_into_bpc (enum 
dc_color_depth display_color_de
        return 0;
 }
 
+static int convert_dc_pixel_encoding_into_drm_color_format(
+       enum dc_pixel_encoding display_pixel_encoding)
+{
+       switch (display_pixel_encoding) {
+       case PIXEL_ENCODING_RGB:
+               return DRM_COLOR_FORMAT_RGB444;
+       case PIXEL_ENCODING_YCBCR422:
+               return DRM_COLOR_FORMAT_YCRCB422;
+       case PIXEL_ENCODING_YCBCR444:
+               return DRM_COLOR_FORMAT_YCRCB444;
+       case PIXEL_ENCODING_YCBCR420:
+               return DRM_COLOR_FORMAT_YCRCB420;
+       default:
+               break;
+       }
+       return 0;
+}
+
 static int dm_encoder_helper_atomic_check(struct drm_encoder *encoder,
                                          struct drm_crtc_state *crtc_state,
                                          struct drm_connector_state 
*conn_state)
@@ -7711,6 +7729,7 @@ void amdgpu_dm_connector_init_helper(struct 
amdgpu_display_manager *dm,
        if (!aconnector->mst_port) {
                drm_connector_attach_max_bpc_property(&aconnector->base, 8, 16);
                drm_connector_attach_active_bpc_property(&aconnector->base, 8, 
16);
+               
drm_connector_attach_active_color_format_property(&aconnector->base);
        }
 
        /* This defaults to the max in the range, but we want 8bpc for non-edp. 
*/
@@ -9090,14 +9109,20 @@ static void amdgpu_dm_atomic_commit_tail(struct 
drm_atomic_state *state)
                        dm_new_crtc_state = to_dm_crtc_state(new_crtc_state);
                        stream = dm_new_crtc_state->stream;
 
-                       if (stream)
+                       if (stream) {
                                drm_connector_set_active_bpc_property(connector,
                                        stream->timing.flags.DSC ?
                                                
stream->timing.dsc_cfg.bits_per_pixel / 16 / 3 :
                                                convert_dc_color_depth_into_bpc(
                                                        
stream->timing.display_color_depth));
-               } else
+                               
drm_connector_set_active_color_format_property(connector,
+                                       
convert_dc_pixel_encoding_into_drm_color_format(
+                                               
dm_new_crtc_state->stream->timing.pixel_encoding));
+                       }
+               } else {
                        drm_connector_set_active_bpc_property(connector, 0);
+                       
drm_connector_set_active_color_format_property(connector, 0);
+               }
        }
 
        /* Count number of newly disabled CRTCs for dropping PM refs later. */
diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c 
b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c
index 0cf38743ec47..13151d13aa73 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c
@@ -413,6 +413,10 @@ dm_dp_add_mst_connector(struct drm_dp_mst_topology_mgr 
*mgr,
        if (connector->active_bpc_property)
                drm_connector_attach_active_bpc_property(&aconnector->base, 8, 
16);
 
+       connector->active_color_format_property = 
master->base.active_color_format_property;
+       if (connector->active_color_format_property)
+               
drm_connector_attach_active_color_format_property(&aconnector->base);
+
        connector->vrr_capable_property = master->base.vrr_capable_property;
        if (connector->vrr_capable_property)
                drm_connector_attach_vrr_capable_property(connector);
-- 
2.25.1

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to