-
Notifications
You must be signed in to change notification settings - Fork 3k
Add MTB MXChip EMW3166 #5809
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
Add MTB MXChip EMW3166 #5809
Conversation
@ashok-rao Can you share the test results ? |
Thanks for approving @0xc0170 .. For greentea, there is no DAPLink support available yet for the STM32F412 that is used on this module. So far, I have been using KEIL to flash the axf onto the target and check the Mbed OS port. Any ideas how to proceed with tests? Edit: PR is also raised for mbedls: ARMmbed/mbed-ls#287 |
We shall wait for daplink firmware to test this addition |
@c1728p9 .. can we please build a DAPLink for the EMW3166 with F412? It is currently built on F411. Thank you. |
@ashok-rao this isn't a supported DAPLink board currently, so I can't simply rebuild it. |
Ok. So, we won't be able to run CI on this then.. |
@0xc0170 .. we got a DAPLink build for it from @c1728p9 for this module .. I ran greentea locally on it and attaching the results below.. All of them seem to be commonly failing in lp_ticker / timer .. I suspect this could be because we are using internal clocks on all these MCB's ? or is there something going on with greentea ? Could you please have a look? Thanks. |
/morph build |
Build : SUCCESSBuild number : 893 Triggering tests/morph test |
Exporter Build : FAILUREBuild number : 566 |
Test : SUCCESSBuild number : 738 |
@0xc0170 ..thanks for triggering the test.. I cannot see the reason why exporter has failed. Looks like all PASS from the logs.. ?? |
/morph export-build |
/morph uvisor-test |
Exporter Build : FAILUREBuild number : 567 |
@0xc0170 Looks to me that it's failing on other different platforms while the MTB's are passing.. @studavekar @kegilbert |
Rerunning export test. Looks like one of the exporter machines went offline too soon. /morph export-build |
@0xc0170, what's the particular reason for the Do Not Merge label? |
Exporter Build : SUCCESSBuild number : 569 |
@0xc0170 ..bump! |
@ashok-rao Is it still the case that this needs hardware in CI? Or is that still not possible due to the board not having a DAPlink interface? |
@cmonr .. we have a DAPLink for this now & the tests have passed above.. It doesn't need new HW in CI.. So, should be good to go from my end. Thanks! |
Description
Adding MXChip EMW3166 as a new target.
Status
READY
Migrations
NO
Todos
Deploy notes
Needs addition of new hardware in CI.
@0xc0170 - can you please review the additions?
cc @screamerbg @chris-styles @studavekar