Hi Mika,
> +enum ofono_gprs_addrconf {
> + OFONO_GPRS_ADDRCONF_NONE,
> + OFONO_GPRS_ADDRCONF_STATIC,
> + OFONO_GPRS_ADDRCONF_DHCP,
> +};
> +
What exactly is the _NONE addrconf for? And why don't we use a gboolean
here like before?
> struct ofono_gprs_primary_context {
> unsigned int cid;
> int direction;
> @@ -94,6 +101,20 @@ struct ofono_modem *ofono_gprs_context_get_modem(struct
> ofono_gprs_context *gc);
>
> void ofono_gprs_context_set_type(struct ofono_gprs_context *gc,
> enum ofono_gprs_context_type type);
> +void ofono_gprs_context_set_interface(struct ofono_gprs_context *gc,
> + const char *interface);
> +void ofono_gprs_context_set_ip_addrconf(struct ofono_gprs_context *gc,
> + enum ofono_gprs_addrconf method);
> +void ofono_gprs_context_set_ip_address(struct ofono_gprs_context *gc,
> + const char *address);
> +void ofono_gprs_context_set_ip_netmask(struct ofono_gprs_context *gc,
> + const char *netmask);
> +void ofono_gprs_context_set_ip_gateway(struct ofono_gprs_context *gc,
> + const char *netmask);
> +void ofono_gprs_context_set_ipv6_address(struct ofono_gprs_context *gc,
> + const char *address);
> +void ofono_gprs_context_set_dns_servers(struct ofono_gprs_context *gc,
> + const char **dns);
>
So I'm actually fine with this approach.
> #ifdef __cplusplus
> }
Regards,
-Denis
_______________________________________________
ofono mailing list
ofono@ofono.org
http://lists.ofono.org/listinfo/ofono
