From 8375d4909aee4c18798f373ecf24a79f040f75fc Mon Sep 17 00:00:00 2001 From: Mauro Carvalho Chehab Date: Tue, 6 Jan 2009 14:43:00 -0800 Subject: edac: driver for i5400 MCH (update) Signed-off-by: Ben Woodard Signed-off-by: Mauro Carvalho Chehab Cc: Doug Thompson Signed-off-by: Andrew Morton Signed-off-by: Linus Torvalds --- drivers/edac/i5400_edac.c | 119 ++++++++++++++++++++++++---------------------- 1 file changed, 62 insertions(+), 57 deletions(-) (limited to 'drivers/edac') diff --git a/drivers/edac/i5400_edac.c b/drivers/edac/i5400_edac.c index 8ec3eca3061..b08b6d8e2dc 100644 --- a/drivers/edac/i5400_edac.c +++ b/drivers/edac/i5400_edac.c @@ -1,5 +1,5 @@ /* - * Intel 5400 class Memory Controllers kernel module + * Intel 5400 class Memory Controllers kernel module (Seaburg) * * This file may be distributed under the terms of the * GNU General Public License. @@ -55,13 +55,11 @@ * Function 1: Memory Branch Map, Control, Errors Register * Function 2: FSB Error Registers * - * All 3 functions of Device 16 (0,1,2) share the SAME DID + * All 3 functions of Device 16 (0,1,2) share the SAME DID and + * uses PCI_DEVICE_ID_INTEL_5400_ERR for device 16 (0,1,2), + * PCI_DEVICE_ID_INTEL_5400_FBD0 and PCI_DEVICE_ID_INTEL_5400_FBD1 + * for device 21 (0,1). */ -#ifndef PCI_DEVICE_ID_INTEL_5400_ERR -#define PCI_DEVICE_ID_INTEL_5400_ERR 0x4030 /* Device 16 (0,1,2) */ -#define PCI_DEVICE_ID_INTEL_5400_FBD0 0x4035 /* Device 21 (0,1) */ -#define PCI_DEVICE_ID_INTEL_5400_FBD1 0x4036 /* Device 21 (0,1) */ -#endif /* OFFSETS for Function 0 */ #define AMBASE 0x48 /* AMB Mem Mapped Reg Region Base */ @@ -174,17 +172,17 @@ enum error_mask { /* * Names to translate bit error into something useful */ -char *error_name[] = { +static const char *error_name[] = { [0] = "Memory Write error on non-redundant retry", [1] = "Memory or FB-DIMM configuration CRC read error", /* Reserved */ [3] = "Uncorrectable Data ECC on Replay", [4] = "Aliased Uncorrectable Non-Mirrored Demand Data ECC", - /* Unsupported on i5400 */ + /* M6 Unsupported on i5400 */ [6] = "Aliased Uncorrectable Resilver- or Spare-Copy Data ECC", [7] = "Aliased Uncorrectable Patrol Data ECC", [8] = "Non-Aliased Uncorrectable Non-Mirrored Demand Data ECC", - /* Unsupported */ + /* M10 Unsupported on i5400 */ [10] = "Non-Aliased Uncorrectable Resilver- or Spare-Copy Data ECC", [11] = "Non-Aliased Uncorrectable Patrol Data ECC", [12] = "Memory Write error on first attempt", @@ -192,7 +190,7 @@ char *error_name[] = { [14] = "Memory or FB-DIMM configuration CRC read error", [15] = "Channel Failed-Over Occurred", [16] = "Correctable Non-Mirrored Demand Data ECC", - /* Unsupported */ + /* M18 Unsupported on i5400 */ [18] = "Correctable Resilver- or Spare-Copy Data ECC", [19] = "Correctable Patrol Data ECC", [20] = "FB-DIMM Northbound parity error on FB-DIMM Sync Status", @@ -259,24 +257,32 @@ char *error_name[] = { #define FERR_FAT_MASK ERROR_FAT_MASK /* masks for non-fatal error register */ -#define TO_NF_MASK(a) (((a) & EMASK_M29) | ((a) >> 3)) -#define FROM_NF_FERR(a) (((a) & EMASK_M29) | (((a) << 3) & ((1 << 30)-1))) +static inline int to_nf_mask(unsigned int mask) +{ + return (mask & EMASK_M29) | (mask >> 3); +}; + +static inline int from_nf_ferr(unsigned int mask) +{ + return (mask & EMASK_M29) | /* Bit 28 */ + (mask & ((1 << 28) - 1) << 3); /* Bits 0 to 27 */ +}; -#define FERR_NF_MASK TO_NF_MASK(ERROR_NF_MASK) -#define FERR_NF_CORRECTABLE TO_NF_MASK(ERROR_NF_CORRECTABLE) -#define FERR_NF_DIMM_SPARE TO_NF_MASK(ERROR_NF_DIMM_SPARE) -#define FERR_NF_SPD_PROTOCOL TO_NF_MASK(ERROR_NF_SPD_PROTOCOL) -#define FERR_NF_NORTH_CRC TO_NF_MASK(ERROR_NF_NORTH_CRC) -#define FERR_NF_RECOVERABLE TO_NF_MASK(ERROR_NF_RECOVERABLE) -#define FERR_NF_UNCORRECTABLE TO_NF_MASK(ERROR_NF_UNCORRECTABLE) +#define FERR_NF_MASK to_nf_mask(ERROR_NF_MASK) +#define FERR_NF_CORRECTABLE to_nf_mask(ERROR_NF_CORRECTABLE) +#define FERR_NF_DIMM_SPARE to_nf_mask(ERROR_NF_DIMM_SPARE) +#define FERR_NF_SPD_PROTOCOL to_nf_mask(ERROR_NF_SPD_PROTOCOL) +#define FERR_NF_NORTH_CRC to_nf_mask(ERROR_NF_NORTH_CRC) +#define FERR_NF_RECOVERABLE to_nf_mask(ERROR_NF_RECOVERABLE) +#define FERR_NF_UNCORRECTABLE to_nf_mask(ERROR_NF_UNCORRECTABLE) /* Defines to extract the vaious fields from the * MTRx - Memory Technology Registers */ #define MTR_DIMMS_PRESENT(mtr) ((mtr) & (1 << 10)) #define MTR_DIMMS_ETHROTTLE(mtr) ((mtr) & (1 << 9)) -#define MTR_DRAM_WIDTH(mtr) (((mtr) & (1<< 8)) ? 8 : 4) -#define MTR_DRAM_BANKS(mtr) (((mtr) & (1<< 6)) ? 8 : 4) +#define MTR_DRAM_WIDTH(mtr) (((mtr) & (1 << 8)) ? 8 : 4) +#define MTR_DRAM_BANKS(mtr) (((mtr) & (1 << 6)) ? 8 : 4) #define MTR_DRAM_BANKS_ADDR_BITS(mtr) ((MTR_DRAM_BANKS(mtr) == 8) ? 3 : 2) #define MTR_DIMM_RANK(mtr) (((mtr) >> 5) & 0x1) #define MTR_DIMM_RANK_ADDR_BITS(mtr) (MTR_DIMM_RANK(mtr) ? 2 : 1) @@ -293,7 +299,7 @@ static inline int extract_fbdchan_indx(u32 x) #ifdef CONFIG_EDAC_DEBUG /* MTR NUMROW */ -static char *numrow_toString[] = { +static const char *numrow_toString[] = { "8,192 - 13 rows", "16,384 - 14 rows", "32,768 - 15 rows", @@ -301,7 +307,7 @@ static char *numrow_toString[] = { }; /* MTR NUMCOL */ -static char *numcol_toString[] = { +static const char *numcol_toString[] = { "1,024 - 10 columns", "2,048 - 11 columns", "4,096 - 12 columns", @@ -341,11 +347,11 @@ struct i5400_pvt { u16 mir0, mir1; - u16 b0_mtr[NUM_MTRS_PER_BRANCH]; /* Memory Technlogy Reg */ + u16 b0_mtr[NUM_MTRS_PER_BRANCH]; /* Memory Technlogy Reg */ u16 b0_ambpresent0; /* Branch 0, Channel 0 */ u16 b0_ambpresent1; /* Brnach 0, Channel 1 */ - u16 b1_mtr[NUM_MTRS_PER_BRANCH]; /* Memory Technlogy Reg */ + u16 b1_mtr[NUM_MTRS_PER_BRANCH]; /* Memory Technlogy Reg */ u16 b1_ambpresent0; /* Branch 1, Channel 8 */ u16 b1_ambpresent1; /* Branch 1, Channel 1 */ @@ -370,7 +376,7 @@ struct i5400_error_info { u16 recmema; /* Recoverable Mem Error log A */ u32 recmemb; /* Recoverable Mem Error log B */ - /* These registers are input ONLY if there was a Non-Recoverable Error */ + /* These registers are input ONLY if there was a Non-Rec Error */ u16 nrecmema; /* Non-Recoverable Mem log A */ u16 nrecmemb; /* Non-Recoverable Mem log B */ @@ -560,10 +566,10 @@ static void i5400_proccess_non_recoverable_info(struct mem_ctl_info *mci, /* Form out message */ snprintf(msg, sizeof(msg), - "%s (Branch=%d DRAM-Bank=%d Buffer ID = %d RDWR=%s RAS=%d CAS=%d " - "%s Err=0x%lx (%s))", - type, branch >> 1, bank, buf_id, rdwr_str(rdwr), ras, cas, type, - allErrors, error_name[errnum]); + "%s (Branch=%d DRAM-Bank=%d Buffer ID = %d RDWR=%s " + "RAS=%d CAS=%d %s Err=0x%lx (%s))", + type, branch >> 1, bank, buf_id, rdwr_str(rdwr), ras, cas, + type, allErrors, error_name[errnum]); /* Call the helper to output message */ edac_mc_handle_fbd_ue(mci, rank, channel, channel + 1, msg); @@ -590,7 +596,7 @@ static void i5400_process_nonfatal_error_info(struct mem_ctl_info *mci, int errnum; /* mask off the Error bits that are possible */ - allErrors = FROM_NF_FERR(info->ferr_nf_fbd & FERR_NF_MASK); + allErrors = from_nf_ferr(info->ferr_nf_fbd & FERR_NF_MASK); if (!allErrors) return; /* if no error, return now */ @@ -631,10 +637,10 @@ static void i5400_process_nonfatal_error_info(struct mem_ctl_info *mci, /* Form out message */ snprintf(msg, sizeof(msg), - "Corrected error (Branch=%d DRAM-Bank=%d RDWR=%s RAS=%d " - "CAS=%d, CE Err=0x%lx (%s))", branch >> 1, bank, - rdwr_str(rdwr), ras, cas, allErrors, - error_name[errnum]); + "Corrected error (Branch=%d DRAM-Bank=%d RDWR=%s " + "RAS=%d CAS=%d, CE Err=0x%lx (%s))", + branch >> 1, bank, rdwr_str(rdwr), ras, cas, + allErrors, error_name[errnum]); /* Call the helper to output message */ edac_mc_handle_fbd_ce(mci, rank, channel, msg); @@ -704,17 +710,10 @@ static void i5400_put_devices(struct mem_ctl_info *mci) pvt = mci->pvt_info; /* Decrement usage count for devices */ - if (pvt->branch_1) - pci_dev_put(pvt->branch_1); - - if (pvt->branch_0) - pci_dev_put(pvt->branch_0); - - if (pvt->fsb_error_regs) - pci_dev_put(pvt->fsb_error_regs); - - if (pvt->branchmap_werrors) - pci_dev_put(pvt->branchmap_werrors); + pci_dev_put(pvt->branch_1); + pci_dev_put(pvt->branch_0); + pci_dev_put(pvt->fsb_error_regs); + pci_dev_put(pvt->branchmap_werrors); } /* @@ -810,9 +809,9 @@ error: /* * determine_amb_present * - * the information is contained in NUM_MTRS_PER_BRANCH different registers - * determining which of the NUM_MTRS_PER_BRANCH requires knowing - * which channel is in question + * the information is contained in NUM_MTRS_PER_BRANCH different + * registers determining which of the NUM_MTRS_PER_BRANCH requires + * knowing which channel is in question * * 2 branches, each with 2 channels * b0_ambpresent0 for channel '0' @@ -842,7 +841,7 @@ static int determine_amb_present_reg(struct i5400_pvt *pvt, int channel) /* * determine_mtr(pvt, csrow, channel) * - * return the proper MTR register as determine by the csrow and channel desired + * return the proper MTR register as determine by the csrow and desired channel */ static int determine_mtr(struct i5400_pvt *pvt, int csrow, int channel) { @@ -857,7 +856,8 @@ static int determine_mtr(struct i5400_pvt *pvt, int csrow, int channel) n = csrow >> 3; if (n >= NUM_MTRS_PER_BRANCH) { - debugf0("ERROR: trying to access an invalid csrow: %d\n", csrow); + debugf0("ERROR: trying to access an invalid csrow: %d\n", + csrow); return 0; } @@ -885,7 +885,7 @@ static void decode_mtr(int slot_row, u16 mtr) debugf2("\t\tWIDTH: x%d\n", MTR_DRAM_WIDTH(mtr)); debugf2("\t\tELECTRICAL THROTTLING is %s\n", - MTR_DIMMS_ETHROTTLE(mtr) ? "enabled": "disabled"); + MTR_DIMMS_ETHROTTLE(mtr) ? "enabled" : "disabled"); debugf2("\t\tNUMBANK: %d bank(s)\n", MTR_DRAM_BANKS(mtr)); debugf2("\t\tNUMRANK: %s\n", MTR_DIMM_RANK(mtr) ? "double" : "single"); @@ -1251,6 +1251,9 @@ static int i5400_probe1(struct pci_dev *pdev, int dev_idx) int num_dimms_per_channel; int num_csrows; + if (dev_idx >= ARRAY_SIZE(i5400_devs)) + return -EINVAL; + debugf0("MC: " __FILE__ ": %s(), pdev bus %u dev=0x%x fn=0x%x\n", __func__, pdev->bus->number, @@ -1425,7 +1428,7 @@ MODULE_DEVICE_TABLE(pci, i5400_pci_tbl); * */ static struct pci_driver i5400_driver = { - .name = KBUILD_BASENAME, + .name = "i5400_edac", .probe = i5400_init_one, .remove = __devexit_p(i5400_remove_one), .id_table = i5400_pci_tbl, @@ -1463,9 +1466,11 @@ module_init(i5400_init); module_exit(i5400_exit); MODULE_LICENSE("GPL"); -MODULE_AUTHOR("Ben Woodard Red Hat Inc. (http://www.redhat.com)"); -MODULE_AUTHOR("Mauro Carvalho Chehab Red Hat Inc. (http://www.redhat.com)"); -MODULE_DESCRIPTION("MC Driver for Intel I5400 memory controllers - " I5400_REVISION); +MODULE_AUTHOR("Ben Woodard "); +MODULE_AUTHOR("Mauro Carvalho Chehab "); +MODULE_AUTHOR("Red Hat Inc. (http://www.redhat.com)"); +MODULE_DESCRIPTION("MC Driver for Intel I5400 memory controllers - " + I5400_REVISION); module_param(edac_op_state, int, 0444); MODULE_PARM_DESC(edac_op_state, "EDAC Error Reporting state: 0=Poll,1=NMI"); -- cgit v1.2.3-70-g09d2