<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Sep 20, 2013 at 4:47 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"><div class="im">On 09/20/2013 07:18 AM, Ben Chan wrote:<br>


>     I do yes. Your Samsung modems are also Icera based, IIRC. Do you want me<br>
>     to try to do it? Or, otherwise, if you do it I can test it with those<br>
>     modems.<br>
><br>
><br>
> Uploaded patch v3. I'm testing with an Icera Y3400 and a Huawei MU736.<br>
><br>
> As it's not a small change, more testing would be good. Could you try<br>
> the patchs with the modems you have? Thanks.<br>
<br>
</div>I updated both patches (v4); please take a look at them. Tested both<br>
with an Option/HSO and an Ericsson MBM modem, and seem to work ok.<br>
<br>
The new changes are mainly refactoring some of the plugin specific<br>
methods, plus making sure that the generic report_connection_status()<br>
methods in MMBearer and MMBroadbandBearer only expect DISCONNECTED.<br>
Receiving any other state is a programmer error (e.g. a bearer can<br>
specify that it is now connected only during a connection attempt). In<br>
other words, each plugin is responsible for handling the connection<br>
status they expect in addition to DISCONNECTED; they may choose to<br>
ignore them or use them to complete an ongoing connection attempt, or<br>
whatever, but it's up to the plugin to do so. Generic implementation<br>
should always handle DISCONNECTED only.<br>
<br>
What do you think of these? Can you test them with the Icera and the Huawei?<br>
<span class=""><font color="#888888"><br></font></span></blockquote><div><br></div><div>Thanks. Looking at the v4 patches. Shouldn't MMBroadbandModemHuawei::<span style="color:rgb(0,0,0)">bearer_report_connection_status() report DISCONNECTING instead of DISCONNECTED? Perhaps we should also keep the comment to explain why DISCONNECTING is chosen instead of DISCONNECTED.</span></div>

<div><br></div><div> </div><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"><span class=""><font color="#888888">
--<br>
Aleksander<br>
</font></span></blockquote></div><br></div></div>