aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/edac
diff options
context:
space:
mode:
authorJason Uhlenkott <juhlenko@akamai.com>2008-02-07 03:14:53 -0500
committerLinus Torvalds <torvalds@woody.linux-foundation.org>2008-02-07 11:42:23 -0500
commitce783d70b95416725e262866f5b11436f35f68a9 (patch)
treee7a951256b83dccf37290430a9ff56521e247d7d /drivers/edac
parent48764e4143c06672fc072eb482fdc4c75ee0f968 (diff)
drivers-edac: i3000 code tidying
Style cleanup, mostly just 80-column fixes. Signed-off-by: Jason Uhlenkott <juhlenko@akamai.com> Cc: Alan Cox <alan@lxorguk.ukuu.org.uk Signed-off-by: Doug Thompson <dougthompson@xmission.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/edac')
-rw-r--r--drivers/edac/i3000_edac.c207
1 files changed, 110 insertions, 97 deletions
diff --git a/drivers/edac/i3000_edac.c b/drivers/edac/i3000_edac.c
index e895f9f887ab..3ccf3b5eecc3 100644
--- a/drivers/edac/i3000_edac.c
+++ b/drivers/edac/i3000_edac.c
@@ -30,105 +30,112 @@
30#define I3000_MCHBAR_MASK 0xffffc000 30#define I3000_MCHBAR_MASK 0xffffc000
31#define I3000_MMR_WINDOW_SIZE 16384 31#define I3000_MMR_WINDOW_SIZE 16384
32 32
33#define I3000_EDEAP 0x70 /* Extended DRAM Error Address Pointer (8b) 33#define I3000_EDEAP 0x70 /* Extended DRAM Error Address Pointer (8b)
34 * 34 *
35 * 7:1 reserved 35 * 7:1 reserved
36 * 0 bit 32 of address 36 * 0 bit 32 of address
37 */ 37 */
38#define I3000_DEAP 0x58 /* DRAM Error Address Pointer (32b) 38#define I3000_DEAP 0x58 /* DRAM Error Address Pointer (32b)
39 * 39 *
40 * 31:7 address 40 * 31:7 address
41 * 6:1 reserved 41 * 6:1 reserved
42 * 0 Error channel 0/1 42 * 0 Error channel 0/1
43 */ 43 */
44#define I3000_DEAP_GRAIN (1 << 7) 44#define I3000_DEAP_GRAIN (1 << 7)
45#define I3000_DEAP_PFN(edeap, deap) ((((edeap) & 1) << (32 - PAGE_SHIFT)) | \ 45#define I3000_DEAP_PFN(edeap, deap) ((((edeap) & 1) << (32 - PAGE_SHIFT)) \
46 ((deap) >> PAGE_SHIFT)) 46 | ((deap) >> PAGE_SHIFT))
47#define I3000_DEAP_OFFSET(deap) ((deap) & ~(I3000_DEAP_GRAIN-1) & ~PAGE_MASK) 47#define I3000_DEAP_OFFSET(deap) ((deap) & ~(I3000_DEAP_GRAIN-1) & \
48 ~PAGE_MASK)
48#define I3000_DEAP_CHANNEL(deap) ((deap) & 1) 49#define I3000_DEAP_CHANNEL(deap) ((deap) & 1)
49 50
50#define I3000_DERRSYN 0x5c /* DRAM Error Syndrome (8b) 51#define I3000_DERRSYN 0x5c /* DRAM Error Syndrome (8b)
51 * 52 *
52 * 7:0 DRAM ECC Syndrome 53 * 7:0 DRAM ECC Syndrome
53 */ 54 */
54 55
55#define I3000_ERRSTS 0xc8 /* Error Status Register (16b) 56#define I3000_ERRSTS 0xc8 /* Error Status Register (16b)
56 * 57 *
57 * 15:12 reserved 58 * 15:12 reserved
58 * 11 MCH Thermal Sensor Event for SMI/SCI/SERR 59 * 11 MCH Thermal Sensor Event
59 * 10 reserved 60 * for SMI/SCI/SERR
60 * 9 LOCK to non-DRAM Memory Flag (LCKF) 61 * 10 reserved
61 * 8 Received Refresh Timeout Flag (RRTOF) 62 * 9 LOCK to non-DRAM Memory Flag (LCKF)
62 * 7:2 reserved 63 * 8 Received Refresh Timeout Flag (RRTOF)
63 * 1 Multiple-bit DRAM ECC Error Flag (DMERR) 64 * 7:2 reserved
64 * 0 Single-bit DRAM ECC Error Flag (DSERR) 65 * 1 Multi-bit DRAM ECC Error Flag (DMERR)
65 */ 66 * 0 Single-bit DRAM ECC Error Flag (DSERR)
67 */
66#define I3000_ERRSTS_BITS 0x0b03 /* bits which indicate errors */ 68#define I3000_ERRSTS_BITS 0x0b03 /* bits which indicate errors */
67#define I3000_ERRSTS_UE 0x0002 69#define I3000_ERRSTS_UE 0x0002
68#define I3000_ERRSTS_CE 0x0001 70#define I3000_ERRSTS_CE 0x0001
69 71
70#define I3000_ERRCMD 0xca /* Error Command (16b) 72#define I3000_ERRCMD 0xca /* Error Command (16b)
71 * 73 *
72 * 15:12 reserved 74 * 15:12 reserved
73 * 11 SERR on MCH Thermal Sensor Event (TSESERR) 75 * 11 SERR on MCH Thermal Sensor Event
74 * 10 reserved 76 * (TSESERR)
75 * 9 SERR on LOCK to non-DRAM Memory (LCKERR) 77 * 10 reserved
76 * 8 SERR on DRAM Refresh Timeout (DRTOERR) 78 * 9 SERR on LOCK to non-DRAM Memory
77 * 7:2 reserved 79 * (LCKERR)
78 * 1 SERR Multiple-Bit DRAM ECC Error (DMERR) 80 * 8 SERR on DRAM Refresh Timeout
79 * 0 SERR on Single-Bit ECC Error (DSERR) 81 * (DRTOERR)
80 */ 82 * 7:2 reserved
83 * 1 SERR Multi-Bit DRAM ECC Error
84 * (DMERR)
85 * 0 SERR on Single-Bit ECC Error
86 * (DSERR)
87 */
81 88
82/* Intel MMIO register space - device 0 function 0 - MMR space */ 89/* Intel MMIO register space - device 0 function 0 - MMR space */
83 90
84#define I3000_DRB_SHIFT 25 /* 32MiB grain */ 91#define I3000_DRB_SHIFT 25 /* 32MiB grain */
85 92
86#define I3000_C0DRB 0x100 /* Channel 0 DRAM Rank Boundary (8b x 4) 93#define I3000_C0DRB 0x100 /* Channel 0 DRAM Rank Boundary (8b x 4)
87 * 94 *
88 * 7:0 Channel 0 DRAM Rank Boundary Address 95 * 7:0 Channel 0 DRAM Rank Boundary Address
89 */ 96 */
90#define I3000_C1DRB 0x180 /* Channel 1 DRAM Rank Boundary (8b x 4) 97#define I3000_C1DRB 0x180 /* Channel 1 DRAM Rank Boundary (8b x 4)
91 * 98 *
92 * 7:0 Channel 1 DRAM Rank Boundary Address 99 * 7:0 Channel 1 DRAM Rank Boundary Address
93 */ 100 */
94 101
95#define I3000_C0DRA 0x108 /* Channel 0 DRAM Rank Attribute (8b x 2) 102#define I3000_C0DRA 0x108 /* Channel 0 DRAM Rank Attribute (8b x 2)
96 * 103 *
97 * 7 reserved 104 * 7 reserved
98 * 6:4 DRAM odd Rank Attribute 105 * 6:4 DRAM odd Rank Attribute
99 * 3 reserved 106 * 3 reserved
100 * 2:0 DRAM even Rank Attribute 107 * 2:0 DRAM even Rank Attribute
101 * 108 *
102 * Each attribute defines the page 109 * Each attribute defines the page
103 * size of the corresponding rank: 110 * size of the corresponding rank:
104 * 000: unpopulated 111 * 000: unpopulated
105 * 001: reserved 112 * 001: reserved
106 * 010: 4 KB 113 * 010: 4 KB
107 * 011: 8 KB 114 * 011: 8 KB
108 * 100: 16 KB 115 * 100: 16 KB
109 * Others: reserved 116 * Others: reserved
110 */ 117 */
111#define I3000_C1DRA 0x188 /* Channel 1 DRAM Rank Attribute (8b x 2) */ 118#define I3000_C1DRA 0x188 /* Channel 1 DRAM Rank Attribute (8b x 2) */
112#define ODD_RANK_ATTRIB(dra) (((dra) & 0x70) >> 4) 119#define ODD_RANK_ATTRIB(dra) (((dra) & 0x70) >> 4)
113#define EVEN_RANK_ATTRIB(dra) ((dra) & 0x07) 120#define EVEN_RANK_ATTRIB(dra) ((dra) & 0x07)
114 121
115#define I3000_C0DRC0 0x120 /* DRAM Controller Mode 0 (32b) 122#define I3000_C0DRC0 0x120 /* DRAM Controller Mode 0 (32b)
116 * 123 *
117 * 31:30 reserved 124 * 31:30 reserved
118 * 29 Initialization Complete (IC) 125 * 29 Initialization Complete (IC)
119 * 28:11 reserved 126 * 28:11 reserved
120 * 10:8 Refresh Mode Select (RMS) 127 * 10:8 Refresh Mode Select (RMS)
121 * 7 reserved 128 * 7 reserved
122 * 6:4 Mode Select (SMS) 129 * 6:4 Mode Select (SMS)
123 * 3:2 reserved 130 * 3:2 reserved
124 * 1:0 DRAM Type (DT) 131 * 1:0 DRAM Type (DT)
125 */ 132 */
126 133
127#define I3000_C0DRC1 0x124 /* DRAM Controller Mode 1 (32b) 134#define I3000_C0DRC1 0x124 /* DRAM Controller Mode 1 (32b)
128 * 135 *
129 * 31 Enhanced Addressing Enable (ENHADE) 136 * 31 Enhanced Addressing Enable (ENHADE)
130 * 30:0 reserved 137 * 30:0 reserved
131 */ 138 */
132 139
133enum i3000p_chips { 140enum i3000p_chips {
134 I3000 = 0, 141 I3000 = 0,
@@ -187,7 +194,8 @@ static void i3000_get_error_info(struct mem_ctl_info *mci,
187 pci_read_config_byte(pdev, I3000_DERRSYN, &info->derrsyn); 194 pci_read_config_byte(pdev, I3000_DERRSYN, &info->derrsyn);
188 } 195 }
189 196
190 /* Clear any error bits. 197 /*
198 * Clear any error bits.
191 * (Yes, we really clear bits by writing 1 to them.) 199 * (Yes, we really clear bits by writing 1 to them.)
192 */ 200 */
193 pci_write_bits16(pdev, I3000_ERRSTS, I3000_ERRSTS_BITS, 201 pci_write_bits16(pdev, I3000_ERRSTS, I3000_ERRSTS_BITS,
@@ -245,7 +253,8 @@ static int i3000_is_interleaved(const unsigned char *c0dra,
245{ 253{
246 int i; 254 int i;
247 255
248 /* If the channels aren't populated identically then 256 /*
257 * If the channels aren't populated identically then
249 * we're not interleaved. 258 * we're not interleaved.
250 */ 259 */
251 for (i = 0; i < I3000_RANKS_PER_CHANNEL / 2; i++) 260 for (i = 0; i < I3000_RANKS_PER_CHANNEL / 2; i++)
@@ -254,7 +263,8 @@ static int i3000_is_interleaved(const unsigned char *c0dra,
254 EVEN_RANK_ATTRIB(c1dra[i])) 263 EVEN_RANK_ATTRIB(c1dra[i]))
255 return 0; 264 return 0;
256 265
257 /* If the rank boundaries for the two channels are different 266 /*
267 * If the rank boundaries for the two channels are different
258 * then we're not interleaved. 268 * then we're not interleaved.
259 */ 269 */
260 for (i = 0; i < I3000_RANKS_PER_CHANNEL; i++) 270 for (i = 0; i < I3000_RANKS_PER_CHANNEL; i++)
@@ -300,7 +310,8 @@ static int i3000_probe1(struct pci_dev *pdev, int dev_idx)
300 310
301 iounmap(window); 311 iounmap(window);
302 312
303 /* Figure out how many channels we have. 313 /*
314 * Figure out how many channels we have.
304 * 315 *
305 * If we have what the datasheet calls "asymmetric channels" 316 * If we have what the datasheet calls "asymmetric channels"
306 * (essentially the same as what was called "virtual single 317 * (essentially the same as what was called "virtual single
@@ -363,7 +374,8 @@ static int i3000_probe1(struct pci_dev *pdev, int dev_idx)
363 csrow->edac_mode = EDAC_UNKNOWN; 374 csrow->edac_mode = EDAC_UNKNOWN;
364 } 375 }
365 376
366 /* Clear any error bits. 377 /*
378 * Clear any error bits.
367 * (Yes, we really clear bits by writing 1 to them.) 379 * (Yes, we really clear bits by writing 1 to them.)
368 */ 380 */
369 pci_write_bits16(pdev, I3000_ERRSTS, I3000_ERRSTS_BITS, 381 pci_write_bits16(pdev, I3000_ERRSTS, I3000_ERRSTS_BITS,
@@ -390,7 +402,7 @@ static int i3000_probe1(struct pci_dev *pdev, int dev_idx)
390 debugf3("MC: %s(): success\n", __func__); 402 debugf3("MC: %s(): success\n", __func__);
391 return 0; 403 return 0;
392 404
393 fail: 405fail:
394 if (mci) 406 if (mci)
395 edac_mc_free(mci); 407 edac_mc_free(mci);
396 408
@@ -409,7 +421,7 @@ static int __devinit i3000_init_one(struct pci_dev *pdev,
409 return -EIO; 421 return -EIO;
410 422
411 rc = i3000_probe1(pdev, ent->driver_data); 423 rc = i3000_probe1(pdev, ent->driver_data);
412 if (mci_pdev == NULL) 424 if (!mci_pdev)
413 mci_pdev = pci_dev_get(pdev); 425 mci_pdev = pci_dev_get(pdev);
414 426
415 return rc; 427 return rc;
@@ -424,7 +436,8 @@ static void __devexit i3000_remove_one(struct pci_dev *pdev)
424 if (i3000_pci) 436 if (i3000_pci)
425 edac_pci_release_generic_ctl(i3000_pci); 437 edac_pci_release_generic_ctl(i3000_pci);
426 438
427 if ((mci = edac_mc_del_mc(&pdev->dev)) == NULL) 439 mci = edac_mc_del_mc(&pdev->dev);
440 if (!mci)
428 return; 441 return;
429 442
430 edac_mc_free(mci); 443 edac_mc_free(mci);
@@ -457,7 +470,7 @@ static int __init i3000_init(void)
457 if (pci_rc < 0) 470 if (pci_rc < 0)
458 goto fail0; 471 goto fail0;
459 472
460 if (mci_pdev == NULL) { 473 if (!mci_pdev) {
461 i3000_registered = 0; 474 i3000_registered = 0;
462 mci_pdev = pci_get_device(PCI_VENDOR_ID_INTEL, 475 mci_pdev = pci_get_device(PCI_VENDOR_ID_INTEL,
463 PCI_DEVICE_ID_INTEL_3000_HB, NULL); 476 PCI_DEVICE_ID_INTEL_3000_HB, NULL);