Skip to content

boards/weact-stm32h750: Don't disable compilation optimization#18850

Merged
lupyuen merged 1 commit intoapache:masterfrom
acassis:fix_weact-stm32h750
May 5, 2026
Merged

boards/weact-stm32h750: Don't disable compilation optimization#18850
lupyuen merged 1 commit intoapache:masterfrom
acassis:fix_weact-stm32h750

Conversation

@acassis
Copy link
Copy Markdown
Contributor

@acassis acassis commented May 4, 2026

Summary

This board was failing because the CONFIG_DEBUG_NOOPT was enabled.

Impact

Fix CI error

Testing

LD: nuttx
Memory region         Used Size  Region Size  %age Used
            itcm:           0 B        64 KB      0.00%
           flash:       72648 B       128 KB     55.43%
           dtcm1:           0 B        64 KB      0.00%
           dtcm2:           0 B        64 KB      0.00%
            sram:        6400 B       512 KB      1.22%
           sram1:           0 B       128 KB      0.00%
           sram2:           0 B       128 KB      0.00%
           sram3:           0 B        32 KB      0.00%
           sram4:           0 B        64 KB      0.00%
           bbram:           0 B         4 KB      0.00%
CP: nuttx.hex
CP: nuttx.bin

@github-actions github-actions Bot added Size: XS The size of the change in this PR is very small Board: arm labels May 4, 2026
This board was failing because the CONFIG_DEBUG_NOOPT was enabled.

Signed-off-by: Alan C. Assis <acassis@gmail.com>
@acassis acassis force-pushed the fix_weact-stm32h750 branch from 1898b5e to 07dcfdc Compare May 4, 2026 19:11
@acassis
Copy link
Copy Markdown
Contributor Author

acassis commented May 4, 2026

@simbit18 In the past Greg never allowed defconfig to include CONFIG_DEBUG_SYMBOLS=y, but seems like we are allowing it. Maybe we could add this check to CI, what do you think?

@lupyuen lupyuen requested a review from simbit18 May 5, 2026 06:54
@lupyuen lupyuen merged commit fc89dfd into apache:master May 5, 2026
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Board: arm Size: XS The size of the change in this PR is very small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants