All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/6] phonesim: Don't use VENDOR_PHONESIM for Calypso.
@ 2010-08-05 17:30 Andrzej Zaborowski
  2010-08-05 17:30 ` [PATCH 2/6] calypso: Check if SIM is present on modem startup Andrzej Zaborowski
                   ` (3 more replies)
  0 siblings, 4 replies; 7+ messages in thread
From: Andrzej Zaborowski @ 2010-08-05 17:30 UTC (permalink / raw)
  To: ofono

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

Also move the STK initialisation to post-sim because it is more logical
there (even if it makes no difference under phonesim).

I'm not sure if there's any user of the "calypso" mode in phonesim
driver now, perhaps it can be removed.
---
 plugins/phonesim.c |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/plugins/phonesim.c b/plugins/phonesim.c
index 9c5b2cb..97fa530 100644
--- a/plugins/phonesim.c
+++ b/plugins/phonesim.c
@@ -289,8 +289,6 @@ static void phonesim_pre_sim(struct ofono_modem *modem)
 	else
 		ofono_voicecall_create(modem, 0, "atmodem", data->chat);
 
-	ofono_stk_create(modem, OFONO_VENDOR_PHONESIM, "atmodem", data->chat);
-
 	if (sim)
 		ofono_sim_inserted_notify(sim, TRUE);
 }
@@ -304,6 +302,10 @@ static void phonesim_post_sim(struct ofono_modem *modem)
 
 	DBG("%p", modem);
 
+	if (!data->calypso)
+		ofono_stk_create(modem, OFONO_VENDOR_PHONESIM,
+					"atmodem", data->chat);
+
 	ofono_ussd_create(modem, 0, "atmodem", data->chat);
 	ofono_call_forwarding_create(modem, 0, "atmodem", data->chat);
 	ofono_call_settings_create(modem, 0, "atmodem", data->chat);
-- 
1.7.1.86.g0e460.dirty


^ permalink raw reply related	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2010-08-06 17:45 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-08-05 17:30 [PATCH 1/6] phonesim: Don't use VENDOR_PHONESIM for Calypso Andrzej Zaborowski
2010-08-05 17:30 ` [PATCH 2/6] calypso: Check if SIM is present on modem startup Andrzej Zaborowski
2010-08-05 17:30 ` [PATCH 3/6] calypsomodem: Add a Calypso STK driver Andrzej Zaborowski
2010-08-06 17:19   ` Denis Kenzior
2010-08-06 17:45     ` Andrzej Zaborowski
2010-08-05 17:30 ` [PATCH 4/6] calypso: Enable AT extended error codes Andrzej Zaborowski
2010-08-06 17:17 ` [PATCH 1/6] phonesim: Don't use VENDOR_PHONESIM for Calypso Denis Kenzior

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.