BUG in nrf52_common.ld?

RE: BUG in nrf52_common.ld?

Sigurd — Fri, 11 Aug 2017 14:01:00 GMT

Hi Dave_couling ,

Yes, this issue applies to the current SDK versions. The current version of the MDK(8.14.1) does not include this fix. I believe it will be included in the next upcoming version of the MDK.

RE: BUG in nrf52_common.ld?

Dave_couling — Tue, 01 Aug 2017 20:28:47 GMT

Does this bug apply to SDK 12.3.0?

RE: BUG in nrf52_common.ld?

David García Polo — Mon, 12 Jun 2017 07:17:48 GMT

No coding without sleeping.

{code} /* Check if text sections + data exceeds FLASH limit */ DataInitFlashUsed = bss_start - data_start; CodeFlashUsed = __etext; TotalFlashUsed = CodeFlashUsed + DataInitFlashUsed; ASSERT(TotalFlashUsed <= LENGTH(FLASH), "region FLASH overflowed with .data and user data") {code}

RE: BUG in nrf52_common.ld?

David García Polo — Fri, 09 Jun 2017 15:57:03 GMT

yes, I had noticed it does not work. thanks.

RE: BUG in nrf52_common.ld?

Torsten Robitzki — Fri, 09 Jun 2017 15:55:48 GMT

Well, I think my fix would be much more straightforward, but when you fix works as well, then it's once to me, how this is fixed :-) Anyway: Thank you!

RE: BUG in nrf52_common.ld?

David García Polo — Fri, 09 Jun 2017 15:28:09 GMT

I have checked the problem and you are entirely right. Nevertheless instead of your proposed solution what do you think of an assert? (Adding some spacings on underscores to avoid text formatting)

{code} Wrong code, see bellow. {endcode}

The symbols used allow for users to add sections after .text (before .ARM.extab) and after .data (before .bss) and still be covered.

I will modify the linker scripts on a future release if you do not think there is any problem.

RE: BUG in nrf52_common.ld?

Sigurd — Mon, 06 Feb 2017 13:49:21 GMT

Hi,

Thank you for reporting this. I have reported this bug internally. An answer will be added with the official solution/fix when it’s ready.