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

boards/stm32f429i-disc1: add initial support #8868

Merged
merged 1 commit into from
Jul 3, 2018

Conversation

aabadie
Copy link
Contributor

@aabadie aabadie commented Apr 3, 2018

Contribution description

This PR adds a very basic support for the STM32F429I-DISC1 board.

This boards embeds an ILI9341 display and could be used with #8623 (not tested yet though).

Only one uart, timer and spi are configured for the moment.

Issues/PRs references

None

@aabadie aabadie added Platform: ARM Platform: This PR/issue effects ARM-based platforms CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Area: boards Area: Board ports labels Apr 3, 2018
@aabadie aabadie force-pushed the pr/boards/stm32f429i-disc1 branch from ad5c959 to e96f3ff Compare April 3, 2018 14:03
@kYc0o
Copy link
Contributor

kYc0o commented Apr 3, 2018

Can we finish first #8475 before adding more ST boards?

@bergzand
Copy link
Member

bergzand commented Apr 3, 2018

Also, can this board/CPU be tested for xtimer backoff issues (#8807)

@dylad
Copy link
Member

dylad commented Apr 3, 2018

just a side question here :
Can we use the big external SDRAM of this board in the current state ? :)

#define LED0_PIN GPIO_PIN(PORT_G, 13)
#define LED1_PIN GPIO_PIN(PORT_G, 14)
#define LED2_PIN GPIO_PIN(PORT_B, 13)
#define LED3_PIN GPIO_PIN(PORT_C, 5)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm wondering if LED2 & LED3 should be declared, there are related to USB OTG hardware right ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are right, the datasheet is not calling them user LEDs by the way, I'll remove them.

@dylad
Copy link
Member

dylad commented Apr 9, 2018

I ran a few tests and the board was working fine. I hope I'll be able to run some test with #8623 soon !

@aabadie
Copy link
Contributor Author

aabadie commented Apr 10, 2018

@dylad, I removed the 2 USB OTG LEDs. @kYc0o, I can also adapt this PR on top of #8475 if you prefer.

@dylad
Copy link
Member

dylad commented Apr 10, 2018

This looks good, I'll try to test the screen whenever I find some times :)

@aabadie
Copy link
Contributor Author

aabadie commented Apr 12, 2018

can this board/CPU be tested for xtimer backoff issues

It works with #8807

@aabadie aabadie force-pushed the pr/boards/stm32f429i-disc1 branch from cf58784 to aa7fad6 Compare June 28, 2018 18:49
@aabadie
Copy link
Contributor Author

aabadie commented Jun 28, 2018

Adapted this PR with #8475 and #8549 merged PRs.

Copy link
Member

@astralien3000 astralien3000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested (saul, uart, timer, spi), ACK.

@astralien3000 astralien3000 merged commit 4fcd58a into RIOT-OS:master Jul 3, 2018
@aabadie aabadie deleted the pr/boards/stm32f429i-disc1 branch July 9, 2018 11:43
@cladmi cladmi added this to the Release 2018.07 milestone Jul 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: boards Area: Board ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: ARM Platform: This PR/issue effects ARM-based platforms
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants