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