diff options
Diffstat (limited to 'libmm-glib')
-rw-r--r-- | libmm-glib/mm-firmware-properties.c | 70 | ||||
-rw-r--r-- | libmm-glib/mm-firmware-properties.h | 6 | ||||
-rw-r--r-- | libmm-glib/mm-modem-firmware.c | 18 | ||||
-rw-r--r-- | libmm-glib/mm-modem-firmware.h | 4 |
4 files changed, 31 insertions, 67 deletions
diff --git a/libmm-glib/mm-firmware-properties.c b/libmm-glib/mm-firmware-properties.c index 44097e4a..57cb8486 100644 --- a/libmm-glib/mm-firmware-properties.c +++ b/libmm-glib/mm-firmware-properties.c @@ -33,15 +33,13 @@ G_DEFINE_TYPE (MMFirmwareProperties, mm_firmware_properties, G_TYPE_OBJECT); -#define PROPERTY_NAME "name" -#define PROPERTY_VERSION "version" +#define PROPERTY_UNIQUE_ID "unique-id" #define PROPERTY_IMAGE_TYPE "image-type" struct _MMFirmwarePropertiesPrivate { /* Mandatory parameters */ MMFirmwareImageType image_type; - gchar *name; - gchar *version; + gchar *unique_id; }; static MMFirmwareProperties *firmware_properties_new_empty (void); @@ -49,37 +47,19 @@ static MMFirmwareProperties *firmware_properties_new_empty (void); /*****************************************************************************/ /** - * mm_firmware_properties_get_name: + * mm_firmware_properties_get_unique_id: * @self: A #MMFirmwareProperties. * - * Gets the unique name of the firmare image. + * Gets the unique ID of the firmare image. * - * Returns: (transfer none): The name of the image. Do not free the returned value, it is owned by @self. + * Returns: (transfer none): The ID of the image. Do not free the returned value, it is owned by @self. */ const gchar * -mm_firmware_properties_get_name (MMFirmwareProperties *self) +mm_firmware_properties_get_unique_id (MMFirmwareProperties *self) { g_return_val_if_fail (MM_IS_FIRMWARE_PROPERTIES (self), NULL); - return self->priv->name; -} - -/*****************************************************************************/ - -/** - * mm_firmware_properties_get_version: - * @self: A #MMFirmwareProperties. - * - * Gets the version string of the firmare image. - * - * Returns: (transfer none): The version of the image. Do not free the returned value, it is owned by @self. - */ -const gchar * -mm_firmware_properties_get_version (MMFirmwareProperties *self) -{ - g_return_val_if_fail (MM_IS_FIRMWARE_PROPERTIES (self), NULL); - - return self->priv->version; + return self->priv->unique_id; } /*****************************************************************************/ @@ -127,13 +107,8 @@ mm_firmware_properties_get_dictionary (MMFirmwareProperties *self) g_variant_builder_add (&builder, "{sv}", - PROPERTY_NAME, - g_variant_new_string (self->priv->name)); - - g_variant_builder_add (&builder, - "{sv}", - PROPERTY_VERSION, - g_variant_new_string (self->priv->version)); + PROPERTY_UNIQUE_ID, + g_variant_new_string (self->priv->unique_id)); g_variant_builder_add (&builder, "{sv}", @@ -151,12 +126,9 @@ consume_variant (MMFirmwareProperties *self, GVariant *value, GError **error) { - if (g_str_equal (key, PROPERTY_NAME)) { - g_free (self->priv->name); - self->priv->name = g_variant_dup_string (value, NULL); - } else if (g_str_equal (key, PROPERTY_VERSION)) { - g_free (self->priv->version); - self->priv->version = g_variant_dup_string (value, NULL); + if (g_str_equal (key, PROPERTY_UNIQUE_ID)) { + g_free (self->priv->unique_id); + self->priv->unique_id = g_variant_dup_string (value, NULL); } else if (g_str_equal (key, PROPERTY_IMAGE_TYPE)) self->priv->image_type = g_variant_get_uint32 (value); else { @@ -230,8 +202,7 @@ mm_firmware_properties_new_from_dictionary (GVariant *dictionary, } /* If mandatory properties missing, destroy the object */ - if (!self->priv->name || - !self->priv->version || + if (!self->priv->unique_id || self->priv->image_type == MM_FIRMWARE_IMAGE_TYPE_UNKNOWN) { g_set_error (error, MM_CORE_ERROR, @@ -250,8 +221,7 @@ mm_firmware_properties_new_from_dictionary (GVariant *dictionary, /** * mm_firmware_properties_new: * @image_type: A #MMFirmwareImageType specifying the type of the image. - * @name: The unique name of the image. - * @version: The version of the image. + * @unique_id: The unique ID of the image. * * Creates a new #MMFirmwareProperties object with the properties specified. * @@ -259,19 +229,16 @@ mm_firmware_properties_new_from_dictionary (GVariant *dictionary, */ MMFirmwareProperties * mm_firmware_properties_new (MMFirmwareImageType image_type, - const gchar *name, - const gchar *version) + const gchar *unique_id) { MMFirmwareProperties *self; g_return_val_if_fail (image_type != MM_FIRMWARE_IMAGE_TYPE_UNKNOWN, NULL); - g_return_val_if_fail (name != NULL, NULL); - g_return_val_if_fail (version != NULL, NULL); + g_return_val_if_fail (unique_id != NULL, NULL); self = firmware_properties_new_empty (); self->priv->image_type = image_type; - self->priv->name = g_strdup (name); - self->priv->version = g_strdup (version); + self->priv->unique_id = g_strdup (unique_id); return self; } @@ -299,8 +266,7 @@ finalize (GObject *object) { MMFirmwareProperties *self = MM_FIRMWARE_PROPERTIES (object); - g_free (self->priv->name); - g_free (self->priv->version); + g_free (self->priv->unique_id); G_OBJECT_CLASS (mm_firmware_properties_parent_class)->finalize (object); } diff --git a/libmm-glib/mm-firmware-properties.h b/libmm-glib/mm-firmware-properties.h index eb7cae68..4cacfd68 100644 --- a/libmm-glib/mm-firmware-properties.h +++ b/libmm-glib/mm-firmware-properties.h @@ -55,8 +55,7 @@ struct _MMFirmwarePropertiesClass { GType mm_firmware_properties_get_type (void); -const gchar *mm_firmware_properties_get_name (MMFirmwareProperties *self); -const gchar *mm_firmware_properties_get_version (MMFirmwareProperties *self); +const gchar *mm_firmware_properties_get_unique_id (MMFirmwareProperties *self); MMFirmwareImageType mm_firmware_properties_get_image_type (MMFirmwareProperties *self); /*****************************************************************************/ @@ -67,8 +66,7 @@ MMFirmwareImageType mm_firmware_properties_get_image_type (MMFirmwareProperties defined (LIBMM_GLIB_COMPILATION) MMFirmwareProperties *mm_firmware_properties_new (MMFirmwareImageType image_type, - const gchar *name, - const gchar *version); + const gchar *unique_id); MMFirmwareProperties *mm_firmware_properties_new_from_dictionary (GVariant *dictionary, GError **error); diff --git a/libmm-glib/mm-modem-firmware.c b/libmm-glib/mm-modem-firmware.c index 0b059dd1..96f74951 100644 --- a/libmm-glib/mm-modem-firmware.c +++ b/libmm-glib/mm-modem-firmware.c @@ -105,7 +105,7 @@ mm_modem_firmware_select_finish (MMModemFirmware *self, /** * mm_modem_firmware_select: * @self: A #MMModemFirmware. - * @name: Unique name of the firmware image to select. + * @unique_id: Unique ID of the firmware image to select. * @cancellable: (allow-none): A #GCancellable or %NULL. * @callback: A #GAsyncReadyCallback to call when the request is satisfied or %NULL. * @user_data: User data to pass to @callback. @@ -122,21 +122,21 @@ mm_modem_firmware_select_finish (MMModemFirmware *self, */ void mm_modem_firmware_select (MMModemFirmware *self, - const gchar *name, + const gchar *unique_id, GCancellable *cancellable, GAsyncReadyCallback callback, gpointer user_data) { g_return_if_fail (MM_IS_MODEM_FIRMWARE (self)); - g_return_if_fail (name != NULL || name[0] == '\0'); + g_return_if_fail (unique_id != NULL || unique_id[0] == '\0'); - mm_gdbus_modem_firmware_call_select (MM_GDBUS_MODEM_FIRMWARE (self), name, cancellable, callback, user_data); + mm_gdbus_modem_firmware_call_select (MM_GDBUS_MODEM_FIRMWARE (self), unique_id, cancellable, callback, user_data); } /** * mm_modem_firmware_select_sync: * @self: A #MMModemFirmware. - * @name: Unique name of the firmware image to select. + * @unique_id: Unique ID of the firmware image to select. * @cancellable: (allow-none): A #GCancellable or %NULL. * @error: Return location for error or %NULL. * @@ -152,14 +152,14 @@ mm_modem_firmware_select (MMModemFirmware *self, */ gboolean mm_modem_firmware_select_sync (MMModemFirmware *self, - const gchar *name, + const gchar *unique_id, GCancellable *cancellable, GError **error) { g_return_val_if_fail (MM_IS_MODEM_FIRMWARE (self), FALSE); - g_return_val_if_fail (name != NULL || name[0] == '\0', FALSE); + g_return_val_if_fail (unique_id != NULL || unique_id[0] == '\0', FALSE); - return mm_gdbus_modem_firmware_call_select_sync (MM_GDBUS_MODEM_FIRMWARE (self), name, cancellable, error); + return mm_gdbus_modem_firmware_call_select_sync (MM_GDBUS_MODEM_FIRMWARE (self), unique_id, cancellable, error); } /*****************************************************************************/ @@ -218,7 +218,7 @@ build_results (const gchar *str_selected, *installed = g_list_append (*installed, firmware); if (str_selected && - g_str_equal (mm_firmware_properties_get_name (firmware), str_selected)) + g_str_equal (mm_firmware_properties_get_unique_id (firmware), str_selected)) *selected = g_object_ref (firmware); } } diff --git a/libmm-glib/mm-modem-firmware.h b/libmm-glib/mm-modem-firmware.h index fd8fbe8c..b600ad8b 100644 --- a/libmm-glib/mm-modem-firmware.h +++ b/libmm-glib/mm-modem-firmware.h @@ -82,7 +82,7 @@ gboolean mm_modem_firmware_list_sync (MMModemFirmware *self, GError **error); void mm_modem_firmware_select (MMModemFirmware *self, - const gchar *name, + const gchar *unique_id, GCancellable *cancellable, GAsyncReadyCallback callback, gpointer user_data); @@ -90,7 +90,7 @@ gboolean mm_modem_firmware_select_finish (MMModemFirmware *self, GAsyncResult *res, GError **error); gboolean mm_modem_firmware_select_sync (MMModemFirmware *self, - const gchar *name, + const gchar *unique_id, GCancellable *cancellable, GError **error); |