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

[AS5812_54X] Support API2.0 #8760

Closed

Conversation

jostar-yang
Copy link
Contributor

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

Support Implement API2.0.

How I did it

Implement API2 needed code.

How to verify it

Test show cmd,
root@as5812-54x:/home/admin# show platform firmware status
Chassis Module Component Version Description


Accton-AS5812-54X N/A CPLD1 8 CPLD 1
CPLD2 6 CPLD 2
CPLD3 6 CPLD 3
BIOS 5.6.5 Basic Input/Output System

root@as5812-54x:/home/admin# show platform psustatus
PSU Model Serial Voltage (V) Current (A) Power (W) Status LED


PSU 1 CPR-4011-4M11 4011411G23T1314 12.20 3.00 36.00 OK green
PSU 2 CPR-4011-4M11 4011411H51T0149 12.10 2.38 29.00 OK green

root@as5812-54x:/home/admin# show platform temperature
Sensor Temperature High TH Low TH Crit High TH Crit Low TH Warning Timestamp


PSU-1 temp sensor 1 29 80 N/A N/A N/A False 20210903 08:39:28
PSU-2 temp sensor 2 32 80 N/A N/A N/A False 20210903 08:39:28
Temp sensor 1 26 80 N/A N/A N/A False 20210903 08:39:28
Temp sensor 2 26 80 N/A N/A N/A False 20210903 08:39:28
Temp sensor 3 25 80 N/A N/A N/A False 20210903 08:39:28

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

  • 201811
  • 201911
  • 202006
  • 202012
    --> Please merge to 202012br because customer use this branch.
  • 202106
    -->Please merge to 202106br because customer will need this branch.

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 Sep 15, 2021

This pull request introduces 2 alerts when merging dede7c1 into 1863e1f - view on LGTM.com

new alerts:

  • 2 for Unreachable code

@lgtm-com
Copy link

lgtm-com bot commented Sep 16, 2021

This pull request introduces 2 alerts when merging 97b5e92 into 83e7876 - view on LGTM.com

new alerts:

  • 2 for Unreachable code

@lgtm-com
Copy link

lgtm-com bot commented Oct 26, 2021

This pull request introduces 1 alert when merging 12eec92 into 1506737 - view on LGTM.com

new alerts:

  • 1 for Unused import

@seanwu-ec seanwu-ec mentioned this pull request Dec 28, 2021
5 tasks
@jostar-yang jostar-yang requested a review from a team as a code owner June 10, 2022 02:01
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Aug 4, 2022

CLA Missing ID CLA Not Signed

@jostar-yang
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@jostar-yang
Copy link
Contributor Author

Close the PR due to some file has modified and merged to master code base. It cause this PR seems not work. So create new PR:#11701 to modify needed files.

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.

2 participants