Skip to content

Commit 5297cfa

Browse files
Sai Krishna Potthurisuryasaimadhu
authored andcommitted
EDAC/synopsys: Fix wrong value type assignment for edac_mode
dimm->edac_mode contains values of type enum edac_type - not the corresponding capability flags. Fix that. Issue caught by Coverity check "enumerated type mixed with another type." [ bp: Rewrite commit message, add tags. ] Fixes: ae9b56e ("EDAC, synps: Add EDAC support for zynq ddr ecc controller") Signed-off-by: Sai Krishna Potthuri <[email protected]> Signed-off-by: Shubhrajyoti Datta <[email protected]> Signed-off-by: Borislav Petkov <[email protected]> Cc: <[email protected]> Link: https://lkml.kernel.org/r/[email protected]
1 parent 6880fa6 commit 5297cfa

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

drivers/edac/synopsys_edac.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -782,7 +782,7 @@ static void init_csrows(struct mem_ctl_info *mci)
782782

783783
for (j = 0; j < csi->nr_channels; j++) {
784784
dimm = csi->channels[j]->dimm;
785-
dimm->edac_mode = EDAC_FLAG_SECDED;
785+
dimm->edac_mode = EDAC_SECDED;
786786
dimm->mtype = p_data->get_mtype(priv->baseaddr);
787787
dimm->nr_pages = (size >> PAGE_SHIFT) / csi->nr_channels;
788788
dimm->grain = SYNPS_EDAC_ERR_GRAIN;

0 commit comments

Comments
 (0)