pkarashchenko commented on a change in pull request #4908: URL: https://github.com/apache/incubator-nuttx/pull/4908#discussion_r757888056
########## File path: boards/arm/stm32h7/nucleo-h743zi/Kconfig ########## @@ -27,4 +27,66 @@ config STM32_ROMFS_IMAGEFILE depends on STM32_ROMFS default "../../../rom.img" + +config PROGMEM_OTA_PARTITION + bool "Progmem ota partitions" + default y + select MTD + select MTD_BYTE_WRITE + select MTD_PARTITION + select MTD_PROGMEM + select MTD_PROGMEM_ERASESTATE + +menuconfig APP_FORMAT_MCUBOOT + bool "MCUboot-bootable format" + default y + select PROGMEM_OTA_PARTITION + ---help--- + The MCUboot support of loading the firmware images. + +if APP_FORMAT_MCUBOOT + +config MCUBOOT_BOOTLOADER + bool "MCUboot bootloader application" + default y + ---help--- + This switch between linker scripts to allow an application be + built to another entry point address. + +comment "MCUboot Application Image OTA Update support" + +config OTA_PRIMARY_SLOT_DEVPATH + string "Application image primary slot device path" + default "/dev/ota0" + +config OTA_SECONDARY_SLOT_DEVPATH + string "Application image secondary slot device path" + default "/dev/ota1" + +config OTA_SCRATCH_DEVPATH + string "Scratch partition device path" + default "/dev/otascratch" + +config OTA_PRIMARY_SLOT_OFFSET + hex "MCUboot application image primary slot offset" + default "0x20000" + +config OTA_SECONDARY_SLOT_OFFSET + hex "MCUboot application image secondary slot offset" + default "0x100000" Review comment: Please check here. I see that `OTA_PRIMARY_SLOT_OFFSET`+`OTA_SLOT_SIZE` -> `0x20000 + 0xc0000 == 0xe0000` so there still is `0x20000` free room left ########## File path: boards/arm/stm32h7/nucleo-h743zi/configs/mcuboot-loader/defconfig ########## @@ -0,0 +1,94 @@ +# +# This file is autogenerated: PLEASE DO NOT EDIT IT. +# +# You can use "make menuconfig" to make any modifications to the installed .config file. +# You can then do "make savedefconfig" to generate a new defconfig file that includes your +# modifications. +# +# CONFIG_SPI_EXCHANGE is not set +# CONFIG_STM32H7_SYSCFG is not set +CONFIG_ARCH="arm" +CONFIG_ARCH_BOARD="nucleo-h743zi" +CONFIG_ARCH_BOARD_NUCLEO_H743ZI=y +CONFIG_ARCH_CHIP="stm32h7" +CONFIG_ARCH_CHIP_STM32H743ZI=y +CONFIG_ARCH_CHIP_STM32H7=y +CONFIG_ARCH_STACKDUMP=y +CONFIG_ARMV7M_BASEPRI_WAR=y +CONFIG_ARMV7M_DCACHE=y +CONFIG_ARMV7M_DTCM=y +CONFIG_ARMV7M_ICACHE=y +CONFIG_ARMV7M_MEMCPY=y +CONFIG_ARMV7M_STACKCHECK=y +CONFIG_ARMV7M_USEBASEPRI=y +CONFIG_BOARDCTL_RESET=y +CONFIG_BOARDCTL_ROMDISK=y +CONFIG_BOARD_CUSTOM_LEDS=y +CONFIG_BOARD_LATE_INITIALIZE=y +CONFIG_BOARD_LOOPSPERMSEC=43103 +CONFIG_BOOT_MCUBOOT=y +CONFIG_BUILTIN=y +CONFIG_CLOCK_MONOTONIC=y +CONFIG_DEBUG_FEATURES=y +CONFIG_DEBUG_IRQ=y +CONFIG_DEBUG_SYMBOLS=y +CONFIG_DRVR_READAHEAD=y +CONFIG_DRVR_WRITEBUFFER=y +CONFIG_EXPERIMENTAL=y +CONFIG_FAT_LCNAMES=y +CONFIG_FAT_LFN=y +CONFIG_FS_FAT=y +CONFIG_FS_PROCFS=y +CONFIG_FTL_WRITEBUFFER=y +CONFIG_HAVE_CXX=y +CONFIG_HAVE_CXXINITIALIZE=y +CONFIG_IOB_NBUFFERS=24 +CONFIG_IOB_NCHAINS=24 +CONFIG_LIBC_LZF=y +CONFIG_LIBM=y +CONFIG_MCUBOOT_BOOTLOADER=y +CONFIG_MCUBOOT_VERSION="ecd34c116873bf187a3d7431151249ec856738d0" Review comment: I think I fixed all warnings with https://github.com/mcu-tools/mcuboot/commit/7c890f4b075aed73e4c825ccf875b2fb9ebf2ded and that hash commit is the latest in nuttx-apps, see: https://github.com/apache/incubator-nuttx-apps/commit/16accc17bfe26956c52b9f886135dc5eb8aaf3b1 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org