Skip to content

[AS4630-54PE]Support API2.0 to 202012br #8241

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

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

Conversation

jostar-yang
Copy link
Contributor

Signed-off-by: Jostar Yang [email protected]

Why I did it

Support API2.0 to 202012 branch

How I did it

Add needed code.

How to verify it

Test sensors. Test show cmd.
root@sonic:/home/admin# show platform fan
Drawer LED FAN Speed Direction Presence Status Timestamp


FanTray1 green FAN-1F 100% exhaust Present OK 20210630 04:08:04
FanTray1 green FAN-1R 100% exhaust Present OK 20210630 04:08:04
FanTray2 green FAN-2F 100% exhaust Present OK 20210630 04:08:04
FanTray2 green FAN-2R 100% exhaust Present OK 20210630 04:08:04
FanTray3 green FAN-3F 100% exhaust Present OK 20210630 04:08:04
FanTray3 green FAN-3R 100% exhaust Present OK 20210630 04:08:04
N/A green PSU-1 FAN-1 49% exhaust Present OK 20210630 04:08:04
N/A green PSU-2 FAN-1 48% exhaust Present OK 20210630 04:08:05

root@sonic:/home/admin# sensors
lm77-i2c-14-48
Adapter: i2c-2-mux (chan_id 4)
temp1: +38.5 C (low = +10.0 C, hyst = +12.0 C)
(high = +64.0 C, hyst = +62.0 C)
(crit = +80.0 C, hyst = +78.0 C)

ype1200am-i2c-11-59
Adapter: i2c-2-mux (chan_id 1)
in3: +0.00 V
fan1: 0 RPM
temp1: +39.0 C
power2: 0.00 W
curr2: +0.00 A

ype1200am-i2c-10-58
Adapter: i2c-2-mux (chan_id 0)
in3: +54.06 V
fan1: 12992 RPM
temp1: +31.0 C
power2: 60.00 W
curr2: +1.11 A

as4630_54pe_cpld-i2c-3-60
Adapter: i2c-1-mux (chan_id 1)
fan1: 6612 RPM
fan2: 6612 RPM
fan3: 6498 RPM

lm75-i2c-24-4b
Adapter: i2c-3-mux (chan_id 6)
temp1: +38.5 C (high = +80.0 C, hyst = +75.0 C)

lm75-i2c-25-4a
Adapter: i2c-3-mux (chan_id 7)
temp1: +34.5 C (high = +80.0 C, hyst = +75.0 C)

coretemp-isa-0000
Adapter: ISA adapter
Package id 0: +51.0 C (high = +71.0 C, crit = +91.0 C)
Core 2: +48.0 C (high = +71.0 C, crit = +91.0 C)
Core 6: +47.0 C (high = +71.0 C, crit = +91.0 C)
Core 8: +46.0 C (high = +71.0 C, crit = +91.0 C)
Core 12: +46.0 C (high = +71.0 C, crit = +91.0 C)

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)

@lguohan
Copy link
Collaborator

lguohan commented Jul 24, 2021

please submit pr to master branch, instead of 202012 branch.

@jostar-yang
Copy link
Contributor Author

In as4630, master use PDDF. But PDDF doesn't support in 202012br. So we implement API2 to 202012br only. API2.0 can't work with PDDF due to both they have the sonic_platform folder , but code content is different.

@lgtm-com
Copy link

lgtm-com bot commented Nov 4, 2021

This pull request fixes 1 alert when merging d1a247e into fb844c7 - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

@lgtm-com
Copy link

lgtm-com bot commented Jan 10, 2022

This pull request fixes 1 alert when merging 13a1238 into a0376a6 - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

@lgtm-com
Copy link

lgtm-com bot commented Apr 15, 2022

This pull request fixes 1 alert when merging 24ba4de into 9fd75ff - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

@lgtm-com
Copy link

lgtm-com bot commented May 10, 2022

This pull request fixes 1 alert when merging b2b0e9c into 8970425 - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

@lgtm-com
Copy link

lgtm-com bot commented May 24, 2022

This pull request fixes 1 alert when merging fa980de into 9caf128 - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

@lgtm-com
Copy link

lgtm-com bot commented Jul 27, 2022

This pull request fixes 1 alert when merging c002b1b into db37325 - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Aug 5, 2022

CLA Missing ID CLA Not Signed

@lgtm-com
Copy link

lgtm-com bot commented Aug 5, 2022

This pull request fixes 1 alert when merging 8a5349e into 8bb3863 - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

@jostar-yang
Copy link
Contributor Author

/easycla

@lgtm-com
Copy link

lgtm-com bot commented Oct 4, 2022

This pull request fixes 1 alert when merging f3d8154 into a16843a - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable

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