hwc: redesign the hwc
[platform/core/uifw/libtdm.git] / src / tdm.c
index effa0f3..bc59c3c 100644 (file)
--- a/src/tdm.c
+++ b/src/tdm.c
@@ -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
@@ -97,6 +97,44 @@ tdm_display_find_output_stamp(tdm_private_display *private_display, double stamp
        return NULL;
 }
 
+tdm_private_hwc *
+tdm_display_find_private_hwc(tdm_private_display *private_display, tdm_hwc *hwc_backend)
+{
+       tdm_private_module *private_module = NULL;
+       tdm_private_output *private_output = NULL;
+       tdm_private_hwc *private_hwc = NULL;
+
+       LIST_FOR_EACH_ENTRY(private_module, &private_display->module_list, link) {
+               LIST_FOR_EACH_ENTRY(private_output, &private_module->output_list, link) {
+                       if (!private_output->private_hwc) continue;
+                       private_hwc = private_output->private_hwc;
+                       if (private_hwc->hwc_backend == hwc_backend)
+                               return private_hwc;
+               }
+       }
+
+       return NULL;
+}
+
+INTERN void *
+tdm_display_find_hwc_stamp(tdm_private_display *private_display, double stamp)
+{
+       tdm_private_module *private_module = NULL;
+       tdm_private_output *private_output = NULL;
+       tdm_private_hwc *private_hwc = NULL;
+
+       LIST_FOR_EACH_ENTRY(private_module, &private_display->module_list, link) {
+               LIST_FOR_EACH_ENTRY(private_output, &private_module->output_list, link) {
+                       if (!private_output->private_hwc) continue;
+                       private_hwc = private_output->private_hwc;
+                       if (private_hwc->stamp == stamp)
+                               return private_hwc;
+               }
+       }
+
+       return NULL;
+}
+
 static void
 _tdm_display_destroy_caps_pp(tdm_caps_pp *caps_pp)
 {
@@ -164,10 +202,12 @@ _tdm_display_destroy_private_output(tdm_private_output *private_output)
 {
        tdm_private_display *private_display = private_output->private_display;
        tdm_private_layer *l = NULL, *ll = NULL;
+       tdm_private_hwc *private_hwc = NULL;
        tdm_private_hwc_window *hw = NULL, *hww = NULL;
        tdm_private_capture *c = NULL, *cc = NULL;
        tdm_private_output_vblank_handler *v = NULL, *vv = NULL;
        tdm_private_output_commit_handler *om = NULL, *omm = NULL;
+       tdm_private_hwc_commit_handler *hm = NULL, *hmm = NULL;
        tdm_private_layer_commit_handler *lm = NULL, *lmm = NULL;
        tdm_private_output_change_handler *h = NULL, *hh = NULL;
 
@@ -186,9 +226,17 @@ _tdm_display_destroy_private_output(tdm_private_output *private_output)
                free(om);
        }
 
-       LIST_FOR_EACH_ENTRY_SAFE(lm, lmm, &private_output->layer_commit_handler_list, link) {
-               LIST_DEL(&lm->link);
-               free(lm);
+       if (private_output->private_hwc) {
+               private_hwc = private_output->private_hwc;
+               LIST_FOR_EACH_ENTRY_SAFE(hm, hmm, &private_hwc->hwc_commit_handler_list, link) {
+                       LIST_DEL(&hm->link);
+                       free(hm);
+               }
+       } else {
+               LIST_FOR_EACH_ENTRY_SAFE(lm, lmm, &private_output->layer_commit_handler_list, link) {
+                       LIST_DEL(&lm->link);
+                       free(lm);
+               }
        }
 
        LIST_FOR_EACH_ENTRY_SAFE(lm, lmm, &private_output->pending_commit_handler_list, link) {
@@ -218,21 +266,20 @@ _tdm_display_destroy_private_output(tdm_private_output *private_output)
        }
 
        LIST_FOR_EACH_ENTRY_SAFE(c, cc, &private_output->capture_list, link)
-       tdm_capture_destroy_internal(c);
-
-       LIST_FOR_EACH_ENTRY_SAFE(hw, hww, &private_output->hwc_window_list, link)
-       tdm_hwc_window_destroy_internal(hw);
+               tdm_capture_destroy_internal(c);
 
-       LIST_FOR_EACH_ENTRY_SAFE(l, ll, &private_output->layer_list, link)
-       _tdm_display_destroy_private_layer(l);
+       if (private_output->private_hwc) {
+               private_hwc = private_output->private_hwc;
+               LIST_FOR_EACH_ENTRY_SAFE(hw, hww, &private_hwc->hwc_window_list, link)
+                       tdm_hwc_window_destroy_internal(hw);
+               free(private_hwc);
+       } else {
+               LIST_FOR_EACH_ENTRY_SAFE(l, ll, &private_output->layer_list, link)
+                       _tdm_display_destroy_private_layer(l);
+       }
 
        _tdm_display_destroy_caps_output(&private_output->caps);
 
-       tdm_thread_cb_remove(private_output, TDM_THREAD_CB_NEED_VALIDATE, NULL,
-                                                tdm_output_need_validate_handler_thread, NULL);
-       tdm_event_loop_source_remove(private_output->need_validate.event_source);
-       close(private_output->need_validate.event_fd);
-
        /* when destroying output, vblank objects are also destroyed. vblank checks
         * if output object is valid. So delete the output's link at last.
         */
@@ -405,6 +452,8 @@ tdm_display_update_output(tdm_private_module *private_module,
        tdm_func_output *func_output = &private_module->func_output;
        tdm_private_output *private_output = NULL;
        tdm_layer **layers = NULL;
+       tdm_private_hwc *private_hwc = NULL;
+       tdm_hwc *hwc;
        int layer_count = 0, i;
        tdm_error ret;
 
@@ -426,12 +475,9 @@ tdm_display_update_output(tdm_private_module *private_module,
                private_output->pipe = pipe;
                private_output->index = pipe;
 
-               LIST_INITHEAD(&private_output->layer_list);
-               LIST_INITHEAD(&private_output->hwc_window_list);
                LIST_INITHEAD(&private_output->capture_list);
                LIST_INITHEAD(&private_output->vblank_handler_list);
                LIST_INITHEAD(&private_output->output_commit_handler_list);
-               LIST_INITHEAD(&private_output->layer_commit_handler_list);
                LIST_INITHEAD(&private_output->pending_commit_handler_list);
                LIST_INITHEAD(&private_output->change_handler_list);
 
@@ -441,50 +487,85 @@ tdm_display_update_output(tdm_private_module *private_module,
                                                                                                   private_output);
                        private_output->regist_change_cb = 1;
                }
-       }
-
-       /* need_new_caps will be true only in case of "disconnected -> connected" and "connected -> disconnected"
-        * because we have to get new modes.
-        */
-       if (need_new_caps) {
-               _tdm_display_destroy_caps_output(&private_output->caps);
 
                ret = _tdm_display_update_caps_output(private_module, pipe, output_backend, &private_output->caps);
                TDM_RETURN_VAL_IF_FAIL(ret == TDM_ERROR_NONE, ret);
-       }
 
-       layers = func_output->output_get_layers(output_backend, &layer_count, &ret);
-       if (ret != TDM_ERROR_NONE) {
-               free(layers);
-               return ret;
+               if (private_output->caps.status == TDM_OUTPUT_CONN_STATUS_DISCONNECTED)
+                       private_output->current_mode = NULL;
+
+               /* get the tdm_hwc object */
+               if (private_output->caps.capabilities&TDM_OUTPUT_CAPABILITY_HWC) {
+                       hwc = func_output->output_get_hwc(output_backend, &ret);
+                       TDM_RETURN_VAL_IF_FAIL(hwc != NULL, ret);
+
+                       private_hwc = calloc(1, sizeof(tdm_private_hwc));
+                       TDM_RETURN_VAL_IF_FAIL(private_hwc != NULL, TDM_ERROR_OUT_OF_MEMORY);
+
+                       private_hwc->private_module = private_module;
+                       private_hwc->private_output = private_output;
+                       private_hwc->hwc_backend = hwc;
+                       private_hwc->index = pipe;
+                       private_hwc->stamp = tdm_helper_get_time();
+                       while (tdm_display_find_hwc_stamp(private_module->private_display, private_hwc->stamp))
+                               private_hwc->stamp++;
+
+                       LIST_INITHEAD(&private_hwc->hwc_window_list);
+                       LIST_INITHEAD(&private_hwc->hwc_commit_handler_list);
+
+                       private_output->private_hwc = private_hwc;
+               } else {
+                       LIST_INITHEAD(&private_output->layer_list);
+                       LIST_INITHEAD(&private_output->layer_commit_handler_list);
+               }
+       } else {
+               /* need_new_caps will be true only in case of "disconnected -> connected" and "connected -> disconnected"
+                * because we have to get new modes.
+                */
+               if (need_new_caps) {
+                       _tdm_display_destroy_caps_output(&private_output->caps);
+
+                       ret = _tdm_display_update_caps_output(private_module, pipe, output_backend, &private_output->caps);
+                       TDM_RETURN_VAL_IF_FAIL(ret == TDM_ERROR_NONE, ret);
+
+                       if (private_output->caps.status == TDM_OUTPUT_CONN_STATUS_DISCONNECTED)
+                               private_output->current_mode = NULL;
+               }
        }
 
-       for (i = 0; i < layer_count; i++) {
-               ret = _tdm_display_update_layer(private_output, layers[i], i);
+       /* do not use the layer object when the tdm_output has the hwc capability */
+       if ((private_output->caps.capabilities&TDM_OUTPUT_CAPABILITY_HWC) == 0) {
+               layers = func_output->output_get_layers(output_backend, &layer_count, &ret);
                if (ret != TDM_ERROR_NONE) {
                        free(layers);
                        return ret;
                }
-       }
 
-       free(layers);
+               for (i = 0; i < layer_count; i++) {
+                       ret = _tdm_display_update_layer(private_output, layers[i], i);
+                       if (ret != TDM_ERROR_NONE) {
+                               free(layers);
+                               return ret;
+                       }
+               }
+
+               free(layers);
+       }
 
        return TDM_ERROR_NONE;
 }
 
-static tdm_output **
+static void
 _tdm_display_set_main_first(tdm_output **outputs, int index)
 {
        tdm_output *output_tmp = NULL;
 
        if (index == 0)
-               return outputs;
+               return;
 
        output_tmp = outputs[0];
        outputs[0] = outputs[index];
        outputs[index] = output_tmp;
-
-       return outputs;
 }
 
 static tdm_output **
@@ -492,7 +573,6 @@ _tdm_display_get_ordered_outputs(tdm_private_module *private_module, int *count)
 {
        tdm_func_display *func_display = &private_module->func_display;
        tdm_output **outputs = NULL;
-       tdm_output **new_outputs = NULL;
        tdm_output *output_dsi = NULL;
        tdm_output *output_lvds = NULL;
        tdm_output *output_hdmia = NULL;
@@ -526,11 +606,6 @@ _tdm_display_get_ordered_outputs(tdm_private_module *private_module, int *count)
                tdm_caps_output caps;
                memset(&caps, 0, sizeof(tdm_caps_output));
 
-               if (!func_output->output_get_capability) {
-                       TDM_ERR("no output_get_capability()");
-                       goto failed_get_outputs;
-               }
-
                ret = func_output->output_get_capability(outputs[i], &caps);
                if (ret != TDM_ERROR_NONE) {
                        TDM_ERR("output_get_capability() failed");
@@ -575,32 +650,28 @@ _tdm_display_get_ordered_outputs(tdm_private_module *private_module, int *count)
        if (output_connected_count == 0) {
                /* hdmi > dsi > lvds > else */
                if (output_hdmia != NULL)
-                       new_outputs = _tdm_display_set_main_first(outputs, index_hdmia);
+                       _tdm_display_set_main_first(outputs, index_hdmia);
                else if (output_hdmib != NULL)
-                       new_outputs = _tdm_display_set_main_first(outputs, index_hdmib);
+                       _tdm_display_set_main_first(outputs, index_hdmib);
                else if (output_dsi != NULL)
-                       new_outputs = _tdm_display_set_main_first(outputs, index_dsi);
+                       _tdm_display_set_main_first(outputs, index_dsi);
                else if (output_lvds != NULL)
-                       new_outputs = _tdm_display_set_main_first(outputs, index_lvds);
-               else
-                       new_outputs = outputs;
+                       _tdm_display_set_main_first(outputs, index_lvds);
        } else { /* (output_connected_count > 1) */
                /* dsi > lvds > hdmi > else */
                if (output_dsi != NULL)
-                       new_outputs = _tdm_display_set_main_first(outputs, index_dsi);
+                       _tdm_display_set_main_first(outputs, index_dsi);
                else if (output_lvds != NULL)
-                       new_outputs = _tdm_display_set_main_first(outputs, index_lvds);
+                       _tdm_display_set_main_first(outputs, index_lvds);
                else if (output_hdmia != NULL)
-                       new_outputs = _tdm_display_set_main_first(outputs, index_hdmia);
+                       _tdm_display_set_main_first(outputs, index_hdmia);
                else if (output_hdmib != NULL)
-                       new_outputs = _tdm_display_set_main_first(outputs, index_hdmib);
-               else
-                       new_outputs = outputs;
+                       _tdm_display_set_main_first(outputs, index_hdmib);
        }
 
-       private_module->outputs = new_outputs;
+       private_module->outputs = outputs;
 
-       return new_outputs;
+       return outputs;
 
 failed_get_outputs:
        free(outputs);
@@ -609,7 +680,7 @@ failed_get_outputs:
 }
 
 static tdm_error
-_tdm_display_update(tdm_private_display *private_display)
+_tdm_display_setup(tdm_private_display *private_display)
 {
        tdm_private_module *private_module = NULL;
        tdm_error ret = TDM_ERROR_NONE;
@@ -656,7 +727,6 @@ EXTERN tdm_error
 tdm_display_update(tdm_display *dpy)
 {
        TDM_RETURN_VAL_IF_FAIL(dpy != NULL, TDM_ERROR_INVALID_PARAMETER);
-       TDM_RETURN_VAL_IF_FAIL(tdm_display_is_valid(dpy), TDM_ERROR_INVALID_PARAMETER);
 
        TDM_DEPRECATED(NULL);
 
@@ -811,7 +881,7 @@ _tdm_display_load_module_with_file(tdm_private_display *private_display,
        TDM_TRACE_BEGIN("TDM_Load_Backend");
        module = dlopen(path, RTLD_LAZY);
        if (!module) {
-               TDM_ERR("failed to load module: %s(%s)", dlerror(), file);
+               TDM_ERR("%s", dlerror());
                TDM_TRACE_END();
                ret = TDM_ERROR_BAD_MODULE;
                goto failed_load;
@@ -886,7 +956,7 @@ static tdm_error
 _tdm_display_load_modules(tdm_private_display *private_display)
 {
        const char *module_names;
-       tdm_error ret = 0;
+       tdm_error ret = TDM_ERROR_NONE;
        char temp[TDM_PATH_LEN];
        char *arg;
        char *end;
@@ -1013,6 +1083,10 @@ tdm_display_init(tdm_error *error)
        if (ret != TDM_ERROR_NONE)
                goto failed_load;
 
+       ret = tdm_hwc_init(private_display);
+       if (ret != TDM_ERROR_NONE)
+               goto failed_load;
+
        ret = tdm_pp_init(private_display);
        if (ret != TDM_ERROR_NONE)
                goto failed_load;
@@ -1048,7 +1122,7 @@ tdm_display_init(tdm_error *error)
 #endif
 
        TDM_TRACE_BEGIN("TDM_Update_Display");
-       ret = _tdm_display_update(private_display);
+       ret = _tdm_display_setup(private_display);
        TDM_TRACE_END();
        if (ret != TDM_ERROR_NONE)
                goto failed_update;
@@ -1062,15 +1136,6 @@ tdm_display_init(tdm_error *error)
        private_display->init_count = 1;
        g_private_display = private_display;
 
-       tdm_private_module *b = NULL;
-       tdm_private_output *o = NULL;
-       LIST_FOR_EACH_ENTRY(b, &private_display->module_list, link) {
-               LIST_FOR_EACH_ENTRY(o, &b->output_list, link) {
-                       if (o->caps.capabilities & TDM_OUTPUT_CAPABILITY_HWC)
-                               tdm_output_need_validate_event_init(o);
-               }
-       }
-
        /* the COMMIT_PER_VBLANK functionality is ability of an output to support
         * several operational modes (commit_per_vblank modes) related to tdm_commit;
         * this functionality can be turned off which means a default mode */
@@ -1128,8 +1193,6 @@ tdm_display_deinit(tdm_display *dpy)
        if (!private_display)
                return;
 
-       TDM_RETURN_IF_FAIL(tdm_display_is_valid(dpy));
-
        pthread_mutex_lock(&gLock);
 
        private_display->init_count--;
@@ -1186,330 +1249,3 @@ tdm_display_get(void)
        return g_private_display;
 }
 
-INTERN int
-tdm_display_is_valid(tdm_display *dpy)
-{
-       if (dpy != NULL && dpy == g_private_display)
-               return 1;
-       return 0;
-}
-
-INTERN int
-tdm_output_is_valid(tdm_output *output)
-{
-       tdm_private_module *b = NULL;
-       tdm_private_output *o = NULL;
-
-       if (!output || !g_private_display)
-               return 0;
-
-       LIST_FOR_EACH_ENTRY(b, &g_private_display->module_list, link) {
-               LIST_FOR_EACH_ENTRY(o, &b->output_list, link)
-                       if (o == output)
-                               return 1;
-       }
-
-       return 0;
-}
-
-/* LCOV_EXCL_START */
-INTERN int
-tdm_module_check_abi(tdm_private_module *private_module, int abimaj, int abimin)
-{
-       tdm_backend_module *module = private_module->module_data;
-
-       if (TDM_BACKEND_GET_ABI_MAJOR(module->abi_version) < abimaj)
-               return 0;
-
-       if (TDM_BACKEND_GET_ABI_MINOR(module->abi_version) < abimin)
-               return 0;
-
-       return 1;
-}
-
-INTERN tdm_error
-tdm_display_enable_debug_module(const char*modules)
-{
-       char temp[TDM_PATH_LEN];
-       char *arg;
-       char *end;
-
-       snprintf(temp, TDM_PATH_LEN, "%s", modules);
-
-       tdm_debug_module = 0;
-
-       arg = strtok_r(temp, TDM_CONFIG_DELIM, &end);
-       while (arg) {
-               if (!strncmp(arg, "none", 4)) {
-                       tdm_debug_module = 0;
-                       return TDM_ERROR_NONE;
-               }
-               if (!strncmp(arg, "all", 3)) {
-                       tdm_debug_module = 0xFFFFFFFF;
-                       return TDM_ERROR_NONE;
-               }
-               if (!strncmp(arg, "buffer", 6))
-                       tdm_debug_module |= TDM_DEBUG_BUFFER;
-               else if (!strncmp(arg, "event", 5))
-                       tdm_debug_module |= TDM_DEBUG_EVENT;
-               else if (!strncmp(arg, "thread", 6))
-                       tdm_debug_module |= TDM_DEBUG_THREAD;
-               else if (!strncmp(arg, "mutex", 5))
-                       tdm_debug_module |= TDM_DEBUG_MUTEX;
-               else if (!strncmp(arg, "vblank", 6))
-                       tdm_debug_module |= TDM_DEBUG_VBLANK;
-               else if (!strncmp(arg, "commit", 6))
-                       tdm_debug_module |= TDM_DEBUG_COMMIT;
-
-               arg = strtok_r(NULL, TDM_CONFIG_DELIM, &end);
-       }
-
-       TDM_INFO("module debugging... '%s'", modules);
-
-       return TDM_ERROR_NONE;
-}
-
-INTERN tdm_error
-tdm_display_enable_dump(tdm_private_display *private_display, const char *dump_str, char *reply, int *len)
-{
-       char temp[TDM_PATH_LEN] = {0,}, temp2[TDM_PATH_LEN] = {0,};
-       char *path = NULL, *path2;
-       char *arg;
-       char *end;
-
-       snprintf(temp2, TDM_PATH_LEN, "%s", dump_str);
-       path2 = strtostr(temp, TDM_PATH_LEN, temp2, "@");
-       if (!path2 || path2[0] == '\0')
-               path2 = TDM_DUMP_DIR;
-       else
-               path2++;
-
-       tdm_debug_dump = 0;
-
-       snprintf(temp, sizeof(temp), "%s", dump_str);
-       arg = strtok_r(temp, ",", &end);
-       TDM_GOTO_IF_FAIL(arg != NULL, done);
-
-       if (!strncmp(arg, "none", 4)) {
-               tdm_debug_dump = 0;
-               TDM_SNPRINTF(reply, len, "path: %s\n", (tdm_debug_dump_dir) ? : "unknown");
-               if (tdm_debug_dump_dir) {
-                       free(tdm_debug_dump_dir);
-                       tdm_debug_dump_dir = NULL;
-               }
-               goto done;
-       }
-
-       path = tdm_helper_dump_make_directory(path2, reply, len);
-       TDM_GOTO_IF_FAIL(path != NULL, done);
-
-       if (!strncmp(arg, "current", 7)) {
-               tdm_private_module *b = NULL;
-               tdm_private_output *o = NULL;
-
-               if (!private_display) {
-                       TDM_WRN("no private_display");
-                       goto done;
-               }
-
-               LIST_FOR_EACH_ENTRY(b, &private_display->module_list, link) {
-                       LIST_FOR_EACH_ENTRY(o, &b->output_list, link) {
-                               tdm_private_layer *l = NULL;
-                               LIST_FOR_EACH_ENTRY(l, &o->layer_list, link) {
-                                       char str[TDM_PATH_LEN];
-                                       if (l->usable || l->caps.capabilities & TDM_LAYER_CAPABILITY_VIDEO)
-                                               continue;
-                                       if (!l->showing_buffer)
-                                               continue;
-                                       snprintf(str, TDM_PATH_LEN, "layer_%d_%d", o->index, l->index);
-                                       tdm_helper_dump_buffer_str(l->showing_buffer->buffer, path, str);
-                               }
-                       }
-               }
-
-               TDM_SNPRINTF(reply, len, "path: %s\n", path);
-               goto done;
-       }
-
-       TDM_SNPRINTF(reply, len, "dump: %s\n", arg);
-
-       while (arg) {
-               if (!strncmp(arg, "all", 3)) {
-                       tdm_debug_dump = 0xFFFFFFFF;
-                       goto done;
-               } else if (!strncmp(arg, "layer", 5)) {
-                       tdm_debug_dump |= TDM_DUMP_FLAG_LAYER;
-               } else if (!strncmp(arg, "pp", 2)) {
-                       tdm_debug_dump |= TDM_DUMP_FLAG_PP;
-               } else if (!strncmp(arg, "capture", 7)) {
-                       tdm_debug_dump |= TDM_DUMP_FLAG_CAPTURE;
-               } else if (!strncmp(arg, "window", 6)) {
-                       tdm_debug_dump |= TDM_DUMP_FLAG_WINDOW;
-               } else
-                       goto done;
-
-               arg = strtok_r(NULL, ",", &end);
-       }
-
-       if (tdm_debug_dump_dir)
-               free(tdm_debug_dump_dir);
-
-       tdm_debug_dump_dir = strndup(path, TDM_PATH_LEN);
-
-       TDM_INFO("dump... '%s'", dump_str);
-
-done:
-       if (path)
-               free(path);
-
-       return TDM_ERROR_NONE;
-}
-
-static void
-_tdm_display_ttrace_vblank_cb(tdm_vblank *vblank, tdm_error error, unsigned int sequence,
-                                                         unsigned int tv_sec, unsigned int tv_usec, void *user_data)
-{
-       tdm_error ret = TDM_ERROR_NONE;
-
-       TDM_TRACE_MARK("VSYNC");
-
-       ret = tdm_vblank_wait(vblank, 0, 0, 1, _tdm_display_ttrace_vblank_cb, NULL);
-       TDM_RETURN_IF_FAIL(ret == TDM_ERROR_NONE);
-}
-
-INTERN tdm_error
-tdm_display_enable_ttrace_vblank(tdm_display *dpy, tdm_output *output, int enable)
-{
-       tdm_private_display *private_display = dpy;
-       tdm_private_module *private_module = NULL;
-       tdm_private_output *private_output = NULL;
-       const tdm_output_mode *mode = NULL;
-       tdm_vblank *vblank = NULL;
-       tdm_error ret = TDM_ERROR_NONE;
-
-       if (!enable) {
-               LIST_FOR_EACH_ENTRY(private_module, &private_display->module_list, link) {
-                       LIST_FOR_EACH_ENTRY(private_output, &private_module->output_list, link) {
-                               if (private_output->ttrace_vblank)
-                                       tdm_vblank_destroy(private_output->ttrace_vblank);
-                               private_output->ttrace_vblank = NULL;
-                       }
-               }
-               return TDM_ERROR_NONE;
-       }
-
-       private_output = output;
-       TDM_RETURN_VAL_IF_FAIL(private_output != NULL, TDM_ERROR_INVALID_PARAMETER);
-
-       if (private_output->ttrace_vblank)
-               return TDM_ERROR_NONE;
-
-       vblank = tdm_vblank_create(private_display, output, &ret);
-       TDM_RETURN_VAL_IF_FAIL(vblank != NULL, ret);
-
-       ret = tdm_output_get_mode(output, &mode);
-       TDM_GOTO_IF_FAIL(mode != NULL, enable_fail);
-
-       ret = tdm_vblank_set_fps(vblank, mode->vrefresh);
-       TDM_GOTO_IF_FAIL(ret == TDM_ERROR_NONE, enable_fail);
-
-       ret = tdm_vblank_set_enable_fake(vblank, 1);
-       TDM_GOTO_IF_FAIL(ret == TDM_ERROR_NONE, enable_fail);
-
-       ret = tdm_vblank_wait(vblank, 0, 0, 1, _tdm_display_ttrace_vblank_cb, NULL);
-       TDM_GOTO_IF_FAIL(ret == TDM_ERROR_NONE, enable_fail);
-
-       private_output->ttrace_vblank = vblank;
-
-       return TDM_ERROR_NONE;
-
-enable_fail:
-       if (vblank)
-               tdm_vblank_destroy(vblank);
-
-       return ret;
-}
-
-INTERN tdm_error
-tdm_display_enable_ttrace(tdm_private_display *private_display, const char *ttrace, int output_id, char *reply, int *len)
-{
-       char temp[TDM_PATH_LEN];
-       char *arg;
-       char *end;
-       tdm_output *output;
-       tdm_error ret;
-       tdm_output_type type;
-
-       snprintf(temp, TDM_PATH_LEN, "%s", ttrace);
-
-       tdm_ttrace_output = output_id;
-       tdm_ttrace_module = 0;
-
-       output = tdm_display_get_output(private_display, output_id, &ret);
-       if (!output) {
-               TDM_SNPRINTF(reply, len, "can't find the output_id(%d)\n", output_id);
-               return ret;
-       }
-
-       ret = tdm_output_get_output_type(output, &type);
-       if (ret != TDM_ERROR_NONE) {
-               TDM_SNPRINTF(reply, len, "can't find the type of output_id(%d)\n", output_id);
-               return ret;
-       }
-
-       arg = strtok_r(temp, TDM_CONFIG_DELIM, &end);
-       while (arg) {
-               if (!strncmp(arg, "none", 4))
-                       tdm_ttrace_module = 0;
-               else if (!strncmp(arg, "all", 3))
-                       tdm_ttrace_module = 0xFFFFFFFF;
-               else if (!strncmp(arg, "vsync", 5))
-                       tdm_ttrace_module |= TDM_TTRACE_VSYNC;
-               else if (!strncmp(arg, "client_vblank", 13))
-                       tdm_ttrace_module |= TDM_TTRACE_CLIENT_VBLANK;
-               else if (!strncmp(arg, "server_vblank", 13))
-                       tdm_ttrace_module |= TDM_TTRACE_SERVER_VBLANK;
-               else if (!strncmp(arg, "vblank", 6))
-                       tdm_ttrace_module |= TDM_TTRACE_VBLANK;
-               else if (!strncmp(arg, "layer", 5))
-                       tdm_ttrace_module |= TDM_TTRACE_LAYER;
-               else if (!strncmp(arg, "pp", 2))
-                       tdm_ttrace_module |= TDM_TTRACE_PP;
-               else if (!strncmp(arg, "capture", 7))
-                       tdm_ttrace_module |= TDM_TTRACE_CAPTURE;
-               else  {
-                       tdm_ttrace_module = 0;
-                       tdm_display_enable_ttrace_vblank(private_display, NULL, 0);
-                       tdm_server_enable_ttrace_client_vblank(private_display, NULL, 0);
-                       TDM_SNPRINTF(reply, len, "unknown option: '%s'\n", arg);
-                       return TDM_ERROR_NONE;
-               }
-
-               arg = strtok_r(NULL, TDM_CONFIG_DELIM, &end);
-       }
-
-       TDM_SNPRINTF(reply, len, "ttrace debugging... '%s' %x\n", ttrace, tdm_ttrace_module);
-
-       if (tdm_ttrace_module & TDM_TTRACE_VSYNC)
-               tdm_display_enable_ttrace_vblank(private_display, output, 1);
-       else
-               tdm_display_enable_ttrace_vblank(private_display, NULL, 0);
-
-       if (tdm_ttrace_module & TDM_TTRACE_CLIENT_VBLANK)
-               tdm_server_enable_ttrace_client_vblank(private_display, output, 1);
-       else
-               tdm_server_enable_ttrace_client_vblank(private_display, NULL, 0);
-
-       return TDM_ERROR_NONE;
-}
-
-INTERN tdm_error
-tdm_display_enable_fps(tdm_private_display *private_display, int enable)
-{
-       private_display->print_fps = enable;
-
-       TDM_INFO("print fps: %s", (enable) ? "enable" : "disable");
-
-       return TDM_ERROR_NONE;
-}
-/* LCOV_EXCL_STOP */