[PATCH 7/8] drm/amd/display: fix 12bpc truncate to 10bpc

[Harry Wentland]

From: Charlene Liu <charlene....@amd.com>

Change-Id: I32427f4c3ea4aa410732aa4837098b5e109a1647
Signed-off-by: Charlene Liu <charlene....@amd.com>
Reviewed-by: Tony Cheng <tony.ch...@amd.com>
Acked-by: Harry Wentland <harry.wentl...@amd.com>
---
 drivers/gpu/drm/amd/display/dc/dce/dce_opp.c | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/dce/dce_opp.c 
b/drivers/gpu/drm/amd/display/dc/dce/dce_opp.c
index a2f57cf603aa..121ccbb1ad2d 100644
--- a/drivers/gpu/drm/amd/display/dc/dce/dce_opp.c
+++ b/drivers/gpu/drm/amd/display/dc/dce/dce_opp.c
@@ -710,32 +710,32 @@ void dce110_opp_set_dyn_expansion(
        enum signal_type signal)
 {
        struct dce110_opp *opp110 = TO_DCE110_OPP(opp);
-       bool enable_dyn_exp = false;
 
        REG_UPDATE_2(FMT_DYNAMIC_EXP_CNTL,
                        FMT_DYNAMIC_EXP_EN, 0,
                        FMT_DYNAMIC_EXP_MODE, 0);
-       /* From HW programming guide:
-               FMT_DYNAMIC_EXP_EN = 0 for limited RGB or YCbCr output
-               FMT_DYNAMIC_EXP_EN = 1 for RGB full range only*/
-       if (color_sp == COLOR_SPACE_SRGB)
-               enable_dyn_exp = true;
 
        /*00 - 10-bit -> 12-bit dynamic expansion*/
        /*01 - 8-bit  -> 12-bit dynamic expansion*/
-       if (signal == SIGNAL_TYPE_HDMI_TYPE_A) {
+       if (signal == SIGNAL_TYPE_HDMI_TYPE_A ||
+               signal == SIGNAL_TYPE_DISPLAY_PORT ||
+               signal == SIGNAL_TYPE_DISPLAY_PORT_MST) {
                switch (color_dpth) {
                case COLOR_DEPTH_888:
                        REG_UPDATE_2(FMT_DYNAMIC_EXP_CNTL,
-                               FMT_DYNAMIC_EXP_EN, enable_dyn_exp ? 1:0,
+                               FMT_DYNAMIC_EXP_EN, 1,
                                FMT_DYNAMIC_EXP_MODE, 1);
                        break;
                case COLOR_DEPTH_101010:
                        REG_UPDATE_2(FMT_DYNAMIC_EXP_CNTL,
-                               FMT_DYNAMIC_EXP_EN, enable_dyn_exp ? 1:0,
+                               FMT_DYNAMIC_EXP_EN, 1,
                                FMT_DYNAMIC_EXP_MODE, 0);
                        break;
                case COLOR_DEPTH_121212:
+                       REG_UPDATE_2(
+                               FMT_DYNAMIC_EXP_CNTL,
+                               FMT_DYNAMIC_EXP_EN, 1,/*otherwise last two bits 
are zero*/
+                               FMT_DYNAMIC_EXP_MODE, 0);
                        break;
                default:
                        break;
-- 
2.9.3

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