<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Sep 19, 2013 at 11:44 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 class="HOEnZb"><div class="h5">On 19/09/13 19:24, Ben Chan wrote:<br>
><br>
>     Instead of passing a MMBearerStatus value; it probably is a good idea to<br>
>     have a new enumeration considering other things. E.g. in the option/hso<br>
>     bearer we have report_connection_status() which is the method used by<br>
>     the modem object to notify about unsolicited messages received during<br>
>     connection/disconnection sequence. This method uses a new enum with the<br>
>     following values:<br>
><br>
>     typedef enum {<br>
>         MM_BROADBAND_BEARER_HSO_CONNECTION_STATUS_UNKNOWN,<br>
>         MM_BROADBAND_BEARER_HSO_CONNECTION_STATUS_CONNECTED,<br>
>         MM_BROADBAND_BEARER_HSO_CONNECTION_STATUS_CONNECTION_FAILED,<br>
>         MM_BROADBAND_BEARER_HSO_CONNECTION_STATUS_DISCONNECTED<br>
>     } MMBroadbandBearerHsoConnectionStatus;<br>
><br>
>     The generic report_status() could handle all these. Something like:<br>
><br>
>     typedef enum {<br>
>         MM_BEARER_CONNECTION_STATUS_UNKNOWN,<br>
>         MM_BEARER_CONNECTION_STATUS_CONNECTED,<br>
>         MM_BEARER_CONNECTION_STATUS_CONNECTION_FAILED,<br>
>         MM_BEARER_CONNECTION_STATUS_DISCONNECTING,<br>
>         MM_BEARER_CONNECTION_STATUS_DISCONNECTED,<br>
>     } MMBearerConnectionStatus.<br>
><br>
>     The 'disconnecting' case could be used by the Huawei plugin to notify<br>
>     about a disconnection that will need further plugin action to finally<br>
>     report disconnected (see the other email).<br>
><br>
><br>
> I guess you're proposing:<br>
> - getting rid of those custom connection status enum in mbm/hso/icera<br>
> - converting those custom mm_broadband_*_report_connection_status() in<br>
> mbm/hso/icera to report_connection_status()<br>
> - instead of calling mm_bearer_report_* in those custom<br>
> mm_broadband_*_report_connection_status(), call parent's<br>
> report_connection_status<br>
><br>
<br>
</div></div>Exactly, yes.<br>
<div class="im"><br>
> I think the changes make sense, but don't have those modems to verify<br>
> such changes. Do you have those modems?<br>
><br>
><br>
<br>
</div>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>
<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>Uploaded patch v3. I'm testing with an Icera Y3400 and a Huawei MU736.</div><div><br></div><div>As it's not a small change, more testing would be good. Could you try the patchs with the modems you have? Thanks.</div>

<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888">
--<br>
Aleksander<br>
</font></span></blockquote></div><br></div></div>