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

Network/Hardware: Upgrade to latest STM32F4xx_HAL_Driver and fix network autonegotiation issues with LAN8742 #4342

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

bkerler
Copy link
Contributor

@bkerler bkerler commented Dec 13, 2024

Follow-up PR to #4171.

This upgrades the STM32F4xx_HAL_Driver to the latest commit 8f20a7dfd0b5902e1a5796be78ed2ceddd342085 and includes needed fixes for adapting the upgrade.

Furthermore, this addresses an issue with incorrect network autonegotiation due to incorrect register usage (STM32F4xx reference network registers were used, however LAN8742 registers were needed). Code has been adapted to xBuddy from STM LAN8742 SDK example code.

The firmware has been tested on two Prusa xBuddy MK4 boards. DHCP functions as well full-duplex LAN functions were confirmed to work correctly by two developers.

Copy link
Member

@danopernis danopernis left a comment

Choose a reason for hiding this comment

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

Overall we agree with suggested upgrade, however this will require extensive testing so we are postponing this until after next minor or major release. Thanks for understanding.

@TeTeHacko
Copy link

TeTeHacko commented Dec 15, 2024

Hello, is possible to test this without breaking the appendix on the mainboard? i'm linux admin so i can test networking properly i think. What about some signed closed alpha FW for testers?

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.

3 participants