This change is required for DSC 1.1 because the current calculation
is for DSC 1.2 and may get a calculated value which is not
recommended by DSC 1.1, for example, the calculated value at 8bpp
becomes 15, not the value of 12 recommened by DSC 1.1.

Cc: Ankit Nautiyal <ankit.k.nauti...@intel.com>
Cc: Juha-Pekka Heikkil <juha-pekka.heikk...@intel.com>
Signed-off-by: William Tseng <william.ts...@intel.com>
---
 drivers/gpu/drm/i915/display/intel_vdsc.c | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_vdsc.c 
b/drivers/gpu/drm/i915/display/intel_vdsc.c
index bd9116d2cd76..9b350f5f8ebb 100644
--- a/drivers/gpu/drm/i915/display/intel_vdsc.c
+++ b/drivers/gpu/drm/i915/display/intel_vdsc.c
@@ -72,11 +72,18 @@ calculate_rc_params(struct drm_dsc_config *vdsc_cfg)
        int qp_bpc_modifier = (bpc - 8) * 2;
        u32 res, buf_i, bpp_i;
 
-       if (vdsc_cfg->slice_height >= 8)
-               vdsc_cfg->first_line_bpg_offset =
-                       12 + DIV_ROUND_UP((9 * min(34, vdsc_cfg->slice_height - 
8)), 100);
-       else
-               vdsc_cfg->first_line_bpg_offset = 2 * (vdsc_cfg->slice_height - 
1);
+       if (vdsc_cfg->dsc_version_minor == 2) {
+               if (vdsc_cfg->slice_height >= 8)
+                       vdsc_cfg->first_line_bpg_offset =
+                               12 + DIV_ROUND_UP((9 * min(34, 
vdsc_cfg->slice_height - 8)), 100);
+               else
+                       vdsc_cfg->first_line_bpg_offset = 2 * 
(vdsc_cfg->slice_height - 1);
+       } else {
+               if (bpp == 8)
+                       rc->first_line_bpg_offset = 12;
+               else
+                       rc->first_line_bpg_offset = 15;
+       }
 
        /* Our hw supports only 444 modes as of today */
        if (bpp >= 12)
-- 
2.34.1

Reply via email to