Skip to content

Commit 1369a1d

Browse files
teburdjhedberg
authored andcommitted
llext: Cleanups noted in initial PR
There were a few small nits that were pointed out in the initial PR. Fixes the LOG_ macro in the arm elf implementation, replaces a few stray mentions of modules in llext.h with extensions. Signed-off-by: Tom Burdick <[email protected]>
1 parent a3eff88 commit 1369a1d

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

arch/arm/core/elf.c

+2-2
Original file line numberDiff line numberDiff line change
@@ -8,13 +8,13 @@
88
#include <zephyr/llext/llext.h>
99
#include <zephyr/logging/log.h>
1010

11-
LOG_MODULE_DECLARE(elf);
11+
LOG_MODULE_REGISTER(elf, CONFIG_LLEXT_LOG_LEVEL);
1212

1313
/**
1414
* @brief Architecture specific function for relocating partially linked (static) elf
1515
*
1616
* Elf files contain a series of relocations described in a section. These relocation
17-
* instructions are architecture specific and each architecture supporting modules
17+
* instructions are architecture specific and each architecture supporting extensions
1818
* must implement this.
1919
*
2020
* The relocation codes for arm are well documented

include/zephyr/llext/llext.h

+2-2
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ struct llext *llext_by_name(const char *name);
7979
* Only relocatable ELF files are currently supported (partially linked).
8080
*
8181
* @param[in] loader An extension loader that provides input data and context
82-
* @param[in] name A string identifier for the module
82+
* @param[in] name A string identifier for the extension
8383
* @param[out] ext A pointer to a statically allocated llext struct
8484
*
8585
* @retval 0 Success
@@ -143,4 +143,4 @@ void arch_elf_relocate(elf_rel_t *rel, uintptr_t opaddr, uintptr_t opval);
143143
}
144144
#endif
145145

146-
#endif /* ZEPHYR_MODULE_H */
146+
#endif /* ZEPHYR_LLEXT_H */

0 commit comments

Comments
 (0)