Re: [MM] Power down during init

2012-09-20 Thread Marius Kotsbak
Den 19. sep. 2012 07:47, skrev Aleksander Morgado: On 09/18/2012 08:03 PM, Dan Williams wrote: Btw, is this expected?: model: '+CGMM: "MF820D"' A bug, the handler code here probably needs to strip out the CGMM if found. Most devices don't actually prefix the CGMM response with "+CGMM" so the

Re: [MM] Power down during init

2012-09-18 Thread Aleksander Morgado
> > We need to run the power-down during init *only* after having checked > current/modem capabilities. Will update the patch. > I pushed a new 'power-down-during-init' branch to the upstream git tree, including the change of when the power down command is run, so that we only run it after havi

Re: [MM] Power down during init

2012-09-18 Thread Aleksander Morgado
>> >> I briefly tested the patch on Samsung/Icera Y3300/Y3400 modems. >> >> Planning to do some more stress tests later this week. >> >> >> > >> > I'm assuming the tests went well? :-) >> > >> >> Except for the issue that CFUN+4 can occasionally take a long time >> to finish :( > > How long?

Re: [MM] Power down during init

2012-09-18 Thread Ben Chan
> > How long? I think the timeout is 3 seconds now so maybe it should be > increased. > could be up to 30s, which is likely a firmware issue. - Ben ___ networkmanager-list mailing list networkmanager-list@gnome.org https://mail.gnome.org/mailman/listinf

Re: [MM] Power down during init

2012-09-18 Thread Marius Kotsbak
On Sep 19, 2012 7:36 AM, "Ben Chan" wrote: > > On Tue, Sep 18, 2012 at 10:05 PM, Aleksander Morgado > wrote: > > > >> I briefly tested the patch on Samsung/Icera Y3300/Y3400 modems. > >> Planning to do some more stress tests later this week. > >> > > > > I'm assuming the tests went well? :-) > >

Re: [MM] Power down during init

2012-09-18 Thread Aleksander Morgado
On 09/18/2012 08:03 PM, Dan Williams wrote: >> > Btw, is this expected?: >> > >> > model: '+CGMM: "MF820D"' > A bug, the handler code here probably needs to strip out the CGMM if > found. Most devices don't actually prefix the CGMM response with > "+CGMM" so the code wasn't built to handle that,

Re: [MM] Power down during init

2012-09-18 Thread Aleksander Morgado
>> >>> I briefly tested the patch on Samsung/Icera Y3300/Y3400 modems. >>> Planning to do some more stress tests later this week. >>> >> >> I'm assuming the tests went well? :-) >> > > Except for the issue that CFUN+4 can occasionally take a long time > to finish :( > > Still haven't found t

Re: [MM] Power down during init

2012-09-18 Thread Ben Chan
On Tue, Sep 18, 2012 at 10:05 PM, Aleksander Morgado wrote: > >> I briefly tested the patch on Samsung/Icera Y3300/Y3400 modems. >> Planning to do some more stress tests later this week. >> > > I'm assuming the tests went well? :-) > Except for the issue that CFUN+4 can occasionally take a lo

Re: [MM] Power down during init

2012-09-18 Thread Aleksander Morgado
> Hey Dan and Marius, > > > > >> Power Up = enabled > >> Power Down = disabled > >> > >> So we already have those states in ModemManager via the > Enable() method > >> and the modem states. It's just that not all devices > a

Re: [MM] Power down during init

2012-09-18 Thread Aleksander Morgado
> I briefly tested the patch on Samsung/Icera Y3300/Y3400 modems. > Planning to do some more stress tests later this week. > I'm assuming the tests went well? :-) > It may not be a bad idea to allow a plugin to override the > power-down-on-initialization behavior. > Not a big deal I guess; gi

Re: [MM] Power down during init (was: Re: [MM master] Re: [MM06] [PATCH] sierra: use +CFUN=4 for powering down)

2012-09-18 Thread Ben Chan
I briefly tested the patch on Samsung/Icera Y3300/Y3400 modems. Planning to do some more stress tests later this week. It may not be a bad idea to allow a plugin to override the power-down-on-initialization behavior. Thanks, Ben On Tue, Sep 18, 2012 at 7:24 AM, Aleksander Morgado wrote: > Hey D

Re: [MM] Power down during init (was: Re: [MM master] Re: [MM06] [PATCH] sierra: use +CFUN=4 for powering down)

2012-09-18 Thread Marius Kotsbak
2012/9/18 Marius Kotsbak > 2012/9/18 Aleksander Morgado > >> Hey Dan and Marius, >> >> > >> >> Power Up = enabled >> >> Power Down = disabled >> >> >> >> So we already have those states in ModemManager via the Enable() method >> >> and the modem states. It's just that not all devices actually >

Re: [MM] Power down during init (was: Re: [MM master] Re: [MM06] [PATCH] sierra: use +CFUN=4 for powering down)

2012-09-18 Thread Dan Williams
On Tue, 2012-09-18 at 19:45 +0200, Marius Kotsbak wrote: > 2012/9/18 Aleksander Morgado > Hey Dan and Marius, > > > > >> Power Up = enabled > >> Power Down = disabled > >> > >> So we already have those states in ModemManager via the >

Re: [MM] Power down during init (was: Re: [MM master] Re: [MM06] [PATCH] sierra: use +CFUN=4 for powering down)

2012-09-18 Thread Marius Kotsbak
2012/9/18 Aleksander Morgado > Hey Dan and Marius, > > > > >> Power Up = enabled > >> Power Down = disabled > >> > >> So we already have those states in ModemManager via the Enable() method > >> and the modem states. It's just that not all devices actually implement > >> the low-power mode when