2009/12/16 Denis Kenzior <denkenz(a)gmail.com>:
You should still be able to set the topics even if powered=false.
store the new list and send it to the modem if powered==true.
This is what the code does afaict.
> - cbs->pending = dbus_message_ref(msg);
> + topic_str = cbs_topics_to_str(cbs, topics);
> cbs->driver->set_topics(cbs, topic_str, cbs_set_topics_cb, cbs);
> return NULL;
> +static void cbs_power_off_cb(const struct ofono_error *error, void *data)
Lets return the reply once the operation has succeeded at the driver to be
consistent with other parts of the system.
Okay, changed the behaviour to reply in the callback. The reason I
chose to reply immediately was that we don't mind it if the command
fails at the modem.