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