-
Notifications
You must be signed in to change notification settings - Fork 7k
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
drivers: eth: fix stm32_hal PHY address resolution #77761
Conversation
Hello @scaprile, and thank you very much for your first pull request to the Zephyr project! |
This wrong macro call is my mistake, sorry for that. |
Please rebase as the CI issue has been fixed in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@scaprile Please remove the merge commit.
Fixes: #77705 Fix wrong macro. Tested with all the boards mentioned in that issue, build now works. Signed-off-by: Sergio R. Caprile <[email protected]>
There are no commits on your |
Yeah, I must have screwed something and blamed Github... added on top |
Hi @scaprile! To celebrate this milestone and showcase your contribution, we'd love to award you the Zephyr Technical Contributor badge. If you're interested, please claim your badge by filling out this form: Claim Your Zephyr Badge. Thank you for your valuable input, and we look forward to seeing more of your contributions in the future! 🪁 |
Fixes: #77705
Fix wrong macro.
Tested with all the boards mentioned in that issue, build now works.