-
Notifications
You must be signed in to change notification settings - Fork 3k
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
latest changes to add arduino support, plus fixes for IOTSS BEID #1576
Conversation
#define DEVICE_SERIAL_FC 1 | ||
|
||
#define DEVICE_I2C 1 | ||
#define DEVICE_I2CSLAVE 1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be set to 0
With all this changes, we would appreciate if you can run singletest script and share the results with us. |
`CORTEX-M0 mbed Test Suite CORTEX-M0 plus mbed Test Suite CORTEX-M3 mbed Test Suite CORTEX-M4 mbed Test Suite CORTEX-M7 mbed Test Suite CORTEX-BEID mbed Test Suite ` |
This one : |
RTC fails cos we don’t have rtc in mps2, |
Feel free to create an issue for people to be aware that RTC fails (did you test some applications if RTC is functional - if it's not , we might just disable it for now until it's fixed? what do you say?) |
RTC test fails because there is no RTC in MPS2. Its disabled in device.h, but the test still runs. |
I misunderstood your previous message, sorry. Those RTC tests should be fixed and should not be run or should print unsupported, which is not a failure? cc @bridadan LGTM |
latest changes to add arduino support, plus fixes for IOTSS BEID
Looks like I missed the RTC test, woops! :) I opened a PR that adds the NOT_SUPPORTED message to that and a few more tests: #1584 |
@PeterConheeney Ok, #1584 was merged so if you rebase your changes on top of master you should see the RTC test (and the FAIL) go away. Would you mind confirming that? |
Hi Brian, if you saw previous message please ignore, my setup was wrong. so far the testing looks good, Regards |
Building project RTC (ARM_MPS2_M0, ARM) The fail is now gone, but i get the above. is there a way to just disable the test if RTC is disabled? |
That error is fine to ignore if you like. But if you really want to make sure the RTC test is completely ignored, add |
Ahh! Just let it fail. The exclude list is like smoke and mirrors |
No description provided.