<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Sep 20, 2013 at 10:21 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>On 09/20/2013 05:44 PM, Ben Chan wrote:<br>
>     >     I do yes. Your Samsung modems are also Icera based, IIRC. Do<br>
>     you want me<br>
>     >     to try to do it? Or, otherwise, if you do it I can test it<br>
>     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>
>     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<br>
>     Huawei?<br>
><br>
><br>
> Thanks. Looking at the v4 patches. Shouldn't<br>
> MMBroadbandModemHuawei::bearer_report_connection_status() report<br>
> DISCONNECTING instead of DISCONNECTED? Perhaps we should also keep the<br>
> comment to explain why DISCONNECTING is chosen instead of DISCONNECTED.<br>
<br>
</div></div>That's done in patch 2/2 if I'm not mistaken.<br></blockquote><div><br></div><div>patch 2/2 doesn't seem to modify mm-broadband-modem-huawei.c</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



<span><font color="#888888"><br>
--<br>
Aleksander<br>
</font></span></blockquote></div><br></div></div>