mcuboot icon indicating copy to clipboard operation
mcuboot copied to clipboard

NuttX mcuboot_config.h missing RAM_START and RAM_SIZE for RAM_LOAD?

Open TimJTi opened this issue 10 months ago • 3 comments

I am wanting to use MCUBOOT with a NuttX project, using RAM load. I believe there is an error in mcuboot_config.h for NuttX.

It is currently:

#ifdef CONFIG_MCUBOOT_RAM_LOAD
#  define MCUBOOT_RAM_LOAD
#endif

but I think it should be more like the Zephyr variant of this file:

#ifdef CONFIG_BOOT_RAM_LOAD
#  define MCUBOOT_RAM_LOAD 1
#  define IMAGE_EXECUTABLE_RAM_START CONFIG_BOOT_IMAGE_EXECUTABLE_RAM_START
#  define IMAGE_EXECUTABLE_RAM_SIZE CONFIG_BOOT_IMAGE_EXECUTABLE_RAM_SIZE
#endif

I have created appropriate Kconfig entries relevant to my board ("platform") for BOOT_IMAGE_EXECUTABLE_RAM_START and BOOT_IMAGE_EXECUTABLE_RAM_SIZE

I am still early on with my journey using mcuboot so if I'm incorrect please advise - but making this change does, at least, allow my early mcuboot/NuttX test code to compile without the #error macro for these config entries.

I can submit a PR if I am right, and if wanted.

TimJTi avatar Jan 15 '25 14:01 TimJTi