[coreboot-gerrit] New patch to review for coreboot: f66091d cbfstool: fix --machine

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Tue Apr 2 00:04:18 CEST 2013


Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3009

-gerrit

commit f66091d5dacdda9efe2817c3d480b293b260abf2
Author: Stefan Reinauer <reinauer at chromium.org>
Date:   Thu Mar 28 16:46:07 2013 -0700

    cbfstool: fix --machine
    
    The help text says --machine, but the code
    actually checked for --arch. Fix it!
    
    Change-Id: Ib9bbf758b82ef070550348e897419513495f154b
    Signed-off-by: Stefan Reinauer <reinauer at google.com>
---
 util/cbfstool/cbfstool.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/util/cbfstool/cbfstool.c b/util/cbfstool/cbfstool.c
index 1a3d25e..bd34960 100644
--- a/util/cbfstool/cbfstool.c
+++ b/util/cbfstool/cbfstool.c
@@ -274,7 +274,7 @@ static int cbfs_create(void)
 
 	// TODO Remove arch or pack into param.
 	if (arch == CBFS_ARCHITECTURE_UNKNOWN) {
-		ERROR("You need to specify -m/--machine arch\n");
+		ERROR("You need to specify -m/--machine arch.\n");
 		return 1;
 	}
 
@@ -474,7 +474,7 @@ static struct option long_options[] = {
 	{"page-size",    required_argument, 0, 'P' },
 	{"offset",       required_argument, 0, 'o' },
 	{"file",         required_argument, 0, 'f' },
-	{"arch",         required_argument, 0, 'm' },
+	{"machine",      required_argument, 0, 'm' },
 	{"empty-fits",   required_argument, 0, 'x' },
 	{"verbose",      no_argument,       0, 'v' },
 	{"help",         no_argument,       0, 'h' },



More information about the coreboot-gerrit mailing list