Hi Sergey, On 1/10/21 1:44 PM, Sergey Matyukevich wrote:
Implement network monitoring driver for gemalto modems that are able to provide serving cell information and basic measurements using AT+CQS and AT^SMONI commands. --- Makefile.am | 3 +- drivers/gemaltomodem/gemaltomodem.c | 2 + drivers/gemaltomodem/gemaltomodem.h | 3 + drivers/gemaltomodem/netmon.c | 609 ++++++++++++++++++++++++++++ 4 files changed, 616 insertions(+), 1 deletion(-) create mode 100644 drivers/gemaltomodem/netmon.c
<snip>
+static int gemalto_netmon_probe(struct ofono_netmon *netmon, + unsigned int vendor, void *user) +{ + GAtChat *chat = user; + struct netmon_driver_data *nmd; + + DBG("gemalto netmon probe"); + + nmd = g_try_new0(struct netmon_driver_data, 1); + if (nmd == NULL) + return -ENOMEM;
Feel free to use g_new0. We have given up using g_try_new0 since these errors don't really occur on Linux.
+ + nmd->chat = g_at_chat_clone(chat); + + ofono_netmon_set_data(netmon, nmd); + + g_idle_add(gemalto_delayed_register, netmon); + + return 0; +}
Other than this, LGTM. Regards, -Denis _______________________________________________ ofono mailing list -- ofono@ofono.org To unsubscribe send an email to ofono-le...@ofono.org