diff options
author | Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net> | 2009-08-19 15:03:28 +0000 |
---|---|---|
committer | Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net> | 2009-08-19 15:03:28 +0000 |
commit | 664e7ad995a86663e12fc0785ffd3cdc939b9311 (patch) | |
tree | 1d7e8d9f99f0753af9aa235bb2c895c6cf09343f /flashrom.c | |
parent | 8ab49e72af8465d4527de2ec37b22cd44f7a1169 (diff) | |
download | ast2050-flashrom-664e7ad995a86663e12fc0785ffd3cdc939b9311.zip ast2050-flashrom-664e7ad995a86663e12fc0785ffd3cdc939b9311.tar.gz |
Change programmer options separator from = to :
Current programmer parameter syntax is -p programmer=parameter
Unfortunately, many parameters are of the form variable=val, so we get
commandlines like this.
flashrom -p it87spi=port=0x820 and this looks horrible.
Using : instead of = would make such parameters look better: flashrom -p
it87spi:port=0x820
As a side benefit, this patch mentions the programmer name in the error
message if it is unknown.
Corresponding to flashrom svn r693.
Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>
Acked-by: Stefan Reinauer <stepan@coresystems.de>
Diffstat (limited to 'flashrom.c')
-rw-r--r-- | flashrom.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -708,7 +708,7 @@ int main(int argc, char *argv[]) namelen = strlen(name); if (strncmp(optarg, name, namelen) == 0) { switch (optarg[namelen]) { - case '=': + case ':': programmer_param = strdup(optarg + namelen + 1); break; case '\0': @@ -725,7 +725,7 @@ int main(int argc, char *argv[]) } } if (programmer == PROGRAMMER_INVALID) { - printf("Error: Unknown programmer.\n"); + printf("Error: Unknown programmer %s.\n", optarg); exit(1); } break; |