Skip to content
Browse files

logging: Fix misspelled CONFIG_RISCV32

The conditional CONFIG_RISCV32 was misspelled in is_rodata() resulting
in the test failing all strings that are in RODATA section.
Additionally, it was using wrong section names for riscv.

Fixes #17065

Signed-off-by: David Leach <>
  • Loading branch information...
dleach02 authored and nashif committed Jun 25, 2019
1 parent 39c8451 commit 70f25eb329b7a1892e73032ba5fa1f93d628bf08
Showing with 2 additions and 3 deletions.
  1. +2 −3 subsys/logging/log_core.c
@@ -125,13 +125,12 @@ static u32_t count_s(const char *str, u32_t nargs)
static bool is_rodata(const void *addr)
#if defined(CONFIG_ARM) || defined(CONFIG_ARC) || defined(CONIFG_RISCV32) || \
#if defined(CONFIG_ARM) || defined(CONFIG_ARC) || defined(CONFIG_X86)
extern const char *_image_rodata_start[];
extern const char *_image_rodata_end[];
#define RO_START _image_rodata_start
#define RO_END _image_rodata_end
#elif defined(CONFIG_NIOS2)
#elif defined(CONFIG_NIOS2) || defined(CONFIG_RISCV32)
extern const char *_image_rom_start[];
extern const char *_image_rom_end[];
#define RO_START _image_rom_start

0 comments on commit 70f25eb

Please sign in to comment.
You can’t perform that action at this time.