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

[AS7712-32X] Add to support PDDF #8040

Merged
merged 3 commits into from
Feb 10, 2022

Conversation

jostar-yang
Copy link
Contributor

Signed-off-by: Jostar Yang jostar_yang@accton.com.tw

Why I did it

Add to support PDDF

How I did it

Implement needed code.

How to verify it

Test via pddf cmd.

root@sonic:/home/admin# pddf_psuutil seninfo
PSU Status Output Voltage (V) Output Current (A) Output Power (W) Temperature1 (C) Fan1 Speed (RPM)


PSU1 NOT PRESENT N/A N/A N/A N/A N/A
PSU2 OK 11.906 17.218 205.0 26.0 6200
root@sonic:/home/admin# pddf_psuutil mfrinfo
PSU Status Manufacturer ID Model Serial Fan Airflow Direction


PSU1 NOT PRESENT N/A N/A N/A N/A
PSU2 OK 3Y POWER YM-2651 SA160N091539046565 EXHAUST

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Jostar Yang <jostar_yang@accton.com.tw>
@lgtm-com
Copy link

lgtm-com bot commented Jul 2, 2021

This pull request introduces 5 alerts when merging 867d386 into 01f51e0 - view on LGTM.com

new alerts:

  • 2 for Unused import
  • 2 for Use of 'global' at module level
  • 1 for Variable defined multiple times

FuzailBrcm
FuzailBrcm previously approved these changes Jul 6, 2021
Copy link
Contributor

@FuzailBrcm FuzailBrcm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good to me.

@prgeor prgeor self-assigned this Feb 9, 2022
@prgeor prgeor merged commit 519dcde into sonic-net:master Feb 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants