Ublox TOBY-L210 needs two retries to connect

[Penalva, Salvador]

Hi,

Thank you for opening the bug and tracking it. Right now, I have not enough time to do the changes and test it. I'm sorry. It's gonna take much of my time to understand how to do the changes and apply them correctly.  And we don't need it to work right now if without SIM of with a retry is still working.

Probably you have too many other important things to do before solving this, but if you find time to solve the bug sometime I will be glad to help you to test it if you need it.

Thank you,

Salvador. 

-----Original Message-----
From: Aleksander Morgado [mailto:aleksander at aleksander.es] 
Sent: martes, 09 de mayo de 2017 23:16
To: Penalva, Salvador
Cc: ModemManager (development)
Subject: Re: Ublox TOBY-L210 needs two retries to connect

Hey Salvador,

>
> I did these changes but it didn't work. I added a debug message to determine when the wait_function was being called. The first attempt of connection that fails, never calls that function, so I think that the timeout is not being applied.  However the second attempt, the one that connects is calling the function.
>
> Here you go the changes I applied, in case I did something wrong.
>

Yes, I realize now that just subclassing that step isn't enough, as that step is a wait time *after* the modem has reported us it's unlocked. In our case, we need to wait time before even checking if the modem is unlocked. What we  probably need is to subclass is the
send_pin() method in the MMBaseSim object. This means we need to create a new MMSimTelit object with MMBaseSim as parent object, and then subclass the send_pin() method only to add a
g_timeout_add_seconds(5,...) call.

Would you be up to doing this? An example of plugin subclasing the
send_pin() in the MMBaseSim object is the "mbm" plugin, where we have a MMSimMbm that subclasses send_pin() and send_puk(). Both those should be subclassed in our case as well.

I opened a bug in bugzilla to track it:
https://bugs.freedesktop.org/show_bug.cgi?id=100980

Cheers!

--
Aleksander
https://aleksander.es