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

STM32 NUCLEO boards with Ethernet connector: solve conflict with D11 arduino pin #4272

Merged
merged 1 commit into from May 15, 2017

Conversation

Projects
None yet
4 participants
@jeromecoutant
Contributor

jeromecoutant commented May 4, 2017

Description

STM32 NUCLEO boards with Ethernet connector are:

  • NUCLEO_F207ZG
  • NUCLEO_F429ZI
  • NUCLEO_F746ZG
  • NUCLEO_F767ZI

If you are using Ethernet, and not SPI Arduino pins:
=> keep your board with the default configuration

If you are using SPI Arduino pins, and not Ethernet:
=> you should remove the JP6 bridge on the front side of the board

If you need SPI Arduino and Ethernet:
=> you have to patch the NUCLEO board on the back side: set solder bridge SB121 to off, and SB122 to on
=> D11 is no more connected to PA_7 but to PB_5
=> enable ETHERNET_SPI_D11_CONFLICT macro in mbed_app.json file

Status

READY

@0xc0170

This comment has been minimized.

Member

0xc0170 commented May 5, 2017

Looks fine to me.

What about ETHERNET_SPI_D11_CONFLICT - is a macro not a config that an app can override??

The idea about config is that a target provides default config, in this case setting D11 to a default configuration. A user can easily find out the default configuration, read the description what it does, and overwrite it if needed.
In case of macros, they are hidden in the codebase, must be documented somewhere (do we have a place designated for this?) ?

Based on this, I would expect this as a target configuration.

STM32 NUCLEO boards with Ethernet connector: solve conflict with D11 …
…arduino pin

If you are using Ethernet, and not SPI Arduino pins:
=> keep your board with the default configuration

If you are using SPI Arduino pins, and not Ethernet:
=> you should remove the JP6 bridge on the front side of the board

If you need SPI Arduino and Ethernet:
=> you have to patch the NUCLEO board on the back side: set solder bridge SB121 to off, and SB122 to on
=> D11 is no more connected to PA_7 but to PB_5
=> overwrite d11_configuration config value in json file

@jeromecoutant jeromecoutant force-pushed the jeromecoutant:PR_F429_D11 branch to 0016306 May 10, 2017

@jeromecoutant

This comment has been minimized.

Contributor

jeromecoutant commented May 10, 2017

Hi @0xc0170

Please, check the new commit
Target configuration is now used as discussed.

@0xc0170

This comment has been minimized.

Member

0xc0170 commented May 10, 2017

@jeromecoutant Thanks for the update, LGTM

@0xc0170 0xc0170 added needs: CI and removed needs: work labels May 10, 2017

@0xc0170

This comment has been minimized.

Member

0xc0170 commented May 10, 2017

/morph test

@mbed-bot

This comment has been minimized.

mbed-bot commented May 10, 2017

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 176

All builds and test passed!

@0xc0170 0xc0170 merged commit 1705d10 into ARMmbed:master May 15, 2017

4 checks passed

Cam-CI uvisor Build & Test Success
Details
ci/morph-test Job has completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@jeromecoutant jeromecoutant deleted the jeromecoutant:PR_F429_D11 branch May 15, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment