Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 8de5c1a1 authored by Konstantin Olifer's avatar Konstantin Olifer Committed by Linus Torvalds
Browse files

edac: e752x fsb ecc 



FSB parity is only supported on the Xeon processor.  Previously it was
incorrectly enabled for the Celeron as well.

Signed-off-by: default avatarKonstantin Olifer <kolifer@gmail.com>
Signed-off-by: default avatarDoug Thompson <dougthompson@xmission.com>
Cc: Peter Tyser <ptyser@xes-inc.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 66ed3f75

drivers/edac/e752x_edac.c

+2 −8
Original line number Diff line number Diff line
@@ -1074,9 +1074,6 @@ static int e752x_get_devs(struct pci_dev *pdev, int dev_idx, 
/* Setup system bus parity mask register.

 * Sysbus parity supported on:

 * e7320/e7520/e7525 + Xeon

 *   i3100 + Xeon/Celeron

 * Sysbus parity not supported on:

 *   i3100 + Pentium M/Celeron M/Core Duo/Core2 Duo

 */

static void e752x_init_sysbus_parity_mask(struct e752x_pvt *pvt)

{
@@ -1087,10 +1084,7 @@ static void e752x_init_sysbus_parity_mask(struct e752x_pvt *pvt) 
	/* Allow module parameter override, else see if CPU supports parity */

	if (sysbus_parity != -1) {

		enable = sysbus_parity;

	} else if (cpu_id[0] &&

		   ((strstr(cpu_id, "Pentium") && strstr(cpu_id, " M ")) ||

		    (strstr(cpu_id, "Celeron") && strstr(cpu_id, " M ")) ||

		    (strstr(cpu_id, "Core") && strstr(cpu_id, "Duo")))) {

	} else if (cpu_id[0] && !strstr(cpu_id, "Xeon")) {

		e752x_printk(KERN_INFO, "System Bus Parity not "

			     "supported by CPU, disabling\n");

		enable = 0;