diff options
author | Aleksander Morgado <aleksander@lanedo.com> | 2011-06-02 12:33:09 +0200 |
---|---|---|
committer | Aleksander Morgado <aleksander@lanedo.com> | 2011-06-06 17:21:47 +0200 |
commit | f9f6d1dfdb070bec05e5e7385f8d4eb95e573b6f (patch) | |
tree | 07e29bd3e00f8a4763e1075e5bda2162ec863f42 /src/mm-plugin-base.c | |
parent | 39215599018a12817a853615ccbc4ee67c234858 (diff) |
plugins: propagate cached probing result to supports task
We need to ensure that the supports task always has the results of the probing,
no matter if the probing was just launched by the plugin grabbing the port, or
by a previous plugin. We do this during supports_port(), by propagating to the
supports task any possible previously cached probing results.
Diffstat (limited to 'src/mm-plugin-base.c')
-rw-r--r-- | src/mm-plugin-base.c | 48 |
1 files changed, 28 insertions, 20 deletions
diff --git a/src/mm-plugin-base.c b/src/mm-plugin-base.c index 113303c5..7c49cd22 100644 --- a/src/mm-plugin-base.c +++ b/src/mm-plugin-base.c @@ -278,6 +278,28 @@ mm_plugin_base_supports_task_get_probed_product (MMPluginBaseSupportsTask *task) return MM_PLUGIN_BASE_SUPPORTS_TASK_GET_PRIVATE (task)->probed_product; } +gboolean +mm_plugin_base_supports_task_propagate_cached (MMPluginBaseSupportsTask *task) +{ + MMPluginBaseSupportsTaskPrivate *priv; + + g_return_val_if_fail (task != NULL, FALSE); + g_return_val_if_fail (MM_IS_PLUGIN_BASE_SUPPORTS_TASK (task), FALSE); + + priv = MM_PLUGIN_BASE_SUPPORTS_TASK_GET_PRIVATE (task); + + g_free (priv->probed_vendor); + g_free (priv->probed_product); + + /* Returns TRUE if a previous supports task already cached probing results. + * It will also store a copy of the cached result. */ + return mm_plugin_base_get_cached_probe_result (priv->plugin, + priv->port, + &priv->probed_caps, + &priv->probed_vendor, + &priv->probed_product); +} + void mm_plugin_base_supports_task_complete (MMPluginBaseSupportsTask *task, guint32 level) @@ -1049,27 +1071,9 @@ mm_plugin_base_probe_port (MMPluginBase *self, } gboolean -mm_plugin_base_get_cached_port_capabilities (MMPluginBase *self, - GUdevDevice *port, - guint32 *capabilities) -{ - MMPluginBaseProbedInfo *info; - - if (g_hash_table_lookup_extended (probed_info, - g_udev_device_get_name (port), - NULL, - (gpointer *)&info)) { - *capabilities = info->capabilities; - return TRUE; - } - - *capabilities = 0; - return FALSE; -} - -gboolean -mm_plugin_base_get_cached_product_info (MMPluginBase *self, +mm_plugin_base_get_cached_probe_result (MMPluginBase *self, GUdevDevice *port, + guint32 *capabilities, gchar **vendor, gchar **product) { @@ -1079,6 +1083,8 @@ mm_plugin_base_get_cached_product_info (MMPluginBase *self, g_udev_device_get_name (port), NULL, (gpointer *)&info)) { + if (capabilities) + *capabilities = info->capabilities; if (vendor) *vendor = (info->vendor ? g_strdup (info->vendor) : NULL); if (product) @@ -1086,6 +1092,8 @@ mm_plugin_base_get_cached_product_info (MMPluginBase *self, return TRUE; } + if (capabilities) + *capabilities = 0; if (vendor) *vendor = NULL; if (product) |