Hi Marit, On 11/03/2010 07:06 AM, Marit Henriksen wrote: > From: Marit Henriksen <[email protected]> > > When adding the use of pin event *EPEV also to at_pin_send_puk_cb, > it becomes identical to at_pin_send_cb, and can be removed. > --- > drivers/atmodem/sim.c | 44 +------------------------------------------- > 1 files changed, 1 insertions(+), 43 deletions(-) > > diff --git a/drivers/atmodem/sim.c b/drivers/atmodem/sim.c > index 3abd1d8..9cfdc65 100644 > --- a/drivers/atmodem/sim.c > +++ b/drivers/atmodem/sim.c > @@ -647,48 +647,6 @@ error: > CALLBACK_WITH_FAILURE(cb, data); > } > > -static void at_pin_send_puk_cb(gboolean ok, GAtResult *result, > - gpointer user_data) > -{ > - struct cb_data *cbd = user_data; > - struct sim_data *sd = cbd->user; > - ofono_sim_lock_unlock_cb_t cb = cbd->cb; > - struct ofono_error error; > - > - decode_at_error(&error, g_at_result_final_response(result)); > - > - if (!ok) > - goto done; > - > - switch (sd->vendor) { > - case OFONO_VENDOR_IFX: > - /* > - * On the IFX modem, AT+CPIN? can return READY too > - * early and so use +XSIM notification to detect > - * the ready state of the SIM. > - */ > - sd->ready_id = g_at_chat_register(sd->chat, "+XSIM", > - at_xsim_notify, > - FALSE, cbd, g_free); > - return; > - case OFONO_VENDOR_MBM: > - /* > - * On the MBM modem, AT+CPIN? keeps returning SIM PIN > - * for a moment after successful AT+CPIN="..", but then > - * sends *EPEV when that changes. > - */ > - sd->ready_id = g_at_chat_register(sd->chat, "*EPEV", > - at_epev_notify, > - FALSE, cbd, g_free); > - return; > - }
I'm fine with this patch but do note that it does not apply. I guess that is because the OFONO_VENDOR_MBM part is not in the upstream repository. Applying: atmodem: Remove at_pin_send_puk_cb, and use at_pin_send_cb instead. fatal: sha1 information is lacking or useless (drivers/atmodem/sim.c). Repository lacks necessary blobs to fall back on 3-way merge. Cannot fall back to three-way merge. Patch failed at 0001 atmodem: Remove at_pin_send_puk_cb, and use at_pin_send_cb instead. When you have resolved this problem run "git am -3 --resolved". If you would prefer to skip this patch, instead run "git am -3 --skip". To restore the original branch and stop patching run "git am -3 --abort". > - > -done: > - cb(&error, cbd->data); > - > - g_free(cbd); > -} > - > static void at_pin_send_puk(struct ofono_sim *sim, const char *puk, > const char *passwd, > ofono_sim_lock_unlock_cb_t cb, void *data) > @@ -706,7 +664,7 @@ static void at_pin_send_puk(struct ofono_sim *sim, const > char *puk, > snprintf(buf, sizeof(buf), "AT+CPIN=\"%s\",\"%s\"", puk, passwd); > > ret = g_at_chat_send(sd->chat, buf, none_prefix, > - at_pin_send_puk_cb, cbd, NULL); > + at_pin_send_cb, cbd, NULL); > > memset(buf, 0, sizeof(buf)); > Regards, -Denis _______________________________________________ ofono mailing list [email protected] http://lists.ofono.org/listinfo/ofono
