casaroli opened a new pull request, #20802:
URL: https://github.com/apache/nuttx/pull/20802

   ## Summary
   
   `same70-qmtech:mcuboot-loader` does not link on master: `region 'flash' 
overflowed by 172 bytes`. The CI job Linux (arm-07) fails on every PR because 
of this.
   
   The cause is apache/nuttx-apps#3865: the NSH `mount` command now takes the 
mount flags out of `-o`, and this adds about 370 bytes. Before that change, the 
loader had 196 bytes free in its 128 KiB flash region.
   
   This configuration starts `mcuboot_loader_main`, not NSH, so the NSH `mount` 
command can not be used. This PR sets `CONFIG_NSH_DISABLE_MOUNT=y` in its 
defconfig.
   
   ## Impact
   
   Only `same70-qmtech:mcuboot-loader`. The loader works as before. The image 
is 130444 bytes, with 628 bytes free.
   
   ## Testing
   
   Built on macOS with Arm GNU Toolchain 13.2.Rel1 (the CI version), NuttX 
master and nuttx-apps master:
   
   - before: `arm-none-eabi-ld: region 'flash' overflowed by 172 bytes`
   - after: links; `arm-none-eabi-size`: text 127528, data 2916.
   
   The defconfig comes from `make savedefconfig`. 
`same70-xplained:mcuboot-loader` (51363 bytes) and `samv71-xult:mcuboot-loader` 
(102500 bytes) are not near the limit. `tools/checkpatch.sh -c -u -m -g` 
passes. Not tested on hardware: the change removes only an NSH command that 
this configuration never runs.
   


-- 
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