9c6f6b65d2
i7300-edac: CodingStyle cleanup
...
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
2010-08-30 14:57:06 -03:00
d091a6eb17
i7300_edac: Improve comments
...
This is basically a cleanup patch, improving the comments for each
function.
While here, do a few cleanups.
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
2010-08-30 14:57:05 -03:00
b4552aceb3
i7300_edac: Cleanup: reorganize the file contents
...
This change should do no functional change. It just rearranges the
contents of the c file, in order to make easier to understand and
maintain it.
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
2010-08-30 14:57:04 -03:00
37b69cf91c
i7300_edac: Properly detect channel on CE errors
...
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
2010-08-30 14:57:03 -03:00
32f9472613
i7300_edac: enrich FBD error info for corrected errors
...
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
2010-08-30 14:57:02 -03:00
8199d8cc65
i7300_edac: enrich FBD error info for fatal errors
...
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
2010-08-30 14:57:01 -03:00
85580ea4f7
i7300_edac: pre-allocate a buffer used to prepare err messages
...
Instead of dynamically allocating a buffer for it where needed,
just allocate it once. As we'll use the same buffer also during
fatal and non-fatal errors, is is very risky to dynamically allocate
it during an error.
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
2010-08-30 14:56:59 -03:00
28c2ce7c8b
i7300_edac: Fix MTR x4/x8 detection logic
...
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
2010-08-30 14:56:58 -03:00
3b330f6758
i7300_edac: Make the debug messages coherent with the others
...
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
2010-08-30 14:56:57 -03:00
f427742248
i7300_edac: Cleanup: remove get_error_info logic
...
As the error logic in this driver came from i5400 driver, it
were using one function to get errors, and another to display.
Let's make it simpler and avoid doing it into two steps.
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
2010-08-30 14:56:56 -03:00
e432760509
i7300_edac: Add a code to cleanup error registers
...
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
2010-08-30 14:56:55 -03:00
57021918aa
i7300_edac: Add support for reporting FBD errors
...
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
2010-08-30 14:56:54 -03:00
15154c57c6
i7300_edac: Properly detect the type of error correction
...
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
2010-08-30 14:56:52 -03:00
bb81a21637
i7300_edac: Detect if the device is on single mode
...
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
2010-08-30 14:56:51 -03:00
d7de2bdb0e
i7300_edac: Adds detection for enhanced scrub mode on x8
...
While here, do some cleanup by adding some macros to check
for device features.
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
2010-08-30 14:56:50 -03:00
86002324cf
i7300_edac: Clear the error bit after reading
...
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
2010-08-30 14:56:49 -03:00
5de6e07ed7
i7300_edac: Add error detection code for global errors
...
There's no mention at the datasheet about how to enable global error
reporting. So, I'm assuming that those errors are always enabled.
Maybe I'm plain wrong about that ;)
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
2010-08-30 14:56:48 -03:00
3e57eef64c
i7300_edac: Better name PCI devices
...
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
2010-08-30 14:56:47 -03:00
116389ed21
i7300_edac: Add a FIXME note about the error correction type
...
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
2010-08-30 14:56:45 -03:00
c3af2eaf7a
i7300_edac: add global error registers
...
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
2010-08-30 14:56:44 -03:00
af3d8831e7
i7300_edac: display info if ECC is enabled or not
...
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
2010-08-30 14:56:43 -03:00
fcaf780b2a
i7300_edac: start a driver for i7300 chipset (Clarksboro)
...
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
2010-08-30 14:56:42 -03:00
Mauro Carvalho Chehab