[STM32F7] Allow IAR compilation #1604
Merged
Conversation
This file contains hidden or 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
targets.py : add IAR for NUCLEO_F746ZG and DISCO_F746NG iar.py: in case of Cortex-M7F, change it into Cortex-M7 and add --fpu VFPv5_sp I have been confirmed that for Cortex-M4F we don't need to add the --fpu VFPv4_sp
| @@ -32,9 +32,12 @@ class IAR(mbedToolchain): | |||
|
|
|||
| def __init__(self, target, options=None, notify=None, macros=None, silent=False, extra_verbose=False): | |||
| mbedToolchain.__init__(self, target, options, notify, macros, silent, extra_verbose=extra_verbose) | |||
|
|
|||
| if target.core == "Cortex-M7F": | |||
| cpuchoice = "Cortex-M7" | |||
There was a problem hiding this comment.
please align the code here, tabs used?
|
Thanks ! I'll fetch this and test. Related note: I am going to release a new patch for progen, which should fix the IAR export for F746ZG target. |
|
@adustm Please update progen, and you can test to export to IAR. |
Let's create an issue, add there details what errors are there. Thanks |
|
There's a typo in targets for F746Z, if you can correct that one, it should work, I'll fix that |
|
Tested locally with the template fix I have just merged, builds without errors. |
|
LGTM |
0xc0170
added a commit
that referenced
this pull request
Mar 11, 2016
[STM32F7] Allow IAR compilation
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.
targets.py : add IAR for NUCLEO_F746ZG and DISCO_F746NG
iar.py: in case of Cortex-M7F, change it into Cortex-M7 and add --fpu VFPv5_sp
I have been confirmed that for Cortex-M4F we don't need to add the --fpu VFPv4_sp
test results
DTCT_1 EXAMPLE_1 MBED_10 MBED_11 MBED_12 MBED_16 MBED_2 MBED_23 MBED_24 MBED_25 MBED_26 MBED_34 MBED_37 MBED_38 MBED_A1 MBED_A21 MBED_A9 MBED_BUSOUT RTOS_1 RTOS_2 RTOS_3 RTOS_4 RTOS_5 RTOS_6 RTOS_7 RTOS_8
ARM DISCO_F746NG OK OK OK OK OK NOT_SUPPORTED OK OK OK OK OK OK OK OK OK OK OK OK OK OK OK OK OK OK OK OK
GCC_ARM DISCO_F746NG OK OK OK OK OK NOT_SUPPORTED OK OK OK OK OK OK OK OK OK OK OK OK OK OK OK OK TIMEOUT TIMEOUT OK OK
IAR DISCO_F746NG OK OK OK OK OK NOT_SUPPORTED OK OK OK OK OK OK OK OK OK OK OK OK BUILD_FAILED BUILD_FAILED BUILD_FAILED BUILD_FAILED BUILD_FAILED BUILD_FAILED BUILD_FAILED BUILD_FAILED
uARM DISCO_F746NG OK OK OK OK OK NOT_SUPPORTED OK OK OK OK OK OK OK OK OK OK OK OK OK OK TIMEOUT OK OK OK OK OK
RTOS tests don't compile for IAR toolchain. I suggest that we investigate this separately.