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

[AS7326-56X] Support API2.0 #7443

Open
wants to merge 34 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

PDDF not support 202012 branch.
So add API2.0 to as7326 platform in 202012 branch.

How I did it

Implement API2.0

How to verify it

root@sonic:/usr/local/bin# sensors
ym2651-i2c-17-59
Adapter: i2c-1-mux (chan_id 0)
in3: +0.00 V
fan1: 0 RPM
temp1: +0.0 C
power2: 0.00 W
curr2: +0.00 A

lm75-i2c-15-4b
Adapter: i2c-1-mux (chan_id 6)
temp1: +32.0 C (high = +80.0 C, hyst = +75.0 C)

ym2651-i2c-13-5b
Adapter: i2c-1-mux (chan_id 4)
in3: +12.12 V
fan1: 6144 RPM
temp1: +40.0 C
power2: 151.00 W
curr2: +12.00 A

lm75-i2c-15-49
Adapter: i2c-1-mux (chan_id 6)
temp1: +62.0 C (high = +80.0 C, hyst = +75.0 C)

as7326_56x_fan-i2c-11-66
Adapter: i2c-1-mux (chan_id 2)
fan1: FAULT
fan2: FAULT
fan3: FAULT
fan4: FAULT
fan5: FAULT
fan6: FAULT
fan11: FAULT
fan12: FAULT
fan13: FAULT
fan14: FAULT
fan15: FAULT
fan16: FAULT

pch_haswell-virtual-0
Adapter: Virtual device
temp1: +54.0 C

lm75-i2c-15-4a
Adapter: i2c-1-mux (chan_id 6)
temp1: +36.0 C (high = +80.0 C, hyst = +75.0 C)

lm75-i2c-15-48
Adapter: i2c-1-mux (chan_id 6)
temp1: +61.0 C (high = +80.0 C, hyst = +75.0 C)

coretemp-isa-0000
Adapter: ISA adapter
Package id 0: +57.0 C (high = +82.0 C, crit = +104.0 C)
Core 0: +57.0 C (high = +82.0 C, crit = +104.0 C)
Core 1: +57.0 C (high = +82.0 C, crit = +104.0 C)
Core 2: +57.0 C (high = +82.0 C, crit = +104.0 C)
Core 3: +57.0 C (high = +82.0 C, crit = +104.0 C)

root@sonic:/usr/local/bin# show platform firmware status
Chassis Module Component Version Description


Accton-AS7326-56X N/A CPLD-1 0 CPLD-1
CPLD-2 1 CPLD-2
CPLD-3 1 CPLD-3
BIOS AS7326 V36 20181228 Basic Input/Output System

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

  • 201811
  • 201911
  • 202006
  • [x ] 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 Apr 27, 2021

This pull request introduces 8 alerts and fixes 1 when merging dbbad6b into 59cae24 - view on LGTM.com

new alerts:

  • 3 for Unused import
  • 2 for Wrong name for an argument in a class instantiation
  • 1 for Unused local variable
  • 1 for Variable defined multiple times
  • 1 for Wrong number of arguments in a class instantiation

fixed alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Apr 27, 2021

This pull request fixes 1 alert when merging 9ce97de into 59cae24 - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Apr 28, 2021

This pull request fixes 1 alert when merging 7dccda3 into 59cae24 - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented May 6, 2021

This pull request fixes 1 alert when merging 6d2b4a2 into 97066cb - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented May 7, 2021

This pull request fixes 1 alert when merging 52d87fd into deb9e67 - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented May 10, 2021

This pull request fixes 1 alert when merging a81b709 into d634a85 - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented May 19, 2021

This pull request fixes 1 alert when merging 5bcfd9a into 3db6572 - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented May 21, 2021

This pull request fixes 1 alert when merging bb1dc4a into 4a53031 - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented May 28, 2021

This pull request fixes 1 alert when merging 0fc6abd into babaaaa - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Jun 11, 2021

This pull request fixes 1 alert when merging bc267e4 into c1b380d - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Jun 11, 2021

This pull request fixes 1 alert when merging 44e58c3 into c1b380d - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Jun 21, 2021

This pull request fixes 1 alert when merging 314f093 into 30a05f8 - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Jul 15, 2021

This pull request fixes 1 alert when merging fac7842 into 1d3939b - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Jul 15, 2021

This pull request fixes 2 alerts when merging 81d57db into 1d3939b - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable
  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Sep 16, 2021

This pull request fixes 2 alerts when merging 85c2529 into ce000ea - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable
  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Sep 17, 2021

This pull request fixes 2 alerts when merging 239c6b5 into 038fda8 - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable
  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Oct 15, 2021

This pull request fixes 2 alerts when merging b868e04 into 967d8ab - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable
  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Apr 15, 2022

This pull request fixes 2 alerts when merging 104e559 into 9fd75ff - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable
  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Apr 15, 2022

This pull request fixes 2 alerts when merging b918b6a into 9fd75ff - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable
  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented May 10, 2022

This pull request fixes 2 alerts when merging 0bde3d7 into 8970425 - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable
  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Jul 14, 2022

This pull request introduces 2 alerts and fixes 2 when merging ca3c39a into aa4379d - view on LGTM.com

new alerts:

  • 2 for Unused import

fixed alerts:

  • 1 for Unused local variable
  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Jul 14, 2022

This pull request fixes 2 alerts when merging c74a2c9 into aa4379d - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable
  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Jul 22, 2022

This pull request fixes 2 alerts when merging ed97225 into e2042e2 - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable
  • 1 for Unused import

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Aug 8, 2022

CLA Missing ID CLA Not Signed

@lgtm-com
Copy link

lgtm-com bot commented Aug 8, 2022

This pull request fixes 2 alerts when merging 3d506e8 into 1c8c1a6 - view on LGTM.com

fixed alerts:

  • 1 for Unused local variable
  • 1 for Unused import

@jostar-yang
Copy link
Contributor Author

/easycla

@lgtm-com
Copy link

lgtm-com bot commented Oct 4, 2022

This pull request introduces 1 alert and fixes 2 when merging cda162f into a16843a - view on LGTM.com

new alerts:

  • 1 for Comparison using is when operands support `__eq__`

fixed alerts:

  • 1 for Unused local variable
  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Oct 14, 2022

This pull request introduces 1 alert and fixes 2 when merging 0b5a6a9 into 479e6ea - view on LGTM.com

new alerts:

  • 1 for Comparison using is when operands support `__eq__`

fixed alerts:

  • 1 for Unused local variable
  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Oct 17, 2022

This pull request introduces 1 alert and fixes 2 when merging 14a0d72 into 678edcb - view on LGTM.com

new alerts:

  • 1 for Comparison using is when operands support `__eq__`

fixed alerts:

  • 1 for Unused local variable
  • 1 for Unused import

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