[Telink] Improve B92 retention target & Update builds to docker version 93 #36552
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
- cut down the ramcost for b92 retention mode. - move the .ramcode to retention . Signed-off-by: Haiwen Xia <[email protected]>
|
Review changes with |
|
PR #36552: Size comparison from 8fe4596 to 107cbcc Full report (3 builds for cc32xx, stm32)
|
s07641069
commented
Nov 19, 2024
s07641069
force-pushed
the
master_b92_retention
branch
from
107cbcc
to
bd976f7
Compare
|
PR #36552: Size comparison from 8fe4596 to bd976f7 Increases above 0.2%:
Full report (81 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
|
This PR require new docker image #36563 |
s07641069
changed the title
s07641069
changed the title
haiwentelink
force-pushed
the
master_b92_retention
branch
from
bd976f7
to
1dff5aa
Compare
|
PR #36552: Size comparison from c197bc2 to 72e4e7d Full report (19 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, nrfconnect, qpg, stm32, tizen)
|
haiwentelink
force-pushed
the
master_b92_retention
branch
from
7f4a89b
to
17fb739
Compare
- change the kconfig.default value . - merge the same setting for save ram. Signed-off-by: Haiwen Xia <[email protected]>
haiwentelink
force-pushed
the
master_b92_retention
branch
from
17fb739
to
e1d1659
Compare
|
PR #36552: Size comparison from c197bc2 to e1d1659 Full report (19 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, nrfconnect, qpg, stm32, tizen)
|
Merge ProtectionsYour pull request matches the following merge protections and will not be merged until they are valid. 🟢 🛡 GitHub branch protections and repository rulesets requirementsWonderful, this rule succeeded.
|
|
PR #36552: Size comparison from c197bc2 to 0adefcb Increases above 0.2%:
Full report (81 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
|
@andy31415 PR is ready for review |
andy31415
approved these changes
Nov 20, 2024
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.
Change overview
After cut down the ram cost by matter's change, it can save about 12k ram, so we can have enough ram to keep ramcode (driver), then it will not need load the ramcode every time in thread mode (retention mode) which will save power in thread mode.
This is a balance between ram and power , so if the customer's ram is not enough , just change it back ,but the power will be bigger than before. we select the initial setting to low down the power.