From: Ville Syrjälä Date: Thu, 9 Apr 2026 10:15:37 +0000 (+0300) Subject: drm/i915/dp: Validate sink format in .mode_valid() X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=5c7a549671897a52f4351cbb7b279162e84a26b6;p=thirdparty%2Fkernel%2Flinux.git drm/i915/dp: Validate sink format in .mode_valid() Make sure the sink supports our chosen sink format. I suppose it might be at least possible that some PCONs might not snoop the EDID hard enough and filter out all the modes that they should. Also if we ever want to add a similar "force DVI" knob to DP outputs that we have for native HDMI, we'd need to manually get rid of anything DVI sinks can't handle. Cc: Nicolas Frattaroli Reviewed-by: Ankit Nautiyal Signed-off-by: Ville Syrjälä Link: https://patch.msgid.link/20260409101539.22032-8-ville.syrjala@linux.intel.com Tested-by: Nicolas Frattaroli --- diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c index e806828345300..945994fe681ba 100644 --- a/drivers/gpu/drm/i915/display/intel_dp.c +++ b/drivers/gpu/drm/i915/display/intel_dp.c @@ -1457,6 +1457,10 @@ intel_dp_mode_valid_format(struct intel_connector *connector, int num_joined_pipes; int link_bpp_x16; + status = intel_dp_sink_format_valid(connector, mode, sink_format); + if (status != MODE_OK) + return status; + output_format = intel_dp_output_format(connector, sink_format); max_link_clock = intel_dp_max_link_rate(intel_dp);