From: Iswara Nagulendran <iswara.nagulend...@amd.com>

[WHY]
There is no way to distinguish
the static backlight control type
being used and the VABC support
without the use of a debugger or
reading DPCD registers.

[HOW]
Add Visual Confirm support
for VESA Aux-based Backlight Control.

Reviewed-by: Harry Vanzylldejong <harry.vanzylldej...@amd.com>
Signed-off-by: Iswara Nagulendran <iswara.nagulend...@amd.com>
Signed-off-by: Roman Li <roman...@amd.com>
Tested-by: Daniel Wheeler <daniel.whee...@amd.com>
---
 drivers/gpu/drm/amd/display/dc/core/dc.c      |  2 +
 .../drm/amd/display/dc/core/dc_hw_sequencer.c | 38 +++++++++++++++++++
 drivers/gpu/drm/amd/display/dc/dc.h           |  1 +
 .../drm/amd/display/dc/hwss/hw_sequencer.h    |  6 +--
 4 files changed, 44 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c 
b/drivers/gpu/drm/amd/display/dc/core/dc.c
index 4742a4ad19bd..12839fa3682d 100644
--- a/drivers/gpu/drm/amd/display/dc/core/dc.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
@@ -1236,6 +1236,8 @@ static void dc_update_visual_confirm_color(struct dc *dc, 
struct dc_state *conte
                                get_mclk_switch_visual_confirm_color(pipe_ctx, 
&(pipe_ctx->visual_confirm_color));
                        else if (dc->debug.visual_confirm == 
VISUAL_CONFIRM_FAMS2)
                                get_fams2_visual_confirm_color(dc, context, 
pipe_ctx, &(pipe_ctx->visual_confirm_color));
+                       else if (dc->debug.visual_confirm == 
VISUAL_CONFIRM_VABC)
+                               get_vabc_visual_confirm_color(pipe_ctx, 
&(pipe_ctx->visual_confirm_color));
                }
        }
 }
diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_hw_sequencer.c 
b/drivers/gpu/drm/amd/display/dc/core/dc_hw_sequencer.c
index 252af83e34a5..6eb9bae3af91 100644
--- a/drivers/gpu/drm/amd/display/dc/core/dc_hw_sequencer.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc_hw_sequencer.c
@@ -425,6 +425,44 @@ void get_hdr_visual_confirm_color(
        }
 }
 
+/* Visual Confirm color definition for VABC */
+void get_vabc_visual_confirm_color(
+       struct pipe_ctx *pipe_ctx,
+       struct tg_color *color)
+{
+       uint32_t color_value = MAX_TG_COLOR_VALUE;
+       struct dc_link *edp_link = NULL;
+
+       if (pipe_ctx && pipe_ctx->stream && pipe_ctx->stream->link) {
+               if (pipe_ctx->stream->link->connector_signal == SIGNAL_TYPE_EDP)
+                       edp_link = pipe_ctx->stream->link;
+       }
+
+       if (edp_link) {
+               switch (edp_link->backlight_control_type) {
+               case BACKLIGHT_CONTROL_PWM:
+                       color->color_r_cr = color_value;
+                       color->color_g_y = 0;
+                       color->color_b_cb = 0;
+                       break;
+               case BACKLIGHT_CONTROL_AMD_AUX:
+                       color->color_r_cr = 0;
+                       color->color_g_y = color_value;
+                       color->color_b_cb = 0;
+                       break;
+               case BACKLIGHT_CONTROL_VESA_AUX:
+                       color->color_r_cr = 0;
+                       color->color_g_y = 0;
+                       color->color_b_cb = color_value;
+                       break;
+               }
+       } else {
+               color->color_r_cr = 0;
+               color->color_g_y = 0;
+               color->color_b_cb = 0;
+       }
+}
+
 void get_subvp_visual_confirm_color(
                struct pipe_ctx *pipe_ctx,
                struct tg_color *color)
diff --git a/drivers/gpu/drm/amd/display/dc/dc.h 
b/drivers/gpu/drm/amd/display/dc/dc.h
index 063f853160d0..23a6d56d0798 100644
--- a/drivers/gpu/drm/amd/display/dc/dc.h
+++ b/drivers/gpu/drm/amd/display/dc/dc.h
@@ -489,6 +489,7 @@ enum visual_confirm {
        VISUAL_CONFIRM_MCLK_SWITCH = 16,
        VISUAL_CONFIRM_FAMS2 = 19,
        VISUAL_CONFIRM_HW_CURSOR = 20,
+       VISUAL_CONFIRM_VABC = 21,
 };
 
 enum dc_psr_power_opts {
diff --git a/drivers/gpu/drm/amd/display/dc/hwss/hw_sequencer.h 
b/drivers/gpu/drm/amd/display/dc/hwss/hw_sequencer.h
index a5bb10d7b160..98d85c7ab3fa 100644
--- a/drivers/gpu/drm/amd/display/dc/hwss/hw_sequencer.h
+++ b/drivers/gpu/drm/amd/display/dc/hwss/hw_sequencer.h
@@ -194,7 +194,6 @@ enum block_sequence_func {
        DMUB_SUBVP_SAVE_SURF_ADDR,
        HUBP_WAIT_FOR_DCC_META_PROP,
        DMUB_FAMS2_GLOBAL_CONTROL_LOCK_FAST,
-
 };
 
 struct block_sequence {
@@ -485,11 +484,12 @@ void get_hdr_visual_confirm_color(
 void get_mpctree_visual_confirm_color(
                struct pipe_ctx *pipe_ctx,
                struct tg_color *color);
-
+void get_vabc_visual_confirm_color(
+       struct pipe_ctx *pipe_ctx,
+       struct tg_color *color);
 void get_subvp_visual_confirm_color(
        struct pipe_ctx *pipe_ctx,
        struct tg_color *color);
-
 void get_fams2_visual_confirm_color(
        struct dc *dc,
        struct dc_state *context,
-- 
2.34.1

Reply via email to