[coreboot] [flashrom] r563 - trunk

svn at coreboot.org svn at coreboot.org
Tue Jun 2 00:07:52 CEST 2009


Author: hailfinger
Date: 2009-06-02 00:07:52 +0200 (Tue, 02 Jun 2009)
New Revision: 563

Modified:
   trunk/flashrom.c
Log:
Use read_flash() when flash chip probe is forced.

Signed-off-by: Stephan Guilloux <stephan.guilloux at free.fr>
Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>


Modified: trunk/flashrom.c
===================================================================
--- trunk/flashrom.c	2009-06-01 21:37:00 UTC (rev 562)
+++ trunk/flashrom.c	2009-06-01 22:07:52 UTC (rev 563)
@@ -314,6 +314,11 @@
 	FILE *image;
 	unsigned long size = flash->total_size * 1024;
 	unsigned char *buf = calloc(size, sizeof(char));
+
+	if (!filename) {
+		printf("Error: No filename specified.\n");
+		return 1;
+	}
 	if ((image = fopen(filename, "w")) == NULL) {
 		perror(filename);
 		exit(1);
@@ -670,35 +675,7 @@
 				printf("Run flashrom -L to view the hardware supported in this flashrom version.\n");
 				exit(1);
 			}
-			if (!filename) {
-				printf("Error: No filename specified.\n");
-				exit(1);
-			}
-			size = flashes[0]->total_size * 1024;
-			buf = (uint8_t *) calloc(size, sizeof(char));
-
-			if ((image = fopen(filename, "w")) == NULL) {
-				perror(filename);
-				exit(1);
-			}
-			printf("Force reading flash... ");
-			if (!flashes[0]->read) {
-				printf("FAILED!\n");
-				fprintf(stderr, "ERROR: flashrom has no read function for this flash chip.\n");
-				return 1;
-			} else
-				flashes[0]->read(flashes[0], buf);
-
-			if (exclude_end_position - exclude_start_position > 0)
-				memset(buf + exclude_start_position, 0,
-				       exclude_end_position -
-				       exclude_start_position);
-
-			numbytes = fwrite(buf, 1, size, image);
-			fclose(image);
-			printf("%s.\n", numbytes == size ? "done" : "FAILED");
-			free(buf);
-			return numbytes != size;
+			return read_flash(flashes[0], filename, exclude_start_position, exclude_end_position);
 		}
 		// FIXME: flash writes stay enabled!
 		exit(1);





More information about the coreboot mailing list