Re: [PATCH v4 3/8] gprs: Add 'stk' gprs context type

[Denis Kenzior <denkenz@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1831 bytes --]

Hi Philippe,

On 05/20/2011 11:26 AM, Philippe Nunes wrote:
> ---
>  include/gprs-context.h |    1 +
>  src/gprs.c             |    7 +++++++
>  2 files changed, 8 insertions(+), 0 deletions(-)
> 

Please refer to 'Submitting patches' section in HACKING.  We do prefer
these to be two separate patches.

> diff --git a/include/gprs-context.h b/include/gprs-context.h
> index f82fcaa..e7976a1 100644
> --- a/include/gprs-context.h
> +++ b/include/gprs-context.h
> @@ -46,6 +46,7 @@ enum ofono_gprs_context_type {
>  	OFONO_GPRS_CONTEXT_TYPE_MMS,
>  	OFONO_GPRS_CONTEXT_TYPE_WAP,
>  	OFONO_GPRS_CONTEXT_TYPE_IMS,
> +	OFONO_GPRS_CONTEXT_TYPE_STK,
>  };
>  
>  struct ofono_gprs_primary_context {
> diff --git a/src/gprs.c b/src/gprs.c
> index deffeb8..9657a3e 100644
> --- a/src/gprs.c
> +++ b/src/gprs.c
> @@ -188,6 +188,8 @@ static const char *gprs_context_default_name(enum ofono_gprs_context_type type)
>  		return "WAP";
>  	case OFONO_GPRS_CONTEXT_TYPE_IMS:
>  		return "IMS";
> +	case OFONO_GPRS_CONTEXT_TYPE_STK:
> +		return "STK";
>  	}
>  
>  	return NULL;
> @@ -207,6 +209,8 @@ static const char *gprs_context_type_to_string(
>  		return "wap";
>  	case OFONO_GPRS_CONTEXT_TYPE_IMS:
>  		return "ims";
> +	case OFONO_GPRS_CONTEXT_TYPE_STK:
> +		return "stk";
>  	}
>  
>  	return NULL;
> @@ -227,6 +231,9 @@ static gboolean gprs_context_string_to_type(const char *str,
>  	} else if (g_str_equal(str, "ims")) {
>  		*out = OFONO_GPRS_CONTEXT_TYPE_IMS;
>  		return TRUE;
> +	} else if (g_str_equal(str, "stk")) {
> +		*out = OFONO_GPRS_CONTEXT_TYPE_STK;
> +		return TRUE;
>  	}
>  
>  	return FALSE;

I don't really see the point of these changes though.  We do not want to
expose STK contexts over D-Bus, so none of these are going to be used.

Regards,
-Denis