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

Incorrect ethernet initialization sequence. PHY: LAN8720 (IDFGH-5035) #6821

Closed
Sejny opened this issue Apr 4, 2021 · 2 comments
Closed

Incorrect ethernet initialization sequence. PHY: LAN8720 (IDFGH-5035) #6821

Sejny opened this issue Apr 4, 2021 · 2 comments
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally

Comments

@Sejny
Copy link

Sejny commented Apr 4, 2021

I found bug in IDF ethernet driver. PHY LAN8720 initialization sequence is not correct. PHY clock is supplied by ESP32 (ETH_CLOCK_GPIO17_OUT), tested on ESP-IDF Release v3.3.5

ESP must deliver PHY clock approx 40 clock cycles before PHY reset go HIGH.

Microchip note in LAN8720 datasheet:
LAN8720_RstTiming

Measured on real setup:
LAN8720_ESP32Timing

Iam not skilled for repair this self :(

@espressif-bot espressif-bot added the Status: Opened Issue is new label Apr 4, 2021
@github-actions github-actions bot changed the title Incorrect ethernet initialization sequence. PHY: LAN8720 Incorrect ethernet initialization sequence. PHY: LAN8720 (IDFGH-5035) Apr 4, 2021
@suda-morris
Copy link
Collaborator

Hi @Sejny May I ask how you do the PHY hardware reset in your code? In IDF 3.3, we didn't provide a "reset GPIO" configuration in the ethernet driver.

@Alvin1Zhang
Copy link
Collaborator

Thanks for reporting, fix is available fc60e09, feel free to reopen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally
Projects
None yet
Development

No branches or pull requests

4 participants