implementation
[platform/core/uifw/libtdm.git] / src / tdm_output.c
index e73de5b..550a750 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
 
        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
+
+#define OUTPUT_FUNC_ENTRY_VOID() \
+       tdm_private_display *private_display; \
+       tdm_private_output *private_output; \
+       TDM_RETURN_IF_FAIL(tdm_output_is_valid(output)); \
        private_output = (tdm_private_output*)output; \
        private_display = private_output->private_display
 
@@ -127,47 +134,151 @@ _tdm_output_vblank_timeout_update(tdm_private_output *private_output, int ms_del
        }
 }
 
-static tdm_private_hwc_window *
-_tdm_output_find_private_hwc_window(tdm_private_output *private_output,
-                                                               tdm_hwc_window *hwc_window_backend)
+INTERN tdm_error
+tdm_output_init(tdm_private_display *private_display)
 {
-       tdm_private_hwc_window *private_hwc_window = NULL;
-
-       LIST_FOR_EACH_ENTRY(private_hwc_window, &private_output->hwc_window_list, link) {
-               if (private_hwc_window->hwc_window_backend == hwc_window_backend)
-                       return private_hwc_window;
-       }
+       tdm_thread_cb_set_find_func(TDM_THREAD_CB_OUTPUT_DESTROY, tdm_display_find_output_stamp);
+       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_STATUS, tdm_display_find_output_stamp);
+       tdm_thread_cb_set_find_func(TDM_THREAD_CB_OUTPUT_DPMS, tdm_display_find_output_stamp);
 
-       return NULL;
+       return TDM_ERROR_NONE;
 }
 
 INTERN tdm_error
-tdm_output_init(tdm_private_display *private_display)
+tdm_output_call_thread_cb_destroy(tdm_private_output *private_output)
 {
-       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_output_destroy output_destroy;
+       tdm_error ret;
+
+       memset(&output_destroy, 0, sizeof output_destroy);
+       output_destroy.base.type = TDM_THREAD_CB_OUTPUT_DESTROY;
+       output_destroy.base.length = sizeof output_destroy;
+       output_destroy.base.object_stamp = private_output->stamp;
+       output_destroy.base.data = NULL;
+       output_destroy.base.sync = 1;
+
+       ret = tdm_thread_cb_call(private_output, &output_destroy.base, 1);
+       TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
 
        return TDM_ERROR_NONE;
 }
 
-EXTERN tdm_backend *
-tdm_output_get_backend(tdm_output *output, tdm_error *error)
+INTERN void
+tdm_output_thread_cb_destroy(tdm_private_display *private_display, void *object, tdm_thread_cb_base *cb_base, void *user_data)
+{
+       tdm_private_output *private_output = object;
+       tdm_private_output_destroy_handler *destroy_handler = user_data;
+
+       TDM_RETURN_IF_FAIL(TDM_MUTEX_IS_LOCKED());
+
+       assert(destroy_handler->owner_tid == syscall(SYS_gettid));
+
+       _pthread_mutex_unlock(&private_display->lock);
+       destroy_handler->func(private_output, destroy_handler->user_data);
+       _pthread_mutex_lock(&private_display->lock);
+}
+
+EXTERN tdm_error
+tdm_output_add_destroy_handler(tdm_output *output,
+                                                          tdm_output_destroy_handler func,
+                                                          void *user_data)
+{
+       tdm_private_output_destroy_handler *destroy_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(destroy_handler, &private_output->destroy_handler_list, link) {
+               if (destroy_handler->func == func && destroy_handler->user_data == user_data) {
+                       TDM_ERR("can't add twice");
+                       _pthread_mutex_unlock(&private_display->lock);
+                       return TDM_ERROR_BAD_REQUEST;
+               }
+       }
+
+       destroy_handler = calloc(1, sizeof(tdm_private_output_destroy_handler));
+       if (!destroy_handler) {
+               TDM_ERR("failed: alloc memory");
+               _pthread_mutex_unlock(&private_display->lock);
+               return TDM_ERROR_OUT_OF_MEMORY;
+       }
+
+       ret = tdm_thread_cb_add(private_output, TDM_THREAD_CB_OUTPUT_DESTROY, NULL, tdm_output_thread_cb_destroy, destroy_handler);
+       if (ret != TDM_ERROR_NONE) {
+               TDM_ERR("tdm_thread_cb_add failed");
+               free(destroy_handler);
+               _pthread_mutex_unlock(&private_display->lock);
+               return TDM_ERROR_OPERATION_FAILED;
+       }
+
+       destroy_handler->private_output = private_output;
+       destroy_handler->func = func;
+       destroy_handler->user_data = user_data;
+       destroy_handler->owner_tid = syscall(SYS_gettid);
+
+       LIST_ADDTAIL(&destroy_handler->link, &private_output->destroy_handler_list);
+
+       _pthread_mutex_unlock(&private_display->lock);
+
+       return ret;
+}
+
+
+EXTERN void
+tdm_output_remove_destroy_handler(tdm_output *output,
+                                                                 tdm_output_destroy_handler func,
+                                                                 void *user_data)
+{
+       tdm_private_display *private_display;
+       tdm_private_output *private_output;
+       tdm_private_output_destroy_handler *destroy_handler = NULL, *hh = NULL;
+
+       TDM_RETURN_IF_FAIL(output != NULL);
+       TDM_RETURN_IF_FAIL(func != NULL);
+
+       private_output = (tdm_private_output*)output;
+       private_display = private_output->private_display;
+
+       _pthread_mutex_lock(&private_display->lock);
+
+       LIST_FOR_EACH_ENTRY_SAFE(destroy_handler, hh, &private_output->destroy_handler_list, link) {
+               if (destroy_handler->func != func || destroy_handler->user_data != user_data)
+                       continue;
+
+               tdm_thread_cb_remove(private_output, TDM_THREAD_CB_OUTPUT_DESTROY, NULL, tdm_output_thread_cb_destroy, destroy_handler);
+
+               LIST_DEL(&destroy_handler->link);
+               free(destroy_handler);
+
+               _pthread_mutex_unlock(&private_display->lock);
+
+               return;
+       }
+
+       _pthread_mutex_unlock(&private_display->lock);
+}
+
+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
@@ -226,34 +337,72 @@ 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;
@@ -263,7 +412,6 @@ 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_RETURN_IF_FAIL(private_output);
@@ -272,15 +420,13 @@ tdm_output_cb_status(tdm_output *output_backend, tdm_output_conn_status 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_backend, output_backend, private_output->pipe, 1);
+               ret = tdm_display_update_output(private_output->private_module, output_backend);
                TDM_RETURN_IF_FAIL(ret == TDM_ERROR_NONE);
        } else {
                private_output->caps.status = status;
        }
 
-       value.u32 = 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);
 }
 
@@ -288,7 +434,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));
@@ -297,9 +442,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);
 }
 
@@ -332,10 +475,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;
@@ -363,7 +517,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;
@@ -375,7 +529,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);
@@ -538,7 +693,7 @@ 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 (!tdm_module_check_abi(private_output->private_module, 1, 5)) {
                _pthread_mutex_unlock(&private_display->lock);
                return TDM_ERROR_BAD_REQUEST;
        }
@@ -632,14 +787,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 */
@@ -660,7 +815,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();
 
@@ -668,8 +823,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 */
@@ -766,7 +921,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);
 }
 
@@ -842,7 +1000,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);
 }
 
@@ -855,15 +1013,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)
@@ -1060,18 +1218,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 */
@@ -1084,35 +1242,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);
@@ -1129,6 +1285,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)
@@ -1205,7 +1370,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();
 
@@ -1213,8 +1378,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 */
@@ -1255,7 +1420,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();
 
@@ -1280,8 +1445,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));
 
@@ -1298,12 +1463,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;
 
@@ -1322,7 +1485,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();
 
@@ -1352,8 +1515,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);
@@ -1403,11 +1566,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;
 
@@ -1421,8 +1584,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 */
@@ -1442,13 +1605,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;
@@ -1470,688 +1631,90 @@ tdm_output_get_dpms(tdm_output *output, tdm_output_dpms *dpms_value)
        return ret;
 }
 
-EXTERN tdm_capture *
-tdm_output_create_capture(tdm_output *output, tdm_error *error)
+EXTERN tdm_error
+tdm_output_has_capture_capability(tdm_output *output, unsigned int *has_capability)
 {
-       tdm_capture *capture = NULL;
+       tdm_private_module *private_module;
 
-       OUTPUT_FUNC_ENTRY_ERROR();
+       OUTPUT_FUNC_ENTRY();
+
+       TDM_RETURN_VAL_IF_FAIL(has_capability != NULL, TDM_ERROR_INVALID_PARAMETER);
 
        _pthread_mutex_lock(&private_display->lock);
 
-       capture = (tdm_capture *)tdm_capture_create_output_internal(private_output, error);
+       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 capture;
+       return ret;
 }
 
-EXTERN tdm_hwc_window *
-tdm_output_hwc_create_window(tdm_output *output, tdm_error *error)
+EXTERN tdm_capture *
+tdm_output_create_capture(tdm_output *output, tdm_error *error)
 {
-       tdm_hwc_window *hwc_window = NULL;
+       tdm_capture *capture = NULL;
 
        OUTPUT_FUNC_ENTRY_ERROR();
 
        _pthread_mutex_lock(&private_display->lock);
 
-       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 */
-       }
+       capture = (tdm_capture *)tdm_capture_create_output_internal(private_output, error);
 
        _pthread_mutex_unlock(&private_display->lock);
 
-       return hwc_window;
+       return capture;
 }
 
-EXTERN tdm_hwc_window *
-tdm_output_hwc_create_video_window(tdm_output *output, tdm_error *error)
+EXTERN tdm_hwc *
+tdm_output_get_hwc(tdm_output *output, tdm_error *error)
 {
-       tdm_hwc_window *hwc_window = NULL;
+       tdm_private_hwc *private_hwc = NULL;
 
        OUTPUT_FUNC_ENTRY_ERROR();
 
        _pthread_mutex_lock(&private_display->lock);
 
-       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);
+       private_hwc = private_output->private_hwc;
+       if (!private_hwc) {
                if (error)
                        *error = TDM_ERROR_BAD_REQUEST;
-               /* LCOV_EXCL_STOP */
-       }
-
-       _pthread_mutex_unlock(&private_display->lock);
-
-       return hwc_window;
-}
-
-EXTERN tdm_error
-tdm_output_hwc_destroy_window(tdm_output *output, tdm_hwc_window *hwc_window)
-{
-       OUTPUT_FUNC_ENTRY();
-
-       TDM_RETURN_VAL_IF_FAIL(hwc_window != NULL, TDM_ERROR_INVALID_PARAMETER);
-
-       _pthread_mutex_lock(&private_display->lock);
-
-       ret = tdm_hwc_window_destroy_internal(hwc_window);
-
-       _pthread_mutex_unlock(&private_display->lock);
-
-       return ret;
-}
-
-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_func_output *func_output = NULL;
-       tdm_private_hwc_window **composited_wnds_frontend = NULL;
-       tdm_hwc_window **composited_wnds_backend = NULL;
-       int i;
-
-       OUTPUT_FUNC_ENTRY();
-
-       TDM_RETURN_VAL_IF_FAIL(num_types != NULL, TDM_ERROR_INVALID_PARAMETER);
-
-       _pthread_mutex_lock(&private_display->lock);
-
-       if (!(private_output->caps.capabilities & TDM_OUTPUT_CAPABILITY_HWC)) {
-               TDM_ERR("output(%p) not support HWC", private_output);
-               _pthread_mutex_unlock(&private_display->lock);
-               return TDM_ERROR_BAD_REQUEST;
-       }
-
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->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) {
-               ret = func_output->output_hwc_validate(private_output->output_backend, NULL, 0, num_types);
-
-               _pthread_mutex_unlock(&private_display->lock);
-
-               return ret;
-       }
-
-       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 */
+               return NULL;
        }
 
-       composited_wnds_frontend = (tdm_private_hwc_window **)composited_wnds;
-
-       for (i = 0; i < num_wnds; i++)
-               composited_wnds_backend[i] = composited_wnds_frontend[i]->hwc_window_backend;
-
-       ret = func_output->output_hwc_validate(private_output->output_backend, composited_wnds_backend, num_wnds, num_types);
-
-       free(composited_wnds_backend);
+       if (error)
+               *error = TDM_ERROR_NONE;
 
        _pthread_mutex_unlock(&private_display->lock);
 
-       return ret;
+       return private_hwc;
 }
 
-EXTERN tdm_error
-tdm_output_hwc_set_need_validate_handler(tdm_output *output,
-               tdm_output_need_validate_handler hndl)
+INTERN tdm_error
+tdm_output_choose_commit_per_vblank_mode(tdm_private_output *private_output, int mode)
 {
-       OUTPUT_FUNC_ENTRY();
-
-       TDM_RETURN_VAL_IF_FAIL(hndl != NULL, TDM_ERROR_INVALID_PARAMETER);
-
-       _pthread_mutex_lock(&private_display->lock);
-
-       if (!(private_output->caps.capabilities & TDM_OUTPUT_CAPABILITY_HWC)) {
-               TDM_ERR("output(%p) not support HWC", private_output);
-               _pthread_mutex_unlock(&private_display->lock);
-               return TDM_ERROR_BAD_REQUEST;
-       }
-
-       /* there's no reason to allow this */
-       if (private_output->need_validate.hndl) {
+       if (!private_output)
+               return TDM_ERROR_INVALID_PARAMETER;
 
-               _pthread_mutex_unlock(&private_display->lock);
-               return TDM_ERROR_OPERATION_FAILED;
-       }
+       if (mode < 0 || mode > 2)
+               return TDM_ERROR_INVALID_PARAMETER;
 
-       private_output->need_validate.hndl = hndl;
+       private_output->commit_per_vblank = mode;
 
-       _pthread_mutex_unlock(&private_display->lock);
+       if (private_output->commit_per_vblank == 0)
+               TDM_INFO("commit per vblank: disable");
+       else if (private_output->commit_per_vblank == 1)
+               TDM_INFO("commit per vblank: enable (1 layer)");
+       else if (private_output->commit_per_vblank == 2)
+               TDM_INFO("commit per vblank: enable (previous commit)");
 
-       return ret;
-}
-
-EXTERN tdm_error
-tdm_output_hwc_get_changed_composition_types(tdm_output *output,
-                                                                                uint32_t *num_elements,
-                                                                                tdm_hwc_window **hwc_window,
-                                                                                tdm_hwc_window_composition *composition_types)
-{
-       tdm_private_backend *private_backend;
-       tdm_func_output *func_output = NULL;
-       tdm_private_hwc_window * private_hwc_window = NULL;
-       int i = 0;
-
-       OUTPUT_FUNC_ENTRY();
-
-       TDM_RETURN_VAL_IF_FAIL(num_elements != NULL, TDM_ERROR_INVALID_PARAMETER);
-
-       _pthread_mutex_lock(&private_display->lock);
-
-       if (!(private_output->caps.capabilities & TDM_OUTPUT_CAPABILITY_HWC)) {
-               TDM_ERR("output(%p) not support HWC", private_output);
-               _pthread_mutex_unlock(&private_display->lock);
-               return TDM_ERROR_BAD_REQUEST;
-       }
-
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->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) {
-               _pthread_mutex_unlock(&private_display->lock);
-               return TDM_ERROR_NONE;
-       }
-
-       for (i = 0; i < *num_elements; i++) {
-
-               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;
-       }
-
-       _pthread_mutex_unlock(&private_display->lock);
-
-       return ret;
-}
-
-tdm_error
-tdm_output_hwc_accept_changes(tdm_output *output)
-{
-       tdm_private_backend *private_backend;
-       tdm_func_output *func_output = NULL;
-
-       OUTPUT_FUNC_ENTRY();
-
-       _pthread_mutex_lock(&private_display->lock);
-
-       if (!(private_output->caps.capabilities & TDM_OUTPUT_CAPABILITY_HWC)) {
-               TDM_ERR("output(%p) not support HWC", private_output);
-               _pthread_mutex_unlock(&private_display->lock);
-               return TDM_ERROR_BAD_REQUEST;
-       }
-
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->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);
-
-       _pthread_mutex_unlock(&private_display->lock);
-
-       return ret;
-}
-
-tbm_surface_queue_h
-tdm_output_hwc_get_target_buffer_queue(tdm_output *output, tdm_error *error)
-{
-       tdm_private_backend *private_backend;
-       tdm_func_output *func_output = NULL;
-       tbm_surface_queue_h queue = NULL;
-
-       OUTPUT_FUNC_ENTRY_ERROR();
-
-       _pthread_mutex_lock(&private_display->lock);
-
-       if (!(private_output->caps.capabilities & TDM_OUTPUT_CAPABILITY_HWC)) {
-               TDM_ERR("output(%p) not support HWC", private_output);
-               if (error)
-                       *error = TDM_ERROR_BAD_REQUEST;
-               _pthread_mutex_unlock(&private_display->lock);
-               return NULL;
-       }
-
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->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);
-
-       _pthread_mutex_unlock(&private_display->lock);
-
-       return queue;
-}
-
-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_func_output *func_output = NULL;
-
-       OUTPUT_FUNC_ENTRY();
-
-       _pthread_mutex_lock(&private_display->lock);
-
-       if (!(private_output->caps.capabilities & TDM_OUTPUT_CAPABILITY_HWC)) {
-               TDM_ERR("output(%p) not support HWC", private_output);
-               _pthread_mutex_unlock(&private_display->lock);
-               return TDM_ERROR_BAD_REQUEST;
-       }
-
-       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;
-
-       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);
-
-       _pthread_mutex_unlock(&private_display->lock);
-
-       return ret;
-}
-
-EXTERN tdm_error
-tdm_output_hwc_unset_client_target_buffer(tdm_output *output)
-{
-       tdm_private_backend *private_backend;
-       tdm_func_output *func_output = NULL;
-
-       OUTPUT_FUNC_ENTRY();
-
-       _pthread_mutex_lock(&private_display->lock);
-
-       if (!(private_output->caps.capabilities & TDM_OUTPUT_CAPABILITY_HWC)) {
-               TDM_ERR("output(%p) not support HWC", private_output);
-               _pthread_mutex_unlock(&private_display->lock);
-               return TDM_ERROR_BAD_REQUEST;
-       }
-
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->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);
-
-       _pthread_mutex_unlock(&private_display->lock);
-
-       return ret;
-}
-
-static void
-_tdm_output_hwc_layer_commit_handler(tdm_layer *layer, unsigned int sequence,
-                                                                                unsigned int tv_sec, unsigned int tv_usec,
-                                                                                void *user_data)
-{
-       tdm_private_output_hwc_target_buffer_commit_handler *output_hwc_target_buffer_commit_handler = (tdm_private_output_hwc_target_buffer_commit_handler *)user_data;
-       tdm_output_hwc_target_buffer_commit_handler func = output_hwc_target_buffer_commit_handler->func;
-       tdm_output *output = (tdm_output *)output_hwc_target_buffer_commit_handler->private_output;
-       void *data = output_hwc_target_buffer_commit_handler->user_data;
-
-       func(output, sequence, tv_sec, tv_usec, data);
-
-       free(output_hwc_target_buffer_commit_handler);
-}
-
-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_func_output *func_output;
-       tdm_private_output_hwc_target_buffer_commit_handler *output_hwc_target_buffer_commit_handler;
-       tdm_layer *layer = NULL;
-       tdm_private_layer *private_layer;
-       const tdm_output_mode *mode;
-       tbm_surface_h buffer;
-
-       OUTPUT_FUNC_ENTRY();
-
-       _pthread_mutex_lock(&private_display->lock);
-
-       if (!(private_output->caps.capabilities & TDM_OUTPUT_CAPABILITY_HWC)) {
-               TDM_ERR("output(%p) not support HWC", private_output);
-               _pthread_mutex_unlock(&private_display->lock);
-               return TDM_ERROR_BAD_REQUEST;
-       }
-
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->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,
-                                                                                                          &ret);
-       if (buffer)
-               ret = tdm_layer_set_buffer_internal(private_layer, buffer);
-       else
-               ret = tdm_layer_unset_buffer_internal(private_layer);
-       if (ret != TDM_ERROR_NONE) {
-               /* LCOV_EXCL_START */
-               TDM_ERR("failed: layer set info(window)");
-               _pthread_mutex_unlock(&private_display->lock);
-               /* LCOV_EXCL_STOP */
-               return ret;
-       }
-
-       if (private_output->need_set_target_info) {
-               mode = private_output->current_mode;
-               private_output->target_buffer_info.src_config.size.h = mode->hdisplay;
-               private_output->target_buffer_info.src_config.size.v = mode->vdisplay;
-               private_output->target_buffer_info.src_config.pos.x = 0;
-               private_output->target_buffer_info.src_config.pos.y = 0;
-               private_output->target_buffer_info.src_config.pos.w = mode->hdisplay;
-               private_output->target_buffer_info.src_config.pos.h = mode->vdisplay;
-               private_output->target_buffer_info.dst_pos.x = 0;
-               private_output->target_buffer_info.dst_pos.y = 0;
-               private_output->target_buffer_info.dst_pos.w = mode->hdisplay;
-               private_output->target_buffer_info.dst_pos.h = mode->vdisplay;
-               private_output->target_buffer_info.transform = TDM_TRANSFORM_NORMAL;
-
-               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)");
-                       _pthread_mutex_unlock(&private_display->lock);
-                       /* LCOV_EXCL_STOP */
-                       return ret;
-               }
-
-               private_output->need_set_target_info = 0;
-       }
-
-       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, 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);
-               _pthread_mutex_unlock(&private_display->lock);
-               /* LCOV_EXCL_STOP */
-               return ret;
-       }
-
-       _pthread_mutex_unlock(&private_display->lock);
-
-       return ret;
-}
-
-tdm_error
-tdm_output_hwc_get_video_supported_formats(tdm_output *output, const tbm_format **formats,
-                                                               int *count)
-{
-       tdm_private_backend *private_backend;
-       tdm_func_output *func_output;
-       OUTPUT_FUNC_ENTRY();
-
-       TDM_RETURN_VAL_IF_FAIL(formats != NULL, TDM_ERROR_INVALID_PARAMETER);
-       TDM_RETURN_VAL_IF_FAIL(count != NULL, TDM_ERROR_INVALID_PARAMETER);
-
-       _pthread_mutex_lock(&private_display->lock);
-
-       private_backend = private_output->private_backend;
-       func_output = &private_backend->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(
-                                                               private_output->output_backend, formats, count);
-
-       _pthread_mutex_unlock(&private_display->lock);
-
-       return ret;
-}
-
-static int
-_is_hwc_output_still_existed(tdm_private_output *private_output)
-{
-       tdm_private_backend *private_backend = private_output->private_backend;
-       tdm_private_output *o = NULL;
-
-       LIST_FOR_EACH_ENTRY(o, &private_backend->output_list, link) {
-               if (!(o->caps.capabilities & TDM_OUTPUT_CAPABILITY_HWC))
-                       continue;
-
-               if (o == private_output)
-                       goto exist;
-       }
-
-       return 0;
-
-exist:
-       return 1;
-}
-
-/* gets called on behalf of the ecore-main-loop thread */
-INTERN void
-tdm_output_need_validate_handler_thread(tdm_private_display *private_display, void *object, tdm_thread_cb_base *cb_base, void *user_data)
-{
-       tdm_private_output *private_output = object;
-
-       TDM_RETURN_IF_FAIL(private_output != NULL);
-
-       _pthread_mutex_lock(&private_display->lock);
-
-       /* as we get 'private_output' within an event, an output this 'private_output'
-        * points to can be destroyed already */
-       if (!_is_hwc_output_still_existed(private_output)) {
-               _pthread_mutex_unlock(&private_display->lock);
-               return;
-       }
-
-       _pthread_mutex_unlock(&private_display->lock);
-
-       TDM_INFO("tdm-backend asks for revalidation for the output:%p.", private_output);
-
-       if (private_output->need_validate.hndl)
-               private_output->need_validate.hndl((tdm_output*)private_output);
-}
-
-/* gets called on behalf of the tdm-thread */
-static tdm_error
-_need_validate_handler(int fd, tdm_event_loop_mask mask, void *user_data)
-{
-       tdm_thread_cb_need_validate ev;
-       tdm_private_output *private_output;
-       tdm_error ret;
-       uint64_t value;
-
-       private_output = (tdm_private_output *)user_data;
-
-       if (read(private_output->need_validate.event_fd, &value, sizeof(value)) < 0) {
-               TDM_ERR("error while trying to read from a need_validate.event_fd fd.");
-               return TDM_ERROR_OPERATION_FAILED;
-       }
-
-       memset(&ev, 0, sizeof ev);
-       ev.base.type = TDM_THREAD_CB_NEED_VALIDATE;
-       ev.base.length = sizeof ev;
-       ev.base.object_stamp = private_output->stamp;
-       ev.base.data = NULL;
-       ev.base.sync = 0;
-
-       ret = tdm_thread_cb_call(private_output, &ev.base);
-       TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
-
-       TDM_INFO("tdm-thread: get a 'need to revalidate' event for the ouptut:%p.", private_output);
-
-       /* who cares about this? */
-       return TDM_ERROR_NONE;
-}
-
-INTERN tdm_error
-tdm_output_need_validate_event_init(tdm_output *output)
-{
-       int fd;
-
-       OUTPUT_FUNC_ENTRY();
-
-       TDM_RETURN_VAL_IF_FAIL(TDM_MUTEX_IS_LOCKED(), TDM_ERROR_OPERATION_FAILED);
-
-       if (!(private_output->caps.capabilities & TDM_OUTPUT_CAPABILITY_HWC)) {
-               TDM_ERR("output(%p) not support HWC", private_output);
-               return TDM_ERROR_BAD_REQUEST;
-       }
-
-       /* build in eventfd fds into event_loop listened & handled by the tdm-thread */
-       fd = eventfd(0, 0);
-       TDM_WARNING_IF_FAIL(fd >= 0);
-
-       private_output->need_validate.event_source = tdm_event_loop_add_fd_handler(private_display,
-                       fd,     TDM_EVENT_LOOP_READABLE, _need_validate_handler, private_output, &ret);
-       TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
-
-       private_output->need_validate.event_fd = fd;
-
-       TDM_INFO("register an output:%p for the revalidation, event_fd:%d.", private_output, fd);
-
-       return ret;
-}
-
-INTERN tdm_error
-tdm_output_choose_commit_per_vblank_mode(tdm_private_output *private_output, int mode)
-{
-       if (!private_output)
-               return TDM_ERROR_INVALID_PARAMETER;
-
-       if (mode < 0 || mode > 2)
-               return TDM_ERROR_INVALID_PARAMETER;
-
-       private_output->commit_per_vblank = mode;
-
-       if (private_output->commit_per_vblank == 0)
-               TDM_INFO("commit per vblank: disable");
-       else if (private_output->commit_per_vblank == 1)
-               TDM_INFO("commit per vblank: enable (1 layer)");
-       else if (private_output->commit_per_vblank == 2)
-               TDM_INFO("commit per vblank: enable (previous commit)");
-
-       return TDM_ERROR_NONE;
+       return TDM_ERROR_NONE;
 }
+/* LCOV_EXCL_STOP */