<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jan 3, 2014 at 3:26 AM, Aleksander Morgado <span dir="ltr"><<a href="mailto:aleksander@aleksander.es" target="_blank">aleksander@aleksander.es</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"><div class="im">On 03/01/14 01:00, Ben Chan wrote:<br>
> ---<br>
>  plugins/altair/mm-broadband-bearer-altair-lte.c | 6 ++++++<br>
>  1 file changed, 6 insertions(+)<br>
><br>
<br>
</div>What's the purpose of keeping a reference to the secondary port around<br>
if it is not going to be used during the connection sequence?<br>
<div class=""><div class="h5"><br></div></div></blockquote><div><br></div><div>IIUC, the connection sequence uses mm_base_modem_at_command, which could potentially pick up the secondary port, if one exists. The existing code works but this patch makes the code more future-proof (and more symmetric to the disconnection sequence, which also references the secondary port). Alternatively, we could explicitly use the primary port, or add g_assert (secondary == NULL) if the modem never has a secondary port.</div>

</div></div></div>