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

Add pin override for NUCLEO_F429ZI. #20

Merged
merged 6 commits into from
May 18, 2017
Merged

Add pin override for NUCLEO_F429ZI. #20

merged 6 commits into from
May 18, 2017

Conversation

simosillankorva
Copy link
Contributor

This is needed as the regular configuration does not work with this board.

@geky
Copy link
Contributor

geky commented May 15, 2017

A quick question, is this only needed when ethernet is in use? It seemed to work fine in the default pinout otherwise.

Copy link
Contributor

@geky geky left a comment

Choose a reason for hiding this comment

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

Also I like the picture 👍

README.md Outdated

The above figure shows how to connect the NUCLEO_F429ZI with the v1.0.0 CI test shield. Note:

- To get the SD Card to work with this platform the CI test shield cannot be connected directly to this board, instead follow the instruction above.
Copy link
Contributor

Choose a reason for hiding this comment

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

Quick nit, could we add that this is only if ethernet is in use?

maybe

- To get the SD Card to work with this platform the CI test shield cannot be connected directly to this board,
+ To get both Ethernet and the SD Card to work with this platform the CI test shield cannot be connected directly to this board,

Choose a reason for hiding this comment

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

@VeliMattiLahtela commented there is a pin conflict also with Wi-Fi, can you please confirm?

Copy link

@Timjun01 Timjun01 left a comment

Choose a reason for hiding this comment

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

Ground wire in CI shield side is in wrong pin. It is now in 5V instead of GND

@MarceloSalazar
Copy link

The image looks good, however the CI test shield is not publicly available nor there is requirement to use it.
Any SD card reader would work, as long as these lines are available through the SPI bus: SCLK, MISO, MOSI, CS. This is the key info to add to the docs.

Copy link

@MarceloSalazar MarceloSalazar left a comment

Choose a reason for hiding this comment

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

Please just add comments about SPI lines required and leave the image as it is.

Janne Kiiskilä added 2 commits May 18, 2017 09:47
Generic pin information, as the CI-shield is just one option. Other adapters will work as well as long as just connect the right pins to the right headers.
One typo fix, any -> Any.
@JanneKiiskila JanneKiiskila merged commit efc6bbc into master May 18, 2017
@JanneKiiskila JanneKiiskila deleted the NUCLEO_F429ZI branch May 18, 2017 07:32
@geky
Copy link
Contributor

geky commented Mar 8, 2018

Hey @JanneKiiskila, @Timjun01, this is low priority, but we're confused why this workaround is needed.

We followed the pins on the CI shield, but it looks like nothing conflicts with the NUCLEO board if you're only using the SD card. Is there a conflict with another peripheral?

cc @kegilbert

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants