Why is *p_flash_info variable ?

[Vincent Belaïche]

Just a naive question about the definition of nrf_fstorage_t , it has a member  nrf_fstorage_info_t *p_flash_info.

Why the type isn't nrf_fstorage_info_t const*  ?

Tell me if I am wrong : doesn't it contain only information known at compile time. OK, which structure you point at is not known in advance, but there are only a few predefined cases so they could be hard coded :  and therefore having the const qualifier would allow to point at a structure residing on the on chip flash, which would save some RAM.

[ovrebekk]

Hi Vincent

I will check with the developers if it there is a particular reason why this field wasn't defined as a const pointer. Normally this would imply that the values might change at runtime, but I agree that doesn't seem very likely for these values. 

Best regards
Torbjørn

[ovrebekk]

Hi again

The developer confirmed that this pointer could have been a const pointer. 

Unfortunately I don't expect this to be changed, as the nRF5 SDK is no longer in active development. 

In order to take advantage of this change yourself I would recommend making a local copy of the nrf_fstorage.h file, and update it with this change. 

Best regards
Torbjørn

[Vincent Belaïche]

Thank you for the feedback. Actually this would be a tiny optimization : I was more seeking confirmation of my understanding that this does not change dynamically.

[ovrebekk]

That makes sense. I will consider the case resolved then