sdk 8.1.0 compilation fails due to redefinition

RE: sdk 8.1.0 compilation fails due to redefinition

MartinBL — Thu, 20 Aug 2015 13:29:54 GMT

Seems like you are right. I have reported it to the SDK team. SD_PPI_RESTRICTED should be 0UL as the PPI is no longer a restricted resource in S110.

These are the defines in the S110 nrf_sd_def.h files in SDK 9:

#define SD_PPI_RESTRICTED               0            /**< 1 if PPI peripheral is restricted, 0 otherwise. */
#define SD_PPI_CHANNELS_USED       		0xFFF0C000uL /**< PPI channels utilized by SotfDevice (not available to th spplication). */
#define SD_PPI_GROUPS_USED              0x0000000CuL /**< PPI groups utilized by SoftDevice (not available to the application). */
#define SD_TIMERS_USED                  0x00000001uL /**< Timers used by SoftDevice. */
#define SD_SWI_USED                     0x0000003CuL /**< Software interrupts used by SoftDevice */

RE: sdk 8.1.0 compilation fails due to redefinition

2R — Sat, 08 Aug 2015 15:27:13 GMT

Same issue in SDK 9.0

Im using a softdevice 110, Now my question;

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

should be

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

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

in nrf_sd_def.h???

RE: sdk 8.1.0 compilation fails due to redefinition

MartinBL — Thu, 21 May 2015 11:03:33 GMT

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.

RE: sdk 8.1.0 compilation fails due to redefinition

Nick Pelis — Wed, 20 May 2015 17:03:39 GMT

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.

RE: sdk 8.1.0 compilation fails due to redefinition

MartinBL — Wed, 20 May 2015 08:58:34 GMT

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.

RE: sdk 8.1.0 compilation fails due to redefinition

Nick Pelis — Tue, 19 May 2015 16:57:20 GMT

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.

RE: sdk 8.1.0 compilation fails due to redefinition

MartinBL — Tue, 19 May 2015 10:17:32 GMT

Hi, Are you running a specific example?