[flashrom] [commit] r1565 - trunk

[repository service]

Author: stefanct
Date: Thu Aug  9 23:09:51 2012
New Revision: 1565
URL: http://flashrom.org/trac/flashrom/changeset/1565

Log:
Simplify processor_enable.c.

Signed-off-by: Peter Lemenkov 
Acked-by: Stefan Tauner

Modified:
   trunk/processor_enable.c

Modified: trunk/processor_enable.c
==============================================================================
--- trunk/processor_enable.c	Wed Aug  8 02:14:58 2012	(r1564)
+++ trunk/processor_enable.c	Thu Aug  9 23:09:51 2012	(r1565)
@@ -24,19 +24,6 @@
 #include "flash.h"
 #include "programmer.h"

-#if defined(__i386__) || defined(__x86_64__)
-
-int processor_flash_enable(void)
-{
-	/* On x86, flash access is not processor specific except on
-	 * AMD Elan SC520, AMD Geode and maybe other SoC-style CPUs.
-	 * FIXME: Move enable_flash_cs5536 and get_flashbase_sc520 here.
-	 */
-	return 0;
-}
-
-#else
-
 #if defined (__MIPSEL__) && defined (__linux)
 #include <stdio.h>
 #include <string.h>
@@ -71,10 +58,8 @@
    	while (*ptr && isspace((unsigned char)*ptr))
    		ptr++;
    	fclose(cpuinfo);
-		return (strncmp(ptr, "ICT Loongson-2 V0.3",
-				strlen("ICT Loongson-2 V0.3")) == 0)
-		    || (strncmp(ptr, "Godson2 V0.3  FPU V0.1",
-				strlen("Godson2 V0.3  FPU V0.1")) == 0);
+		return (strncmp(ptr, "ICT Loongson-2 V0.3", strlen("ICT Loongson-2 V0.3")) == 0) ||
+		       (strncmp(ptr, "Godson2 V0.3  FPU V0.1", strlen("Godson2 V0.3  FPU V0.1")) == 0);
    }
    fclose(cpuinfo);
    return 0;
@@ -83,15 +68,21 @@

int processor_flash_enable(void)
 {
+	/* Default to 1 to catch not implemented architectures. */
+	int ret = 1;
+
    /* FIXME: detect loongson on FreeBSD and OpenBSD as well.  */
 #if defined (__MIPSEL__) && defined (__linux)
    if (is_loongson()) {
    	flashbase = 0x1fc00000;
-		return 0;
+		ret = 0;
    }
+#elif defined(__i386__) || defined(__x86_64__)
+	/* On x86, flash access is not processor specific except on
+	 * AMD Elan SC520, AMD Geode and maybe other SoC-style CPUs.
+	 * FIXME: Move enable_flash_cs5536 and get_flashbase_sc520 here.
+	 */
+	ret = 0;
 #endif
-	/* Not implemented yet. Oh well. */
-	return 1;
+	return ret;
 }
-
-#endif