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-28xb platform #11119

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-28xb 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)

  • 202211

@brandonchuang brandonchuang requested review from a team and lguohan as code owners June 13, 2022 09:23
@brandonchuang brandonchuang force-pushed the as7535-28xb-sfp-refactor branch from 22747e1 to 8494193 Compare June 13, 2022 09:40
@brandonchuang brandonchuang force-pushed the as7535-28xb-sfp-refactor branch from 8494193 to 7f68c49 Compare October 25, 2022 07:53
@lgtm-com
Copy link

lgtm-com bot commented Oct 25, 2022

This pull request introduces 2 alerts when merging 7f68c49a3fe417b9376bd56a83d914f21e7582eb into 158371d - view on LGTM.com

new alerts:

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

@brandonchuang brandonchuang force-pushed the as7535-28xb-sfp-refactor branch from 7f68c49 to 5faa8fa Compare October 25, 2022 09:56
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.

Switch Vendor: Accton
Switch SKU: Accton-AS7535-28XB
CPU: BROADWELL-DE
ASIC Vendor: Broadcom
Switch ASIC: Broadcom BCM88483
Port Configuration: 2x QSFP28 + 2x QSFP-DD + 24x SFP28

Why I did it
Added support to as7535-28xb 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.

Description for the changelog
Initial commit

Signed-off-by: Brandon Chuang <brandon_chuang@edge-core.com>
@brandonchuang brandonchuang force-pushed the as7535-28xb-sfp-refactor branch from 5faa8fa to 4db8413 Compare May 22, 2023 08:24
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