Skip to content

Commit d4c61c4

Browse files
author
Manasi Navare
committed
drm/i915/dp: Fix DSC enable code to use cpu_transcoder instead of encoder->type
This patch fixes the intel_configure_pps_for_dsc_encoder() function to use cpu_transcoder instead of encoder->type to select the correct DSC registers that was wrongly used in the original patch for one DSC register isntance. Fixes: 7182414 ("drm/i915/dp: Configure i915 Picture parameter Set registers during DSC enabling") Cc: Ville Syrjala <ville.syrjala@linux.intel.com> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Cc: Jani Nikula <jani.nikula@linux.intel.com> Cc: Ville Syrjala <ville.syrjala@linux.intel.com> Cc: <stable@vger.kernel.org> # v5.0+ Signed-off-by: Manasi Navare <manasi.d.navare@intel.com> Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Link: https://patchwork.freedesktop.org/patch/msgid/20190821215950.24223-1-manasi.d.navare@intel.com
1 parent cee508a commit d4c61c4

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

drivers/gpu/drm/i915/display/intel_vdsc.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -547,7 +547,7 @@ static void intel_configure_pps_for_dsc_encoder(struct intel_encoder *encoder,
547547
pps_val |= DSC_PIC_HEIGHT(vdsc_cfg->pic_height) |
548548
DSC_PIC_WIDTH(vdsc_cfg->pic_width / num_vdsc_instances);
549549
DRM_INFO("PPS2 = 0x%08x\n", pps_val);
550-
if (encoder->type == INTEL_OUTPUT_EDP) {
550+
if (cpu_transcoder == TRANSCODER_EDP) {
551551
I915_WRITE(DSCA_PICTURE_PARAMETER_SET_2, pps_val);
552552
/*
553553
* If 2 VDSC instances are needed, configure PPS for second

0 commit comments

Comments
 (0)