-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Ameba] Implement reading and decoding factorydata from flash #23758
Conversation
pankore
commented
Nov 25, 2022
- Ameba implementation of reading and decoding factorydata from flash
- Add FactoryDataDecoder
- Revise FactoryDataProvider to use factorydata from flash
- Change chipinterface initialization order
- Add CHIP_ENABLE_AMEBA_FACTORY_DATA macro to enable/disable flash factorydata
- Revise filepath in mbedtls.cmake
- Requires docker update first
PR #23758: Size comparison from 733afaa to 746f1d0 Increases (6 builds for bl602, bl702, efr32, esp32, psoc6, telink)
Decreases (6 builds for bl602, bl702, psoc6, telink)
Full report (52 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
@pankore it looks like we have an ameba failure on this PR. |
PR #23758: Size comparison from 04c20f6 to 19aa3ca Increases (3 builds for bl602, cc13x2_26x2, psoc6)
Decreases (9 builds for bl702, cc13x2_26x2, esp32, nrfconnect, psoc6, telink)
Full report (42 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
- Add FactoryDataDecoder - Revise FactoryDataProvider - Change chipinterface initialization order - Add CHIP_ENABLE_AMEBA_FACTORY_DATA macro to enable/disable flash factorydata - Revise filepath in mbedtls.cmake
PR #23758: Size comparison from 19c8ea4 to 2909168 Increases (8 builds for bl702, cc13x2_26x2, esp32, nrfconnect, psoc6, telink)
Decreases (5 builds for bl602, cc13x2_26x2, esp32, qpg, telink)
Full report (53 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
PR #23758: Size comparison from 19c8ea4 to e84f7d4 Increases (9 builds for bl602, bl702, cc13x2_26x2, nrfconnect, psoc6, telink)
Decreases (3 builds for esp32, telink)
Full report (53 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
PR #23758: Size comparison from 89df4ac to ab0a66e Increases (10 builds for bl602, bl702, cc13x2_26x2, esp32, qpg, telink)
Decreases (3 builds for cc13x2_26x2, efr32, telink)
Full report (54 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
Changes are in Ameba only and ameba compilation passed. pullapprove seems stuck ("given usersnames could not be requested") so I will bypass CI checks and merge this. |
…t-chip#23758) * [FactoryData] Implement reading and decoding factorydata from flash - Add FactoryDataDecoder - Revise FactoryDataProvider - Change chipinterface initialization order - Add CHIP_ENABLE_AMEBA_FACTORY_DATA macro to enable/disable flash factorydata - Revise filepath in mbedtls.cmake * [Build] Fix warning and errors * [Build] ignore warning for unused label * [factorydata] dont read factorydata if CONFIG_ENABLE_AMEBA_FACTORY_DATA is disabled Co-authored-by: Andrei Litvin <[email protected]>
…t-chip#23758) * [FactoryData] Implement reading and decoding factorydata from flash - Add FactoryDataDecoder - Revise FactoryDataProvider - Change chipinterface initialization order - Add CHIP_ENABLE_AMEBA_FACTORY_DATA macro to enable/disable flash factorydata - Revise filepath in mbedtls.cmake * [Build] Fix warning and errors * [Build] ignore warning for unused label * [factorydata] dont read factorydata if CONFIG_ENABLE_AMEBA_FACTORY_DATA is disabled Co-authored-by: Andrei Litvin <[email protected]>