Hi Sjur,

>  drivers/atmodem/sim.c |    9 ++++++---
>  1 files changed, 6 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/atmodem/sim.c b/drivers/atmodem/sim.c
> index eb40ad7..8e7c403 100644
> --- a/drivers/atmodem/sim.c
> +++ b/drivers/atmodem/sim.c
> @@ -567,10 +567,12 @@ static void at_pin_send_cb(gboolean ok, GAtResult 
> *result,
>       decode_at_error(&error, g_at_result_final_response(result));
>  
>       /*
> -      * On the MBM modem, AT+CPIN? keeps returning SIM PIN for a moment
> -      * after successful AT+CPIN="..", but sends *EPEV when that changes.
> +      * On the MBM and STE modem, AT+CPIN? keeps returning SIM PIN for a
> +      * moment after successful AT+CPIN="..", but sends *EPEV when that
> +      * changes.
>        */
> -     if (ok && sd->vendor == OFONO_VENDOR_MBM) {
> +     if (ok && (sd->vendor == OFONO_VENDOR_MBM ||
> +                     sd->vendor == OFONO_VENDOR_STE)) {
>               sd->epev_id = g_at_chat_register(sd->chat, "*EPEV",
>                                                       at_epev_notify,
>                                                       FALSE, cbd, g_free);
> @@ -817,6 +819,7 @@ static int at_sim_probe(struct ofono_sim *sim, unsigned 
> int vendor,
>       case OFONO_VENDOR_WAVECOM:
>               g_at_chat_add_terminator(sd->chat, "+CPIN:", 6, TRUE);
>               break;
> +     case OFONO_VENDOR_STE:
>       case OFONO_VENDOR_MBM:
>               g_at_chat_send(sd->chat, "AT*EPEE=1", NULL, NULL, NULL, NULL);
>               break;

do we really wanna create another vendor quirk here. If both modems
behave identical then just pass OFONO_VENDOR_MBM in the SIM atom
registration and be done with it.

Regards

Marcel


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

Reply via email to