New Defects reported by Coverity Scan for coreboot

Hi,

Please find the latest report on new defect(s) introduced to coreboot found with Coverity Scan.

1 new defect(s) introduced to coreboot found with Coverity Scan.

New defect(s) Reported-by: Coverity Scan
Showing 1 of 1 defect(s)

** CID 1398603:    (CONSTANT_EXPRESSION_RESULT)
/src/soc/mediatek/mt8183/dramc_pi_basic_api.c: 124 in transfer_pll_to_spm_control()
/src/soc/mediatek/mt8183/dramc_pi_basic_api.c: 125 in transfer_pll_to_spm_control()

________________________________________________________________________________________________________
*** CID 1398603:    (CONSTANT_EXPRESSION_RESULT)
/src/soc/mediatek/mt8183/dramc_pi_basic_api.c: 124 in transfer_pll_to_spm_control()
118     	clrsetbits_le32(&mtk_spm->poweron_config_set,
119     		(0xffff << 16) | (0x1 << 0),
120     		(0xb16 << 16) | (0x1 << 0));
121     
122     	/* Set SPM pinmux */
123     	clrbits_le32(&mtk_spm->pcm_pwr_io_en, (0xff << 0) | (0xff << 16));
> > > CID 1398603:    (CONSTANT_EXPRESSION_RESULT)
> > > "((uint32_t)read32(&mtk_spm->dramc_dpy_clk_sw_con_sel) & 4294967295U /* ~((uint32_t)0) */) | 0xffffffffU" is always 0xffffffff regardless of the values of its operands. This occurs as an argument to a function call.

124     	setbits_le32(&mtk_spm->dramc_dpy_clk_sw_con_sel, 0xffffffff);
125     	setbits_le32(&mtk_spm->dramc_dpy_clk_sw_con_sel2, 0xffffffff);
126     
127     	setbits_le32(&mtk_spm->spm_power_on_val0, (0x1 << 8) | (0xf << 12));
128     	setbits_le32(&mtk_spm->spm_s1_mode_ch, 0x3 << 0);
129     
/src/soc/mediatek/mt8183/dramc_pi_basic_api.c: 125 in transfer_pll_to_spm_control()
119     		(0xffff << 16) | (0x1 << 0),
120     		(0xb16 << 16) | (0x1 << 0));
121     
122     	/* Set SPM pinmux */
123     	clrbits_le32(&mtk_spm->pcm_pwr_io_en, (0xff << 0) | (0xff << 16));
124     	setbits_le32(&mtk_spm->dramc_dpy_clk_sw_con_sel, 0xffffffff);
> > > CID 1398603:    (CONSTANT_EXPRESSION_RESULT)
> > > "((uint32_t)read32(&mtk_spm->dramc_dpy_clk_sw_con_sel2) & 4294967295U /* ~((uint32_t)0) */) | 0xffffffffU" is always 0xffffffff regardless of the values of its operands. This occurs as an argument to a function call.

125     	setbits_le32(&mtk_spm->dramc_dpy_clk_sw_con_sel2, 0xffffffff);
126     
127     	setbits_le32(&mtk_spm->spm_power_on_val0, (0x1 << 8) | (0xf << 12));
128     	setbits_le32(&mtk_spm->spm_s1_mode_ch, 0x3 << 0);
129     
130     	shu_lev = (shu_lev == 1) ? 2 : 1;

________________________________________________________________________________________________________
To view the defects in Coverity Scan visit, https://u2389337.ct.sendgrid.net/wf/click?upn=08onrYu34A-2BWcWUl-2F-2BfV0V05...