drm/amd/display: Remove redundant initialization of variable eng_id

Colin Ian King · alexdeucher · commit ed7c28c77103 · 2021-08-06T16:12:27.000-04:00
The variable eng_id is being initialized with a value that is never
read, it is being re-assigned on the next statment. The assignment
is redundant and can be removed.

Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King &lt;colin.king@canonical.com&gt;
Signed-off-by: Alex Deucher &lt;alexander.deucher@amd.com&gt;
diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_link_enc_cfg.c b/drivers/gpu/drm/amd/display/dc/core/dc_link_enc_cfg.c
@@ -305,7 +305,7 @@ struct link_encoder *link_enc_cfg_get_next_avail_link_enc(
 	const struct dc_state *state)
 {
 	struct link_encoder *link_enc = NULL;
-	enum engine_id eng_id = ENGINE_ID_UNKNOWN;
+	enum engine_id eng_id;
 
 	eng_id = find_first_avail_link_enc(dc->ctx, state);
 	if (eng_id != ENGINE_ID_UNKNOWN)

Original file line number	Diff line number	Diff line change
`@@ -305,7 +305,7 @@ struct link_encoder *link_enc_cfg_get_next_avail_link_enc(`
`305`	`305`	`const struct dc_state *state)`
`306`	`306`	`{`
`307`	`307`	`struct link_encoder *link_enc = NULL;`
`308`		`- enum engine_id eng_id = ENGINE_ID_UNKNOWN;`
	`308`	`+ enum engine_id eng_id;`
`309`	`309`
`310`	`310`	`eng_id = find_first_avail_link_enc(dc->ctx, state);`
`311`	`311`	`if (eng_id != ENGINE_ID_UNKNOWN)`