drivers: pinmux: stm32: STM32F7 added ETH support to pinmux #8663
Conversation
This patch adds support for Ethernet in the STM32F7 famlily microcontrollers Signed-off-by: Daniel Egger <[email protected]>
Codecov Report
@@ Coverage Diff @@
## master #8663 +/- ##
==========================================
- Coverage 64.3% 64.29% -0.01%
==========================================
Files 440 440
Lines 42093 42093
Branches 7046 7046
==========================================
- Hits 27066 27065 -1
- Misses 11827 11828 +1
Partials 3200 3200
Continue to review full report at Codecov.
|
|
@erwango Can you review please? This is a broken out patch from a larger patch set to bring up the Nucleo-F746ZG including networking; I tend to get easily confused with too many open construction sites. ;) |
erwango
left a comment
erwango
left a comment
There was a problem hiding this comment.
If this is to add ETH pinmux to the familly, why not adding PC2, PC3?
|
@erwango Because they're not needed. This is modelled after the pinmux configuration for the F4 which also doesn't define them. I also have a similar patch for the F2 once support lands. I don't know what PC2 and PC3 do from the top of my head but I can look it up and add them if you prefer. |
|
@therealprof , well looking to a F7 datasheet, these also have ethernet function |
|
@erwango Those are for MII operation, you should find a few more pins required for MII mode but I'm not sure whether the driver actually supports MII mode at the moment, the typical implementations (also the one on the Nucleos) use RMII mode. |
|
@therealprof , ok, my bad, I'm not too much familiar with eth |
4 participants
This patch adds support for Ethernet in the STM32F7 famlily
microcontrollers
Signed-off-by: Daniel Egger [email protected]