target-utils: added 10 ms delay at the end of abb_power_off() The deosmification of the ABB access code (replacement of osmo_delay_ms() bogus delays with correctly-timed ones, which are significantly shorter) had one annoying side effect: when executing the poweroff command from any of the programs, one last '=' prompt character was being sent (and received by the x86 host) as the Calypso board powers off. With delays being shorter now, the abb_power_off() function was returning and the standalone program's main loop was printing its prompt before the Iota chip fully executed the switch-off sequence! I thought about inserting an endless tight loop at the end of the abb_power_off() function, but the implemented solution of a 10 ms delay is a little nicer IMO because if the DEVOFF operation doesn't happen for some reason in a manual hacking scenario, there won't be an artificial blocker in the form of a tight loop keeping us from further poking around.

/*
 * Parsing of structured responses to AT commands
 */

#include <ctype.h>
#include <stdio.h>
#include <stdlib.h>
#include "resp_parse.h"

parse_structured_response(start, fields, max_fields)
	char *start;
	struct resp_field *fields;
	int max_fields;
{
	char *cp;
	int fieldcnt;

	cp = start;
	for (fieldcnt = 0; fieldcnt < max_fields; fieldcnt++) {
		if (*cp == '\0')
			break;
		if (isdigit(*cp)) {
			fields[fieldcnt].type = RESP_FIELD_NUMBER;
			fields[fieldcnt].num = strtoul(cp, 0, 10);
			while (isdigit(*cp))
				cp++;
		} else if (*cp == '"') {
			cp++;
			fields[fieldcnt].type = RESP_FIELD_STRING;
			fields[fieldcnt].str = cp;
			while (*cp && *cp != '"')
				cp++;
			if (*cp != '"')
				return(-1);
			*cp++ = '\0';
		} else if (*cp == ',')
			fields[fieldcnt].type = RESP_FIELD_EMPTY;
		else
			return(-1);
		if (*cp == ',')
			cp++;
		else if (*cp)
			return(-1);
	}
	return fieldcnt;
}