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

add support for YT8512 phy #9495

Merged
merged 1 commit into from Jul 10, 2023
Merged

Conversation

freakishness
Copy link
Contributor

Summary

Add support for YT8512C / YT8512H (10/100 Mbps Ethernet phy)

Impact

None

Testing

Test on imxrt board(mimxrt1062)

@xiaoxiang781216
Copy link
Contributor

@freakishness could you fix the follow warn:

Error: /home/runner/work/nuttx/nuttx/nuttx/arch/arm/src/imxrt/imxrt_enet.c:1898:1: error: Blank line contains whitespace
Error: /home/runner/work/nuttx/nuttx/nuttx/arch/arm/src/imxrt/imxrt_enet.c:2441:83: error: Long line found
Error: /home/runner/work/nuttx/nuttx/nuttx/arch/arm/src/imxrt/imxrt_enet.c:2445:83: error: Long line found
Error: /home/runner/work/nuttx/nuttx/nuttx/arch/arm/src/imxrt/imxrt_enet.c:2449:83: error: Long line found
Error: /home/runner/work/nuttx/nuttx/nuttx/arch/arm/src/imxrt/imxrt_enet.c:2453:83: error: Long line found
Error: /home/runner/work/nuttx/nuttx/nuttx/arch/arm/src/imxrt/imxrt_enet.c:2455:66: error: Dangling whitespace at the end of line
Error: /home/runner/work/nuttx/nuttx/nuttx/include/nuttx/net/mii.h:1:77: error: Block comments have different lengths
Error: /home/runner/work/nuttx/nuttx/nuttx/include/nuttx/net/mii.h:873:76: error: Block comments have different lengths
Error: /home/runner/work/nuttx/nuttx/nuttx/include/nuttx/net/mii.h:880:1: error: Missing blank line after comment
Error: /home/runner/work/nuttx/nuttx/nuttx/include/nuttx/net/mii.h:891:1: error: Missing blank line after comment
Error: /home/runner/work/nuttx/nuttx/nuttx/include/nuttx/net/mii.h:892:1: error: Missing blank line after comment

@xiaoxiang781216
Copy link
Contributor

@freakishness please rebase and sqaush your patch into one.

@freakishness
Copy link
Contributor Author

@xiaoxiang781216 Sorry about that. I lost my github 2fa key.
I also have learned how to check source format use 'checkpatch.sh'.

@xiaoxiang781216
Copy link
Contributor

@xiaoxiang781216 Sorry about that. I lost my github 2fa key. I also have learned how to check source format use 'checkpatch.sh'.

Ok, I squash your patch, please take a look.

@xiaoxiang781216 xiaoxiang781216 merged commit 899471c into apache:master Jul 10, 2023
26 checks passed
@jerpelea jerpelea added this to To-Add in Release Notes - 12.3.0 Sep 26, 2023
@jerpelea jerpelea moved this from To-Add to done in Release Notes - 12.3.0 Oct 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants