diff options
author | Urja Rannikko <urjaman@gmail.com> | 2016-01-04 03:05:23 +0000 |
---|---|---|
committer | Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at> | 2016-01-04 03:05:23 +0000 |
commit | e3a41897d898322f93e7aec0f9673a91cab8335c (patch) | |
tree | cccdb3718ce2b2399efa0e011c6cca0ba41bc45e | |
parent | 6e734d8c2a479e8cc9f4ab13f7f24e1f560dc063 (diff) | |
download | flashrom-e3a41897d898322f93e7aec0f9673a91cab8335c.zip flashrom-e3a41897d898322f93e7aec0f9673a91cab8335c.tar.gz |
serprog: fix missing device parameter error message on Windows
Previously, flashrom -p serprog didnt notice a completely missing dev=
string on Windows. Also, the error message shown if no device name
followed contained a misleading "/dev/...".
Corresponding to flashrom svn r1908.
Signed-off-by: Urja Rannikko <urjaman@gmail.com>
Acked-by: Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at>
-rw-r--r-- | serprog.c | 10 |
1 files changed, 8 insertions, 2 deletions
@@ -371,6 +371,8 @@ int serprog_init(void) have_device++; } } + +#if !IS_WINDOWS if (device && !strlen(device)) { msg_perr("Error: No device specified.\n" "Use flashrom -p serprog:dev=/dev/device[:baud]\n"); @@ -379,7 +381,6 @@ int serprog_init(void) } free(device); -#if !IS_WINDOWS device = extract_programmer_param("ip"); if (have_device && device) { msg_perr("Error: Both host and device specified.\n" @@ -415,15 +416,20 @@ int serprog_init(void) free(device); return 1; } +#endif free(device); if (!have_device) { +#if IS_WINDOWS + msg_perr("Error: No device specified.\n" + "Use flashrom -p serprog:dev=comN[:baud]\n"); +#else msg_perr("Error: Neither host nor device specified.\n" "Use flashrom -p serprog:dev=/dev/device:baud or " "flashrom -p serprog:ip=ipaddr:port\n"); +#endif return 1; } -#endif if (register_shutdown(serprog_shutdown, NULL)) return 1; |