Hi Anirudh,

On 09/10/2018 10:52 PM, Anirudh Gargi wrote:
> CREG status 6 and 7 added in network registration status, sms atom
> to consider new states also.
> ---
>   src/common.h | 14 ++++++++------
>   src/sms.c    |  2 ++
>   2 files changed, 10 insertions(+), 6 deletions(-)
> 
> diff --git a/src/common.h b/src/common.h
> index 1b6b01d..b826228 100644
> --- a/src/common.h
> +++ b/src/common.h
> @@ -37,12 +37,14 @@ enum access_technology {
>   
>   /* 27.007 Section 7.2 <stat> */
>   enum network_registration_status {
> -	NETWORK_REGISTRATION_STATUS_NOT_REGISTERED =	0,
> -	NETWORK_REGISTRATION_STATUS_REGISTERED =	1,
> -	NETWORK_REGISTRATION_STATUS_SEARCHING =		2,
> -	NETWORK_REGISTRATION_STATUS_DENIED =		3,
> -	NETWORK_REGISTRATION_STATUS_UNKNOWN =		4,
> -	NETWORK_REGISTRATION_STATUS_ROAMING =		5,
> +	NETWORK_REGISTRATION_STATUS_NOT_REGISTERED =		0,
> +	NETWORK_REGISTRATION_STATUS_REGISTERED =		1,
> +	NETWORK_REGISTRATION_STATUS_SEARCHING =			2,
> +	NETWORK_REGISTRATION_STATUS_DENIED =			3,
> +	NETWORK_REGISTRATION_STATUS_UNKNOWN =			4,
> +	NETWORK_REGISTRATION_STATUS_ROAMING =			5,
> +	NETWORK_REGISTRATION_STATUS_REGISTERED_SMS_EUTRAN =	6,
> +	NETWORK_REGISTRATION_STATUS_ROAMING_SMS_EUTRAN = 	7,
>   };
>   
>   /* 27.007 Section 7.3 <stat> */
> diff --git a/src/sms.c b/src/sms.c
> index b86158e..c604e05 100644
> --- a/src/sms.c
> +++ b/src/sms.c
> @@ -782,6 +782,8 @@ static void netreg_status_watch(int status, int lac, int ci, int tech,
>   	switch (status) {
>   	case NETWORK_REGISTRATION_STATUS_REGISTERED:
>   	case NETWORK_REGISTRATION_STATUS_ROAMING:
> +	case NETWORK_REGISTRATION_STATUS_REGISTERED_SMS_EUTRAN:
> +	case NETWORK_REGISTRATION_STATUS_ROAMING_SMS_EUTRAN:
>   		sms->registered = TRUE;
>   		break;
>   	default:
> 

So while this patch is fine, don't we have a bunch of places that use 
network registration status that need to be updated as well?  I mean 
even registration_status_to_string should be updated...

So I'd like some patches that address these issues before I can take 
this one.

Regards,
-Denis