| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 1 | /* | 
|  | 2 | * Intel 82860 Memory Controller kernel module | 
|  | 3 | * (C) 2005 Red Hat (http://www.redhat.com) | 
|  | 4 | * This file may be distributed under the terms of the | 
|  | 5 | * GNU General Public License. | 
|  | 6 | * | 
|  | 7 | * Written by Ben Woodard <woodard@redhat.com> | 
|  | 8 | * shamelessly copied from and based upon the edac_i82875 driver | 
|  | 9 | * by Thayne Harbaugh of Linux Networx. (http://lnxi.com) | 
|  | 10 | */ | 
|  | 11 |  | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 12 | #include <linux/module.h> | 
|  | 13 | #include <linux/init.h> | 
|  | 14 | #include <linux/pci.h> | 
|  | 15 | #include <linux/pci_ids.h> | 
|  | 16 | #include <linux/slab.h> | 
|  | 17 | #include "edac_mc.h" | 
|  | 18 |  | 
| Doug Thompson | 929a40e | 2006-07-01 04:35:45 -0700 | [diff] [blame] | 19 | #define  I82860_REVISION " Ver: 2.0.1 " __DATE__ | 
|  | 20 | #define EDAC_MOD_STR	"i82860_edac" | 
| Doug Thompson | 37f0458 | 2006-06-30 01:56:07 -0700 | [diff] [blame] | 21 |  | 
| Dave Peterson | 537fba2 | 2006-03-26 01:38:40 -0800 | [diff] [blame] | 22 | #define i82860_printk(level, fmt, arg...) \ | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 23 | edac_printk(level, "i82860", fmt, ##arg) | 
| Dave Peterson | 537fba2 | 2006-03-26 01:38:40 -0800 | [diff] [blame] | 24 |  | 
|  | 25 | #define i82860_mc_printk(mci, level, fmt, arg...) \ | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 26 | edac_mc_chipset_printk(mci, level, "i82860", fmt, ##arg) | 
| Dave Peterson | 537fba2 | 2006-03-26 01:38:40 -0800 | [diff] [blame] | 27 |  | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 28 | #ifndef PCI_DEVICE_ID_INTEL_82860_0 | 
|  | 29 | #define PCI_DEVICE_ID_INTEL_82860_0	0x2531 | 
|  | 30 | #endif				/* PCI_DEVICE_ID_INTEL_82860_0 */ | 
|  | 31 |  | 
|  | 32 | #define I82860_MCHCFG 0x50 | 
|  | 33 | #define I82860_GBA 0x60 | 
|  | 34 | #define I82860_GBA_MASK 0x7FF | 
|  | 35 | #define I82860_GBA_SHIFT 24 | 
|  | 36 | #define I82860_ERRSTS 0xC8 | 
|  | 37 | #define I82860_EAP 0xE4 | 
|  | 38 | #define I82860_DERRCTL_STS 0xE2 | 
|  | 39 |  | 
|  | 40 | enum i82860_chips { | 
|  | 41 | I82860 = 0, | 
|  | 42 | }; | 
|  | 43 |  | 
|  | 44 | struct i82860_dev_info { | 
|  | 45 | const char *ctl_name; | 
|  | 46 | }; | 
|  | 47 |  | 
|  | 48 | struct i82860_error_info { | 
|  | 49 | u16 errsts; | 
|  | 50 | u32 eap; | 
|  | 51 | u16 derrsyn; | 
|  | 52 | u16 errsts2; | 
|  | 53 | }; | 
|  | 54 |  | 
|  | 55 | static const struct i82860_dev_info i82860_devs[] = { | 
|  | 56 | [I82860] = { | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 57 | .ctl_name = "i82860" | 
|  | 58 | }, | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 59 | }; | 
|  | 60 |  | 
|  | 61 | static struct pci_dev *mci_pdev = NULL;	/* init dev: in case that AGP code | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 62 | * has already registered driver | 
|  | 63 | */ | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 64 |  | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 65 | static void i82860_get_error_info(struct mem_ctl_info *mci, | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 66 | struct i82860_error_info *info) | 
|  | 67 | { | 
| Doug Thompson | 37f0458 | 2006-06-30 01:56:07 -0700 | [diff] [blame] | 68 | struct pci_dev *pdev; | 
|  | 69 |  | 
|  | 70 | pdev = to_pci_dev(mci->dev); | 
|  | 71 |  | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 72 | /* | 
|  | 73 | * This is a mess because there is no atomic way to read all the | 
|  | 74 | * registers at once and the registers can transition from CE being | 
|  | 75 | * overwritten by UE. | 
|  | 76 | */ | 
| Doug Thompson | 37f0458 | 2006-06-30 01:56:07 -0700 | [diff] [blame] | 77 | pci_read_config_word(pdev, I82860_ERRSTS, &info->errsts); | 
|  | 78 | pci_read_config_dword(pdev, I82860_EAP, &info->eap); | 
|  | 79 | pci_read_config_word(pdev, I82860_DERRCTL_STS, &info->derrsyn); | 
|  | 80 | pci_read_config_word(pdev, I82860_ERRSTS, &info->errsts2); | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 81 |  | 
| Doug Thompson | 37f0458 | 2006-06-30 01:56:07 -0700 | [diff] [blame] | 82 | pci_write_bits16(pdev, I82860_ERRSTS, 0x0003, 0x0003); | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 83 |  | 
|  | 84 | /* | 
|  | 85 | * If the error is the same for both reads then the first set of reads | 
|  | 86 | * is valid.  If there is a change then there is a CE no info and the | 
|  | 87 | * second set of reads is valid and should be UE info. | 
|  | 88 | */ | 
|  | 89 | if (!(info->errsts2 & 0x0003)) | 
|  | 90 | return; | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 91 |  | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 92 | if ((info->errsts ^ info->errsts2) & 0x0003) { | 
| Doug Thompson | 37f0458 | 2006-06-30 01:56:07 -0700 | [diff] [blame] | 93 | pci_read_config_dword(pdev, I82860_EAP, &info->eap); | 
|  | 94 | pci_read_config_word(pdev, I82860_DERRCTL_STS, | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 95 | &info->derrsyn); | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 96 | } | 
|  | 97 | } | 
|  | 98 |  | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 99 | static int i82860_process_error_info(struct mem_ctl_info *mci, | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 100 | struct i82860_error_info *info, int handle_errors) | 
|  | 101 | { | 
|  | 102 | int row; | 
|  | 103 |  | 
|  | 104 | if (!(info->errsts2 & 0x0003)) | 
|  | 105 | return 0; | 
|  | 106 |  | 
|  | 107 | if (!handle_errors) | 
|  | 108 | return 1; | 
|  | 109 |  | 
|  | 110 | if ((info->errsts ^ info->errsts2) & 0x0003) { | 
|  | 111 | edac_mc_handle_ce_no_info(mci, "UE overwrote CE"); | 
|  | 112 | info->errsts = info->errsts2; | 
|  | 113 | } | 
|  | 114 |  | 
|  | 115 | info->eap >>= PAGE_SHIFT; | 
|  | 116 | row = edac_mc_find_csrow_by_page(mci, info->eap); | 
|  | 117 |  | 
|  | 118 | if (info->errsts & 0x0002) | 
|  | 119 | edac_mc_handle_ue(mci, info->eap, 0, row, "i82860 UE"); | 
|  | 120 | else | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 121 | edac_mc_handle_ce(mci, info->eap, 0, info->derrsyn, row, 0, | 
|  | 122 | "i82860 UE"); | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 123 |  | 
|  | 124 | return 1; | 
|  | 125 | } | 
|  | 126 |  | 
|  | 127 | static void i82860_check(struct mem_ctl_info *mci) | 
|  | 128 | { | 
|  | 129 | struct i82860_error_info info; | 
|  | 130 |  | 
| Dave Peterson | 537fba2 | 2006-03-26 01:38:40 -0800 | [diff] [blame] | 131 | debugf1("MC%d: %s()\n", mci->mc_idx, __func__); | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 132 | i82860_get_error_info(mci, &info); | 
|  | 133 | i82860_process_error_info(mci, &info, 1); | 
|  | 134 | } | 
|  | 135 |  | 
| Doug Thompson | 1318952 | 2006-06-30 01:56:08 -0700 | [diff] [blame] | 136 | static void i82860_init_csrows(struct mem_ctl_info *mci, struct pci_dev *pdev) | 
|  | 137 | { | 
|  | 138 | unsigned long last_cumul_size; | 
|  | 139 | u16 mchcfg_ddim;  /* DRAM Data Integrity Mode 0=none, 2=edac */ | 
|  | 140 | u16 value; | 
|  | 141 | u32 cumul_size; | 
|  | 142 | struct csrow_info *csrow; | 
|  | 143 | int index; | 
|  | 144 |  | 
|  | 145 | pci_read_config_word(pdev, I82860_MCHCFG, &mchcfg_ddim); | 
|  | 146 | mchcfg_ddim = mchcfg_ddim & 0x180; | 
|  | 147 | last_cumul_size = 0; | 
|  | 148 |  | 
|  | 149 | /* The group row boundary (GRA) reg values are boundary address | 
|  | 150 | * for each DRAM row with a granularity of 16MB.  GRA regs are | 
|  | 151 | * cumulative; therefore GRA15 will contain the total memory contained | 
|  | 152 | * in all eight rows. | 
|  | 153 | */ | 
|  | 154 | for (index = 0; index < mci->nr_csrows; index++) { | 
|  | 155 | csrow = &mci->csrows[index]; | 
|  | 156 | pci_read_config_word(pdev, I82860_GBA + index * 2, &value); | 
|  | 157 | cumul_size = (value & I82860_GBA_MASK) << | 
|  | 158 | (I82860_GBA_SHIFT - PAGE_SHIFT); | 
|  | 159 | debugf3("%s(): (%d) cumul_size 0x%x\n", __func__, index, | 
|  | 160 | cumul_size); | 
|  | 161 |  | 
|  | 162 | if (cumul_size == last_cumul_size) | 
|  | 163 | continue;	/* not populated */ | 
|  | 164 |  | 
|  | 165 | csrow->first_page = last_cumul_size; | 
|  | 166 | csrow->last_page = cumul_size - 1; | 
|  | 167 | csrow->nr_pages = cumul_size - last_cumul_size; | 
|  | 168 | last_cumul_size = cumul_size; | 
|  | 169 | csrow->grain = 1 << 12;	/* I82860_EAP has 4KiB reolution */ | 
|  | 170 | csrow->mtype = MEM_RMBS; | 
|  | 171 | csrow->dtype = DEV_UNKNOWN; | 
|  | 172 | csrow->edac_mode = mchcfg_ddim ? EDAC_SECDED : EDAC_NONE; | 
|  | 173 | } | 
|  | 174 | } | 
|  | 175 |  | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 176 | static int i82860_probe1(struct pci_dev *pdev, int dev_idx) | 
|  | 177 | { | 
| Doug Thompson | 1318952 | 2006-06-30 01:56:08 -0700 | [diff] [blame] | 178 | struct mem_ctl_info *mci; | 
| Dave Peterson | 749ede5 | 2006-03-26 01:38:45 -0800 | [diff] [blame] | 179 | struct i82860_error_info discard; | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 180 |  | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 181 | /* RDRAM has channels but these don't map onto the abstractions that | 
|  | 182 | edac uses. | 
|  | 183 | The device groups from the GRA registers seem to map reasonably | 
|  | 184 | well onto the notion of a chip select row. | 
|  | 185 | There are 16 GRA registers and since the name is associated with | 
|  | 186 | the channel and the GRA registers map to physical devices so we are | 
|  | 187 | going to make 1 channel for group. | 
|  | 188 | */ | 
|  | 189 | mci = edac_mc_alloc(0, 16, 1); | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 190 |  | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 191 | if (!mci) | 
|  | 192 | return -ENOMEM; | 
|  | 193 |  | 
| Dave Peterson | 537fba2 | 2006-03-26 01:38:40 -0800 | [diff] [blame] | 194 | debugf3("%s(): init mci\n", __func__); | 
| Doug Thompson | 37f0458 | 2006-06-30 01:56:07 -0700 | [diff] [blame] | 195 | mci->dev = &pdev->dev; | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 196 | mci->mtype_cap = MEM_FLAG_DDR; | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 197 | mci->edac_ctl_cap = EDAC_FLAG_NONE | EDAC_FLAG_SECDED; | 
|  | 198 | /* I"m not sure about this but I think that all RDRAM is SECDED */ | 
|  | 199 | mci->edac_cap = EDAC_FLAG_SECDED; | 
| Dave Peterson | 680cbbb | 2006-03-26 01:38:41 -0800 | [diff] [blame] | 200 | mci->mod_name = EDAC_MOD_STR; | 
| Doug Thompson | 37f0458 | 2006-06-30 01:56:07 -0700 | [diff] [blame] | 201 | mci->mod_ver = I82860_REVISION; | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 202 | mci->ctl_name = i82860_devs[dev_idx].ctl_name; | 
|  | 203 | mci->edac_check = i82860_check; | 
|  | 204 | mci->ctl_page_to_phys = NULL; | 
| Doug Thompson | 1318952 | 2006-06-30 01:56:08 -0700 | [diff] [blame] | 205 | i82860_init_csrows(mci, pdev); | 
| Dave Peterson | 749ede5 | 2006-03-26 01:38:45 -0800 | [diff] [blame] | 206 | i82860_get_error_info(mci, &discard);  /* clear counters */ | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 207 |  | 
| Doug Thompson | 2d7bbb9 | 2006-06-30 01:56:08 -0700 | [diff] [blame] | 208 | /* Here we assume that we will never see multiple instances of this | 
|  | 209 | * type of memory controller.  The ID is therefore hardcoded to 0. | 
|  | 210 | */ | 
|  | 211 | if (edac_mc_add_mc(mci,0)) { | 
| Dave Peterson | 537fba2 | 2006-03-26 01:38:40 -0800 | [diff] [blame] | 212 | debugf3("%s(): failed edac_mc_add_mc()\n", __func__); | 
| Doug Thompson | 1318952 | 2006-06-30 01:56:08 -0700 | [diff] [blame] | 213 | goto fail; | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 214 | } | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 215 |  | 
| Doug Thompson | 1318952 | 2006-06-30 01:56:08 -0700 | [diff] [blame] | 216 | /* get this far and it's successful */ | 
|  | 217 | debugf3("%s(): success\n", __func__); | 
|  | 218 |  | 
|  | 219 | return 0; | 
|  | 220 |  | 
|  | 221 | fail: | 
|  | 222 | edac_mc_free(mci); | 
|  | 223 | return -ENODEV; | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 224 | } | 
|  | 225 |  | 
|  | 226 | /* returns count (>= 0), or negative on error */ | 
|  | 227 | static int __devinit i82860_init_one(struct pci_dev *pdev, | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 228 | const struct pci_device_id *ent) | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 229 | { | 
|  | 230 | int rc; | 
|  | 231 |  | 
| Dave Peterson | 537fba2 | 2006-03-26 01:38:40 -0800 | [diff] [blame] | 232 | debugf0("%s()\n", __func__); | 
| Dave Peterson | 537fba2 | 2006-03-26 01:38:40 -0800 | [diff] [blame] | 233 | i82860_printk(KERN_INFO, "i82860 init one\n"); | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 234 |  | 
|  | 235 | if (pci_enable_device(pdev) < 0) | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 236 | return -EIO; | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 237 |  | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 238 | rc = i82860_probe1(pdev, ent->driver_data); | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 239 |  | 
|  | 240 | if (rc == 0) | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 241 | mci_pdev = pci_dev_get(pdev); | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 242 |  | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 243 | return rc; | 
|  | 244 | } | 
|  | 245 |  | 
|  | 246 | static void __devexit i82860_remove_one(struct pci_dev *pdev) | 
|  | 247 | { | 
|  | 248 | struct mem_ctl_info *mci; | 
|  | 249 |  | 
| Dave Peterson | 537fba2 | 2006-03-26 01:38:40 -0800 | [diff] [blame] | 250 | debugf0("%s()\n", __func__); | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 251 |  | 
| Doug Thompson | 37f0458 | 2006-06-30 01:56:07 -0700 | [diff] [blame] | 252 | if ((mci = edac_mc_del_mc(&pdev->dev)) == NULL) | 
| Dave Peterson | 18dbc33 | 2006-03-26 01:38:50 -0800 | [diff] [blame] | 253 | return; | 
|  | 254 |  | 
|  | 255 | edac_mc_free(mci); | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 256 | } | 
|  | 257 |  | 
|  | 258 | static const struct pci_device_id i82860_pci_tbl[] __devinitdata = { | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 259 | { | 
|  | 260 | PCI_VEND_DEV(INTEL, 82860_0), PCI_ANY_ID, PCI_ANY_ID, 0, 0, | 
|  | 261 | I82860 | 
|  | 262 | }, | 
|  | 263 | { | 
|  | 264 | 0, | 
|  | 265 | }	/* 0 terminated list. */ | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 266 | }; | 
|  | 267 |  | 
|  | 268 | MODULE_DEVICE_TABLE(pci, i82860_pci_tbl); | 
|  | 269 |  | 
|  | 270 | static struct pci_driver i82860_driver = { | 
| Dave Peterson | 680cbbb | 2006-03-26 01:38:41 -0800 | [diff] [blame] | 271 | .name = EDAC_MOD_STR, | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 272 | .probe = i82860_init_one, | 
|  | 273 | .remove = __devexit_p(i82860_remove_one), | 
|  | 274 | .id_table = i82860_pci_tbl, | 
|  | 275 | }; | 
|  | 276 |  | 
| Alan Cox | da9bb1d | 2006-01-18 17:44:13 -0800 | [diff] [blame] | 277 | static int __init i82860_init(void) | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 278 | { | 
|  | 279 | int pci_rc; | 
|  | 280 |  | 
| Dave Peterson | 537fba2 | 2006-03-26 01:38:40 -0800 | [diff] [blame] | 281 | debugf3("%s()\n", __func__); | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 282 |  | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 283 | if ((pci_rc = pci_register_driver(&i82860_driver)) < 0) | 
| Dave Peterson | e8a491b | 2006-03-26 01:38:43 -0800 | [diff] [blame] | 284 | goto fail0; | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 285 |  | 
|  | 286 | if (!mci_pdev) { | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 287 | mci_pdev = pci_get_device(PCI_VENDOR_ID_INTEL, | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 288 | PCI_DEVICE_ID_INTEL_82860_0, NULL); | 
|  | 289 |  | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 290 | if (mci_pdev == NULL) { | 
|  | 291 | debugf0("860 pci_get_device fail\n"); | 
| Dave Peterson | e8a491b | 2006-03-26 01:38:43 -0800 | [diff] [blame] | 292 | pci_rc = -ENODEV; | 
|  | 293 | goto fail1; | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 294 | } | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 295 |  | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 296 | pci_rc = i82860_init_one(mci_pdev, i82860_pci_tbl); | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 297 |  | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 298 | if (pci_rc < 0) { | 
|  | 299 | debugf0("860 init fail\n"); | 
| Dave Peterson | e8a491b | 2006-03-26 01:38:43 -0800 | [diff] [blame] | 300 | pci_rc = -ENODEV; | 
|  | 301 | goto fail1; | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 302 | } | 
|  | 303 | } | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 304 |  | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 305 | return 0; | 
| Dave Peterson | e8a491b | 2006-03-26 01:38:43 -0800 | [diff] [blame] | 306 |  | 
|  | 307 | fail1: | 
|  | 308 | pci_unregister_driver(&i82860_driver); | 
|  | 309 |  | 
|  | 310 | fail0: | 
|  | 311 | if (mci_pdev != NULL) | 
|  | 312 | pci_dev_put(mci_pdev); | 
|  | 313 |  | 
|  | 314 | return pci_rc; | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 315 | } | 
|  | 316 |  | 
|  | 317 | static void __exit i82860_exit(void) | 
|  | 318 | { | 
| Dave Peterson | 537fba2 | 2006-03-26 01:38:40 -0800 | [diff] [blame] | 319 | debugf3("%s()\n", __func__); | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 320 |  | 
|  | 321 | pci_unregister_driver(&i82860_driver); | 
| Dave Peterson | e8a491b | 2006-03-26 01:38:43 -0800 | [diff] [blame] | 322 |  | 
|  | 323 | if (mci_pdev != NULL) | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 324 | pci_dev_put(mci_pdev); | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 325 | } | 
|  | 326 |  | 
|  | 327 | module_init(i82860_init); | 
|  | 328 | module_exit(i82860_exit); | 
|  | 329 |  | 
|  | 330 | MODULE_LICENSE("GPL"); | 
| Dave Peterson | e7ecd89 | 2006-03-26 01:38:52 -0800 | [diff] [blame] | 331 | MODULE_AUTHOR("Red Hat Inc. (http://www.redhat.com) " | 
|  | 332 | "Ben Woodard <woodard@redhat.com>"); | 
| Alan Cox | 0d88a10 | 2006-01-18 17:44:10 -0800 | [diff] [blame] | 333 | MODULE_DESCRIPTION("ECC support for Intel 82860 memory hub controllers"); |