-
Notifications
You must be signed in to change notification settings - Fork 3k
Add support for DHCP server with WHD #11320
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
Conversation
@kyle-cypress, thank you for your changes. |
|
||
#include "cy_result.h" | ||
#include "cynetwork_utils.h" | ||
#include "mbed.h" |
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.
please include the required headers instead
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.
Agreed. Will remove "mbed.h" and use only necessary dependency.
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.
Fix pushed
@adbridge Please check why this team was not added to reviewers (I manually added it). |
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.
otherwise looks good
} | ||
} | ||
return temp; | ||
} |
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.
Please add new lines at the end of files
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.
Fix pushed
The source changes are not tagged as Cypress specific in the reviewers config file. |
Integrated via #11367 |
Description
Enable DHCP server for Cypress targets with WHD.
Pull request type
Reviewers
@ARMmbed/team-cypress
Release Notes