Skip to content

[Edgecore][device][platform] Add new platform as4630-54npe#14293

Closed
roger530-ho wants to merge 12 commits intosonic-net:masterfrom
roger530-ho:add_as4630-54npe_api2
Closed

[Edgecore][device][platform] Add new platform as4630-54npe#14293
roger530-ho wants to merge 12 commits intosonic-net:masterfrom
roger530-ho:add_as4630-54npe_api2

Conversation

@roger530-ho
Copy link
Copy Markdown
Contributor

Why I did it

Need to support new platform as4630-54npe

How I did it

Add device files for new platform as4630-54npe
ex: sonic_platform

How to verify it

  1. sonic-cli cmd:
    show platform {fan, temperature, firmware status, syseeprom, psustatus}
  2. sfputil cmd:
    show {presence, eeprom, lpmode}
  3. Verify function by sonic-mgmt tools:
    Test file path: /tests/platform_tests/api/
    Test items: test_chassis.py, test_component.py, test_fan_drawer.py, test_psu.py, test_thermal.py, test_sfp.py

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

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

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@roger530-ho roger530-ho requested a review from lguohan as a code owner March 17, 2023 08:58
roger530-ho and others added 4 commits March 21, 2023 03:36
New Temperature Policy:
    If any fan fail , set fan speed register to 16(100%)
    Default system fan speed set to 12(75%)
    The max value of fan speed register is 16
       LM77(48)+LM75(4B)+LM75(4A) > 145, Set 16
       LM77(48)+LM75(4B)+LM75(4A) < 105, Set 12
       Shutdown DUT:LM77(48)>=75C

The thermal shutdown feature is available in CPLD v6.0 and above.

Signed-off-by: brandonchuang <brandon_chuang@edge-core.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

@StormLiangMS,@liushilongbuaa PR: #14293 is conflict with MS internal repo
Please complete the following PR by pushing fix commit to sonicbld/conflict_prefix/14293-fix
https://msazure.visualstudio.com/One/_git/Networking-acs-buildimage/pullrequest/9244049
Then comment "/azpw ms_conflict" to rerun PR checker.

@liushilongbuaa
Copy link
Copy Markdown
Contributor

@ec-michael-shih , PR is too old.

@ec-michael-shih
Copy link
Copy Markdown
Contributor

@liushilongbuaa Thanks for the advice. I've fixed it.

@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla bot commented Mar 22, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

@roger530-ho
Copy link
Copy Markdown
Contributor Author

Migrated to PR #23933 for a combined review.

@roger530-ho roger530-ho closed this Sep 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants