RE: Bootloader misinterpets gpregret bit fields

testy — Wed, 24 Jun 2020 14:50:38 GMT

[quote userid="15146" url="~/f/nordic-q-a/62674/bootloader-misinterpets-gpregret-bit-fields/256515"]I will report that to the developers.[/quote]

Thanks for looking, but issue still persists (although reported several times) and SDK 17.0.0 is out uncorrected.

With above mentioned rectifications my bootloader runs, when GPREGRET = 0xB1, or 0xB3 or 0xB5 or 0xB7 (application sees then LSbit cleared, i.e. 0xB0, 0xB2, 0xB4, 0xB6), doesn't run, when GPREGRET = 0xB9, 0xBB, 0xBD, 0xBF.

But bootloader doesn't hang forever (contrary to SDK 15-17), when GPREGRET != 0xB1 (fortunately).

RE: Bootloader misinterpets gpregret bit fields

Sigurd — Tue, 23 Jun 2020 14:40:19 GMT

Hi,

Thank you for reporting this.

[quote userid="90822" url="~/f/nordic-q-a/62674/bootloader-misinterpets-gpregret-bit-fields"]think you could argue the first two, although not convincingly. The third is just wrong - it's not checking the bit fields correctly.[/quote]

BOOTLOADER_DFU_START and BOOTLOADER_DFU_SKIP_CRC is defined like this,

#define BOOTLOADER_DFU_START    (BOOTLOADER_DFU_GPREGRET | BOOTLOADER_DFU_START_BIT_MASK)      /**< Magic number to signal that bootloader should enter DFU mode because of signal from Buttonless DFU in main app.*/
#define BOOTLOADER_DFU_SKIP_CRC (BOOTLOADER_DFU_GPREGRET2 | BOOTLOADER_DFU_SKIP_CRC_BIT_MASK)  /**< Magic number to signal that CRC can be skipped due to low power modes.*/

nrf_power_gpregret2_set(nrf_power_gpregret2_get() & ~BOOTLOADER_DFU_SKIP_CRC);

nrf_power_gpregret_set(nrf_power_gpregret_get() & ~BOOTLOADER_DFU_START);

should be correct.

But I agree with you that the third one looks wrong. I will report that to the developers.