-
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
Fix m7core #1467
Fix m7core #1467
Conversation
@@ -44,7 +46,7 @@ def __init__(self, target, options=None, notify=None, macros=None, silent=False, | |||
if target.core.startswith("Cortex"): | |||
self.cpu.append("-mthumb") | |||
|
|||
if target.core == "Cortex-M4F": | |||
if target.core == "Cortex-M4F" or target.core == "Cortex-M7F": |
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.
isn't M7 FPU FPUv5? I used -mfpu=fpv5-d16 while building with GCC some time ago
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.
Also, I remember there are two FPU implementations Simple and Double, aren't they?
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.
Good catch, will fix
But in this changelist also i see M7 and M7F defined with SoftFP only and not hardfp? |
@sandeepach it should be hardware floating-point using the soft-float ABI. @bcostm Looking at this, is F746 double or single? In armcc for M7F core, any m7 is considered as single. Now here in gcc as double :(
|
STM F7 has a single fpu but in general a Cortex M7 mcu can have a double fpu - this feature is optional |
@ohagendorf A change here for gcc |
Atmel SAMS70/E70/V70/V71 have double. Better not to have something fixed in case of. On 14 December 2015 09:42:25 CET, Olaf Hagendorf notifications@github.com wrote:
Envoyé de mon téléphone Android avec K-9 Mail. Excusez la brièveté. |
Yes, defintely. But will happen separately to this changeset. We should add a new issue for this. |
I tried with the version 111 of mbed with GNU GCC compiler. |
I was however able to get it working with the mbed-src. Not with the MBED Library. |
uLib does not support floating, does not make sense to have it enabled.
I added new label RTOS_M4_M7 as they share the same files currently. Here are test results:
There are those timeouts , @bcostm might know more? |
Tested today on GCC. Same issue with RTOS_5 and 6. I don't know why.
Tested with uARM and ARM, all tests are in TIMEOUT. I got this message for all the tests:
Tested with IAR, cannot build at all.
|
IAR - can you build a library , does your IAR support m7 ? I had some license issues which I have to resolve, for now as IAR is not released for the board. I receive an exception for older version that cortex m7 is not recognized cpu type. Unknown property means something is wrong with the host tests, wrong baudrate? Use verbose and print all messages. Please create a new issue for this if you are having problems , we will look at this. |
New tests done with latest compilers versions.
Build with IAR v7.40.3.8938 still not working.
|
Can you find out what is IAR saying, print exception on the line 445:
|
I put a print e before the "raise e" but nothing is displayed. |
@bcostm please test