Commit f851b078 authored by Alex Hung's avatar Alex Hung Committed by Alex Deucher

drm/amd/display: Fix uninitialized variables in DC

This fixes 29 UNINIT issues reported by Coverity.
Reviewed-by: default avatarHersen Wu <hersenxs.wu@amd.com>
Acked-by: default avatarWayne Lin <wayne.lin@amd.com>
Signed-off-by: default avatarAlex Hung <alex.hung@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent ba3193fa
...@@ -399,7 +399,7 @@ static enum bp_result transmitter_control_v1_6( ...@@ -399,7 +399,7 @@ static enum bp_result transmitter_control_v1_6(
static void init_transmitter_control(struct bios_parser *bp) static void init_transmitter_control(struct bios_parser *bp)
{ {
uint8_t frev; uint8_t frev;
uint8_t crev; uint8_t crev = 0;
if (BIOS_CMD_TABLE_REVISION(UNIPHYTransmitterControl, if (BIOS_CMD_TABLE_REVISION(UNIPHYTransmitterControl,
frev, crev) == false) frev, crev) == false)
......
...@@ -225,7 +225,7 @@ static enum bp_result transmitter_control_fallback( ...@@ -225,7 +225,7 @@ static enum bp_result transmitter_control_fallback(
static void init_transmitter_control(struct bios_parser *bp) static void init_transmitter_control(struct bios_parser *bp)
{ {
uint8_t frev; uint8_t frev;
uint8_t crev; uint8_t crev = 0;
BIOS_CMD_TABLE_REVISION(dig1transmittercontrol, frev, crev); BIOS_CMD_TABLE_REVISION(dig1transmittercontrol, frev, crev);
......
...@@ -2372,7 +2372,7 @@ bool dcn21_validate_bandwidth_fp(struct dc *dc, struct dc_state *context, ...@@ -2372,7 +2372,7 @@ bool dcn21_validate_bandwidth_fp(struct dc *dc, struct dc_state *context,
static struct _vcs_dpi_voltage_scaling_st construct_low_pstate_lvl(struct clk_limit_table *clk_table, unsigned int high_voltage_lvl) static struct _vcs_dpi_voltage_scaling_st construct_low_pstate_lvl(struct clk_limit_table *clk_table, unsigned int high_voltage_lvl)
{ {
struct _vcs_dpi_voltage_scaling_st low_pstate_lvl; struct _vcs_dpi_voltage_scaling_st low_pstate_lvl = {0};
int i; int i;
low_pstate_lvl.state = 1; low_pstate_lvl.state = 1;
...@@ -2477,7 +2477,7 @@ void dcn201_populate_dml_writeback_from_context_fpu(struct dc *dc, ...@@ -2477,7 +2477,7 @@ void dcn201_populate_dml_writeback_from_context_fpu(struct dc *dc,
int pipe_cnt, i, j; int pipe_cnt, i, j;
double max_calc_writeback_dispclk; double max_calc_writeback_dispclk;
double writeback_dispclk; double writeback_dispclk;
struct writeback_st dout_wb; struct writeback_st dout_wb = {0};
dc_assert_fp_enabled(); dc_assert_fp_enabled();
......
...@@ -260,7 +260,7 @@ void dcn30_fpu_populate_dml_writeback_from_context( ...@@ -260,7 +260,7 @@ void dcn30_fpu_populate_dml_writeback_from_context(
int pipe_cnt, i, j; int pipe_cnt, i, j;
double max_calc_writeback_dispclk; double max_calc_writeback_dispclk;
double writeback_dispclk; double writeback_dispclk;
struct writeback_st dout_wb; struct writeback_st dout_wb = {0};
dc_assert_fp_enabled(); dc_assert_fp_enabled();
......
...@@ -723,7 +723,7 @@ static bool dcn32_enough_pipes_for_subvp(struct dc *dc, struct dc_state *context ...@@ -723,7 +723,7 @@ static bool dcn32_enough_pipes_for_subvp(struct dc *dc, struct dc_state *context
*/ */
static bool subvp_subvp_schedulable(struct dc *dc, struct dc_state *context) static bool subvp_subvp_schedulable(struct dc *dc, struct dc_state *context)
{ {
struct pipe_ctx *subvp_pipes[2]; struct pipe_ctx *subvp_pipes[2] = {0};
struct dc_stream_state *phantom = NULL; struct dc_stream_state *phantom = NULL;
uint32_t microschedule_lines = 0; uint32_t microschedule_lines = 0;
uint32_t index = 0; uint32_t index = 0;
......
...@@ -1973,8 +1973,8 @@ void dml32_CalculateVMRowAndSwath( ...@@ -1973,8 +1973,8 @@ void dml32_CalculateVMRowAndSwath(
unsigned int PTEBufferSizeInRequestsForChroma[DC__NUM_DPP__MAX]; unsigned int PTEBufferSizeInRequestsForChroma[DC__NUM_DPP__MAX];
unsigned int PDEAndMetaPTEBytesFrameY; unsigned int PDEAndMetaPTEBytesFrameY;
unsigned int PDEAndMetaPTEBytesFrameC; unsigned int PDEAndMetaPTEBytesFrameC;
unsigned int MetaRowByteY[DC__NUM_DPP__MAX]; unsigned int MetaRowByteY[DC__NUM_DPP__MAX] = {0};
unsigned int MetaRowByteC[DC__NUM_DPP__MAX]; unsigned int MetaRowByteC[DC__NUM_DPP__MAX] = {0};
unsigned int PixelPTEBytesPerRowY[DC__NUM_DPP__MAX]; unsigned int PixelPTEBytesPerRowY[DC__NUM_DPP__MAX];
unsigned int PixelPTEBytesPerRowC[DC__NUM_DPP__MAX]; unsigned int PixelPTEBytesPerRowC[DC__NUM_DPP__MAX];
unsigned int PixelPTEBytesPerRowY_one_row_per_frame[DC__NUM_DPP__MAX]; unsigned int PixelPTEBytesPerRowY_one_row_per_frame[DC__NUM_DPP__MAX];
......
...@@ -250,8 +250,8 @@ void dml2_init_soc_states(struct dml2_context *dml2, const struct dc *in_dc, ...@@ -250,8 +250,8 @@ void dml2_init_soc_states(struct dml2_context *dml2, const struct dc *in_dc,
{ {
struct dml2_policy_build_synthetic_soc_states_scratch *s = &dml2->v20.scratch.create_scratch.build_synthetic_socbb_scratch; struct dml2_policy_build_synthetic_soc_states_scratch *s = &dml2->v20.scratch.create_scratch.build_synthetic_socbb_scratch;
struct dml2_policy_build_synthetic_soc_states_params *p = &dml2->v20.scratch.build_synthetic_socbb_params; struct dml2_policy_build_synthetic_soc_states_params *p = &dml2->v20.scratch.build_synthetic_socbb_params;
unsigned int dcfclk_stas_mhz[NUM_DCFCLK_STAS]; unsigned int dcfclk_stas_mhz[NUM_DCFCLK_STAS] = {0};
unsigned int dcfclk_stas_mhz_new[NUM_DCFCLK_STAS_NEW]; unsigned int dcfclk_stas_mhz_new[NUM_DCFCLK_STAS_NEW] = {0};
unsigned int dml_project = dml2->v20.dml_core_ctx.project; unsigned int dml_project = dml2->v20.dml_core_ctx.project;
unsigned int i = 0; unsigned int i = 0;
......
...@@ -403,7 +403,7 @@ void dcn20_init_blank( ...@@ -403,7 +403,7 @@ void dcn20_init_blank(
struct output_pixel_processor *opp = NULL; struct output_pixel_processor *opp = NULL;
struct output_pixel_processor *bottom_opp = NULL; struct output_pixel_processor *bottom_opp = NULL;
uint32_t num_opps, opp_id_src0, opp_id_src1; uint32_t num_opps, opp_id_src0, opp_id_src1;
uint32_t otg_active_width, otg_active_height; uint32_t otg_active_width = 0, otg_active_height = 0;
/* program opp dpg blank color */ /* program opp dpg blank color */
color_space = COLOR_SPACE_SRGB; color_space = COLOR_SPACE_SRGB;
......
...@@ -82,7 +82,7 @@ static void update_dsc_on_stream(struct pipe_ctx *pipe_ctx, bool enable) ...@@ -82,7 +82,7 @@ static void update_dsc_on_stream(struct pipe_ctx *pipe_ctx, bool enable)
if (enable) { if (enable) {
struct dsc_config dsc_cfg; struct dsc_config dsc_cfg;
struct dsc_optc_config dsc_optc_cfg; struct dsc_optc_config dsc_optc_cfg = {0};
enum optc_dsc_mode optc_dsc_mode; enum optc_dsc_mode optc_dsc_mode;
/* Enable DSC hw block */ /* Enable DSC hw block */
......
...@@ -989,7 +989,7 @@ static void update_dsc_on_stream(struct pipe_ctx *pipe_ctx, bool enable) ...@@ -989,7 +989,7 @@ static void update_dsc_on_stream(struct pipe_ctx *pipe_ctx, bool enable)
if (enable) { if (enable) {
struct dsc_config dsc_cfg; struct dsc_config dsc_cfg;
struct dsc_optc_config dsc_optc_cfg; struct dsc_optc_config dsc_optc_cfg = {0};
enum optc_dsc_mode optc_dsc_mode; enum optc_dsc_mode optc_dsc_mode;
/* Enable DSC hw block */ /* Enable DSC hw block */
...@@ -1542,7 +1542,7 @@ void dcn32_init_blank( ...@@ -1542,7 +1542,7 @@ void dcn32_init_blank(
struct output_pixel_processor *opp = NULL; struct output_pixel_processor *opp = NULL;
struct output_pixel_processor *bottom_opp = NULL; struct output_pixel_processor *bottom_opp = NULL;
uint32_t num_opps, opp_id_src0, opp_id_src1; uint32_t num_opps, opp_id_src0, opp_id_src1;
uint32_t otg_active_width, otg_active_height; uint32_t otg_active_width = 0, otg_active_height = 0;
uint32_t i; uint32_t i;
/* program opp dpg blank color */ /* program opp dpg blank color */
......
...@@ -373,7 +373,7 @@ static void update_dsc_on_stream(struct pipe_ctx *pipe_ctx, bool enable) ...@@ -373,7 +373,7 @@ static void update_dsc_on_stream(struct pipe_ctx *pipe_ctx, bool enable)
if (enable) { if (enable) {
struct dsc_config dsc_cfg; struct dsc_config dsc_cfg;
struct dsc_optc_config dsc_optc_cfg; struct dsc_optc_config dsc_optc_cfg = {0};
enum optc_dsc_mode optc_dsc_mode; enum optc_dsc_mode optc_dsc_mode;
/* Enable DSC hw block */ /* Enable DSC hw block */
......
...@@ -516,8 +516,8 @@ static void query_hdcp_capability(enum signal_type signal, struct dc_link *link) ...@@ -516,8 +516,8 @@ static void query_hdcp_capability(enum signal_type signal, struct dc_link *link)
static void read_current_link_settings_on_detect(struct dc_link *link) static void read_current_link_settings_on_detect(struct dc_link *link)
{ {
union lane_count_set lane_count_set = {0}; union lane_count_set lane_count_set = {0};
uint8_t link_bw_set; uint8_t link_bw_set = 0;
uint8_t link_rate_set; uint8_t link_rate_set = 0;
uint32_t read_dpcd_retry_cnt = 10; uint32_t read_dpcd_retry_cnt = 10;
enum dc_status status = DC_ERROR_UNEXPECTED; enum dc_status status = DC_ERROR_UNEXPECTED;
int i; int i;
......
...@@ -1071,7 +1071,7 @@ enum dc_status dpcd_set_link_settings( ...@@ -1071,7 +1071,7 @@ enum dc_status dpcd_set_link_settings(
* MUX chip gets link rate set back before link training. * MUX chip gets link rate set back before link training.
*/ */
if (link->connector_signal == SIGNAL_TYPE_EDP) { if (link->connector_signal == SIGNAL_TYPE_EDP) {
uint8_t supported_link_rates[16]; uint8_t supported_link_rates[16] = {0};
core_link_read_dpcd(link, DP_SUPPORTED_LINK_RATES, core_link_read_dpcd(link, DP_SUPPORTED_LINK_RATES,
supported_link_rates, sizeof(supported_link_rates)); supported_link_rates, sizeof(supported_link_rates));
......
...@@ -1143,7 +1143,7 @@ static bool dcn303_resource_construct( ...@@ -1143,7 +1143,7 @@ static bool dcn303_resource_construct(
int i; int i;
struct dc_context *ctx = dc->ctx; struct dc_context *ctx = dc->ctx;
struct irq_service_init_data init_data; struct irq_service_init_data init_data;
struct ddc_service_init_data ddc_init_data; struct ddc_service_init_data ddc_init_data = {0};
ctx->dc_bios->regs = &bios_regs; ctx->dc_bios->regs = &bios_regs;
......
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