<div dir="ltr"><div><div><div><div>Hi Daniele,<br><br></div>it looks like the last commit introduced a some kind of problem. HE910 initialization ends with error<br><br>> Modem couldn't be initialized: couldn't load current capabilities: Failed to determine modem capabilities<br><br></div>Here's the pastebin link of the logs of this test, <br><br>> <a href="http://pastebin.com/h5PkSb92">http://pastebin.com/h5PkSb92</a><br><br>while the following's the logs of a good run made with version 8a386218690aeff7e2c923a14f91da7bbc046ed2 (HEAD~1)<br><br>> <a href="http://pastebin.com/0fqrW52k">http://pastebin.com/0fqrW52k</a><br><br></div>Apparently, the parser mm-broadband-modem.c:parse_caps_gcap isn't receiving the reply from the serial device (the last line is a debug log I added, which prints out the content of the "response" that the function is parsing).<br><br><debug> [1460381746.474901] [mm-port-serial-at.c:459] debug_log(): (ttyACM0): --> 'AT+GCAP<CR>'
<br><debug> [1460381746.521723] [mm-port-serial-at.c:459] debug_log(): (ttyACM0): <-- '<CR><LF>'
<br><debug> [1460381746.550323] [mm-port-serial-at.c:459] debug_log(): (ttyACM0): <-- '+GCAP: +CGSM,+DS,+FCLASS,+MS,+ES<CR><LF><CR><LF>OK<CR><LF>'
<br><debug> [1460381746.583914] [mm-broadband-modem.c:448] parse_caps_gcap(): response is ''<br><br></div><div>On LE910 SV V2 the behavior's the same.<br></div><div><br></div><div>Carlo<br></div><div><br><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, 11 Apr 2016 at 15:52 Daniele Palmas <<a href="mailto:dnlplm@gmail.com">dnlplm@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Carlo,<br>
<br>
2016-04-11 10:55 GMT+02:00 Daniele Palmas <<a href="mailto:dnlplm@gmail.com" target="_blank">dnlplm@gmail.com</a>>:<br>
> Hi Aleksander,<br>
><br>
> 2016-04-08 17:14 GMT+02:00 Aleksander Morgado <<a href="mailto:aleksander@aleksander.es" target="_blank">aleksander@aleksander.es</a>>:<br>
>> On Fri, Apr 8, 2016 at 4:55 PM, Daniele Palmas <<a href="mailto:dnlplm@gmail.com" target="_blank">dnlplm@gmail.com</a>> wrote:<br>
>>> 2016-04-08 16:42 GMT+02:00 Aleksander Morgado <<a href="mailto:aleksander@aleksander.es" target="_blank">aleksander@aleksander.es</a>>:<br>
>>>> When we were completing tasks in idle, the logic was like this:<br>
>>>><br>
>>>>  * Schedule task completion in idle<br>
>>>>  * self->priv->task = NULL<br>
>>>>  * (idle) Task completion callback called<br>
>>>><br>
>>>> This meant that the self->priv->task was always set to NULL before the<br>
>>>> completion callback was called, which is what we wanted as a new task may be<br>
>>>> scheduled in the callback itself.<br>
>>>><br>
>>>> Now, without completing in idle, we were wrongly doing:<br>
>>>><br>
>>>>  * Task completion callback called<br>
>>>>  * self->priv->task = NULL<br>
>>>><br>
>>>> This commit fixes the logic by making sure self->priv->task = NULL before any<br>
>>>> task completion.<br>
>>>> ---<br>
>>>><br>
>>>> Daniele, can you validate that this patch fixes your issue?<br>
>>>><br>
>>><br>
>>> According to a quick test I did, it seems to fix the issue on LE910.<br>
>><br>
>> I'll get the patch merged to git master, and if your tests on Monday<br>
>> go well, I'll release a new rc with this fix.<br>
>><br>
><br>
> I confirm that the issue I was seeing has been fixed by your patch.<br>
> Thanks again!<br>
><br>
<br>
Can you please also test with HE910 and LE910 V2?<br>
<br>
Thanks,<br>
Daniele<br>
<br>
> Regards,<br>
> Daniele<br>
><br>
>> --<br>
>> Aleksander<br>
>> <a href="https://aleksander.es" rel="noreferrer" target="_blank">https://aleksander.es</a><br>
</blockquote></div>