<div dir="ltr"><div class="gmail_extra">Uggg I am very sorry but that patch is not the right patch. It has none of the fixes I worked on :(. Very sorry and terribly embarrassed. As soon as I get back to my home computer I will mail out the correct one. I'd avoid doing any work on this one for the next 24h.</div><div><br><div class="gmail_quote">On Thu, Nov 24, 2016 at 8:54 AM, matthew stanger <span dir="ltr"><<a href="mailto:stangerm2@gmail.com" target="_blank">stangerm2@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><p dir="ltr">Alexsander,</p>
<p dir="ltr">I'm traveling right now will test as soon as I get back tomorrow. </p><div class="gmail-HOEnZb"><div class="gmail-h5">
<div class="gmail_extra"><br><div class="gmail_quote">On Nov 24, 2016 2:58 AM, "Aleksander Morgado" <<a href="mailto:aleksander@aleksander.es" target="_blank">aleksander@aleksander.es</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hey Matthew,<br>
<br>
On Wed, Nov 23, 2016 at 10:28 PM, Aleksander Morgado<br>
<<a href="mailto:aleksander@aleksander.es" target="_blank">aleksander@aleksander.es</a>> wrote:<br>
> Great work! it looks really good now, although there are some small<br>
> things that could still be fixed. Let me prepare a branch with your<br>
> patch and I'll include some additional patches for you to review.<br>
<br>
Please check the 'stanger/cinterion' branch in the upstream git repository:<br>
<a href="https://cgit.freedesktop.org/ModemManager/ModemManager/log/?h=stanger/cinterion" rel="noreferrer" target="_blank">https://cgit.freedesktop.org/M<wbr>odemManager/ModemManager/log/?<wbr>h=stanger/cinterion</a><br>
<br>
I've included several changes that I suggested in previous reviews,<br>
like avoiding the enabling phase for the SWWAN feature support check,<br>
or the simpler ^SWWAN response parser. Let me know what you think, and<br>
please test the branch with a real device and let me know if you find<br>
any issue.<br>
<br>
BTW; I'm worried that we're binding the logic too much to a specific<br>
device with 2 interfaces; I've seen some online AT command reference<br>
that doesn't even talk about the third parameter to specify interface<br>
index in AT^SWWAN= set operations:<br>
  <a href="http://www.seapraha.cz/download/pls8-e_atc_v01000_tcm216-155309.pdf" rel="noreferrer" target="_blank">http://www.seapraha.cz/downloa<wbr>d/pls8-e_atc_v01000_tcm216-<wbr>155309.pdf</a><br>
But for now I guess we're good.<br>
<br>
--<br>
Aleksander<br>
<a href="https://aleksander.es" rel="noreferrer" target="_blank">https://aleksander.es</a><br>
</blockquote></div></div>
</div></div></blockquote></div><br></div></div>