Commit 252f3d95 authored by Harry Wentland's avatar Harry Wentland Committed by Alex Deucher

drm/amd/display: Remove unnecessary NULL check in set_preferred_link_settings

[Why]
link_stream is never NULL here as we've dereferenced it a couple lines before
and have done so for a couple months now.

[How]
- Drop the NULL check.
- Initialize where we know link_stream is non-NULL
Signed-off-by: default avatarHarry Wentland <harry.wentland@amd.com>
Reviewed-by: default avatarNicholas Kazlauskas <Nicholas.Kazlauskas@amd.com>
Acked-by: default avatarLeo Li <sunpeng.li@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 03f3e40c
...@@ -3023,22 +3023,21 @@ void dc_link_set_preferred_link_settings(struct dc *dc, ...@@ -3023,22 +3023,21 @@ void dc_link_set_preferred_link_settings(struct dc *dc,
for (i = 0; i < MAX_PIPES; i++) { for (i = 0; i < MAX_PIPES; i++) {
pipe = &dc->current_state->res_ctx.pipe_ctx[i]; pipe = &dc->current_state->res_ctx.pipe_ctx[i];
if (pipe->stream && pipe->stream->link) { if (pipe->stream && pipe->stream->link) {
if (pipe->stream->link == link) if (pipe->stream->link == link) {
link_stream = pipe->stream;
break; break;
} }
} }
}
/* Stream not found */ /* Stream not found */
if (i == MAX_PIPES) if (i == MAX_PIPES)
return; return;
link_stream = link->dc->current_state->res_ctx.pipe_ctx[i].stream;
/* Cannot retrain link if backend is off */ /* Cannot retrain link if backend is off */
if (link_stream->dpms_off) if (link_stream->dpms_off)
return; return;
if (link_stream)
decide_link_settings(link_stream, &store_settings); decide_link_settings(link_stream, &store_settings);
if ((store_settings.lane_count != LANE_COUNT_UNKNOWN) && if ((store_settings.lane_count != LANE_COUNT_UNKNOWN) &&
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment