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

[AS7816-64X]Support API2.0 #7634

Open
wants to merge 26 commits into
base: 202012
Choose a base branch
from

Conversation

jostar-yang
Copy link
Contributor

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

Why I did it

Support API2.0

How I did it

Implement code for API2.0

How to verify it

Test cmd "sensors", "show platform firmware status", "show platform psustatus"
root@sonic:/home/admin# show platform psustatus
PSU Model Serial Voltage (V) Current (A) Power (W) Status LED


PSU 1 N/A N/A 0.00 0.00 0.00 NOT OK red
PSU 2 YM-2851F SA070U461823010101 12.00 18.09 218.00 OK green

show platform firmware status

Chassis Module Component Version Description


Accton-AS7816-64X N/A CPLD1 8 CPLD 1
CPLD2 3 CPLD 2
CPLD3 3 CPLD 3
CPLD4 3 CPLD 4

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

  • 201811
  • 201911
  • 202006
  • 202012

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 May 18, 2021

This pull request introduces 6 alerts when merging c5e4141 into 3db6572 - view on LGTM.com

new alerts:

  • 4 for Unused import
  • 2 for Variable defined multiple times

@lgtm-com
Copy link

lgtm-com bot commented May 20, 2021

This pull request introduces 1 alert when merging d138b88 into d1043e3 - view on LGTM.com

new alerts:

  • 1 for Variable defined multiple times

@lgtm-com
Copy link

lgtm-com bot commented Jul 23, 2021

This pull request introduces 3 alerts when merging 99c8cee into a90280f - view on LGTM.com

new alerts:

  • 2 for Unused import
  • 1 for Unused local variable

@lgtm-com
Copy link

lgtm-com bot commented Aug 17, 2021

This pull request introduces 1 alert when merging 8672369 into feab18a - view on LGTM.com

new alerts:

  • 1 for Unused local variable

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Aug 8, 2022

CLA Missing ID CLA Not Signed

@jostar-yang
Copy link
Contributor Author

/easycla

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.

1 participant