[Silabs] Adds fix for hardfault in SiWx917 SoC and generator firmware updates #36610
+30
−6
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Review changes with |
rosahay-silabs
force-pushed
the
feature/update-tinycrypt-support
branch
from
0fdb481 to
6eea9cd
Compare
|
PR #36610: Size comparison from 403d595 to 6eea9cd Full report (66 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
|
PR #36610: Size comparison from 403d595 to 7ed80e2 Full report (65 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, esp32, linux, nrfconnect, nxp, psoc6, qpg, telink, tizen)
|
|
PR #36610: Size comparison from 403d595 to f7577c8 Full report (50 builds for cyw30739, esp32, linux, nrfconnect, nxp, psoc6, telink)
|
rosahay-silabs
force-pushed
the
feature/update-tinycrypt-support
branch
from
f7577c8 to
b546599
Compare
rosahay-silabs
force-pushed
the
feature/update-tinycrypt-support
branch
from
b546599 to
008c961
Compare
|
PR #36610: Size comparison from e402b96 to 008c961 Full report (66 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
|
PR #36610: Size comparison from e402b96 to 4e0170e Full report (66 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
|
PR #36610: Size comparison from 3f62505 to cc8ce84 Full report (26 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, nrfconnect, qpg, stm32, telink, tizen)
|
rosahay-silabs
force-pushed
the
feature/update-tinycrypt-support
branch
from
cc8ce84 to
1868611
Compare
|
PR #36610: Size comparison from d5e9829 to 1868611 Full report (67 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
|
PR #36610: Size comparison from 839a11d to d35bc74 Full report (9 builds for cc13x4_26x4, cc32xx, stm32, tizen)
|
rosahay-silabs
force-pushed
the
feature/update-tinycrypt-support
branch
from
d35bc74 to
5c33dd4
Compare
rosahay-silabs
force-pushed
the
feature/update-tinycrypt-support
branch
from
5c33dd4 to
017548b
Compare
rosahay-silabs
changed the title
|
PR #36610: Size comparison from 839a11d to 18a9e04 Increases above 0.2%:
Full report (71 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
|
PR #36610: Size comparison from 839a11d to 7943888 Full report (3 builds for cc32xx, stm32)
|
rosahay-silabs
force-pushed
the
feature/update-tinycrypt-support
branch
from
7943888 to
18f9c81
Compare
rosahay-silabs
force-pushed
the
feature/update-tinycrypt-support
branch
from
6c3aa07 to
1e5a700
Compare
rosahay-silabs
changed the title
jmartinez-silabs
approved these changes
Jan 16, 2025
jmartinez-silabs
added
the
changing-submodules-on-purpose
|
PR #36610: Size comparison from 283d764 to 1e5a700 Increases above 0.2%:
Full report (56 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
andy31415
approved these changes
Jan 16, 2025
rcasallas-silabs
approved these changes
Jan 16, 2025
|
PR #36610: Size comparison from f2ee6d2 to f466704 Increases above 0.2%:
Full report (71 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changelog
Adds fixes for the hardfault in SiWx917 SoC causing LCD and BTN to not work on the SoC.
Ports over compilation changes from the SLC build system between the SDK updates.
Ports over the changes required for the mbedTLS 3.x support on the generator firmware.
Fixes
The hardfault was caused due to missing DEFINE between build systems. Primarily
SI917_MEMLCDwas causing an issue as without LCD no hardfault was seen.Additionally, including missing source files from the SLC build system.
Testing
This was done manually.
Commissioning was tested with default certificates and with certificate injection using the provisioning scripts.
Additionally, manual button press with LCD changes were tested, and sanity commands for toggle and reading device states was tested on SiWx917 SoC, since these changes are restricted to SiWx917 SoC.