Skip to content

Updates to Maxim Integrated targets. #2832

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

Merged
merged 11 commits into from
Oct 12, 2016
Merged

Conversation

jeremybrodt
Copy link
Contributor

@jeremybrodt jeremybrodt commented Sep 27, 2016

Clean up, updates and fixes for issues uncovered during further testing. All mbed tests pass with one exception. The MAX32620HSP platform only has one LED, causing failure on the two tests that use LED2. Test results are attached.
max32620hsp_test_results_20160927.txt
max32600mbed_test_results_20160927.txt
maxwsnenv_test_results_20160927.txt

@sg- sg- added the needs: CI label Sep 28, 2016
@sg-
Copy link
Contributor

sg- commented Sep 28, 2016

/morph test

@mbed-bot
Copy link

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 973

Build failed!

@bridadan
Copy link
Contributor

Looks like both MAXWSNENV and MAX32600MBED are running out of RAM for the STL test when compiling with GCC:

00:12:24 Building project stl_features (MAXWSNENV, GCC_ARM)
00:12:24 Scan: GCC_ARM
00:12:24 Scan: stl_features
00:12:24 Compile [100.0%]: main.cpp
00:12:24 Link: stl_features
00:12:25 /home/jenkins/bin/gcc-arm-none-eabi-4_9-2015q3/bin/../lib/gcc/arm-none-eabi/4.9.3/../../../../arm-none-eabi/bin/ld: region RAM overflowed with stack
00:12:25 collect2: error: ld returned 1 exit status
00:20:16 Building project stl_features (MAX32600MBED, GCC_ARM)
00:20:16 Scan: GCC_ARM
00:20:16 Scan: stl_features
00:20:16 Compile [100.0%]: main.cpp
00:20:16 Link: stl_features
00:20:16 /home/jenkins/bin/gcc-arm-none-eabi-4_9-2015q3/bin/../lib/gcc/arm-none-eabi/4.9.3/../../../../arm-none-eabi/bin/ld: region RAM overflowed with stack
00:20:16 collect2: error: ld returned 1 exit status

@jeremybrodt Can you reproduce this locally when you merge with master?

@jeremybrodt
Copy link
Contributor Author

Yes. After rebasing, I am getting this compile error. I will adjust the stack size and push another commit.

@jeremybrodt
Copy link
Contributor Author

jeremybrodt commented Oct 5, 2016

This appears to be a side effect of the moving of the 'targets' directory. I moved the library files that I added to the new location.

@jeremybrodt
Copy link
Contributor Author

I reran the mbed test suite with these new commits. All tests pass.
max32620hsp_test_results_20161006.txt

@mazimkhan
Copy link

retest uvisor

@bridadan
Copy link
Contributor

bridadan commented Oct 7, 2016

/morph test

@mbed-bot
Copy link

mbed-bot commented Oct 7, 2016

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test

@bridadan
Copy link
Contributor

bridadan commented Oct 7, 2016

@jeremybrodt Can you please rebase? A cmsis restructure was just merged

@jeremybrodt
Copy link
Contributor Author

Ok. I have rebased and addressed the merge conflicts.

@sg-
Copy link
Contributor

sg- commented Oct 7, 2016

/morph test

@sg- sg- added needs: CI and removed needs: work labels Oct 7, 2016
@mbed-bot
Copy link

mbed-bot commented Oct 7, 2016

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1083

Build failed!

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 10, 2016

@jeremybrodt Maxim targets fail to compile. One of the failures: Error: L6218E: Undefined symbol NVIC_SetVector (referred from hci_drv.o).

MAXWSNENV: ARM and GCC_ARM
MAX32600MBED: GCC_ARM

@jeremybrodt
Copy link
Contributor Author

@0xc0170 I have resolved the issue with MAXWSNENV. I am rerunning mbed tests now. I do not see a build failure on MAX32600MBED, and hci_drv.c does not exist for that platform. Is there a different compile error you are seeing for that platform?

@sg-
Copy link
Contributor

sg- commented Oct 10, 2016

/morph test

@sg- sg- added needs: CI and removed needs: work labels Oct 10, 2016
@mbed-bot
Copy link

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1094

Build failed!

@bridadan
Copy link
Contributor

Getting this error:

14:25:53 Building project stl_features (MAX32600MBED, GCC_ARM)
14:25:53 Scan: GCC_ARM
14:25:53 Scan: stl_features
14:25:53 Compile [100.0%]: main.cpp
14:25:53 Link: stl_features
14:25:54 /home/jenkins/bin/gcc-arm-none-eabi-4_9-2015q3/bin/../lib/gcc/arm-none-eabi/4.9.3/../../../../arm-none-eabi/bin/ld: region RAM overflowed with stack
14:25:54 collect2: error: ld returned 1 exit status

We use this command in CI to turn on some extra metrics:

mbed test --compile -m MAX32600MBED -t GCC_ARM -DMBED_HEAP_STATS_ENABLED=1 -DMBED_STACK_STATS_ENABLED=1

Can you reproduce this error locally?

@jeremybrodt
Copy link
Contributor Author

All tests compile successfully for me using this command. Perhaps we are using different GCC versions.
arm-none-eabi-gcc (GNU Tools for ARM Embedded Processors) 5.4.1 20160609 (release) [ARM/embedded-5-branch revision 237715]

@bridadan
Copy link
Contributor

/morph test

@bridadan
Copy link
Contributor

retest uvisor

@mbed-bot
Copy link

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 0

All builds and test passed!

@sg- sg- merged commit fefde9d into ARMmbed:master Oct 12, 2016
@jeremybrodt jeremybrodt deleted the max326xx_updates branch October 12, 2016 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants