sdk 8.1.0 compilation fails due to redefinition

[Nick Pelis]

Hey Nordic--

I updated my Keil project from SDK v8.0.0 to v8.1.0 and immediately get some compilation errors due to redefinition of symbols.

In nrf_sd_def.h, I think you meant for:

#define NRF_PPI_RESTRICTED              0                        /**< 1 if PPI peripheral is restricted, 0 otherwise. */

to be:

#define SD_PPI_RESTRICTED              0                        /**< 1 if PPI peripheral is restricted, 0 otherwise. */

and similarly for the rest of the #defines in that file.

Then I run into another error about SD_PPI_CHANNELS_USED not being defined anywhere, and it's not, since in my project SOFTDEVICE_PRESENT is switched on and nrf_sd_def.h doesn't contain that symbol.

So what gives? Where should I put the missing SD_PPI_CHANNELS_USED definition? Why wasn't it included in the SDK release?

[MartinBL]

Hi, Are you running a specific example?

[Nick Pelis]

My code started out with the HID mouse example but has since morphed into a completely separate project.

BTW, the issue is with the NordicSemiconductor Pack in Keil. The change I described above I made to nrf_sd_def.h, which on my system is located in C:\Keil_v5\ARM\Pack\NordicSemiconductor\nRF_Softdevice_S110\8.0.0\headers\nrf_sd_def.h.

Perhaps the problem is that there should be an nrf_sd_def.h under an "8.1.0" directory, like \ARM\Pack\NordicSemiconductor\nRF_SoftDevice_S110*8.1.0*\headers? But when I go to the Keil Pack Installer and click the "Check for Updates" option, 8.0.0 is the latest version for the NordicSemiconductor::nRF_SoftDevice_S110 component.

[MartinBL]

Are you able to use the zipped folder structure of the SDK instead of the Pack? We see a lot of these kinds of troubles with people using the packs. In general we recommend the zipped version.

[Nick Pelis]

Ah-ha! I found the issue.

When I go to https://developer.nordicsemi.com/nRF51_SDK/pieces/nRF_SoftDevice_S110/--the same URL the Pack Installer downloads its packs from--the most recent version is NordicSemiconductor.nRF_SoftDevice_S110.8.0.0.pack. I downloaded this .pack file, changed the extension to .zip, and looked at the contents of nrf_sd_def.h, which does NOT contain the v8.1.0 changes.

I also downloaded https://developer.nordicsemi.com/nRF51_SDK/nRF51_SDK_v8.x.x/packs/nRF51_SDK_8.1.0_b6ed55f_packs.zip and confirmed that the changes are missing from this .zip of packs, despite the name of the file.

However, when I go to the link provided and look at the zipped folder structure of the entire SDK, the version of nrf_sd_def.h there does contain the missing #defines.

Therefore this is a release management bug. Whatever process at Nordic is responsible for turning the SDK into packs did not run correctly, so there is a mismatch between what different portions of the SDK contain.

[MartinBL]

Hi,

You are right, this is a bug and is now reported to our SDK team. Thank you for your input.

PS: I'll recommend using the zipped folder structure and Softdevice from here. The pack system is great when it works, but is a very common source of complicated file and source errors like this.