patrick/zephyr
1
0
Fork 0
Code Issues Pull requests Actions 8 Packages Projects Releases Wiki Activity

llext: Cleanups noted in initial PR

Browse source 
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 <thomas.burdick@intel.com>
This commit is contained in:
Tom Burdick 2023-09-30 08:01:50 -05:00 committed by Johan Hedberg
parent a3eff88792
commit 1369a1d152
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
arch/arm/core/elf.c
View file

@ -8,13 +8,13 @@
#include <zephyr/llext/llext.h>
#include <zephyr/logging/log.h>
LOG_MODULE_DECLARE(elf);
LOG_MODULE_REGISTER(elf, CONFIG_LLEXT_LOG_LEVEL);
/**
* @brief Architecture specific function for relocating partially linked (static) elf
*
* Elf files contain a series of relocations described in a section. These relocation
* instructions are architecture specific and each architecture supporting modules
* instructions are architecture specific and each architecture supporting extensions
* must implement this.
*
* The relocation codes for arm are well documented

4
include/zephyr/llext/llext.h
View file

@ -79,7 +79,7 @@ struct llext *llext_by_name(const char *name);
* Only relocatable ELF files are currently supported (partially linked).
*
* @param[in] loader An extension loader that provides input data and context
* @param[in] name A string identifier for the module
* @param[in] name A string identifier for the extension
* @param[out] ext A pointer to a statically allocated llext struct
*
* @retval 0 Success
@ -143,4 +143,4 @@ void arch_elf_relocate(elf_rel_t *rel, uintptr_t opaddr, uintptr_t opval);
}
#endif
#endif /* ZEPHYR_MODULE_H */
#endif /* ZEPHYR_LLEXT_H */