-
Notifications
You must be signed in to change notification settings - Fork 520
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
[swss] Add support for PoE feature #3238
base: master
Are you sure you want to change the base?
Conversation
|
|
||
SWSS_LOG_NOTICE("POE: Initialize"); | ||
if (!poe.isPoeEnabled()) { | ||
SWSS_LOG_WARN("POE: not enabled"); |
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.
For non-poe platforms we may not want log a warning message. Maybe we can skip this log if "!platformHasPoe"
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.
added an additional check, so that if the platform does not not support PoE (doesn't have a poe_config.json file present) then we do not print anything
acc1489
to
17ff200
Compare
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.
Looks good
Add poesyncd daemon Parse PoE-related configs *What I did* Add support for PoE based on sonic-sairedis support of multiple switches. *How I verified it* Builds and tests using VS builds. Signed-off-by: Serhiy Boiko <serhiy.boiko@plvision.eu>
17ff200
to
f3377d8
Compare
What I did
Add support for PoE based on sonic-sairedis support of multiple switches
This PR depends on sonic-net/sonic-swss-common#894
Why I did it
How I verified it
Builds and tests using VS builds.
Details if related