* [PATCH 0/1] Patch Description @ 2010-09-17 8:50 Yang Gu 2010-09-17 8:50 ` [PATCH 1/1] Enable some atoms for hso modem Yang Gu 0 siblings, 1 reply; 5+ messages in thread From: Yang Gu @ 2010-09-17 8:50 UTC (permalink / raw) To: ofono [-- Attachment #1: Type: text/plain, Size: 430 bytes --] I just got a new Option modem (iCON 452), and this patch is to enable several atoms for it. The stk support is still not avaiable, for I couldn't find any appropriate manual for this part. If you happen to know this, please kindly share the info. Yang Gu (1): Enable some atoms for hso modem plugins/hso.c | 34 ++++++++++++++++++++++++++++++++-- 1 files changed, 32 insertions(+), 2 deletions(-) -- 1.7.2.3 ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 1/1] Enable some atoms for hso modem 2010-09-17 8:50 [PATCH 0/1] Patch Description Yang Gu @ 2010-09-17 8:50 ` Yang Gu 2010-09-17 9:00 ` Marcel Holtmann 2010-09-17 12:59 ` Denis Kenzior 0 siblings, 2 replies; 5+ messages in thread From: Yang Gu @ 2010-09-17 8:50 UTC (permalink / raw) To: ofono [-- Attachment #1: Type: text/plain, Size: 2945 bytes --] --- plugins/hso.c | 34 ++++++++++++++++++++++++++++++++-- 1 files changed, 32 insertions(+), 2 deletions(-) diff --git a/plugins/hso.c b/plugins/hso.c index 25ca157..1fbde58 100644 --- a/plugins/hso.c +++ b/plugins/hso.c @@ -34,14 +34,23 @@ #define OFONO_API_SUBJECT_TO_CHANGE #include <ofono/plugin.h> #include <ofono/modem.h> +#include <ofono/call-barring.h> +#include <ofono/call-forwarding.h> +#include <ofono/call-meter.h> +#include <ofono/call-settings.h> +#include <ofono/call-volume.h> #include <ofono/devinfo.h> +#include <ofono/message-waiting.h> #include <ofono/netreg.h> +#include <ofono/phonebook.h> #include <ofono/sim.h> #include <ofono/cbs.h> #include <ofono/sms.h> +#include <ofono/ssn.h> #include <ofono/ussd.h> #include <ofono/gprs.h> #include <ofono/gprs-context.h> +#include <ofono/voicecall.h> #include <ofono/radio-settings.h> #include <ofono/log.h> @@ -257,24 +266,40 @@ static void hso_pre_sim(struct ofono_modem *modem) ofono_devinfo_create(modem, 0, "atmodem", data->control); sim = ofono_sim_create(modem, OFONO_VENDOR_OPTION_HSO, "atmodem", data->control); + ofono_voicecall_create(modem, 0, "atmodem", data->app); if (sim) ofono_sim_inserted_notify(sim, TRUE); } + +static void hso_post_sim(struct ofono_modem *modem) +{ + struct hso_data *data = ofono_modem_get_data(modem); + + DBG("%p", modem); + + ofono_phonebook_create(modem, 0, "atmodem", data->app); +} + static void hso_post_online(struct ofono_modem *modem) { struct hso_data *data = ofono_modem_get_data(modem); + struct ofono_message_waiting *mw; struct ofono_gprs *gprs; struct ofono_gprs_context *gc; DBG("%p", modem); + ofono_call_forwarding_create(modem, 0, "atmodem", data->app); + ofono_call_settings_create(modem, 0, "atmodem", data->app); ofono_netreg_create(modem, OFONO_VENDOR_OPTION_HSO, "atmodem", data->app); - + ofono_call_meter_create(modem, 0, "atmodem", data->app); + ofono_call_barring_create(modem, 0, "atmodem", data->app); + ofono_ssn_create(modem, 0, "atmodem", data->app); + ofono_call_volume_create(modem, 0, "atmodem", data->app); ofono_radio_settings_create(modem, 0, "hsomodem", data->app); - ofono_sms_create(modem, OFONO_VENDOR_OPTION_HSO, "atmodem", data->app); ofono_cbs_create(modem, OFONO_VENDOR_QUALCOMM_MSM, "atmodem", data->app); @@ -286,6 +311,10 @@ static void hso_post_online(struct ofono_modem *modem) if (gprs && gc) ofono_gprs_add_context(gprs, gc); + + mw = ofono_message_waiting_create(modem); + if (mw) + ofono_message_waiting_register(mw); } static struct ofono_modem_driver hso_driver = { @@ -296,6 +325,7 @@ static struct ofono_modem_driver hso_driver = { .disable = hso_disable, .set_online = hso_set_online, .pre_sim = hso_pre_sim, + .post_sim = hso_post_sim, .post_online = hso_post_online, }; -- 1.7.2.3 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 1/1] Enable some atoms for hso modem 2010-09-17 8:50 ` [PATCH 1/1] Enable some atoms for hso modem Yang Gu @ 2010-09-17 9:00 ` Marcel Holtmann 2010-09-17 12:59 ` Denis Kenzior 1 sibling, 0 replies; 5+ messages in thread From: Marcel Holtmann @ 2010-09-17 9:00 UTC (permalink / raw) To: ofono [-- Attachment #1: Type: text/plain, Size: 1957 bytes --] Hi Yang, > plugins/hso.c | 34 ++++++++++++++++++++++++++++++++-- > 1 files changed, 32 insertions(+), 2 deletions(-) > > diff --git a/plugins/hso.c b/plugins/hso.c > index 25ca157..1fbde58 100644 > --- a/plugins/hso.c > +++ b/plugins/hso.c > @@ -34,14 +34,23 @@ > #define OFONO_API_SUBJECT_TO_CHANGE > #include <ofono/plugin.h> > #include <ofono/modem.h> > +#include <ofono/call-barring.h> > +#include <ofono/call-forwarding.h> > +#include <ofono/call-meter.h> > +#include <ofono/call-settings.h> > +#include <ofono/call-volume.h> > #include <ofono/devinfo.h> > +#include <ofono/message-waiting.h> > #include <ofono/netreg.h> > +#include <ofono/phonebook.h> > #include <ofono/sim.h> > #include <ofono/cbs.h> > #include <ofono/sms.h> > +#include <ofono/ssn.h> > #include <ofono/ussd.h> > #include <ofono/gprs.h> > #include <ofono/gprs-context.h> > +#include <ofono/voicecall.h> > #include <ofono/radio-settings.h> > #include <ofono/log.h> > > @@ -257,24 +266,40 @@ static void hso_pre_sim(struct ofono_modem *modem) > ofono_devinfo_create(modem, 0, "atmodem", data->control); > sim = ofono_sim_create(modem, OFONO_VENDOR_OPTION_HSO, > "atmodem", data->control); > + ofono_voicecall_create(modem, 0, "atmodem", data->app); > > if (sim) > ofono_sim_inserted_notify(sim, TRUE); > } > > + > +static void hso_post_sim(struct ofono_modem *modem) > +{ > + struct hso_data *data = ofono_modem_get_data(modem); > + > + DBG("%p", modem); > + > + ofono_phonebook_create(modem, 0, "atmodem", data->app); > +} > + this part is fine with me. However why should we enable any kind of voice call support for this modem? It is a data dongle and until you tell me that the voice is routed to the host PC via USB or some external microphone, it makes no sense to enable these parts. If they are data only dongle, I prefer to leave them like that. Regards Marcel ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/1] Enable some atoms for hso modem 2010-09-17 8:50 ` [PATCH 1/1] Enable some atoms for hso modem Yang Gu 2010-09-17 9:00 ` Marcel Holtmann @ 2010-09-17 12:59 ` Denis Kenzior 2010-09-19 7:37 ` Gu, Yang 1 sibling, 1 reply; 5+ messages in thread From: Denis Kenzior @ 2010-09-17 12:59 UTC (permalink / raw) To: ofono [-- Attachment #1: Type: text/plain, Size: 2063 bytes --] Hi Yang, > @@ -257,24 +266,40 @@ static void hso_pre_sim(struct ofono_modem *modem) > ofono_devinfo_create(modem, 0, "atmodem", data->control); > sim = ofono_sim_create(modem, OFONO_VENDOR_OPTION_HSO, > "atmodem", data->control); > + ofono_voicecall_create(modem, 0, "atmodem", data->app); > The Option hardware can never do audio, so enabling voice functionality is not a good idea. > if (sim) > ofono_sim_inserted_notify(sim, TRUE); > } > > + > +static void hso_post_sim(struct ofono_modem *modem) > +{ > + struct hso_data *data = ofono_modem_get_data(modem); > + > + DBG("%p", modem); > + > + ofono_phonebook_create(modem, 0, "atmodem", data->app); > +} > + This is OK > static void hso_post_online(struct ofono_modem *modem) > { > struct hso_data *data = ofono_modem_get_data(modem); > + struct ofono_message_waiting *mw; > struct ofono_gprs *gprs; > struct ofono_gprs_context *gc; > > DBG("%p", modem); > > + ofono_call_forwarding_create(modem, 0, "atmodem", data->app); > + ofono_call_settings_create(modem, 0, "atmodem", data->app); > ofono_netreg_create(modem, OFONO_VENDOR_OPTION_HSO, > "atmodem", data->app); > - > + ofono_call_meter_create(modem, 0, "atmodem", data->app); > + ofono_call_barring_create(modem, 0, "atmodem", data->app); > + ofono_ssn_create(modem, 0, "atmodem", data->app); > + ofono_call_volume_create(modem, 0, "atmodem", data->app); > ofono_radio_settings_create(modem, 0, "hsomodem", data->app); > - > ofono_sms_create(modem, OFONO_VENDOR_OPTION_HSO, "atmodem", data->app); > ofono_cbs_create(modem, OFONO_VENDOR_QUALCOMM_MSM, > "atmodem", data->app); > @@ -286,6 +311,10 @@ static void hso_post_online(struct ofono_modem *modem) > > if (gprs && gc) > ofono_gprs_add_context(gprs, gc); > + > + mw = ofono_message_waiting_create(modem); > + if (mw) > + ofono_message_waiting_register(mw); Same thing here, since voicecalls are not enabled, all the call-* related atoms are not needed. Regards, -Denis ^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH 1/1] Enable some atoms for hso modem 2010-09-17 12:59 ` Denis Kenzior @ 2010-09-19 7:37 ` Gu, Yang 0 siblings, 0 replies; 5+ messages in thread From: Gu, Yang @ 2010-09-19 7:37 UTC (permalink / raw) To: ofono [-- Attachment #1: Type: text/plain, Size: 356 bytes --] Hi Denis and Marcel, >> + >> +static void hso_post_sim(struct ofono_modem *modem) >> +{ >> + struct hso_data *data = ofono_modem_get_data(modem); >> + >> + DBG("%p", modem); >> + >> + ofono_phonebook_create(modem, 0, "atmodem", data->app); >> +} >> + > >This is OK So I will send the patch to enable phonebook only:) Regards, -Yang ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2010-09-19 7:37 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2010-09-17 8:50 [PATCH 0/1] Patch Description Yang Gu 2010-09-17 8:50 ` [PATCH 1/1] Enable some atoms for hso modem Yang Gu 2010-09-17 9:00 ` Marcel Holtmann 2010-09-17 12:59 ` Denis Kenzior 2010-09-19 7:37 ` Gu, Yang
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.