github-mirror/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-07-27 17:47:41 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity Graph

drm/i915/pfit: Call intel_pfit_compute_config() unconditionally on (e)DP/HDMI

Browse Source 
We now have three different reasons for calling
intel_pfit_compute_config():
- actual pfit scaling/centering
- YCbCr 4:2:0 output
- sharpness filter

So let's just call intel_pfit_compute_config() unconditionally
from both the DP and HDMI code. Both gmch and ilk+ pfit code
should be capable of judging whether anything actually
needs the pfit.

The only slightly questionable thing in the gmch code is
the dithering knob, but that's only a thing on gen2/3 which
don't even have HDMI/DP outputs, and so not an issue here.

Reviewed-by: Michał Grzelak <michal.grzelak@intel.com>
Reviewed-by: Nemesa Garg <nemesa.garg@intel.com>
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patch.msgid.link/20260407175244.19654-9-ville.syrjala@linux.intel.com
This commit is contained in:
Ville Syrjälä 2026-04-07 20:52:42 +03:00
parent 9a459db0ae
commit d4686f34bb
2 changed files with 6 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
drivers/gpu/drm/i915/display/intel_dp.c
View File

@ -3545,12 +3545,9 @@ intel_dp_compute_config(struct intel_encoder *encoder,
if (ret) if (ret)
return ret; return ret;
if ((intel_dp_is_edp(intel_dp) && fixed_mode) || ret = intel_pfit_compute_config(pipe_config, conn_state);
pipe_config->output_format == INTEL_OUTPUT_FORMAT_YCBCR420) { if (ret)
ret = intel_pfit_compute_config(pipe_config, conn_state); return ret;
if (ret)
return ret;
}
pipe_config->limited_color_range = pipe_config->limited_color_range =
intel_dp_limited_color_range(pipe_config, conn_state); intel_dp_limited_color_range(pipe_config, conn_state);

8
drivers/gpu/drm/i915/display/intel_hdmi.c
View File

@ -2380,11 +2380,9 @@ int intel_hdmi_compute_config(struct intel_encoder *encoder,
return ret; return ret;
} }
if (intel_hdmi_is_ycbcr420(pipe_config)) { ret = intel_pfit_compute_config(pipe_config, conn_state);
ret = intel_pfit_compute_config(pipe_config, conn_state); if (ret)
if (ret) return ret;
return ret;
}
pipe_config->limited_color_range = pipe_config->limited_color_range =
intel_hdmi_limited_color_range(pipe_config, conn_state); intel_hdmi_limited_color_range(pipe_config, conn_state);