package version up to 1.16.13
[platform/core/uifw/libtdm.git] / src / tdm_output.c
index 53ea938..9b9d200 100644 (file)
@@ -9,7 +9,7 @@
  *          Taeheon Kim <th908.kim@samsung.com>,
  *          YoungJun Cho <yj44.cho@samsung.com>,
  *          SooChan Lim <sc1.lim@samsung.com>,
- *          Boram Park <sc1.lim@samsung.com>
+ *          Boram Park <boram1288.park@samsung.com>
  *
  * Permission is hereby granted, free of charge, to any person obtaining a
  * copy of this software and associated documentation files (the
@@ -45,7 +45,7 @@
        tdm_private_display *private_display; \
        tdm_private_output *private_output; \
        tdm_error ret = TDM_ERROR_NONE; /* default TDM_ERROR_NONE */\
-       TDM_RETURN_VAL_IF_FAIL(tdm_output_is_valid(output), TDM_ERROR_INVALID_PARAMETER); \
+       TDM_RETURN_VAL_IF_FAIL(output != NULL, TDM_ERROR_INVALID_PARAMETER); \
        private_output = (tdm_private_output*)output; \
        private_display = private_output->private_display
 
@@ -53,7 +53,7 @@
        tdm_private_display *private_display; \
        tdm_private_output *private_output; \
        tdm_error ret = TDM_ERROR_NONE; /* default TDM_ERROR_NONE */\
-       TDM_RETURN_VAL_IF_FAIL_WITH_ERROR(tdm_output_is_valid(output), TDM_ERROR_INVALID_PARAMETER, NULL); \
+       TDM_RETURN_VAL_IF_FAIL_WITH_ERROR(output != NULL, TDM_ERROR_INVALID_PARAMETER, NULL); \
        private_output = (tdm_private_output*)output; \
        private_display = private_output->private_display
 
@@ -146,28 +146,30 @@ tdm_output_init(tdm_private_display *private_display)
 {
        tdm_thread_cb_set_find_func(TDM_THREAD_CB_OUTPUT_COMMIT, tdm_display_find_output_stamp);
        tdm_thread_cb_set_find_func(TDM_THREAD_CB_OUTPUT_VBLANK, tdm_display_find_output_stamp);
-       tdm_thread_cb_set_find_func(TDM_THREAD_CB_OUTPUT_CHANGE, tdm_display_find_output_stamp);
+       tdm_thread_cb_set_find_func(TDM_THREAD_CB_OUTPUT_STATUS, tdm_display_find_output_stamp);
+       tdm_thread_cb_set_find_func(TDM_THREAD_CB_OUTPUT_DPMS, tdm_display_find_output_stamp);
+
 
        return TDM_ERROR_NONE;
 }
 
-EXTERN tdm_backend *
-tdm_output_get_backend(tdm_output *output, tdm_error *error)
+EXTERN tdm_module *
+tdm_output_get_backend_module(tdm_output *output, tdm_error *error)
 {
-       tdm_private_backend *private_backend;
+       tdm_private_module *private_module;
 
        OUTPUT_FUNC_ENTRY_ERROR();
 
        _pthread_mutex_lock(&private_display->lock);
 
-       private_backend = private_output->private_backend;
+       private_module = private_output->private_module;
 
        if (error)
                *error = TDM_ERROR_NONE;
 
        _pthread_mutex_unlock(&private_display->lock);
 
-       return private_backend;
+       return private_module;
 }
 
 EXTERN tdm_error
@@ -222,52 +224,76 @@ tdm_output_get_conn_status(tdm_output *output, tdm_output_conn_status *status)
        return ret;
 }
 
-/* LCOV_EXCL_START */
-static void
-_tdm_output_update(tdm_output *output_backend, void *user_data)
-{
-       tdm_private_output *private_output = user_data;
-       tdm_error ret;
-
-       TDM_RETURN_IF_FAIL(private_output);
-
-       ret = tdm_display_update_output(private_output->private_backend, output_backend, private_output->pipe);
-       TDM_RETURN_IF_FAIL(ret == TDM_ERROR_NONE);
-}
-/* LCOV_EXCL_STOP */
-
 INTERN void
 tdm_output_thread_cb_change(tdm_private_display *private_display, void *object, tdm_thread_cb_base *cb_base, void *user_data)
 {
        tdm_private_output *private_output = object;
-       tdm_thread_cb_output_change *output_change = (tdm_thread_cb_output_change *)cb_base;
        tdm_private_output_change_handler *change_handler = user_data;
+       tdm_output_change_type type = TDM_OUTPUT_CHANGE_CONNECTION;
+       tdm_value value = {.u32 = 0 };
 
        TDM_RETURN_IF_FAIL(TDM_MUTEX_IS_LOCKED());
 
        assert(change_handler->owner_tid == syscall(SYS_gettid));
 
+       if (cb_base->type == TDM_THREAD_CB_OUTPUT_STATUS) {
+               tdm_thread_cb_output_status *output_status = (tdm_thread_cb_output_status *)cb_base;
+               type = TDM_OUTPUT_CHANGE_CONNECTION;
+               value.u32 = output_status->status;
+       } else if (cb_base->type == TDM_THREAD_CB_OUTPUT_DPMS) {
+               tdm_thread_cb_output_dpms *output_dpms = (tdm_thread_cb_output_dpms *)cb_base;
+               type = TDM_OUTPUT_CHANGE_DPMS;
+               value.u32 = output_dpms->dpms;
+       } else {
+               TDM_NEVER_GET_HERE();
+               return;
+       }
+
        _pthread_mutex_unlock(&private_display->lock);
-       change_handler->func(private_output, output_change->type, output_change->value, change_handler->user_data);
+       change_handler->func(private_output, type, value, change_handler->user_data);
        _pthread_mutex_lock(&private_display->lock);
+
+}
+
+static tdm_error
+_tdm_output_call_thread_cb_status(tdm_private_output *private_output, tdm_output_conn_status status)
+{
+       tdm_thread_cb_output_status output_status;
+       pid_t caller_tid = syscall(SYS_gettid);
+       tdm_error ret;
+
+       memset(&output_status, 0, sizeof output_status);
+       output_status.base.type = TDM_THREAD_CB_OUTPUT_STATUS;
+       output_status.base.length = sizeof output_status;
+       output_status.base.object_stamp = private_output->stamp;
+       output_status.base.data = NULL;
+       if (tdm_thread_in_display_thread(caller_tid))
+               output_status.base.sync = 0;
+       else
+               output_status.base.sync = 1;
+       output_status.status = status;
+
+       ret = tdm_thread_cb_call(private_output, &output_status.base, 1);
+       TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
+
+       return TDM_ERROR_NONE;
 }
 
 static tdm_error
-_tdm_output_call_thread_cb_change(tdm_private_output *private_output, tdm_output_change_type type, tdm_value value)
+_tdm_output_call_thread_cb_dpms(tdm_private_output *private_output, tdm_output_dpms dpms)
 {
-       tdm_thread_cb_output_change output_change;
+       tdm_thread_cb_output_dpms output_dpms;
        tdm_error ret;
 
-       memset(&output_change, 0, sizeof output_change);
-       output_change.base.type = TDM_THREAD_CB_OUTPUT_CHANGE;
-       output_change.base.length = sizeof output_change;
-       output_change.base.object_stamp = private_output->stamp;
-       output_change.base.data = NULL;
-       output_change.base.sync = 1;
-       output_change.type = type;
-       output_change.value = value;
+       memset(&output_dpms, 0, sizeof output_dpms);
+       output_dpms.base.type = TDM_THREAD_CB_OUTPUT_DPMS;
+       output_dpms.base.length = sizeof output_dpms;
+       output_dpms.base.object_stamp = private_output->stamp;
+       output_dpms.base.data = NULL;
+       output_dpms.base.sync = 0;
+       output_dpms.dpms = dpms;
 
-       ret = tdm_thread_cb_call(private_output, &output_change.base);
+       ret = tdm_thread_cb_call(private_output, &output_dpms.base, 1);
        TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
 
        return TDM_ERROR_NONE;
@@ -277,16 +303,21 @@ INTERN void
 tdm_output_cb_status(tdm_output *output_backend, tdm_output_conn_status status, void *user_data)
 {
        tdm_private_output *private_output = user_data;
-       tdm_value value;
        tdm_error ret;
 
-       TDM_INFO("output(%d) main %s", private_output->pipe, tdm_status_str(status));
+       TDM_RETURN_IF_FAIL(private_output);
 
-       _tdm_output_update(output_backend, user_data);
+       TDM_INFO("output(%d) %s", private_output->pipe, tdm_status_str(status));
 
-       value.u32 = status;
+       if ((private_output->caps.status == TDM_OUTPUT_CONN_STATUS_DISCONNECTED && status != TDM_OUTPUT_CONN_STATUS_DISCONNECTED) ||
+               (private_output->caps.status != TDM_OUTPUT_CONN_STATUS_DISCONNECTED && status == TDM_OUTPUT_CONN_STATUS_DISCONNECTED)) {
+               ret = tdm_display_update_output(private_output->private_module, output_backend, private_output->pipe, 1);
+               TDM_RETURN_IF_FAIL(ret == TDM_ERROR_NONE);
+       } else {
+               private_output->caps.status = status;
+       }
 
-       ret = _tdm_output_call_thread_cb_change(private_output, TDM_OUTPUT_CHANGE_CONNECTION, value);
+       ret = _tdm_output_call_thread_cb_status(private_output, status);
        TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
 }
 
@@ -294,7 +325,6 @@ INTERN void
 tdm_output_cb_dpms(tdm_output *output_backend, tdm_output_dpms dpms, void *user_data)
 {
        tdm_private_output *private_output = user_data;
-       tdm_value value;
        tdm_error ret;
 
        TDM_INFO("output(%d) %s", private_output->pipe, tdm_status_str(dpms));
@@ -303,9 +333,7 @@ tdm_output_cb_dpms(tdm_output *output_backend, tdm_output_dpms dpms, void *user_
        private_output->waiting_dpms_change = 0;
        TDM_INFO("output(%d) dpms async '%s' done", private_output->pipe, tdm_dpms_str(dpms));
 
-       value.u32 = dpms;
-
-       ret = _tdm_output_call_thread_cb_change(private_output, TDM_OUTPUT_CHANGE_DPMS, value);
+       ret = _tdm_output_call_thread_cb_dpms(private_output, dpms);
        TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
 }
 
@@ -314,13 +342,21 @@ tdm_output_add_change_handler(tdm_output *output,
                                                          tdm_output_change_handler func,
                                                          void *user_data)
 {
-       tdm_private_output_change_handler *change_handler;
+       tdm_private_output_change_handler *change_handler = NULL;
        OUTPUT_FUNC_ENTRY();
 
        TDM_RETURN_VAL_IF_FAIL(func != NULL, TDM_ERROR_INVALID_PARAMETER);
 
        _pthread_mutex_lock(&private_display->lock);
 
+       LIST_FOR_EACH_ENTRY(change_handler, &private_output->change_handler_list, link) {
+               if (change_handler->func == func && change_handler->user_data == user_data) {
+                       TDM_ERR("can't add twice");
+                       _pthread_mutex_unlock(&private_display->lock);
+                       return TDM_ERROR_BAD_REQUEST;
+               }
+       }
+
        change_handler = calloc(1, sizeof(tdm_private_output_change_handler));
        if (!change_handler) {
                /* LCOV_EXCL_START */
@@ -330,10 +366,21 @@ tdm_output_add_change_handler(tdm_output *output,
                /* LCOV_EXCL_STOP */
        }
 
-       ret = tdm_thread_cb_add(private_output, TDM_THREAD_CB_OUTPUT_CHANGE, NULL, tdm_output_thread_cb_change, change_handler);
+       ret = tdm_thread_cb_add(private_output, TDM_THREAD_CB_OUTPUT_DPMS, NULL, tdm_output_thread_cb_change, change_handler);
+       if (ret != TDM_ERROR_NONE) {
+               /* LCOV_EXCL_START */
+               TDM_ERR("tdm_thread_cb_add failed");
+               free(change_handler);
+               _pthread_mutex_unlock(&private_display->lock);
+               return TDM_ERROR_OPERATION_FAILED;
+               /* LCOV_EXCL_STOP */
+       }
+
+       ret = tdm_thread_cb_add(private_output, TDM_THREAD_CB_OUTPUT_STATUS, NULL, tdm_output_thread_cb_change, change_handler);
        if (ret != TDM_ERROR_NONE) {
                /* LCOV_EXCL_START */
                TDM_ERR("tdm_thread_cb_add failed");
+               tdm_thread_cb_remove(private_output, TDM_THREAD_CB_OUTPUT_DPMS, NULL, tdm_output_thread_cb_change, change_handler);
                free(change_handler);
                _pthread_mutex_unlock(&private_display->lock);
                return TDM_ERROR_OPERATION_FAILED;
@@ -361,7 +408,7 @@ tdm_output_remove_change_handler(tdm_output *output,
        tdm_private_output *private_output;
        tdm_private_output_change_handler *change_handler = NULL, *hh = NULL;
 
-       TDM_RETURN_IF_FAIL(tdm_output_is_valid(output));
+       TDM_RETURN_IF_FAIL(output != NULL);
        TDM_RETURN_IF_FAIL(func != NULL);
 
        private_output = (tdm_private_output*)output;
@@ -373,7 +420,8 @@ tdm_output_remove_change_handler(tdm_output *output,
                if (change_handler->func != func || change_handler->user_data != user_data)
                        continue;
 
-               tdm_thread_cb_remove(private_output, TDM_THREAD_CB_OUTPUT_CHANGE, NULL, tdm_output_thread_cb_change, change_handler);
+               tdm_thread_cb_remove(private_output, TDM_THREAD_CB_OUTPUT_DPMS, NULL, tdm_output_thread_cb_change, change_handler);
+               tdm_thread_cb_remove(private_output, TDM_THREAD_CB_OUTPUT_STATUS, NULL, tdm_output_thread_cb_change, change_handler);
 
                LIST_DEL(&change_handler->link);
                free(change_handler);
@@ -536,21 +584,8 @@ tdm_output_get_cursor_available_size(tdm_output *output, int *min_w, int *min_h,
 
        _pthread_mutex_lock(&private_display->lock);
 
-       if (!tdm_backend_check_module_abi(private_output->private_backend, 1, 5)) {
-
-               if (min_w)
-                       *min_w = -1;
-               if (min_h)
-                       *min_h = -1;
-               if (max_w)
-                       *max_w = -1;
-               if (max_h)
-                       *max_h = -1;
-               if (preferred_align)
-                       *preferred_align = -1;
-
+       if (!tdm_module_check_abi(private_output->private_module, 1, 5)) {
                _pthread_mutex_unlock(&private_display->lock);
-
                return TDM_ERROR_BAD_REQUEST;
        }
 
@@ -643,14 +678,14 @@ tdm_output_get_primary_index(tdm_output *output, int *index)
 EXTERN tdm_error
 tdm_output_set_property(tdm_output *output, unsigned int id, tdm_value value)
 {
-       tdm_private_backend *private_backend;
+       tdm_private_module *private_module;
        tdm_func_output *func_output;
        OUTPUT_FUNC_ENTRY();
 
        _pthread_mutex_lock(&private_display->lock);
 
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->func_output;
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
 
        if (!func_output->output_set_property) {
                /* LCOV_EXCL_START */
@@ -671,7 +706,7 @@ tdm_output_set_property(tdm_output *output, unsigned int id, tdm_value value)
 EXTERN tdm_error
 tdm_output_get_property(tdm_output *output, unsigned int id, tdm_value *value)
 {
-       tdm_private_backend *private_backend;
+       tdm_private_module *private_module;
        tdm_func_output *func_output;
        OUTPUT_FUNC_ENTRY();
 
@@ -679,8 +714,8 @@ tdm_output_get_property(tdm_output *output, unsigned int id, tdm_value *value)
 
        _pthread_mutex_lock(&private_display->lock);
 
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->func_output;
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
 
        if (!func_output->output_get_property) {
                /* LCOV_EXCL_START */
@@ -777,7 +812,10 @@ _tdm_output_cb_vblank(tdm_output *output_backend, unsigned int sequence,
 
        vblank_handler->sent_to_frontend = 1;
 
-       ret = tdm_thread_cb_call(vblank_handler->private_output, &output_vblank.base);
+       if (tdm_debug_module & TDM_DEBUG_COMMIT)
+               TDM_INFO("output(%d) wait_vblank: handler(%p)", vblank_handler->private_output->pipe, vblank_handler);
+
+       ret = tdm_thread_cb_call(vblank_handler->private_output, &output_vblank.base, 1);
        TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
 }
 
@@ -853,7 +891,7 @@ _tdm_output_cb_commit(tdm_output *output_backend, unsigned int sequence,
        output_commit.tv_sec = tv_sec;
        output_commit.tv_usec = tv_usec;
 
-       ret = tdm_thread_cb_call(private_output, &output_commit.base);
+       ret = tdm_thread_cb_call(private_output, &output_commit.base, 1);
        TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
 }
 
@@ -866,15 +904,15 @@ _tdm_output_wait_vblank(tdm_private_output *private_output, int interval, int sy
                                                tdm_output_vblank_handler func, void *user_data,
                                                unsigned int add_front)
 {
-       tdm_private_backend *private_backend;
+       tdm_private_module *private_module;
        tdm_func_output *func_output;
        tdm_private_output_vblank_handler *vblank_handler = NULL, *v = NULL;
        unsigned int skip_request = 0;
        pid_t tid = syscall(SYS_gettid);
        tdm_error ret = TDM_ERROR_NONE;
 
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->func_output;
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
 
        /* interval SHOULD be at least 1 */
        if (interval <= 0)
@@ -959,6 +997,7 @@ tdm_output_wait_vblank(tdm_output *output, int interval, int sync,
                                           tdm_output_vblank_handler func, void *user_data)
 {
        OUTPUT_FUNC_ENTRY();
+       TDM_RETURN_VAL_IF_FAIL(interval > 0, TDM_ERROR_INVALID_PARAMETER);
 
        _pthread_mutex_lock(&private_display->lock);
 
@@ -1070,18 +1109,18 @@ INTERN tdm_error
 tdm_output_commit_internal(tdm_output *output, int sync, tdm_output_commit_handler func, void *user_data)
 {
        tdm_private_output *private_output;
-       tdm_private_backend *private_backend;
+       tdm_private_module *private_module;
        tdm_func_output *func_output;
        tdm_private_output_commit_handler *output_commit_handler = NULL;
        tdm_private_layer *private_layer = NULL;
        tdm_output_dpms dpms_value = TDM_OUTPUT_DPMS_ON;
        tdm_error ret = TDM_ERROR_NONE;
 
-       TDM_RETURN_VAL_IF_FAIL(tdm_output_is_valid(output), TDM_ERROR_INVALID_PARAMETER);
+       TDM_RETURN_VAL_IF_FAIL(output != NULL, TDM_ERROR_INVALID_PARAMETER);
 
        private_output = (tdm_private_output*)output;
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->func_output;
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
 
        if (!func_output->output_commit) {
                /* LCOV_EXCL_START */
@@ -1094,35 +1133,33 @@ tdm_output_commit_internal(tdm_output *output, int sync, tdm_output_commit_handl
        TDM_RETURN_VAL_IF_FAIL(ret == TDM_ERROR_NONE, ret);
 
        if (!TDM_OUTPUT_DPMS_VSYNC_IS_OFF(dpms_value)) {
-               if (func) {
-                       if (!private_output->regist_commit_cb) {
-                               private_output->regist_commit_cb = 1;
-                               ret = func_output->output_set_commit_handler(private_output->output_backend, _tdm_output_cb_commit);
-                               TDM_GOTO_IF_FAIL(ret == TDM_ERROR_NONE, commit_failed);
-                       }
-
-                       output_commit_handler = calloc(1, sizeof(tdm_private_output_commit_handler));
-                       if (!output_commit_handler) {
-                               /* LCOV_EXCL_START */
-                               TDM_ERR("failed: alloc memory");
-                               return TDM_ERROR_OUT_OF_MEMORY;
-                               /* LCOV_EXCL_STOP */
-                       }
-
-                       ret = tdm_thread_cb_add(private_output, TDM_THREAD_CB_OUTPUT_COMMIT, output_commit_handler, _tdm_output_thread_cb_commit, NULL);
-                       if (ret != TDM_ERROR_NONE) {
-                               TDM_ERR("tdm_thread_cb_add failed");
-                               free(output_commit_handler);
-                               return ret;
-                       }
-
-                       LIST_ADDTAIL(&output_commit_handler->link, &private_output->output_commit_handler_list);
-                       output_commit_handler->private_output = private_output;
-                       output_commit_handler->func = func;
-                       output_commit_handler->user_data = user_data;
-                       output_commit_handler->owner_tid = syscall(SYS_gettid);
+               if (!private_output->regist_commit_cb) {
+                       private_output->regist_commit_cb = 1;
+                       ret = func_output->output_set_commit_handler(private_output->output_backend, _tdm_output_cb_commit);
+                       TDM_GOTO_IF_FAIL(ret == TDM_ERROR_NONE, commit_failed);
+               }
+
+               output_commit_handler = calloc(1, sizeof(tdm_private_output_commit_handler));
+               if (!output_commit_handler) {
+                       /* LCOV_EXCL_START */
+                       TDM_ERR("failed: alloc memory");
+                       return TDM_ERROR_OUT_OF_MEMORY;
+                       /* LCOV_EXCL_STOP */
                }
 
+               ret = tdm_thread_cb_add(private_output, TDM_THREAD_CB_OUTPUT_COMMIT, output_commit_handler, _tdm_output_thread_cb_commit, NULL);
+               if (ret != TDM_ERROR_NONE) {
+                       TDM_ERR("tdm_thread_cb_add failed");
+                       free(output_commit_handler);
+                       return ret;
+               }
+
+               LIST_ADDTAIL(&output_commit_handler->link, &private_output->output_commit_handler_list);
+               output_commit_handler->private_output = private_output;
+               output_commit_handler->func = func;
+               output_commit_handler->user_data = user_data;
+               output_commit_handler->owner_tid = syscall(SYS_gettid);
+
                ret = func_output->output_commit(private_output->output_backend, sync,
                                output_commit_handler);
                TDM_GOTO_IF_FAIL(ret == TDM_ERROR_NONE, commit_failed);
@@ -1139,6 +1176,15 @@ tdm_output_commit_internal(tdm_output *output, int sync, tdm_output_commit_handl
                if (!private_layer->waiting_buffer)
                        continue;
 
+               if (private_layer->committed_buffer) {
+                       tdm_layer_free_buffer(private_layer, private_layer->committed_buffer);
+                       private_layer->committed_buffer = NULL;
+                       if (tdm_debug_module & TDM_DEBUG_BUFFER)
+                               TDM_INFO("layer(%p) waiting_buffer(%p) committed_buffer(%p)",
+                                                private_layer, private_layer->waiting_buffer->buffer,
+                                                private_layer->committed_buffer);
+               }
+
                private_layer->committed_buffer = private_layer->waiting_buffer;
                private_layer->waiting_buffer = NULL;
                if (tdm_debug_module & TDM_DEBUG_BUFFER)
@@ -1215,7 +1261,7 @@ tdm_output_commit(tdm_output *output, int sync, tdm_output_commit_handler func,
 EXTERN tdm_error
 tdm_output_set_mode(tdm_output *output, const tdm_output_mode *mode)
 {
-       tdm_private_backend *private_backend;
+       tdm_private_module *private_module;
        tdm_func_output *func_output;
        OUTPUT_FUNC_ENTRY();
 
@@ -1223,8 +1269,8 @@ tdm_output_set_mode(tdm_output *output, const tdm_output_mode *mode)
 
        _pthread_mutex_lock(&private_display->lock);
 
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->func_output;
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
 
        if (!func_output->output_set_mode) {
                /* LCOV_EXCL_START */
@@ -1265,7 +1311,7 @@ tdm_output_get_mode(tdm_output *output, const tdm_output_mode **mode)
 EXTERN tdm_error
 tdm_output_set_dpms(tdm_output *output, tdm_output_dpms dpms_value)
 {
-       tdm_private_backend *private_backend;
+       tdm_private_module *private_module;
        tdm_func_output *func_output;
        OUTPUT_FUNC_ENTRY();
 
@@ -1290,8 +1336,8 @@ tdm_output_set_dpms(tdm_output *output, tdm_output_dpms dpms_value)
                return TDM_ERROR_BAD_REQUEST;
        }
 
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->func_output;
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
 
        TDM_INFO("output(%d) dpms '%s'", private_output->pipe, tdm_dpms_str(dpms_value));
 
@@ -1308,12 +1354,10 @@ tdm_output_set_dpms(tdm_output *output, tdm_output_dpms dpms_value)
 done:
        if (ret == TDM_ERROR_NONE) {
                if (private_output->current_dpms_value != dpms_value) {
-                       tdm_value value;
                        private_output->current_dpms_value = dpms_value;
-                       value.u32 = dpms_value;
-                       _tdm_output_call_thread_cb_change(private_output, TDM_OUTPUT_CHANGE_DPMS, value);
-                       TDM_INFO("output(%d) dpms '%s' done", private_output->pipe, tdm_dpms_str(dpms_value));
+                       _tdm_output_call_thread_cb_dpms(private_output, dpms_value);
                }
+               TDM_INFO("output(%d) dpms '%s' done", private_output->pipe, tdm_dpms_str(dpms_value));
        } else {
                tdm_output_dpms temp = TDM_OUTPUT_DPMS_OFF;
 
@@ -1332,7 +1376,7 @@ done:
 EXTERN tdm_error
 tdm_output_set_dpms_async(tdm_output *output, tdm_output_dpms dpms_value)
 {
-       tdm_private_backend *private_backend;
+       tdm_private_module *private_module;
        tdm_func_output *func_output;
        OUTPUT_FUNC_ENTRY();
 
@@ -1362,8 +1406,8 @@ tdm_output_set_dpms_async(tdm_output *output, tdm_output_dpms dpms_value)
                return TDM_ERROR_BAD_REQUEST;
        }
 
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->func_output;
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
        if (!func_output->output_set_dpms_handler) {
                TDM_WRN("not implemented: output_set_dpms_handler");
                _pthread_mutex_unlock(&private_display->lock);
@@ -1413,11 +1457,11 @@ INTERN tdm_error
 tdm_output_get_dpms_internal(tdm_output *output, tdm_output_dpms *dpms_value)
 {
        tdm_private_output *private_output;
-       tdm_private_backend *private_backend;
+       tdm_private_module *private_module;
        tdm_func_output *func_output;
        tdm_error ret = TDM_ERROR_NONE;
 
-       TDM_RETURN_VAL_IF_FAIL(tdm_output_is_valid(output), TDM_ERROR_INVALID_PARAMETER);
+       TDM_RETURN_VAL_IF_FAIL(output != NULL, TDM_ERROR_INVALID_PARAMETER);
 
        private_output = (tdm_private_output*)output;
 
@@ -1431,8 +1475,8 @@ tdm_output_get_dpms_internal(tdm_output *output, tdm_output_dpms *dpms_value)
                return TDM_ERROR_NONE;
        }
 
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->func_output;
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
 
        if (!func_output->output_get_dpms) {
                /* LCOV_EXCL_START */
@@ -1452,13 +1496,11 @@ tdm_output_get_dpms_internal(tdm_output *output, tdm_output_dpms *dpms_value)
 
        /* checking with backend's value */
        if (*dpms_value != private_output->current_dpms_value) {
-               tdm_value value;
                TDM_ERR("output(%d) dpms changed suddenly: %s -> %s",
-                               private_output->pipe, private_output->current_dpms_value,
+                               private_output->pipe, tdm_dpms_str(private_output->current_dpms_value),
                                tdm_dpms_str(*dpms_value));
                private_output->current_dpms_value = *dpms_value;
-               value.u32 = *dpms_value;
-               _tdm_output_call_thread_cb_change(private_output, TDM_OUTPUT_CHANGE_DPMS, value);
+               _tdm_output_call_thread_cb_dpms(private_output, *dpms_value);
        }
 
        return ret;
@@ -1480,6 +1522,31 @@ tdm_output_get_dpms(tdm_output *output, tdm_output_dpms *dpms_value)
        return ret;
 }
 
+EXTERN tdm_error
+tdm_output_has_capture_capability(tdm_output *output, unsigned int *has_capability)
+{
+       tdm_private_module *private_module;
+
+       OUTPUT_FUNC_ENTRY();
+
+       TDM_RETURN_VAL_IF_FAIL(has_capability != NULL, TDM_ERROR_INVALID_PARAMETER);
+
+       _pthread_mutex_lock(&private_display->lock);
+
+       private_module = private_output->private_module;
+
+       if (!(private_module->capabilities & TDM_DISPLAY_CAPABILITY_CAPTURE))
+               *has_capability = 0;
+       else if (!(private_module->caps_capture.capabilities & TDM_CAPTURE_CAPABILITY_OUTPUT))
+               *has_capability = 0;
+       else
+               *has_capability = 1;
+
+       _pthread_mutex_unlock(&private_display->lock);
+
+       return ret;
+}
+
 EXTERN tdm_capture *
 tdm_output_create_capture(tdm_output *output, tdm_error *error)
 {
@@ -1496,6 +1563,7 @@ tdm_output_create_capture(tdm_output *output, tdm_error *error)
        return capture;
 }
 
+/* LCOV_EXCL_START */
 EXTERN tdm_hwc_window *
 tdm_output_hwc_create_window(tdm_output *output, tdm_error *error)
 {
@@ -1508,11 +1576,9 @@ tdm_output_hwc_create_window(tdm_output *output, tdm_error *error)
        if (private_output->caps.capabilities & TDM_OUTPUT_CAPABILITY_HWC)
                hwc_window = (tdm_hwc_window *)tdm_hwc_window_create_internal(private_output, 0, error);
        else {
-               /* LCOV_EXCL_START */
                TDM_ERR("output(%p) not support HWC", private_output);
                if (error)
                        *error = TDM_ERROR_BAD_REQUEST;
-               /* LCOV_EXCL_STOP */
        }
 
        _pthread_mutex_unlock(&private_display->lock);
@@ -1532,11 +1598,9 @@ tdm_output_hwc_create_video_window(tdm_output *output, tdm_error *error)
        if (private_output->caps.capabilities & TDM_OUTPUT_CAPABILITY_HWC)
                hwc_window = (tdm_hwc_window *)tdm_hwc_window_create_internal(private_output, 1, error);
        else {
-               /* LCOV_EXCL_START */
                TDM_ERR("output(%p) not support HWC", private_output);
                if (error)
                        *error = TDM_ERROR_BAD_REQUEST;
-               /* LCOV_EXCL_STOP */
        }
 
        _pthread_mutex_unlock(&private_display->lock);
@@ -1564,7 +1628,7 @@ EXTERN tdm_error
 tdm_output_hwc_validate(tdm_output *output, tdm_hwc_window **composited_wnds,
                                                                        uint32_t num_wnds, uint32_t *num_types)
 {
-       tdm_private_backend *private_backend;
+       tdm_private_module *private_module;
        tdm_func_output *func_output = NULL;
        tdm_private_hwc_window **composited_wnds_frontend = NULL;
        tdm_hwc_window **composited_wnds_backend = NULL;
@@ -1582,15 +1646,13 @@ tdm_output_hwc_validate(tdm_output *output, tdm_hwc_window **composited_wnds,
                return TDM_ERROR_BAD_REQUEST;
        }
 
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->func_output;
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
 
        if (!func_output->output_hwc_validate) {
-               /* LCOV_EXCL_START */
                _pthread_mutex_unlock(&private_display->lock);
                TDM_WRN("not implemented!!");
                return TDM_ERROR_NOT_IMPLEMENTED;
-               /* LCOV_EXCL_STOP */
        }
 
        if (num_wnds == 0) {
@@ -1603,10 +1665,8 @@ tdm_output_hwc_validate(tdm_output *output, tdm_hwc_window **composited_wnds,
 
        composited_wnds_backend = calloc(num_wnds, sizeof(tdm_hwc_window *));
        if (!composited_wnds_backend) {
-               /* LCOV_EXCL_START */
                _pthread_mutex_unlock(&private_display->lock);
                return TDM_ERROR_OUT_OF_MEMORY;
-               /* LCOV_EXCL_STOP */
        }
 
        composited_wnds_frontend = (tdm_private_hwc_window **)composited_wnds;
@@ -1659,7 +1719,7 @@ tdm_output_hwc_get_changed_composition_types(tdm_output *output,
                                                                                 tdm_hwc_window **hwc_window,
                                                                                 tdm_hwc_window_composition *composition_types)
 {
-       tdm_private_backend *private_backend;
+       tdm_private_module *private_module;
        tdm_func_output *func_output = NULL;
        tdm_private_hwc_window * private_hwc_window = NULL;
        int i = 0;
@@ -1676,24 +1736,20 @@ tdm_output_hwc_get_changed_composition_types(tdm_output *output,
                return TDM_ERROR_BAD_REQUEST;
        }
 
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->func_output;
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
 
        if (!func_output->output_hwc_get_changed_composition_types) {
-               /* LCOV_EXCL_START */
                _pthread_mutex_unlock(&private_display->lock);
                TDM_WRN("not implemented!!");
                return TDM_ERROR_NOT_IMPLEMENTED;
-               /* LCOV_EXCL_STOP */
        }
 
        ret = func_output->output_hwc_get_changed_composition_types(private_output->output_backend,
                                                        num_elements, hwc_window, composition_types);
        if (ret != TDM_ERROR_NONE) {
-               /* LCOV_EXCL_START */
                _pthread_mutex_unlock(&private_display->lock);
                return ret;
-               /* LCOV_EXCL_STOP */
        }
 
        if (hwc_window == NULL || composition_types == NULL) {
@@ -1706,13 +1762,11 @@ tdm_output_hwc_get_changed_composition_types(tdm_output *output,
                private_hwc_window = _tdm_output_find_private_hwc_window(private_output, hwc_window[i]);
 
                if (private_hwc_window == NULL) {
-                       /* LCOV_EXCL_START */
                        TDM_ERR("failed! This should never happen!");
                        func_output->output_hwc_destroy_window(private_output->output_backend, hwc_window[i]);
                        *num_elements = 0;
                        _pthread_mutex_unlock(&private_display->lock);
                        return TDM_ERROR_OPERATION_FAILED;
-                       /* LCOV_EXCL_STOP */
                }
 
                hwc_window[i] = (tdm_hwc_window*)private_hwc_window;
@@ -1726,7 +1780,7 @@ tdm_output_hwc_get_changed_composition_types(tdm_output *output,
 tdm_error
 tdm_output_hwc_accept_changes(tdm_output *output)
 {
-       tdm_private_backend *private_backend;
+       tdm_private_module *private_module;
        tdm_func_output *func_output = NULL;
 
        OUTPUT_FUNC_ENTRY();
@@ -1739,15 +1793,13 @@ tdm_output_hwc_accept_changes(tdm_output *output)
                return TDM_ERROR_BAD_REQUEST;
        }
 
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->func_output;
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
 
        if (!func_output->output_hwc_validate) {
-               /* LCOV_EXCL_START */
                _pthread_mutex_unlock(&private_display->lock);
                TDM_WRN("not implemented!!");
                return TDM_ERROR_NOT_IMPLEMENTED;
-               /* LCOV_EXCL_STOP */
        }
 
        ret = func_output->output_hwc_accept_changes(private_output->output_backend);
@@ -1760,7 +1812,7 @@ tdm_output_hwc_accept_changes(tdm_output *output)
 tbm_surface_queue_h
 tdm_output_hwc_get_target_buffer_queue(tdm_output *output, tdm_error *error)
 {
-       tdm_private_backend *private_backend;
+       tdm_private_module *private_module;
        tdm_func_output *func_output = NULL;
        tbm_surface_queue_h queue = NULL;
 
@@ -1776,15 +1828,13 @@ tdm_output_hwc_get_target_buffer_queue(tdm_output *output, tdm_error *error)
                return NULL;
        }
 
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->func_output;
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
 
        if (!func_output->output_hwc_get_target_buffer_queue) {
-               /* LCOV_EXCL_START */
                _pthread_mutex_unlock(&private_display->lock);
                TDM_WRN("not implemented!!");
                return NULL;
-               /* LCOV_EXCL_STOP */
        }
 
        queue = func_output->output_hwc_get_target_buffer_queue(private_output->output_backend, error);
@@ -1797,7 +1847,7 @@ tdm_output_hwc_get_target_buffer_queue(tdm_output *output, tdm_error *error)
 EXTERN tdm_error
 tdm_output_hwc_set_client_target_buffer(tdm_output *output, tbm_surface_h target_buffer, tdm_hwc_region damage)
 {
-       tdm_private_backend *private_backend;
+       tdm_private_module *private_module;
        tdm_func_output *func_output = NULL;
 
        OUTPUT_FUNC_ENTRY();
@@ -1811,24 +1861,20 @@ tdm_output_hwc_set_client_target_buffer(tdm_output *output, tbm_surface_h target
        }
 
        if (tdm_debug_dump & TDM_DUMP_FLAG_WINDOW) {
-               /* LCOV_EXCL_START */
                char str[TDM_PATH_LEN];
                static int i;
                snprintf(str, TDM_PATH_LEN, "target_window_%d_%03d",
                                 private_output->index, i++);
                tdm_helper_dump_buffer_str(target_buffer, tdm_debug_dump_dir, str);
-               /* LCOV_EXCL_STOP */
        }
 
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->func_output;
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
 
        if (!func_output->output_hwc_set_client_target_buffer) {
-               /* LCOV_EXCL_START */
                _pthread_mutex_unlock(&private_display->lock);
                TDM_WRN("not implemented!!");
                return TDM_ERROR_NOT_IMPLEMENTED;
-               /* LCOV_EXCL_STOP */
        }
 
        ret = func_output->output_hwc_set_client_target_buffer(private_output->output_backend, target_buffer, damage);
@@ -1841,7 +1887,7 @@ tdm_output_hwc_set_client_target_buffer(tdm_output *output, tbm_surface_h target
 EXTERN tdm_error
 tdm_output_hwc_unset_client_target_buffer(tdm_output *output)
 {
-       tdm_private_backend *private_backend;
+       tdm_private_module *private_module;
        tdm_func_output *func_output = NULL;
 
        OUTPUT_FUNC_ENTRY();
@@ -1854,15 +1900,13 @@ tdm_output_hwc_unset_client_target_buffer(tdm_output *output)
                return TDM_ERROR_BAD_REQUEST;
        }
 
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->func_output;
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
 
        if (!func_output->output_hwc_unset_client_target_buffer) {
-               /* LCOV_EXCL_START */
                _pthread_mutex_unlock(&private_display->lock);
                TDM_ERR("not implemented!!");
                return TDM_ERROR_NOT_IMPLEMENTED;
-               /* LCOV_EXCL_STOP */
        }
 
        ret = func_output->output_hwc_unset_client_target_buffer(private_output->output_backend);
@@ -1890,7 +1934,7 @@ _tdm_output_hwc_layer_commit_handler(tdm_layer *layer, unsigned int sequence,
 tdm_error
 tdm_output_hwc_commit_client_target_buffer(tdm_output *output, tdm_output_hwc_target_buffer_commit_handler func, void *user_data)
 {
-       tdm_private_backend *private_backend;
+       tdm_private_module *private_module;
        tdm_func_output *func_output;
        tdm_private_output_hwc_target_buffer_commit_handler *output_hwc_target_buffer_commit_handler;
        tdm_layer *layer = NULL;
@@ -1908,35 +1952,29 @@ tdm_output_hwc_commit_client_target_buffer(tdm_output *output, tdm_output_hwc_ta
                return TDM_ERROR_BAD_REQUEST;
        }
 
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->func_output;
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
 
        if (!func_output->output_hwc_get_client_target_buffer_layer) {
-               /* LCOV_EXCL_START */
                _pthread_mutex_unlock(&private_display->lock);
                TDM_ERR("not implemented!!");
                return TDM_ERROR_NOT_IMPLEMENTED;
-               /* LCOV_EXCL_STOP */
        }
 
        layer = func_output->output_hwc_get_client_target_buffer_layer(private_output->output_backend,
                                                                                                                   &ret);
        if (!layer) {
-               /* LCOV_EXCL_START */
                _pthread_mutex_unlock(&private_display->lock);
                TDM_ERR("no assigned layer!!");
                return TDM_ERROR_INVALID_PARAMETER;
-               /* LCOV_EXCL_STOP */
        }
 
        private_layer = (tdm_private_layer*)layer;
 
        if (!func_output->output_hwc_get_client_target_buffer) {
-               /* LCOV_EXCL_START */
                _pthread_mutex_unlock(&private_display->lock);
                TDM_ERR("not implemented!!");
                return TDM_ERROR_NOT_IMPLEMENTED;
-               /* LCOV_EXCL_STOP */
        }
 
        buffer = func_output->output_hwc_get_client_target_buffer(private_output->output_backend,
@@ -1946,9 +1984,8 @@ tdm_output_hwc_commit_client_target_buffer(tdm_output *output, tdm_output_hwc_ta
        else
                ret = tdm_layer_unset_buffer_internal(private_layer);
        if (ret != TDM_ERROR_NONE) {
-               /* LCOV_EXCL_START */
                TDM_ERR("failed: layer set info(window)");
-               /* LCOV_EXCL_STOP */
+               _pthread_mutex_unlock(&private_display->lock);
                return ret;
        }
 
@@ -1968,9 +2005,8 @@ tdm_output_hwc_commit_client_target_buffer(tdm_output *output, tdm_output_hwc_ta
 
                ret = tdm_layer_set_info_internal(private_layer, &private_output->target_buffer_info);
                if (ret != TDM_ERROR_NONE) {
-                       /* LCOV_EXCL_START */
                        TDM_ERR("failed: layer set info(window)");
-                       /* LCOV_EXCL_STOP */
+                       _pthread_mutex_unlock(&private_display->lock);
                        return ret;
                }
 
@@ -1979,22 +2015,20 @@ tdm_output_hwc_commit_client_target_buffer(tdm_output *output, tdm_output_hwc_ta
 
        output_hwc_target_buffer_commit_handler = calloc(1, sizeof(tdm_private_output_hwc_target_buffer_commit_handler));
        if (!output_hwc_target_buffer_commit_handler) {
-               /* LCOV_EXCL_START */
                TDM_ERR("failed: alloc memory");
+               _pthread_mutex_unlock(&private_display->lock);
                return TDM_ERROR_OUT_OF_MEMORY;
-               /* LCOV_EXCL_STOP */
        }
 
        output_hwc_target_buffer_commit_handler->private_output = private_output;
        output_hwc_target_buffer_commit_handler->func = func;
        output_hwc_target_buffer_commit_handler->user_data = user_data;
 
-       ret = tdm_layer_commit_internal(private_layer, _tdm_output_hwc_layer_commit_handler, user_data);
+       ret = tdm_layer_commit_internal(private_layer, _tdm_output_hwc_layer_commit_handler, output_hwc_target_buffer_commit_handler);
        if (ret != TDM_ERROR_NONE) {
-               /* LCOV_EXCL_START */
                TDM_ERR("failed: commit layer(target buffer)");
                free(output_hwc_target_buffer_commit_handler);
-               /* LCOV_EXCL_STOP */
+               _pthread_mutex_unlock(&private_display->lock);
                return ret;
        }
 
@@ -2007,7 +2041,7 @@ tdm_error
 tdm_output_hwc_get_video_supported_formats(tdm_output *output, const tbm_format **formats,
                                                                int *count)
 {
-       tdm_private_backend *private_backend;
+       tdm_private_module *private_module;
        tdm_func_output *func_output;
        OUTPUT_FUNC_ENTRY();
 
@@ -2016,15 +2050,13 @@ tdm_output_hwc_get_video_supported_formats(tdm_output *output, const tbm_format
 
        _pthread_mutex_lock(&private_display->lock);
 
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->func_output;
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
 
        if (!func_output->output_hwc_get_video_supported_formats) {
-               /* LCOV_EXCL_START */
                _pthread_mutex_unlock(&private_display->lock);
                TDM_WRN("not implemented!!");
                return TDM_ERROR_NOT_IMPLEMENTED;
-               /* LCOV_EXCL_STOP */
        }
 
        ret = func_output->output_hwc_get_video_supported_formats(
@@ -2038,10 +2070,10 @@ tdm_output_hwc_get_video_supported_formats(tdm_output *output, const tbm_format
 static int
 _is_hwc_output_still_existed(tdm_private_output *private_output)
 {
-       tdm_private_backend *private_backend = private_output->private_backend;
+       tdm_private_module *private_module = private_output->private_module;
        tdm_private_output *o = NULL;
 
-       LIST_FOR_EACH_ENTRY(o, &private_backend->output_list, link) {
+       LIST_FOR_EACH_ENTRY(o, &private_module->output_list, link) {
                if (!(o->caps.capabilities & TDM_OUTPUT_CAPABILITY_HWC))
                        continue;
 
@@ -2103,7 +2135,7 @@ _need_validate_handler(int fd, tdm_event_loop_mask mask, void *user_data)
        ev.base.data = NULL;
        ev.base.sync = 0;
 
-       ret = tdm_thread_cb_call(private_output, &ev.base);
+       ret = tdm_thread_cb_call(private_output, &ev.base, 1);
        TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
 
        TDM_INFO("tdm-thread: get a 'need to revalidate' event for the ouptut:%p.", private_output);
@@ -2161,3 +2193,4 @@ tdm_output_choose_commit_per_vblank_mode(tdm_private_output *private_output, int
 
        return TDM_ERROR_NONE;
 }
+/* LCOV_EXCL_STOP */