<div dir="ltr">I've addressed all of the code review comments.  Will send another patch set with debug logs.<div><br></div><div><div class="gmail_extra"><div>The %STATCM=1 command is mandatory because that is used to signal when the modem has established a connection to the network so we want to stop here if we cannot enable STATCM.</div>
<div><br></div><div class="gmail_quote">On Tue, Nov 19, 2013 at 12:57 AM, Aleksander Morgado <span dir="ltr"><<a href="mailto:aleksander@lanedo.com" target="_blank">aleksander@lanedo.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br><div><div class="h5">> +static const MMBaseModemAtCommand unsolicited_events_enable_sequence[] = {<br>
> +  { "%STATCM=1", 10, FALSE, response_processor_no_result_stop_on_error },<br>
> +  { "%NOTIFYEV=\"SIMREFRESH\",1", 10, FALSE, NULL },<br>
> +  { NULL }<br>
> +};<br>
<br>
</div></div>If it is not mandatory; I would avoid 'stopping on error' in the AT<br>
sequence (i.e. just set a NULL callback). What do you think?<br>
<div><div class="h5"><br></div></div></blockquote><div><br></div></div></div></div></div>