github-mirror/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-09-25 17:42:03 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity Graph

drm/i915/dp: Iterate configurations via link_caps for SST DSC

Browse Source 
Use the link caps configuration iterator for DP SST link configuration
computation for DSC mode. This is a step towards unifying configuration
selection and iteration across connector types and between compute and
fallback paths.

The iteration preserves the DP SST connector rate/lane ordering used by
the current code.

This also allows removing the now unused common rate count helper.

v2:
- Rebase on changes using a filter object instead of a mask of
  configuration indices.
- Rebase on changes using an iteration object.

Reviewed-by: Luca Coelho <luciano.coelho@intel.com>
Signed-off-by: Imre Deak <imre.deak@intel.com>
Link: https://patch.msgid.link/20260701153204.4124150-19-imre.deak@intel.com
This commit is contained in:
Imre Deak 2026-07-01 18:31:47 +03:00
parent 7266df62ed
commit 4f104fc10a
3 changed files with 44 additions and 54 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -1988,20 +1988,16 @@ static int dsc_compute_link_config(struct intel_dp *intel_dp,
const struct link_config_limits *limits, const struct link_config_limits *limits,
int dsc_bpp_x16) int dsc_bpp_x16)
{ {
struct intel_dp_link_caps *link_caps = intel_dp->link.caps;
const struct drm_display_mode *adjusted_mode = &pipe_config->hw.adjusted_mode; const struct drm_display_mode *adjusted_mode = &pipe_config->hw.adjusted_mode;
int link_rate, lane_count; struct intel_connector *connector = to_intel_connector(conn_state->connector);
int i; struct intel_dp_link_caps *link_caps = intel_dp->link.caps;
struct intel_dp_link_caps_order order =
for (i = 0; i < intel_dp_link_caps_num_common_rates(intel_dp->link.caps); i++) { intel_dp_link_caps_connector_compute_order(connector);
link_rate = intel_dp_common_rate(link_caps, i); struct intel_dp_link_config link_config;
if (link_rate < limits->min_rate || link_rate > limits->max_rate) struct intel_dp_link_caps_iter iter;
continue;
for (lane_count = limits->min_lane_count;
lane_count <= limits->max_lane_count;
lane_count <<= 1) {
intel_dp_link_caps_iter_start(&iter, link_caps, order, limits->link_config_filter);
for_each_dp_link_config(&iter, &link_config) {
/* /*
* FIXME: intel_dp_mtp_tu_compute_config() requires * FIXME: intel_dp_mtp_tu_compute_config() requires
* ->lane_count and ->port_clock set before we know * ->lane_count and ->port_clock set before we know
@ -2010,10 +2006,10 @@ static int dsc_compute_link_config(struct intel_dp *intel_dp,
* as we'd then bail out from compute config, but it's * as we'd then bail out from compute config, but it's
* just ugly. * just ugly.
*/ */
pipe_config->lane_count = lane_count; pipe_config->lane_count = link_config.lane_count;
pipe_config->port_clock = link_rate; pipe_config->port_clock = link_config.rate;
if (drm_dp_is_uhbr_rate(link_rate)) { if (drm_dp_is_uhbr_rate(link_config.rate)) {
int ret; int ret;
ret = intel_dp_mtp_tu_compute_config(intel_dp, ret = intel_dp_mtp_tu_compute_config(intel_dp,
@ -2031,7 +2027,8 @@ static int dsc_compute_link_config(struct intel_dp *intel_dp,
intel_dsc_line_slice_count(&pipe_config->dsc.slice_config); intel_dsc_line_slice_count(&pipe_config->dsc.slice_config);
if (!is_bw_sufficient_for_dsc_config(intel_dp, if (!is_bw_sufficient_for_dsc_config(intel_dp,
link_rate, lane_count, link_config.rate,
link_config.lane_count,
adjusted_mode->crtc_clock, adjusted_mode->crtc_clock,
adjusted_mode->hdisplay, adjusted_mode->hdisplay,
line_slice_count, line_slice_count,
@ -2042,7 +2039,6 @@ static int dsc_compute_link_config(struct intel_dp *intel_dp,
return 0; return 0;
} }
}
return -EINVAL; return -EINVAL;
} }

5
drivers/gpu/drm/i915/display/intel_dp_link_caps.c
View File

@ -273,11 +273,6 @@ int intel_dp_max_common_rate(struct intel_dp_link_caps *link_caps)
return intel_dp_common_rate(link_caps, link_caps->num_rates - 1); return intel_dp_common_rate(link_caps, link_caps->num_rates - 1);
} }
int intel_dp_link_caps_num_common_rates(struct intel_dp_link_caps *link_caps)
{
return link_caps->num_rates;
}
void intel_dp_link_caps_print_common_rates(struct intel_dp_link_caps *link_caps) void intel_dp_link_caps_print_common_rates(struct intel_dp_link_caps *link_caps)
{ {
struct intel_display *display = to_intel_display(link_caps->dp); struct intel_display *display = to_intel_display(link_caps->dp);

1
drivers/gpu/drm/i915/display/intel_dp_link_caps.h
View File

@ -124,7 +124,6 @@ int intel_dp_common_len_rate_limit(struct intel_dp_link_caps *link_caps,
int intel_dp_common_rate(struct intel_dp_link_caps *link_caps, int index); int intel_dp_common_rate(struct intel_dp_link_caps *link_caps, int index);
int intel_dp_link_caps_common_rate_idx(struct intel_dp_link_caps *link_caps, int rate); int intel_dp_link_caps_common_rate_idx(struct intel_dp_link_caps *link_caps, int rate);
int intel_dp_max_common_rate(struct intel_dp_link_caps *link_caps); int intel_dp_max_common_rate(struct intel_dp_link_caps *link_caps);
int intel_dp_link_caps_num_common_rates(struct intel_dp_link_caps *link_caps);
void intel_dp_link_caps_print_common_rates(struct intel_dp_link_caps *link_caps); void intel_dp_link_caps_print_common_rates(struct intel_dp_link_caps *link_caps);