feat(test_driver): implementation with a weak functions only

Author: erhankur

CMakeLists.txt

@@ -15,7 +15,6 @@ else()
 endif()
 
 set(srcs
-    src/core/core.c
     src/flash.c
     src/mem_utils.c
     src/rom_wrappers.c

example/stub_main.c

@@ -9,7 +9,6 @@
 #include <stddef.h>
 #include <string.h>
 
-#include <esp-stub-lib/core.h>
 #include <esp-stub-lib/log.h>
 #include <esp-stub-lib/flash.h>
 #include <esp-stub-lib/err.h>
@@ -137,14 +136,10 @@ int stub_main(int cmd, ...)
         return lib_ret;
     }
 
-    struct stub_target *target = stub_lib_new_target("esp32");
-    if (!target) {
-        return ESP_STUB_FAIL;
-    }
-    STUB_LOGD("Target created: %s\n", target->target_name);
-    stub_lib_test_drv_init(target);
-    stub_lib_test_drv_set(target, 147);
-    stub_lib_test_drv_whoami(target);
+    stub_lib_test_drv_init();
+    stub_lib_test_drv_set(147);
+    stub_lib_test_drv_get();
+    stub_lib_test_drv_whoami();
 
     const struct stub_cmd_handler *handler = cmd_handlers;
     while (handler->handler) {

include/esp-stub-lib/core.h

@@ -6,16 +6,15 @@
 
 #pragma once
 
-#include <esp-stub-lib/core.h>
+#ifdef __cplusplus
+extern "C" {
+#endif // __cplusplus
 
-struct stub_test_drv_ops {
-    int (*init)(void);
-    int (*set)(int value);
-    int (*get)(void);
-    void (*whoami)(void);
-};
+int stub_lib_test_drv_init(void);
+int stub_lib_test_drv_set(int value);
+int stub_lib_test_drv_get(void);
+void stub_lib_test_drv_whoami(void);
 
-int stub_lib_test_drv_init(struct stub_target *target);
-int stub_lib_test_drv_set(struct stub_target *target, int value);
-int stub_lib_test_drv_get(struct stub_target *target);
-void stub_lib_test_drv_whoami(struct stub_target *target);
+#ifdef __cplusplus
+}
+#endif // __cplusplus

include/esp-stub-lib/test_drv.h

@@ -4,20 +4,10 @@
  * SPDX-License-Identifier: Apache-2.0 OR MIT
  */
 
-#include <esp-stub-lib/core.h>
 #include <esp-stub-lib/log.h>
-#include <esp-stub-lib/test_drv.h>
+#include <target/test_drv.h>
 
-static void esp32_test_drv_whoami(void)
+void stub_target_common_test_drv_whoami(void)
 {
     STUB_LOGD("I am esp32 test driver\n");
 }
-
-static const struct stub_test_drv_ops s_esp32_test_drv_ops = {
-    .whoami = esp32_test_drv_whoami,
-};
-
-void stub_target_test_drv_init(struct stub_target *target)
-{
-    target->test_drv_ops = (void *)&s_esp32_test_drv_ops;
-}

src/core/core.c

@@ -4,51 +4,25 @@
  * SPDX-License-Identifier: Apache-2.0 OR MIT
  */
 
-#include <esp-stub-lib/core.h>
 #include <esp-stub-lib/test_drv.h>
 #include <target/test_drv.h>
 
-int stub_lib_test_drv_init(struct stub_target *target)
+int stub_lib_test_drv_init(void)
 {
-    struct stub_test_drv_ops *ops = target->test_drv_ops;
-
-    if (ops && ops->init) {
-        return ops->init();
-    }
-
     return stub_target_common_test_drv_init();
 }
 
-int stub_lib_test_drv_set(struct stub_target *target, int value)
+int stub_lib_test_drv_set(int value)
 {
-    struct stub_test_drv_ops *ops = target->test_drv_ops;
-
-    if (ops && ops->set) {
-        return ops->set(value);
-    }
-
-    // Fallback to common implementation
     return stub_target_common_test_drv_set(value);
 }
 
-int stub_lib_test_drv_get(struct stub_target *target)
+int stub_lib_test_drv_get(void)
 {
-    struct stub_test_drv_ops *ops = target->test_drv_ops;
-
-    if (ops && ops->get) {
-        return ops->get();
-    }
-
     return stub_target_common_test_drv_get();
 }
 
-void stub_lib_test_drv_whoami(struct stub_target *target)
+void stub_lib_test_drv_whoami(void)
 {
-    struct stub_test_drv_ops *ops = target->test_drv_ops;
-
-    if (ops && ops->whoami) {
-        ops->whoami();
-    }
-
     stub_target_common_test_drv_whoami();
 }