**** BEGIN LOGGING AT Mon Feb 07 02:59:56 2011
Feb 07 11:32:09 <Jeevaka> holtmann: ping
Feb 07 14:32:15 <OlivierG> nbertrand: ping ?
Feb 07 15:31:14 <holtmann> Jeevaka: pongf
Feb 07 15:33:25 <Jeevaka> Currently, ifxmodem/voicecall.c has code for reporting MT emergency call
Feb 07 15:34:05 <Jeevaka> holtmann: is there any need for that? 
Feb 07 15:34:59 <holtmann> Jeevaka: No idea. In general our conclusion was that this is not relevant.
Feb 07 15:35:17 <holtmann> However emergency calls are not my strong suit when it comes to use case analysis.
Feb 07 15:37:19 <denkenz> Jeevaka: FYI we use CRING to identify the call type, e.g. voice vs data
Feb 07 15:37:30 <denkenz> Are you sure removing it is a good idea?
Feb 07 15:37:34 <Jeevaka> yes
Feb 07 15:37:46 <Jeevaka> XCALLSTAT reports the call status of only voice calls
Feb 07 15:38:17 <Jeevaka> since we are not supporting other than voice, do we really need to have that
Feb 07 15:38:31 <denkenz> hm, funny, don't remember them mentioning that in their docs
Feb 07 15:38:38 <denkenz> If this is only for voice calls, then that is fine
Feb 07 15:39:10 <Jeevaka> denkenz: This(XCALLSTAT) command allows enabling / disabling the reporting voice call status on DTE using an unsolicited result code
Feb 07 15:39:49 <Jeevaka> denkenz: thats what is there in the version of the ifx spec I'm having
Feb 07 15:40:09 <holtmann> denkenz: Btw. we should be storing the SMS bearer as string in the config file and not a magic number.
Feb 07 15:40:21 <Jeevaka> denkenz: but we never know ;)
Feb 07 15:43:07 <denkenz> Jeevaka: Exactly, they're not very clear
Feb 07 15:48:37 <Jeevaka> denkenz: you are correct, it seems that we need to rely on CRING
Feb 07 15:48:54 <Jeevaka> denkenz: ifx is sending XCALLSTAT for video call as well
Feb 07 16:10:58 <wagi> hi, I'll try to get HFP working. ofono complains with 
Feb 07 16:10:59 <wagi> ofonod[17232]: plugins/hfp.c:hfp_enable() 0x239cac0
Feb 07 16:10:59 <wagi> ofonod[17232]: plugins/hfp.c:hfp_connect_reply() Connect reply: Method "Connect" with signature "" on interface "org.bluez.HandsfreeGateway" doesn't exist
Feb 07 16:11:22 <wagi> I guess I'm missing something
Feb 07 16:11:36 <wagi> any ideas, of course I'll start to debug ;)
Feb 07 16:12:09 <wagi> bluez too old?
Feb 07 16:12:32 <denkenz> you might not have enabled the gateway in your bluez conf
Feb 07 16:13:42 <denkenz> wagi: see audio/audio.conf
Feb 07 16:14:05 <wagi> denkenz: okay, thanks for the input
Feb 07 16:16:44 <Jeevaka> denkenz: currently, there is no code available in ifxmodem/voicecall.c for handing cnap even when we have enabled for notifications
Feb 07 16:17:11 <Jeevaka> denkenz: can i go ahead and add the notification handling?
Feb 07 16:17:27 <denkenz> of course
Feb 07 16:17:51 <denkenz> And I'm not sure what to do with your patches now
Feb 07 16:17:57 <denkenz> The XCOLP one looks good
Feb 07 16:18:03 <denkenz> And the ATD probably makes sense
Feb 07 16:18:07 <denkenz> care to resend?
Feb 07 16:18:45 <Jeevaka> definitely, I'm modifying the cring a little bit
Feb 07 16:19:54 <Jeevaka> I'll resend it pretty soon
Feb 07 16:20:48 <Jeevaka> denkenz: creating a call in cring doesn't make sense when the call id comes in xcallstat, what do you think?
Feb 07 16:21:16 <denkenz> Jeevaka: There are pretty clear rules on id management in at modems
Feb 07 16:21:40 <denkenz> So it doesn't matter that much, and I didn't know the order of xcallstat vs cring when I wrote the driver
Feb 07 16:21:47 <Jeevaka> incase of ifxmodem, XCALLSTAT comes first and then CRING
Feb 07 16:22:08 <holtmann> Jeevaka: That was never ever documented anywhere in the IFX specs ;)
Feb 07 16:22:19 <Jeevaka> also CRING comes only once
Feb 07 16:22:29 <denkenz> Which is against the spec ;)
Feb 07 16:22:30 <holtmann> Jeevaka: Denis and I were flying blind when we wrote the IFX MAL ;)
Feb 07 16:22:44 <Jeevaka> holtmann, denkenz: agree
Feb 07 16:22:55 <denkenz> If IFX guarantees that xcallstat comes first, then its fine to take the id from there
Feb 07 16:23:23 <Jeevaka> I tested few use cases including call waiting and video call waiting, everthing conforms the same that XCALLSTAT comes first
Feb 07 16:23:30 <holtmann> Jeevaka: You need to have IFX to guarantee that in writing btw. They need to update their AT command specs. and spell this out.
Feb 07 16:23:59 <Jeevaka> holtmann, denkenz: yep, I'll make sure that happens
Feb 07 16:24:23 <holtmann> We could then not enable CRING notifications ;)
Feb 07 16:25:27 <denkenz> Then you can modify cring to just signal the call if the type matches
Feb 07 16:25:40 <denkenz> But then this is all pointless anyway since if you have cring, then you have a single call
Feb 07 16:25:44 <denkenz> and the id is always 1 anyway
Feb 07 16:26:02 <denkenz> So it might not make any sense to touch it ;)
Feb 07 16:27:09 <Jeevaka> no, incase of waiting call its different
Feb 07 16:27:20 <denkenz> waiting is done via CCWA
Feb 07 16:27:25 <denkenz> and you need that one as well
Feb 07 16:27:36 <Jeevaka> yes, but the XCALLSTAT comes before that as well
Feb 07 16:27:38 <denkenz> Otherwise you don't know the caller id
Feb 07 16:28:26 <denkenz> Again, if you're worried about ids, then go for it
Feb 07 16:29:00 <denkenz> But as I mentioned before, ids are pretty much guaranteed to be allocated in a well-known manner by 3GPP
Feb 07 16:29:59 <Jeevaka> yep, agreed
Feb 07 17:11:20 <wagi> denkenz: got it working. I had to copy audio.conf to /etc/bluetooth and add the entry Enable=Gateway into the General section.
Feb 07 17:11:27 <wagi> denkenz: thanks for the tip
Feb 07 17:15:38 <denkenz> Yep, that's the one, wish that was actually documented ;)
Feb 07 19:19:00 <Jeevaka> denkenz: is the v2 refactoring part ok?
Feb 07 19:31:31 <denkenz> Jeevaka: I'm lost in that one
Feb 07 19:32:24 <denkenz> You're mashing too much into that patch
Feb 07 19:32:34 <denkenz> I suggest one for XCOLP, one for ATD, one for the rest
Feb 07 19:48:19 <Jeevaka> ok
Feb 07 21:47:59 <denkenz> Jeevaka: Funny but Gmail delivered 0, 1 and 3 of your patch set
Feb 07 21:48:03 <denkenz> 2 and 4 are still missing
Feb 07 21:48:10 <denkenz> Must be odd numbers only today
Feb 07 21:50:51 <Jeevaka> do you want me to send it again?
Feb 07 21:51:21 <Jeevaka> i can resend the complete set
Feb 07 21:53:16 <denkenz> well 4/4 came through
Feb 07 21:53:18 <denkenz> still missing 2/4
Feb 07 21:53:34 <denkenz> Give it another 10 minutes I guess
Feb 07 21:54:19 <Jeevaka> i didnt get it in my gmail account too
Feb 07 21:58:41 <holtmann> denkenz: I have all 4 patches.
Feb 07 21:59:02 <denkenz> Then your mail server is sane
Feb 07 21:59:11 <denkenz> Gmail has been dropping patches like crazy lately
Feb 07 21:59:17 <holtmann> Of course it is. Since I build it by myself ;)
Feb 07 21:59:32 <denkenz> There was a whole series from Lucas that just got dropped on the floor
Feb 07 21:59:52 <holtmann> Hah. That is bad.
Feb 07 22:02:50 <denkenz> Jeevaka: So just resend 2/4 to me privately or the entire series, whatever is easier
Feb 07 22:04:36 <Jeevaka> denkenz: sent 2/4 alone to your gmail id
Feb 07 22:08:02 <Jeevaka> denkenz: are we going to have the MT emergency callback in ofono core?
Feb 07 22:09:20 <Jeevaka> denkenz: do we need to handle the XEMC notifications?
Feb 07 22:24:17 <denkenz> Don't think so
Feb 07 22:24:36 <denkenz> If the emergency list is updated properly by the driver, then XEMC is not needed
Feb 07 22:25:53 <Jeevaka> then removal of that handling will be my next patch ;)
Feb 07 22:27:13 <holtmann> Jeevaka: What is actually the proposal for updating the emergency number list.
Feb 07 22:29:19 <Jeevaka> latest info is that there will be URC containing the emergency list
Feb 07 22:29:48 <holtmann> That should be fine, but you do need to propose also the oFono core changes for making this work.
Feb 07 22:30:56 <Jeevaka> once the modem side is confirmed, will take care of core side changes as well
Feb 07 22:31:25 <holtmann> Sounds good. Until then leave the XEMC in.
Feb 07 22:31:43 <holtmann> If anybody tests this, I like to see it in the debug traces.
Feb 07 22:31:51 <holtmann> That was the only reason why I put it there.
Feb 07 22:33:11 <Jeevaka> ok
Feb 07 22:46:28 <Jeevaka> denkenz: do you know why huaweimodem is not creating and also notifying the call incase of waiting call?
Feb 07 22:48:07 <holtmann> Jeevaka: The Huawei voicecall driver is pretty limited. Since the hardware is kinda broken.
Feb 07 22:48:21 <holtmann> More then one call never really worked for me.
Feb 07 22:50:23 <Jeevaka> nice
Feb 07 23:31:24 <demarchi> denkenz: what's the problem with my latest patches?
Feb 07 23:57:37 <denkenz> demarchi: problem?
Feb 07 23:57:54 <denkenz> Jeevaka: Can CCWA repeat on ifx?
Feb 08 00:04:36 <demarchi> 19:58    denkenz> There was a whole series from Lucas that just got dropped on  the floor
Feb 08 00:42:03 <denkenz> demarchi: Remember when your patches were MIA due to Gmail not delivering them?
Feb 08 00:42:09 <denkenz> That's what I was referring to
Feb 08 01:24:51 <demarchi> denkenz: ahhn.. ok
**** ENDING LOGGING AT Tue Feb 08 02:59:57 2011