halapi: common: Change return type of hal_common_put_backend and backend->exit

Change the return type of hal_common_put_backend() and backend->exit()
from void to int in order to pass the error value.

Change-Id: I155a33e469511319303bd56d3866d3bbfbec786c
Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>

diff --git a/include/hal-common-interface.h b/include/hal-common-interface.h
index 56b1c3cba2668fbcfac0e5d3a3b6ef1cc11aaebd..39831b94e5762c5cc3fde811e7a0ff77e14a9636 100644 (file)
--- a/include/hal-common-interface.h
+++ b/include/hal-common-interface.h
@@ -34,7 +34,7 @@ typedef struct __hal_backend {
        const char *vendor;
        const unsigned int abi_version;
        int (*init) (void **data);
-       void (*exit) (void *data);
+       int (*exit) (void *data);
 } hal_backend;
 
 #ifdef __cplusplus

diff --git a/include/hal-common.h b/include/hal-common.h
index e6d025c501e6be536b8f1f4023d3a6531210d7b6..e25503bb936ffc1eb3ca9a9a121d3bcc83f0e963 100644 (file)
--- a/include/hal-common.h
+++ b/include/hal-common.h
@@ -128,8 +128,9 @@ int hal_common_get_backend(enum hal_module handle, void **data);
  * @brief Put the backend data according to the type of HAL module
  * @param[in] HAL module id among enum hal_moudle
  * @param[in] Data pointer where 'hal_backend_[module]_funcs' instance
+ * @return @c 0 on success, otherwise a negative error value
  */
-void hal_common_put_backend(enum hal_module handle, void *data);
+int hal_common_put_backend(enum hal_module handle, void *data);
 
 /**
  * @brief Check HAL ABI version whehter is suppored or not on current platform

diff --git a/src/hal-api-common.c b/src/hal-api-common.c
index 03cb953a6c423da9c0740995bf836e661d960a36..9be72fae30fe3640cf72c9779da48548bb6d5568 100644 (file)
--- a/src/hal-api-common.c
+++ b/src/hal-api-common.c
@@ -149,7 +149,7 @@ err:
 }
 
 EXPORT
-void hal_common_put_backend(enum hal_module module, void *data)
+int hal_common_put_backend(enum hal_module module, void *data)
 {
        hal_backend *backend = NULL;
        void *handle = NULL;
@@ -158,22 +158,28 @@ void hal_common_put_backend(enum hal_module module, void *data)
        /* Check parameter whether is valid or not */
        if (module <= HAL_MODULE_UNKNOWN || module >= HAL_MODULE_END) {
                _E("Invalid parameter of HAL module (%d)\n", module);
-               return;
+               return TIZEN_ERROR_INVALID_PARAMETER;
        }
 
        backend = hal_module_info[module].library_backend;
        handle = hal_module_info[module].library_handle;
 
-       if (backend) {
-               if (backend->exit)
-                       backend->exit(data);
-
-               _I("Put HAL backend: name(%s)/vendor(%s)\n",
-                       backend->name, backend->vendor);
+       if (backend && backend->exit) {
+               ret = backend->exit(data);
+               if (ret < 0) {
+                       _E("Failed to exit backend: name(%s)/vendor(%s)\n",
+                                               backend->name, backend->vendor);
+                       return TIZEN_ERROR_INVALID_PARAMETER;
+               }
        }
 
        if (handle)
                dlclose(handle);
+
+       _I("Put HAL backend: name(%s)/vendor(%s)\n",
+                       backend->name, backend->vendor);
+
+       return TIZEN_ERROR_NONE;
 }
 
 EXPORT