Re: [PATCH] isimodem: use a specific call id when answering

[Denis Kenzior <denkenz@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1479 bytes --]

Hi Pekka,

On 04/27/2011 07:45 AM, Pekka.Pessi(a)nokia.com wrote:
> From: Pekka Pessi <Pekka.Pessi@nokia.com>
> 
> ---
>  drivers/isimodem/voicecall.c |   22 +++++++++++++++++++++-
>  1 files changed, 21 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/isimodem/voicecall.c b/drivers/isimodem/voicecall.c
> index 333f9b8..4d4e16b 100644
> --- a/drivers/isimodem/voicecall.c
> +++ b/drivers/isimodem/voicecall.c
> @@ -1480,7 +1480,27 @@ static void isi_dial(struct ofono_voicecall *ovc,
>  static void isi_answer(struct ofono_voicecall *ovc, ofono_voicecall_cb_t cb,
>  			void *data)
>  {
> -	isi_call_answer_req(ovc, CALL_ID_ALL, cb, data);
> +	struct isi_voicecall *ivc = ofono_voicecall_get_data(ovc);
> +	int id;
> +
> +	for (id = 1; id <= 7; id++) {
> +		switch (ivc->calls[id].status) {
> +		case CALL_STATUS_COMING:
> +		case CALL_STATUS_MT_ALERTING:
> +		case CALL_STATUS_WAITING:

oFono expects answer to only work on 'incoming' calls in the traditional
sense.  Is waiting status equivalent to CCWA here? If so, it might be
better to leave it out here.

> +			goto answer;
> +		case CALL_STATUS_PROCEEDING:
> +			if (ivc->calls[id].mode_info & CALL_MODE_ORIGINATOR)
> +				goto answer;
> +			break;
> +		}
> +	}
> +
> +	CALLBACK_WITH_FAILURE(cb, data);
> +	return;
> +
> +answer:
> +	isi_call_answer_req(ovc, id, cb, data);
>  }
>  
>  static void isi_hangup_current(struct ofono_voicecall *ovc,

Regards,
-Denis