Skip to content
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

Spi f7 #480

Merged
merged 5 commits into from Sep 2, 2017
Merged

Spi f7 #480

merged 5 commits into from Sep 2, 2017

Conversation

ghost
Copy link

@ghost ghost commented Sep 1, 2017

Description

Added specific code for STM32F7 MCU

Motivation and Context

I am very motivated and the context is good as well ;)

How Has This Been Tested? (if applicable)

Result not 100% accurate on Nucleo-F746ZG so far.

Note : too bad that the comments erased my initial comment :(

Christophe Gerbier added 2 commits September 1, 2017 09:34
Backup of changes for SPI with F7 MCU

Signed-off-by: Christophe Gerbier <[email protected]>
Still not working as expected, however.

Signed-off-by: Christophe Gerbier <[email protected]>
@nfbot
Copy link
Member

nfbot commented Sep 1, 2017

Hi @MikroBusNet,

I'm nanoFramework bot.
Thank you for your contribution!

A human will be reviewing it shortly. 😉

in nf-overlay

Signed-off-by: Christophe Gerbier <[email protected]>
@josesimoes
Copy link
Member

Note : too bad that the comments erased my initial comment :(

Know that your original comments are not erased, they remain at the top. The hint text is added afterwards. 😉

@ghost
Copy link
Author

ghost commented Sep 1, 2017

Ok. Perhaps I did not notice. I will see with my next PR ;)

Anyway, Travis is not happy once again. Does it have the SWO option ?

@josesimoes
Copy link
Member

Travis doesn't have SWO enabled because that's a feature typically used during debug. Plus it increases RAM and FLASH usage.

It seems that you've left some SWO call in SPI. That shouldn't be there...

Christophe Gerbier and others added 2 commits September 1, 2017 16:49
To please Travis

Signed-off-by: Christophe Gerbier <[email protected]>
@josesimoes josesimoes added Area: Interpreter Everything related with the interpreter, execution engine and such Series: STM32xx Everything related specifically with STM32 targets labels Sep 2, 2017
@josesimoes josesimoes merged commit 07f98f0 into nanoframework:master Sep 2, 2017
@ghost ghost deleted the spi_F7 branch September 2, 2017 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Interpreter Everything related with the interpreter, execution engine and such Series: STM32xx Everything related specifically with STM32 targets
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants