Re: [gprs-provision RFCPATCHv6 0/4] Plugin API for provisioning of GPRS context settings

2011-01-26 Thread Aki Niemi
Hi Marcel, 2011/1/26 Marcel Holtmann mar...@holtmann.org: lets get this merged without support for SPN for now. We can easily add this later. So please fix Denis' comments and re-submit this without the SPN change. Andrew is currently looking into fixing the SIM reading race. Once that is

Re: [gprs-provision RFCPATCHv6 0/4] Plugin API for provisioning of GPRS context settings

2011-01-26 Thread Marcel Holtmann
Hi Aki, lets get this merged without support for SPN for now. We can easily add this later. So please fix Denis' comments and re-submit this without the SPN change. Andrew is currently looking into fixing the SIM reading race. Once that is done we can tackle the SPN part. Feel free to

[gprs-provision RFCPATCHv6 0/4] Plugin API for provisioning of GPRS context settings

2011-01-25 Thread Jukka Saunamaki
Hello Another patchset about automatic provisioning of GPRS context settings. This time, provisioning driver API is synchronous, and __ofono_gprs_provision_get_settings includes MCC,MNC and SPN as in parameters. Per modem probing and all related complexity was removed, since there was really

Re: [gprs-provision RFCPATCHv6 0/4] Plugin API for provisioning of GPRS context settings

2011-01-25 Thread Marcel Holtmann
Hi Jukka, Another patchset about automatic provisioning of GPRS context settings. This time, provisioning driver API is synchronous, and __ofono_gprs_provision_get_settings includes MCC,MNC and SPN as in parameters. Per modem probing and all related complexity was removed, since there

Re: [gprs-provision RFCPATCHv6 0/4] Plugin API for provisioning of GPRS context settings

2011-01-25 Thread Jukka Saunamaki
Hi On Tue, 2011-01-25 at 13:28 +0100, ext Marcel Holtmann wrote: I think that our only leftover is how we handle the SPN detail. Denis, what is your preference here? My take would be that we store it inside network registration and provide it to external modules. I don't see a point in