Hi Mingli,

On 12/17/2012 10:22 AM, Mingli Wu wrote:
> For Telit HE910,  ofono always got "\r\n+CME ERROR: 14\r\n"
> after issuing "AT+CMER=3,0,0,2\r" command. "CME ERROR: 14"
> means that SIM is busy. Now we igonre the result of the
> command "AT+CMER=3,0,0,2\r", and register network anyway.
> ---
>   drivers/atmodem/network-registration.c |    2 --
>   1 file changed, 2 deletions(-)
>
> diff --git a/drivers/atmodem/network-registration.c b/drivers/atmodem/network-registration.c
> index 19b19b2..518a548 100644
> --- a/drivers/atmodem/network-registration.c
> +++ b/drivers/atmodem/network-registration.c
> @@ -1480,7 +1480,6 @@ static void at_cmer_not_supported(struct ofono_netreg *netreg)
>   	ofono_error("+CMER not supported by this modem.  If this is an error"
>   			" please submit patches to support this hardware");
>
> -	ofono_netreg_remove(netreg);

You are changing the global behavior of the driver here.  This affects 
other modems than HE910...

>   }
>
>   static void at_cmer_set_cb(gboolean ok, GAtResult *result, gpointer user_data)
> @@ -1490,7 +1489,6 @@ static void at_cmer_set_cb(gboolean ok, GAtResult *result, gpointer user_data)
>
>   	if (!ok) {
>   		at_cmer_not_supported(netreg);
> -		return;
>   	}
>
>   	/*

Regards,
-Denis