github-mirror/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-09-12 04:23:03 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity Graph

EDAC/ie31200: Decouple DIMM width decoding from enum order

Browse Source 
The current method to get DIMM width relied on DEV_* enum ordering via a
linear offset (+ DEV_X8), tightly coupling hardware encoding to enum layout.

Replace it with explicit decoding to remove this dependency, as the
enum is expected to grow with additional device widths.

Signed-off-by: Qiuxu Zhuo <qiuxu.zhuo@intel.com>
Signed-off-by: Tony Luck <tony.luck@intel.com>
Link: https://patch.msgid.link/20260730024238.4096623-2-qiuxu.zhuo@intel.com
This commit is contained in:
Qiuxu Zhuo 2026-07-30 10:42:29 +08:00 committed by Tony Luck
parent 36a6518e74
commit 141556543c
1 changed files with 17 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
drivers/edac/ie31200_edac.c
View File

@ -416,7 +416,23 @@ static void populate_dimm_info(struct dimm_data *dd, u32 addr_decode, int dimm,
{
dd->size = field_get(cfg->reg_mad_dimm_size_mask[dimm], addr_decode) * cfg->reg_mad_dimm_size_granularity;
dd->ranks = field_get(cfg->reg_mad_dimm_rank_mask[dimm], addr_decode) + 1;
dd->dtype = field_get(cfg->reg_mad_dimm_width_mask[dimm], addr_decode) + DEV_X8;
switch (field_get(cfg->reg_mad_dimm_width_mask[dimm], addr_decode)) {
case 0:
dd->dtype = DEV_X8;
break;
case 1:
dd->dtype = DEV_X16;
break;
case 2:
dd->dtype = DEV_X32;
break;
case 3:
dd->dtype = DEV_X64;
break;
default:
dd->dtype = DEV_UNKNOWN;
}
}
static void ie31200_get_dimm_config(struct mem_ctl_info *mci, void __iomem *window,