From: Youngjae Cho Date: Mon, 17 Feb 2025 04:41:57 +0000 (+0900) Subject: common: halcc: Remove xml tag X-Git-Tag: accepted/tizen/unified/20250226.102358~2 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=54420aabc0a89c29e882621304f606e6734d29f8;p=platform%2Fhal%2Fapi%2Fcommon.git common: halcc: Remove xml tag It was orignally designed for specifying instances that could come as an instantiated backend with designated name and id. However, this scheme has been abandoned and now backend cannot specify its name and id. Therefore, removed and its related operations. Change-Id: I1516670feaaa88d7e24fd51b961e638073093911 Signed-off-by: Youngjae Cho --- diff --git a/src/hal-api-compatibility-checker-object.c b/src/hal-api-compatibility-checker-object.c index 918a230..13b13e9 100644 --- a/src/hal-api-compatibility-checker-object.c +++ b/src/hal-api-compatibility-checker-object.c @@ -35,17 +35,11 @@ typedef struct halcc_version { int minor; } halcc_version; -typedef struct halcc_interface { - char *name; - char *instance_id; -} halcc_interface; - typedef struct halcc_hal { char *name; halcc_version version_list[HALCC_NUM_VERSION_LIST_MAX]; int num_version_list; halcc_transport_e transport; - GHashTable *interfaces; } halcc_hal; typedef struct halcc_manifest { @@ -117,91 +111,6 @@ static void hashtable_hal_remove(GHashTable *hal_table, const char *name) g_hash_table_remove(hal_table, name); } -static guint interface_hash(gconstpointer key) -{ - const halcc_interface *interface = (const halcc_interface *) key; - char interface_str[HALCC_NAME_MAX * 2] = { 0 , }; - - assert(interface); - assert(interface->name); - - if (interface->instance_id) - snprintf(interface_str, sizeof(interface_str), "%s.%s", interface->name, interface->instance_id); - else - snprintf(interface_str, sizeof(interface_str), "%s.default", interface->name); - - return g_str_hash(interface_str); -} - -static gboolean interface_hash_equal(gconstpointer a, gconstpointer b) -{ - const halcc_interface *interface = (const halcc_interface *) a; - const halcc_interface *key = (const halcc_interface *) b; - - assert(interface); - assert(key); - - return strncmp(interface->name, key->name, strlen(interface->name) + 1) == 0; -} - -static void interface_hash_value_free(gpointer data) -{ - halcc_interface *interface = (halcc_interface *) data; - - halcc_interface_free(interface); -} - -static GHashTable* hashtable_interface_new(void) -{ - return g_hash_table_new_full(interface_hash, - interface_hash_equal, NULL, interface_hash_value_free); -} - -static int hashtable_interface_insert(GHashTable *interface_table, halcc_interface *interface) -{ - g_hash_table_insert(interface_table, interface, interface); - - return 0; -} - -/* -static halcc_interface* hashtable_interface_lookup(GHashTable *interface_table, - const char *name, const char *instance_id) -{ - halcc_interface key; - - if (!interface_table || !name) { - _E("Invalid parameter\n"); - return NULL; - } - - key = (halcc_interface) { - .name = (char *) name, - .instance_id = (char *) instance_id, - }; - - return g_hash_table_lookup(interface_table, &key); -} -*/ - -static void hashtable_interface_remove(GHashTable *interface_table, - const char *name, const char *instance_id) -{ - halcc_interface key; - - if (!interface_table || !name) { - _E("Invalid parameter\n"); - return; - } - - key = (halcc_interface) { - .name = (char *) name, - .instance_id = (char *) instance_id, - }; - - g_hash_table_remove(interface_table, &key); -} - int halcc_manifest_new(halcc_manifest **manifest) { halcc_manifest *m; @@ -345,7 +254,6 @@ int halcc_hal_new(halcc_hal **hal) } h->transport = HALCC_TRANSPORT_DEFAULT; - h->interfaces = hashtable_interface_new(); *hal = g_steal_pointer(&h); @@ -360,7 +268,6 @@ void halcc_hal_free(halcc_hal *hal) } free(g_steal_pointer(&hal->name)); - g_hash_table_destroy(g_steal_pointer(&hal->interfaces)); free(hal); } @@ -485,37 +392,6 @@ int halcc_hal_get_transport(halcc_hal *hal, halcc_transport_e *transport) return 0; } -int halcc_hal_add_interface(halcc_hal *hal, halcc_interface *interface) -{ - if (!hal || !interface) { - _E("Invalid parameter\n"); - return -EINVAL; - } - - return hashtable_interface_insert(hal->interfaces, interface); -} - -void halcc_hal_remove_interface(halcc_hal *hal, - const char *interface_name, const char *instance_id) -{ - if (!hal || !interface_name) { - _E("Invalid parameter\n"); - return; - } - - hashtable_interface_remove(hal->interfaces, interface_name, instance_id); -} - -void halcc_hal_foreach_interface(halcc_hal *hal, halcc_iter_cb cb, void *user_data) -{ - if (!hal || !hal->interfaces || !cb) { - _E("Invalid parameter\n"); - return; - } - - hashtable_foreach(hal->interfaces, cb, user_data); -} - bool halcc_hal_is_compatible_with_version(halcc_hal *hal, int major, int minor) { if (!hal) { @@ -535,99 +411,3 @@ bool halcc_hal_is_compatible_with_version(halcc_hal *hal, int major, int minor) return false; } - -int halcc_interface_new(halcc_interface **interface) -{ - halcc_interface *iface; - - if (!interface) { - _E("Invalid parameter\n"); - return -EINVAL; - } - - iface = calloc(1, sizeof(halcc_interface)); - if (!iface) { - _E("Failed to allocate halcc_interface\n"); - return -ENOMEM; - } - - *interface = g_steal_pointer(&iface); - - return 0; -} - -void halcc_interface_free(halcc_interface *interface) -{ - if (!interface) { - _E("Invalid parameter\n"); - return; - } - - free(g_steal_pointer(&interface->name)); - free(g_steal_pointer(&interface->instance_id)); - free(g_steal_pointer(&interface)); -} - -int halcc_interface_set_name(halcc_interface *interface, const char *interface_name) -{ - char *n; - - if (!interface || !interface_name) { - _E("Invalid parameter\n"); - return -EINVAL; - } - - n = strndup(interface_name, HALCC_NAME_MAX); - if (!n) { - _E("Failed to allocate interface_name\n"); - return -ENOMEM; - } - - interface->name = g_steal_pointer(&n); - - return 0; -} - -int halcc_interface_get_name(halcc_interface *interface, const char **interface_name) -{ - if (!interface || !interface_name) { - _E("Invalid parameter\n"); - return -EINVAL; - } - - *interface_name = interface->name; - - return 0; -} - -int halcc_interface_set_instance_id(halcc_interface *interface, const char *instance_id) -{ - char *id = NULL; - - if (!interface || !instance_id) { - _E("Invalid parameter\n"); - return -EINVAL; - } - - id = strndup(instance_id, HALCC_NAME_MAX); - if (!id) { - _E("Failed to allocate instance_id\n"); - return -ENOMEM; - } - - interface->instance_id = g_steal_pointer(&id); - - return 0; -} - -int halcc_interface_get_instance_id(halcc_interface *interface, const char **instance_id) -{ - if (!interface || !instance_id) { - _E("Invalid parameter\n"); - return -EINVAL; - } - - *instance_id = interface->instance_id; - - return 0; -} diff --git a/src/hal-api-compatibility-checker-object.h b/src/hal-api-compatibility-checker-object.h index 43492c1..4a016a4 100644 --- a/src/hal-api-compatibility-checker-object.h +++ b/src/hal-api-compatibility-checker-object.h @@ -38,7 +38,6 @@ typedef enum halcc_transport_e { typedef struct halcc_manifest halcc_manifest; typedef struct halcc_hal halcc_hal; -typedef struct halcc_interface halcc_interface; typedef void (*halcc_iter_cb) (void *, void *); @@ -61,19 +60,8 @@ int halcc_hal_get_version_list(halcc_hal *hal, int version_list[][2], int max_num_version_list, int *num_version_list); int halcc_hal_set_transport(halcc_hal *hal, halcc_transport_e transport); int halcc_hal_get_transport(halcc_hal *hal, halcc_transport_e *transport); -int halcc_hal_add_interface(halcc_hal *hal, halcc_interface *interface); -void halcc_hal_remove_interface(halcc_hal *hal, - const char *interface_name, const char *instance_id); -void halcc_hal_foreach_interface(halcc_hal *hal, halcc_iter_cb cb, void *user_data); bool halcc_hal_is_compatible_with_version(halcc_hal *hal, int major, int minor); -int halcc_interface_new(halcc_interface **interface); -void halcc_interface_free(halcc_interface *interface); -int halcc_interface_set_name(halcc_interface *interface, const char *interface_name); -int halcc_interface_get_name(halcc_interface *interface, const char **interface_name); -int halcc_interface_set_instance_id(halcc_interface *interface, const char *instance_id); -int halcc_interface_get_instance_id(halcc_interface *interface, const char **instance_id); - #ifdef __cplusplus } #endif diff --git a/src/hal-api-compatibility-checker-parser.c b/src/hal-api-compatibility-checker-parser.c index 8b9ee74..f923fc2 100644 --- a/src/hal-api-compatibility-checker-parser.c +++ b/src/hal-api-compatibility-checker-parser.c @@ -38,29 +38,6 @@ static void free_xmlstr(unsigned char **str) xmlFree(*str); } -static int parse_interface(xmlNode *node, halcc_interface *interface) -{ - assert(node); - assert(interface); - - if (!xmlStrEqual(node->name, "interface")) { - _E("Invalid interface node, %s\n", node->name); - return -EINVAL; - } - - for (xmlNode *child = node->children; child; child = child->next) { - if (xmlStrEqual(child->name, "name")) { - __xmlchar__ xmlChar *name = xmlNodeGetContent(child); - halcc_interface_set_name(interface, name); - } else if (xmlStrEqual(child->name, "instance")) { - __xmlchar__ xmlChar *instance = xmlNodeGetContent(child); - halcc_interface_set_instance_id(interface, (char *) instance); - } - } - - return 0; -} - static int parse_hal(xmlNode *node, halcc_manifest *manifest) { int ret = 0; @@ -131,24 +108,6 @@ static int parse_hal(xmlNode *node, halcc_manifest *manifest) if (ret != 0) _E("Failed to halcc_hal_set_transport(), %s, ret=%d\n", transport, ret); - } else if (xmlStrEqual(child->name, "interface")) { - halcc_interface *iface; - - ret = halcc_interface_new(&iface); - if (ret != 0) { - _E("Failed to halcc_interface_new(), ret=%d\n", ret); - continue; - } - - ret = parse_interface(child, iface); - if (ret != 0) { - _E("Failed to parse_interface(), ret=%d\n", ret); - halcc_interface_free(iface); - continue; - } - - halcc_hal_add_interface(hal, iface); - iface = NULL; } }