SDK 14.2: NRF_LOG prefix is garbage

RE: SDK 14.2: NRF_LOG prefix is garbage

Jørgen Holmefjord — Tue, 06 Mar 2018 08:29:43 GMT

Do you mean that this issue is present in the SDK examples? I can't find this issue in any of the linker files in SDK 14.2. Could you give an example of a file that have this issue?

RE: SDK 14.2: NRF_LOG prefix is garbage

Lee Daniel Crocker — Fri, 02 Mar 2018 19:24:05 GMT

I ran into this problem as well. It caused nrf_log_init() to overwrite the .bss section--very bad.

There are 431 .ld files in SDK14.2; I did a random sampling and found the error in about 1/3 of them.

RE: SDK 14.2: NRF_LOG prefix is garbage

blueshifting — Sat, 27 Jan 2018 04:21:13 GMT

@carlosderseher yes that fixed the issue for me too. Thanks to you and @joh2 for the explanation!

RE: SDK 14.2: NRF_LOG prefix is garbage

Jørgen Holmefjord — Fri, 26 Jan 2018 10:02:15 GMT

What's making the difference is the missing asterisk at the end of .log_const_data/.log_dynamic_data. When you name a logging module, for instance my_module, it will append a section .log_dynamic_data_my_module to store the logging data. In your original code you were only placing section named exactly .log_dynamic_data, while the fixed code will place all sections starting with .log_dynamic_data.

RE: SDK 14.2: NRF_LOG prefix is garbage

Carlos — Fri, 26 Jan 2018 09:20:21 GMT

@blueshifting is your code working with this solution too?

RE: SDK 14.2: NRF_LOG prefix is garbage

Carlos — Fri, 26 Jan 2018 09:19:07 GMT

As I said, I tried to strip down my project to a minimum working example (MWE). When I could compile it, I didn't get out any debug messages at all. So I took a look at my working bootloader, which is actually the one from the SDK examples, with a few changes for signaling through a buzzer and a main.cpp instead of main.c . There I kicked out everything but log module initialization and it still gave me debug messages. So I thought maybe the sdk_config is wrong. Copy pasting it to my MWE didn't bring logs to live. Next step was to compare the Makefiles, but I couldn't find anything suspicous there. Exchanging the linker scripts though brought up debug messages and they were correct too. So I did compare those two files with notepad++ and there you go, I found the line which did the trick. As I said, I don't know why this makes such a difference, maybe someone else can explain that to us?

RE: SDK 14.2: NRF_LOG prefix is garbage

blueshifting — Thu, 25 Jan 2018 19:59:42 GMT

Thank you @CarlosDerseher (and @joh2)!! For my own curiosity and desire to become a better C/C++ developer (new to this), how did you figure this out?

RE: SDK 14.2: NRF_LOG prefix is garbage

Jørgen Holmefjord — Thu, 25 Jan 2018 19:46:54 GMT

This is great! Thank you for reporting back the solution.

RE: SDK 14.2: NRF_LOG prefix is garbage

Carlos — Thu, 25 Jan 2018 19:01:54 GMT

I have to thank you @Jørgen Holmefjord because while trying to create this minimum working example I stumbled upon the solution. Turns out, there was an error in my linker script. I am not sure why it makes a difference, but with this setting ind the linker script everything works just fine now. Here is the wrong part

.log_dynamic_data :
{
    PROVIDE(__start_log_dynamic_data = .);
    KEEP(*(.log_const_data))  /*THIS LINE IS WRONG*/
    PROVIDE(__stop_log_dynamic_data = .);
} > RAM

.log_const_data :
{
    PROVIDE(__start_log_const_data = .);
    KEEP(*(.log_const_data))  /*THIS LINE IS WRONG*/
    PROVIDE(__stop_log_const_data = .);
} > FLASH

I had to change it to:

.log_dynamic_data :
{
    PROVIDE(__start_log_dynamic_data = .);
    KEEP(*(SORT(.log_dynamic_data*)))  /*CORRECT*/
    PROVIDE(__stop_log_dynamic_data = .);
} > RAM

.log_const_data :
{
    PROVIDE(__start_log_const_data = .);
    KEEP(*(SORT(.log_const_data*)))  /*CORRECT*/
    PROVIDE(__stop_log_const_data = .);
} > FLASH

I'll attach my complete linker script as a reference here: gcc_nrf52.ld

And also my akefile from the minimum working example: Makefile

And here my main.cpp from the minimum working example: main.cpp

And here the sdk_config.h: sdk_config.h

Thank you for your input, I hope this helps some people from loosing their hair :)

RE: SDK 14.2: NRF_LOG prefix is garbage

blueshifting — Tue, 23 Jan 2018 20:16:11 GMT

I'm also having this issue. Something about using C++?