pkarashchenko commented on a change in pull request #4880:
URL: https://github.com/apache/incubator-nuttx/pull/4880#discussion_r756346766



##########
File path: boards/arm/samv7/same70-xplained/Kconfig
##########
@@ -90,4 +90,38 @@ config SAME70XPLAINED_HSMCI0_AUTOMOUNT_UDELAY
 
 endif # SAME70XPLAINED_HSMCI0_AUTOMOUNT
 
+menuconfig SAME70XPLAINED_APP_FORMAT_MCUBOOT
+       bool "MCUboot-bootable format"
+       default n || MCUBOOT_BOOTLOADER

Review comment:
       Ok. I will redo and separate those parts. 
`CONFIG_SAME70XPLAINED_APP_FORMAT_MCUBOOT` will only be an option for linker 
file selection and probably I will add additional option like 
`CONFIG_SAME70XPLAINED_HAVE_OTA_PARTITION`.
   But then comes a question if I can still use `#if 
defined(CONFIG_MCUBOOT_BOOTLOADER)` in board specific files because it is an 
application level config?




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to