Hi Jeevaka,

On Tue, 2011-03-08 at 10:59 +0200, ext jeevaka.badrap...@elektrobit.com
wrote:
> Hi Antti,
> 
> ofono-boun...@ofono.org wrote:
> > ---
> >  drivers/atmodem/network-registration.c |    2 ++
> >  1 files changed, 2 insertions(+), 0 deletions(-)
> > 
> > diff --git a/drivers/atmodem/network-registration.c
> > b/drivers/atmodem/network-registration.c
> > index 4913611..2d589f0 100644
> > --- a/drivers/atmodem/network-registration.c
> > +++ b/drivers/atmodem/network-registration.c
> > @@ -722,6 +722,8 @@ static void ifx_ctzv_notify(GAtResult *result,
> >     gpointer user_data) nd->time.mday = mday;
> >     nd->time.mon = mon;
> >     nd->time.year = 2000 + year;
> > +
> > +   ofono_netreg_time_notify(netreg, &nd->time);
> >  }
> > 
> >  static void ifx_ctzdst_notify(GAtResult *result, gpointer user_data)
> 
> Since the fix is more to do with the ifx modem, it would be nice if we
> have the
> commit message changed to reflect that.

I suppose the person applying the patch can do the modifications to the commit 
message if needed. 

> Whenever there is a timezone change, CTZV, CTZDST and XNITZINFO URCs
> will be reported. That is the main
> reason behind calling the ofono_netreg_time_notify from CTZDST. Are you
> pretty sure
> this is not the case?

I don't know the details of ifx modem internals. If the time changes, is
it always guaranteed that CTZDST is sent and that CTZDST comes after the
CTZV? All I know that when testing NITZ with ifx modem, the time
notification is not emitted without my fix.

BR,
  Antti

_______________________________________________
ofono mailing list
ofono@ofono.org
http://lists.ofono.org/listinfo/ofono

Reply via email to