-
Notifications
You must be signed in to change notification settings - Fork 1.7k
[PDDF] multifpgapci driver - Add mdio support #24348
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
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Collaborator
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
cb6be8c to
f22e365
Compare
Collaborator
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
9 tasks
Collaborator
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command. |
89b74ae to
c34941f
Compare
Collaborator
|
/azp run Azure.sonic-buildimage |
Collaborator
|
/azp run Azure.sonic-buildimage |
1 similar comment
Collaborator
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
1 similar comment
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
17a7495 to
99f98de
Compare
Collaborator
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
geans-pin
approved these changes
Nov 10, 2025
Praveen-Brcm
approved these changes
Nov 11, 2025
pavannaregundi
pushed a commit
to pavannaregundi/sonic-buildimage
that referenced
this pull request
Nov 22, 2025
* add mdio protocol support * remove unused variable * undo unrelated change --------- Co-authored-by: Roy Wen <[email protected]>
ashutosh-agrawal
pushed a commit
to AnantKishorSharma/sonic-buildimage
that referenced
this pull request
Nov 30, 2025
* add mdio protocol support * remove unused variable * undo unrelated change --------- Co-authored-by: Roy Wen <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Why I did it
NH-5010 uses the FPGA as an MDIO controller
We desire to use the PDDF FPGA driver to create the kernel MDIO buses
Example usage: https://github.com/sonic-net/sonic-buildimage/blob/master/device/nexthop/x86_64-nexthop_5010-r0/pddf/pddf-device.json.j2#L2005-L2011
Resulting pddf_parse.py commands run for mdio
Resulting mdio buses in sysfs
Through the
mdio_fpga_algo_opsvendors can override the callbacks for the mdio reads and writes without changing the core driver.In addtion an
mdio_accesper an mdio bus sysfs file is exposed for debugging and development purposes.mdio accessAPI:echo "write <phy_addr> <reg_num> <value>" > mdio_accessecho "read <phy_addr> <reg_num>" > mdio_accesscat mdio_accessExample
Work item tracking
How I did it
How to verify it
Which release branch to backport (provide reason below if selected)
Tested branch (Please provide the tested image version)
Description for the changelog
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)