Re: [PATCH 1/1] add set_online interface to he910 modem

[Denis Kenzior <denkenz@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1345 bytes --]

Hi Alexey,

On 12/01/2014 05:35 AM, Alexey Mednyy wrote:
> Had to add set_online interface to he910 modem connman now, working
> properly if disable he910 cellular (was not implemented error before).
> ---
>   plugins/he910.c | 25 +++++++++++++++++++++++++
>   1 file changed, 25 insertions(+)
>

Please be aware that HE910s are shipped with no support for online mode. 
  It was a feature added by later firmware.  On earlier firmware, the 
HE910 would turn off the SIM when CFUN=4 is sent.  This is obviously not 
what oFono expects.

By now this patch is probably okay to accept as the new firmware is more 
widely available.  However, you might want to double check that your 
firmware is recent enough and is working as expected.

> diff --git a/plugins/he910.c b/plugins/he910.c
> index 69362b8..add2243 100644
> --- a/plugins/he910.c
> +++ b/plugins/he910.c
> @@ -322,6 +322,30 @@ static void he910_pre_sim(struct ofono_modem *modem)
>          ofono_location_reporting_create(modem, 0, "telitmodem", data->chat);
>   }
>
> +
> +static void set_online_cb(gboolean ok, GAtResult *result, gpointer
> user_data)

Please re-read our coding style guidelines in doc/coding-style.txt.  In 
particular, pay attention to indentation rules and how to break up long 
lines (over 80 characters).

Regards,
-Denis