Re: [PATCH] alcatel: create sim atom with vendor alcatel

[Denis Kenzior <denkenz@gmail.com>]

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

Hi Cedric,

On 11/14/2012 05:08 AM, Cedric Jehasse wrote:
> From: Cedric Jehasse<cedric.jehasse@softathome.com>
>
> ---
>   plugins/alcatel.c |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)

Patch has been applied, but I had to amend it.

>
> diff --git a/plugins/alcatel.c b/plugins/alcatel.c
> index 8005170..e3e7086 100644
> --- a/plugins/alcatel.c
> +++ b/plugins/alcatel.c
> @@ -230,7 +230,7 @@ static void alcatel_pre_sim(struct ofono_modem *modem)
>   	DBG("%p", modem);
>
>   	ofono_devinfo_create(modem, 0, "atmodem", data->aux);
> -	sim = ofono_sim_create(modem, 0, "atmodem", data->aux);
> +	sim = ofono_sim_create(modem, OFONO_VENDOR_ALCATEL, "atmodem", data->aux);

Please make sure to stay within 80 characters per line.

>
>   	if (sim&&  data->have_sim == TRUE)
>   		ofono_sim_inserted_notify(sim, TRUE);

Regards,
-Denis