Skip to content

MACHINE: resolve unsafe 'STM32MP_SSBL1_PROGRAMMER_NAME' empty override#108

Open
AdrianDC wants to merge 2 commits intoSTMicroelectronics:scarthgapfrom
AdrianDC:scarthgap/machine-variables-empty
Open

MACHINE: resolve unsafe 'STM32MP_SSBL1_PROGRAMMER_NAME' empty override#108
AdrianDC wants to merge 2 commits intoSTMicroelectronics:scarthgapfrom
AdrianDC:scarthgap/machine-variables-empty

Conversation

@AdrianDC
Copy link

Continuation of the PR #101 by bhstalel

Three commits :

  • Fix misuse of STM32MP_METADATA_NAME :

    • Rebased from the original MR over current scarthgap changes
  • MACHINE: resolve 'STM32MP_METADATA_ENABLE' integer conditions :

    • Fix the conditional changes because if '0' is always true being a string, needs to be an integer evaluation
  • MACHINE: resolve unsafe 'STM32MP_SSBL1_PROGRAMMER_NAME' empty override :

    • Apply the same logic for STM32MP_SSBL1_PROGRAMMER_NAME, empty by default, and related to the original PR issues

As usual, feel free to integrate the commits to ST internal releases and to preserve the commits current emails 👍 .

Thanks !

Signed-off-by: Adrian DC <radian.dc@gmail.com>
Resolve empty default variable ':{VARIABLE}' overrides
> WARNING: <..>.bb: Variable key FLASHLAYOUT_PARTITION_ENABLE
> :${STM32MP_SSBL1_PROGRAMMER_NAME} (-) replaces original key
---

Signed-off-by: Adrian DC <radian.dc@gmail.com>
@AdrianDC AdrianDC force-pushed the scarthgap/machine-variables-empty branch from 99efb7b to 0d3b875 Compare February 18, 2026 22:35
@BernardPuel
Copy link
Contributor

Thanks a lot for your contribution !
Your corrections (2 commits) were integrated on top of the v6.2.0 release.

Have a nice day !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants