[PATCH] cputable: Correct IBM/750CXe rev3.x setup

[PATCH] cputable: Correct IBM/750CXe rev3.x setup

[Nicolas DET]

[-- Attachment #1: Type: text/plain, Size: 163 bytes --]

This patch simply add an entry in the cputable to recognize correctly the IBM750CXe rev3.1. Only rev 2.x was correctly detected before.

Your sincerly
Nicolas DET

[-- Attachment #2: powerpc_750cxe_rev3.patch --]
[-- Type: application/octet-stream, Size: 684 bytes --]

--- linux-2.6.17.7.orig/arch/powerpc/kernel/cputable.c	2006-07-25 05:36:01.000000000 +0200
+++ linux-2.6.17.7_nico/arch/powerpc/kernel/cputable.c	2006-08-06 07:50:29.000000000 +0200
@@ -447,6 +447,18 @@
 		.cpu_setup		= __setup_cpu_750cx,
 		.platform		= "ppc750",
 	},
+	{	/* 750CXe rev 3.1 (0008x311) */
+		.pvr_mask		= 0xffff0fff,
+		.pvr_value		= 0x00080311,
+		.cpu_name		= "750CXe",
+		.cpu_features		= CPU_FTRS_750,
+		.cpu_user_features	= COMMON_USER,
+		.icache_bsize		= 32,
+		.dcache_bsize		= 32,
+		.num_pmcs		= 4,
+		.cpu_setup		= __setup_cpu_750cx,
+		.platform		= "ppc750",
+	},
 	{	/* 750CXe "Gekko" (83214) */
 		.pvr_mask		= 0xffffffff,
 		.pvr_value		= 0x00083214,

Re: [PATCH] cputable: Correct IBM/750CXe rev3.x setup

[Paul Mackerras]

Nicolas DET writes:

> This patch simply add an entry in the cputable to recognize
> correctly the IBM750CXe rev3.1. Only rev 2.x was correctly detected
> before.

[Patch annoyingly as an attachment...]

+	{	/* 750CXe rev 3.1 (0008x311) */
+		.pvr_mask		= 0xffff0fff,
+		.pvr_value		= 0x00080311,
+		.cpu_name		= "750CXe",

Any reason to make this so specific?  The rest of the entry is
identical to the ones we have for PVR values 0x0008221x and
0x00083214 (except that you missed out PPC_FEATURE_PPC_LE).
It looks to me like we could collapse all three into a single entry
with a suitable pvr_mask value (say 0xffff0e00?).

(akpm - consider this a nack)

Paul.