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

[device][platform] Added support to as7535-32xb platform #11118

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

brandonchuang
Copy link
Contributor

Signed-off-by: Brandon Chuang brandon_chuang@edge-core.com

Why I did it

Added support to as7535-32xb platform

How I did it

Implement sysfs and sonic_platform

How to verify it

show platform firmware status
show platform syseeprom
show platform summary
show platform psustatus
show platform fan
show platform temperature
show system-health
psuutil
sfputil
sensors

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205

@brandonchuang brandonchuang requested review from a team and lguohan as code owners June 13, 2022 08:21
@lgtm-com
Copy link

lgtm-com bot commented Jun 13, 2022

This pull request introduces 9 alerts when merging fd5cf04f06934fdcd042a1547be9e2ece7b73967 into 201792f - view on LGTM.com

new alerts:

  • 9 for Unnecessary 'else' clause in loop

@brandonchuang brandonchuang force-pushed the as7535-32xb-sfp-refactor branch from fd5cf04 to a5425d3 Compare June 13, 2022 09:43
@brandonchuang brandonchuang force-pushed the as7535-32xb-sfp-refactor branch from a5425d3 to fb0db5d Compare October 25, 2022 09:22
Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

CodeQL found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.

@lgtm-com
Copy link

lgtm-com bot commented Oct 25, 2022

This pull request introduces 2 alerts when merging fb0db5d196319c3e0531a2b70dddf4e29e28b913 into 81ee948 - view on LGTM.com

new alerts:

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

@brandonchuang brandonchuang force-pushed the as7535-32xb-sfp-refactor branch from fb0db5d to 734dbc7 Compare October 25, 2022 09:54
Switch Vendor: Accton
Switch SKU: Accton-AS7535-32XB
CPU: BROADWELL-DE
ASIC Vendor: Broadcom
Switch ASIC: Broadcom BCM88483
Port Configuration: 2x QSFP28 + 2x QSFP-DD + 26x SFP28

Why I did it
Added support to as7535-32xb platform

How I did it
Implement sysfs and sonic_platform

How to verify it
Test sysfs / sensors cmd / sonic_platform / show platform cmd

show platform firmware status
show platform syseeprom
show platform summary
show platform psustatus
show platform fan
show platform temperature
show system-health
psuutil
sfputil
sensors
cat/echo sysfs attributes.

Which release branch to backport (provide reason below if selected)
- [X] 202205

Description for the changelog
Initial commit

Signed-off-by: Brandon Chuang <brandon_chuang@edge-core.com>
@brandonchuang brandonchuang force-pushed the as7535-32xb-sfp-refactor branch from 734dbc7 to aba2d12 Compare May 22, 2023 08:26
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