feat(elf_loader): Add support for linking other components to ELF file

espressif · Dec 19, 2024 · 2f956b7 · 2f956b7
1 parent 4638426
commit 2f956b7
Show file tree

Hide file tree

Showing 9 changed files with 19 additions and 7 deletions.
diff --git a/components/elf_loader/CHANGELOG.md b/components/elf_loader/CHANGELOG.md
@@ -1,5 +1,10 @@
 # ChangeLog
 
+## v1.0.0 - 2024-12-09
+
+* Added support for the following RISC-V chips: ESP32-P4 and ESP32-C6
+* Added support for linking other components to ELF file
+
 ## v0.1.0 - 2023-08-14
 
 * Add basic ELF loader component
diff --git a/components/elf_loader/README.md b/components/elf_loader/README.md
@@ -22,7 +22,7 @@ Add a dependency on this component in your component or project's idf_component.
 
     ```yml
     dependencies:
-      espressif/elf_loader: "0.*"
+      espressif/elf_loader: "1.*"
     ```
 
 Enable ELF loader in the menuconfig:

diff --git a/components/elf_loader/elf_loader.cmake b/components/elf_loader/elf_loader.cmake
@@ -48,9 +48,9 @@ macro(project_elf project_name)
     endif()
 
     # Link input list of libraries to ELF
-    list(APPEND ELF_COMPONENTS "main")
+    list(PREPEND ELF_COMPONENTS "main")
     if(ELF_COMPONENTS)
-        foreach(c "${ELF_COMPONENTS}")
+        foreach(c ${ELF_COMPONENTS})
             list(APPEND elf_libs "esp-idf/${c}/lib${c}.a")
             list(APPEND elf_dependeces "idf::${c}")
         endforeach()

diff --git a/components/elf_loader/idf_component.yml b/components/elf_loader/idf_component.yml
@@ -1,4 +1,4 @@
-version: "0.1.0"
+version: "1.0.0"
 description: Espressif ELF(Executable and Linkable Format) Loader
 url: https://github.com/espressif/esp-iot-solution/tree/master/components/elf_loader
 dependencies:

diff --git a/components/elf_loader/include/private/elf_symbol.h b/components/elf_loader/include/private/elf_symbol.h
@@ -6,7 +6,7 @@
 
 #pragma once
 
-#include "private/elf_types.h"
+#include <stdint.h>
 
 #ifdef __cplusplus
 extern "C" {

diff --git a/components/elf_loader/src/esp_elf.c b/components/elf_loader/src/esp_elf.c
@@ -477,6 +477,10 @@ int esp_elf_relocate(esp_elf_t *elf, const uint8_t *pbuf)
  */
 int esp_elf_request(esp_elf_t *elf, int opt, int argc, char *argv[])
 {
+    if (!elf || !(elf->entry)) {
+        return -EINVAL;
+    }
+
     elf->entry(argc, argv);
 
     return 0;

diff --git a/examples/elf_loader/build_elf_file_example/CMakeLists.txt b/examples/elf_loader/build_elf_file_example/CMakeLists.txt
@@ -7,5 +7,8 @@ cmake_minimum_required(VERSION 3.5)
 include($ENV{IDF_PATH}/tools/cmake/project.cmake)
 project(hello_world)
 
+# Set other components to link to the ELF file
+# e.g: set(ELF_COMPONENTS "log" "esp_wifi")
+
 include(elf_loader)
 project_elf(hello_world)
diff --git a/examples/elf_loader/build_elf_file_example/main/idf_component.yml b/examples/elf_loader/build_elf_file_example/main/idf_component.yml
@@ -1,4 +1,4 @@
 dependencies:
   elf_loader:
-    version: "0.*"
+    version: "1.*"
     override_path: "../../../../components/elf_loader"
diff --git a/examples/elf_loader/elf_loader_example/main/idf_component.yml b/examples/elf_loader/elf_loader_example/main/idf_component.yml
@@ -6,5 +6,5 @@ targets:
   - esp32p4
 dependencies:
   elf_loader:
-    version: "0.*"
+    version: "1.*"
     override_path: "../../../../components/elf_loader"