summaryrefslogtreecommitdiffstats
path: root/serprog.c
diff options
context:
space:
mode:
authorCarl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>2010-07-06 09:55:48 +0000
committerCarl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>2010-07-06 09:55:48 +0000
commit744132af4b9f629716394f9c22f6add71cc73ef9 (patch)
tree46de6c5dc108fc2868603fdc606e7328a83f1703 /serprog.c
parentd1be52d545329debd2128c1aa8685b31dfaa0a0e (diff)
downloadast2050-flashrom-744132af4b9f629716394f9c22f6add71cc73ef9.zip
ast2050-flashrom-744132af4b9f629716394f9c22f6add71cc73ef9.tar.gz
Various places in the flashrom source feature custom parameter extraction from programmer_param
This led to wildly differing syntax for programmer parameters, and it also voids pretty much every assumption you could make about programmer_param. The latter is a problem for libflashrom. Use extract_param everywhere, clean up related code and make it more foolproof. Add two instances of exit(1) where we have no option to return an error. Remove six instances of exit(1) where returning an error was possible. WARNING: This changes programmer parameter syntax for a few programmers! Corresponding to flashrom svn r1070. Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net> Acked-by: Michael Karcher <flashrom@mkarcher.dialup.fu-berlin.de>
Diffstat (limited to 'serprog.c')
-rw-r--r--serprog.c101
1 files changed, 69 insertions, 32 deletions
diff --git a/serprog.c b/serprog.c
index efcf9d8..b2f5787 100644
--- a/serprog.c
+++ b/serprog.c
@@ -298,43 +298,80 @@ static int sp_stream_buffer_op(uint8_t cmd, uint32_t parmlen, uint8_t * parms)
int serprog_init(void)
{
uint16_t iface;
- int len;
unsigned char pgmname[17];
unsigned char rbuf[3];
unsigned char c;
- char *num;
- char *dev;
- msg_pspew("%s\n", __func__);
- /* the parameter is either of format "/dev/device:baud" or "ip:port" */
- if ((!programmer_param) || (!strlen(programmer_param))) {
- nodevice:
- msg_perr("Error: No device/host given for the serial programmer driver.\n"
- "Use flashrom -p serprog=/dev/device:baud or flashrom -p serprog=ip:port\n");
- exit(1);
+ char *device;
+ char *baudport;
+ int have_device = 0;
+
+ /* the parameter is either of format "dev=/dev/device:baud" or "ip=ip:port" */
+ device = extract_param(&programmer_param, "dev", ",");
+ if (device && strlen(device)) {
+ baudport = strstr(device, ":");
+ if (baudport) {
+ /* Split device from baudrate. */
+ *baudport = '\0';
+ baudport++;
+ }
+ if (!baudport || !strlen(baudport)) {
+ msg_perr("Error: No baudrate specified.\n"
+ "Use flashrom -p serprog:dev=/dev/device:baud\n");
+ free(device);
+ return 1;
+ }
+ if (strlen(device)) {
+ sp_fd = sp_openserport(device, atoi(baudport));
+ have_device++;
+ }
}
- num = strstr(programmer_param, ":");
- len = num - programmer_param;
- if (!len) goto nodevice;
- if (!num) {
- msg_perr("Error: No port or baudrate specified to serial programmer driver.\n"
- "Use flashrom -p serprog=/dev/device:baud or flashrom -p serprog=ip:port\n");
- exit(1);
+ if (device && !strlen(device)) {
+ msg_perr("Error: No device specified.\n"
+ "Use flashrom -p serprog:dev=/dev/device:baud\n");
+ free(device);
+ return 1;
+ }
+ free(device);
+
+ device = extract_param(&programmer_param, "ip", ",");
+ if (have_device && device) {
+ msg_perr("Error: Both host and device specified.\n"
+ "Please use either dev= or ip= but not both.\n");
+ free(device);
+ return 1;
+ }
+ if (device && strlen(device)) {
+ baudport = strstr(device, ":");
+ if (baudport) {
+ /* Split host from port. */
+ *baudport = '\0';
+ baudport++;
+ }
+ if (!baudport || !strlen(baudport)) {
+ msg_perr("Error: No port specified.\n"
+ "Use flashrom -p serprog:ip=ipaddr:port\n");
+ free(device);
+ return 1;
+ }
+ if (strlen(device)) {
+ sp_fd = sp_opensocket(device, atoi(baudport));
+ have_device++;
+ }
+ }
+ if (device && !strlen(device)) {
+ msg_perr("Error: No host specified.\n"
+ "Use flashrom -p serprog:ip=ipaddr:port\n");
+ free(device);
+ return 1;
+ }
+ free(device);
+
+ if (!have_device) {
+ 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");
+ return 1;
}
- len = num - programmer_param;
- dev = malloc(len + 1);
- if (!dev) sp_die("Error: memory allocation failure");
- memcpy(dev, programmer_param, len);
- dev[len] = 0;
- num = strdup(num + 1);
- if (!num) sp_die("Error: memory allocation failure");
- free(programmer_param);
- programmer_param = NULL;
-
- if (dev[0] == '/') sp_fd = sp_openserport(dev, atoi(num));
- else sp_fd = sp_opensocket(dev, atoi(num));
-
- free(dev); dev = NULL;
- free(num); num = NULL;
msg_pdbg(MSGHEADER "connected - attempting to synchronize\n");
OpenPOWER on IntegriCloud