diff options
author | hailfinger <hailfinger@2b7e53f0-3cfb-0310-b3e9-8179ed1497e1> | 2012-01-04 00:48:27 +0000 |
---|---|---|
committer | hailfinger <hailfinger@2b7e53f0-3cfb-0310-b3e9-8179ed1497e1> | 2012-01-04 00:48:27 +0000 |
commit | 234fb84eb42ed6718a23b99e822ea3d726e8b995 (patch) | |
tree | 7f14a4305d587d4ff56c1b3f764a9016dba5d03b /internal.c | |
parent | cf3abbf99f950681e9601a1da17c0709179ce76e (diff) | |
download | flashrom-234fb84eb42ed6718a23b99e822ea3d726e8b995.tar.gz |
Replace --mainboard with -p internal:mainboard
--mainboard is a relic from a time before external programmers and makes
the CLI inconsistent.
Use a programmer parameter instead and free up the short option -m.
NOTE:
The --list-supported-wiki output changed to use -p internal:mainboard=
instead of -m
The --list-supported output changed the heading of the mainboard list
from
Vendor Board Status Required option
to
Vendor Board Status Required value for
-p internal:mainboard=
Fix lb_vendor_dev_from_string() not to write to the supplied string.
Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>
Acked-by: Stefan Tauner <stefan.tauner@student.tuwien.ac.at>
git-svn-id: https://code.coreboot.org/svn/flashrom/trunk@1483 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
Diffstat (limited to 'internal.c')
-rw-r--r-- | internal.c | 10 |
1 files changed, 10 insertions, 0 deletions
@@ -213,6 +213,16 @@ int internal_init(void) } free(arg); + arg = extract_programmer_param("mainboard"); + if (arg && strlen(arg)) { + lb_vendor_dev_from_string(arg); + } else if (arg && !strlen(arg)) { + msg_perr("Missing argument for mainboard.\n"); + free(arg); + return 1; + } + free(arg); + get_io_perms(); if (register_shutdown(internal_shutdown, NULL)) return 1; |