Skip to content

drivers: Add utilities for Ethernet Media-Independent Interface (MII) and use them in STM32 periph_eth#15200

Merged
benpicco merged 3 commits intoRIOT-OS:masterfrom
maribu:mii
Oct 12, 2020
Merged

drivers: Add utilities for Ethernet Media-Independent Interface (MII) and use them in STM32 periph_eth#15200
benpicco merged 3 commits intoRIOT-OS:masterfrom
maribu:mii

Conversation

@maribu
Copy link
Copy Markdown
Member

@maribu maribu commented Oct 9, 2020

Contribution description

The Ethernet Media-Independent Interface (MII) provides a vendor-independent specification of how to interact with Ethernet PHYs. Thus, we should have those definitions in a more generic place than the STM32 specific headers.

This PR creates a utilities header similar to the mii.h in Linux and updates the STM32 code to use it.

Testing procedure

This should not change generated binaries (compared to #15193).

Issues/PRs references

@maribu maribu added Area: cpu Area: CPU/MCU ports Area: drivers Area: Device drivers Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation State: waiting for other PR State: The PR requires another PR to be merged first labels Oct 9, 2020
benpicco
benpicco previously approved these changes Oct 11, 2020
@benpicco benpicco dismissed their stale review October 11, 2020 17:20

wrong PR

@benpicco benpicco removed the State: waiting for other PR State: The PR requires another PR to be merged first label Oct 11, 2020
Add utilities for working with Ethernet Media-Independent Interface for use by
Ethernet drivers.
Use shared MII definitions and utilities instead own definitions.
Use shared MII register definitions from mii.h rather than own definitions.
@maribu
Copy link
Copy Markdown
Member Author

maribu commented Oct 12, 2020

This is now ready for review

@benpicco benpicco added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Oct 12, 2020
Copy link
Copy Markdown
Contributor

@benpicco benpicco left a comment

Choose a reason for hiding this comment

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

Now it's clear where the values from the previous defines are coming from.
The used values do not change.

@benpicco benpicco merged commit c1b0e47 into RIOT-OS:master Oct 12, 2020
@maribu
Copy link
Copy Markdown
Member Author

maribu commented Oct 12, 2020

Thanks! :-)

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

Labels

Area: cpu Area: CPU/MCU ports Area: drivers Area: Device drivers CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants