incorrect define in nrf_sdm.h

[Masterbar]

Hello,

The following define is present in file nrf_sdm.h, SDK 17.0.2:

#define SD_FLASH_SIZE 0x26000

I think it's wrong, it shall be 0x27000. But it's maybe not used at all.

[Håkon Alseth]

Hi,

 

Correct you are! Here's the release notes for S140 v7.2.0, where its clearly stated:

 

Thank you very much for pointing this out to us, and taking the time to file a report on the matter. thank you very much for helping us improve our delivery! 

I will make sure that our SDK team is aware of this bug.

I hope you have a nice weekend!

 

Kind regards,

Håkon

 

*edit*:

It looks like we both read the header definitions incorrect. 

The define doesn't include the size of the MBR region, which is 0x1000. It looks to be correct in that sense:

/** @brief Defines the amount of flash that is used by the SoftDevice.
 *         Add @ref MBR_SIZE to find the first available flash address when the SoftDevice is installed
 *         just above the MBR (the usual case).
 */
#define SD_FLASH_SIZE 0x26000

Where MBR_SIZE is define to:

/** @brief The size that must be reserved for the MBR when a SoftDevice is written to flash.
This is the offset where the first byte of the SoftDevice hex file is written. */
#define MBR_SIZE                (0x1000)

 

Since the "SD_FLASH_SIZE" excludes the MBR region, it is correctly defined.