[PATCH 20/22] drm/amd/display: Query all entries in assignment table during updates.

[Anson Jacob]

From: Jimmy Kizito <jimmy.kiz...@amd.com>

[Why]
Stream ordering and count can vary from one state to the next. Only
checking a subset of entries in the encoder assignment table can lead to
invalid encoder assignments.

[How]
Check all entries in encoder assignment table when querying it.

Reviewed-by: Jun Lei <jun....@amd.com>
Acked-by: Anson Jacob <anson.ja...@amd.com>
Signed-off-by: Jimmy Kizito <jimmy.kiz...@amd.com>
---
 .../gpu/drm/amd/display/dc/core/dc_link_enc_cfg.c   | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_link_enc_cfg.c 
b/drivers/gpu/drm/amd/display/dc/core/dc_link_enc_cfg.c
index b96e301f64f2..787aaa661a29 100644
--- a/drivers/gpu/drm/amd/display/dc/core/dc_link_enc_cfg.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc_link_enc_cfg.c
@@ -232,7 +232,7 @@ static struct link_encoder *get_link_enc_used_by_link(
                .link_id = link->link_id,
                .ep_type = link->ep_type};
 
-       for (i = 0; i < state->stream_count; i++) {
+       for (i = 0; i < MAX_PIPES; i++) {
                struct link_enc_assignment assignment = 
state->res_ctx.link_enc_cfg_ctx.link_enc_assignments[i];
 
                if (assignment.valid == true && 
are_ep_ids_equal(&assignment.ep_id, &ep_id))
@@ -538,6 +538,7 @@ bool link_enc_cfg_validate(struct dc *dc, struct dc_state 
*state)
        uint8_t dig_stream_count = 0;
        int matching_stream_ptrs = 0;
        int eng_ids_per_ep_id[MAX_PIPES] = {0};
+       int valid_bitmap = 0;
 
        /* (1) No. valid entries same as stream count. */
        for (i = 0; i < MAX_PIPES; i++) {
@@ -619,5 +620,15 @@ bool link_enc_cfg_validate(struct dc *dc, struct dc_state 
*state)
        is_valid = valid_entries && valid_stream_ptrs && valid_uniqueness && 
valid_avail && valid_streams;
        ASSERT(is_valid);
 
+       if (is_valid == false) {
+               valid_bitmap =
+                       (valid_entries & 0x1) |
+                       ((valid_stream_ptrs & 0x1) << 1) |
+                       ((valid_uniqueness & 0x1) << 2) |
+                       ((valid_avail & 0x1) << 3) |
+                       ((valid_streams & 0x1) << 4);
+               dm_error("Invalid link encoder assignments: 0x%x\n", 
valid_bitmap);
+       }
+
        return is_valid;
 }
-- 
2.25.1