Re: [MM] [PATCH] novatel-lte: increase number of retries for connection status checks

2013-05-28 Thread Ben Chan
Sounds good to me. Changed to 60s. Ben On Tue, May 28, 2013 at 12:47 AM, Aleksander Morgado wrote: > On 28/05/13 06:04, Ben Chan wrote: > > 60s seems a bit excessive as users most likely give up earlier. Revised > > the patch to have a 30s timeout. > > > > Remember that MM lets the connection

Re: [MM] [PATCH] novatel-lte: increase number of retries for connection status checks

2013-05-28 Thread Aleksander Morgado
On 28/05/13 06:04, Ben Chan wrote: > 60s seems a bit excessive as users most likely give up earlier. Revised > the patch to have a 30s timeout. > Remember that MM lets the connection attempt to be cancelled at any time by Disconnect()-ing the Bearer during the connection attempt (best effort is

Re: [MM] [PATCH] novatel-lte: increase number of retries for connection status checks

2013-05-27 Thread Ben Chan
60s seems a bit excessive as users most likely give up earlier. Revised the patch to have a 30s timeout. Thanks, Ben On Mon, May 27, 2013 at 12:52 AM, Aleksander Morgado wrote: > On 26/05/13 08:45, Ben Chan wrote: > > This patch increases the number of retries, from 4 to 10, for connection > >

Re: [MM] [PATCH] novatel-lte: increase number of retries for connection status checks

2013-05-27 Thread Aleksander Morgado
On 26/05/13 08:45, Ben Chan wrote: > This patch increases the number of retries, from 4 to 10, for connection > status check during a connection / disconnection request, which handles > some scenario when the connection / disconnection request takes more > than 5 seconds to complete. We recently u

[MM] [PATCH] novatel-lte: increase number of retries for connection status checks

2013-05-25 Thread Ben Chan
This patch increases the number of retries, from 4 to 10, for connection status check during a connection / disconnection request, which handles some scenario when the connection / disconnection request takes more than 5 seconds to complete. --- plugins/novatel/mm-broadband-bearer-novatel-lte.c |