Hi Philippe,

>  drivers/atmodem/sim.c |   95 
> +++++++++++++++++++++++++++++++++++++++++++++++++
>  1 files changed, 95 insertions(+), 0 deletions(-)

please include an OFONO_AT_DEBUG trace of this AT command.

> diff --git a/drivers/atmodem/sim.c b/drivers/atmodem/sim.c
> index b28f3f8..49f5245 100644
> --- a/drivers/atmodem/sim.c
> +++ b/drivers/atmodem/sim.c
> @@ -667,6 +667,95 @@ static void at_cpinr_cb(gboolean ok, GAtResult *result, 
> gpointer user_data)
>       cb(&error, retries, cbd->data);
>  }
>  
> +static gboolean skip_to_field(GAtResultIter *iter, const char *field)
> +{
> +     char *line;
> +     char *offset;
> +     int field_len = field ? strlen(field) : 0;
> +
> +     if (iter == NULL)
> +             return FALSE;
> +
> +     if (iter->l == NULL)
> +             return FALSE;
> +
> +     line = iter->l->data;
> +
> +     if (field_len == 0) {
> +             iter->line_pos = 0;
> +             return TRUE;
> +     }
> +
> +     offset = g_strrstr(line, field);
> +     if (offset == NULL)
> +             return FALSE;
> +
> +     iter->line_pos = (offset - line) + field_len;
> +
> +     return TRUE;
> +}

What is this one doing?

Denis, can you have a look please.

> +static void at_cpnnum_cb(gboolean ok, GAtResult *result, gpointer user_data)
> +{
> +     struct cb_data *cbd = user_data;
> +     ofono_sim_pin_retries_cb_t cb = cbd->cb;
> +     GAtResultIter iter;
> +     struct ofono_error error;
> +     int retries[OFONO_SIM_PASSWORD_INVALID];
> +     size_t i;
> +
> +     decode_at_error(&error, g_at_result_final_response(result));
> +
> +     if (!ok) {
> +             cb(&error, NULL, cbd->data);
> +             return;
> +     }
> +
> +     for (i = 0; i < OFONO_SIM_PASSWORD_INVALID; i++)
> +             retries[i] = -1;
> +
> +     g_at_result_iter_init(&iter, result);
> +
> +     /*
> +      * Even if this is not really a suffix, we rely on the tag PIN1 to
> +      * recognize the correct response line.
> +      */
> +     if (!g_at_result_iter_next(&iter, "PIN1="))
> +             goto error;
> +
> +     if (!g_at_result_iter_next_number(&iter,
> +                                     &retries[OFONO_SIM_PASSWORD_SIM_PIN]))
> +             goto error;
> +
> +     if (!skip_to_field(&iter, "PUK1="))
> +             goto error;
> +
> +     if (!g_at_result_iter_next_number(&iter,
> +                                     &retries[OFONO_SIM_PASSWORD_SIM_PUK]))
> +             goto error;
> +
> +     if (!skip_to_field(&iter, "PIN2="))
> +             goto error;
> +
> +     if (!g_at_result_iter_next_number(&iter,
> +                                     &retries[OFONO_SIM_PASSWORD_SIM_PIN2]))
> +             goto error;
> +
> +     if (!skip_to_field(&iter, "PUK2="))
> +             goto error;
> +
> +     if (!g_at_result_iter_next_number(&iter,
> +                                     &retries[OFONO_SIM_PASSWORD_SIM_PUK2]))
> +             goto error;
> +
> +     cb(&error, retries, cbd->data);
> +
> +     return;
> +
> +error:
> +     CALLBACK_WITH_FAILURE(cb, NULL, cbd->data);
> +}
> +
>  static void at_pin_retries_query(struct ofono_sim *sim,
>                                       ofono_sim_pin_retries_cb_t cb,
>                                       void *data)
> @@ -695,6 +784,12 @@ static void at_pin_retries_query(struct ofono_sim *sim,
>                       return;
>  
>               break;
> +     case OFONO_VENDOR_SPEEDUP:
> +             if (g_at_chat_send(sd->chat, "AT+CPNNUM", NULL,
> +                                     at_cpnnum_cb, cbd, g_free) > 0)
> +                     return;
> +
> +             break;
>       default:
>               if (g_at_chat_send(sd->chat, "AT+CPINR", cpinr_prefixes,
>                                       at_cpinr_cb, cbd, g_free) > 0)

Regards

Marcel


_______________________________________________
ofono mailing list
[email protected]
http://lists.ofono.org/listinfo/ofono

Reply via email to