sysmocom
/
ofono
mirror of git://git.sysmocom.de/ofono
11
0
Fork 0
Code Issues Releases Activity

ifx: Don't bother polling on an ATD 

The core can handle stateless drivers that do not keep clip state, in
the case of IFX we can skip the poll after ATD for finding out our CLIP
and just keep it as invalid / obtained from COLP.  The core should do
the right thing.
This commit is contained in:
Denis Kenzior 2010-09-24 00:22:06 -05:00
parent 09c0ae8478
commit 4b134bd15f
1 changed files with 9 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
drivers/ifxmodem/voicecall.c
View File

@ -324,7 +324,6 @@ static void atd_cb(gboolean ok, GAtResult *result, gpointer user_data)
{
struct cb_data *cbd = user_data;
struct ofono_voicecall *vc = cbd->user;
struct voicecall_data *vd = ofono_voicecall_get_data(vc);
ofono_voicecall_cb_t cb = cbd->cb;
GAtResultIter iter;
const char *num;
@ -335,8 +334,10 @@ static void atd_cb(gboolean ok, GAtResult *result, gpointer user_data)
decode_at_error(&error, g_at_result_final_response(result));
if (!ok)
goto out;
if (!ok) {
cb(&error, cbd->data);
return;
}
g_at_result_iter_init(&iter, result);
@ -360,20 +361,14 @@ static void atd_cb(gboolean ok, GAtResult *result, gpointer user_data)
return;
}
/* oFono core will generate a call with the dialed number
* inside its dial callback. Unless we got COLP information
* we do not need to communicate that a call is being
* dialed
/* Let oFono core will generate a call with the dialed number
* inside its dial callback.
*/
cb(&error, cbd->data);
/* If we got COLP information, then notify the core */
if (validity != 2)
ofono_voicecall_notify(vc, call);
if (!vd->clcc_source)
vd->clcc_source = g_timeout_add(POLL_CLCC_INTERVAL,
poll_clcc, vc);
out:
cb(&error, cbd->data);
}
static void ifx_dial(struct ofono_voicecall *vc,