Skip to content

[Edgecore][PDDF/as5835-54x] Support PDDF#13700

Closed
jostar-yang wants to merge 24 commits intosonic-net:masterfrom
jostar-yang:as5835_54x_20230207_pddf
Closed

[Edgecore][PDDF/as5835-54x] Support PDDF#13700
jostar-yang wants to merge 24 commits intosonic-net:masterfrom
jostar-yang:as5835_54x_20230207_pddf

Conversation

@jostar-yang
Copy link
Contributor

Signed-off-by: jostar-yang [email protected]

Why I did it

Support PDDF to as5835-54x

How I did it

Implement PDDF related code.

How to verify it

Test show cmd and they all work.
root@as5835-54x:/home/admin# sensors

fan_ctrl-i2c-3-63
Adapter: i2c-1-mux (chan_id 1)
fan1: 0 RPM
fan2: 0 RPM
fan3: 20400 RPM
fan4: 17850 RPM
fan5: 20550 RPM
fan6: 17700 RPM
fan7: 20700 RPM
fan8: 18000 RPM
fan9: 21300 RPM
fan10: 17700 RPM

lm75-i2c-29-4a
Adapter: i2c-2-mux (chan_id 3)
MB_RearRight_temp: +25.0 C (high = +80.0 C, hyst = +75.0 C)

lm75-i2c-28-49
Adapter: i2c-2-mux (chan_id 2)
MB_RearLeft_temp: +26.0 C (high = +80.0 C, hyst = +75.0 C)

psu_pmbus-i2c-12-5b
Adapter: i2c-2-mux (chan_id 2)
in3: 11.82 V
fan1: 3100 RPM
temp1: +31.0 C
power2: 150.00 W
curr2: 10.67 A

psu_pmbus-i2c-11-58
Adapter: i2c-2-mux (chan_id 1)
in3: 1.02 kV
fan1: 0 RPM
temp1: +0.0 C
power2: 0.00 W
curr2: 0.00 A

acpitz-acpi-0
Adapter: ACPI interface
temp1: +0.0 C (crit = +91.0 C)

lm75-i2c-27-4c
Adapter: i2c-2-mux (chan_id 1)
MB_FrontMiddle_temp: +23.5 C (high = +80.0 C, hyst = +75.0 C)

lm75-i2c-26-4b
Adapter: i2c-2-mux (chan_id 0)
CpuBoard_temp: +27.5 C (high = +80.0 C, hyst = +75.0 C)

coretemp-isa-0000
Adapter: ISA adapter
Package id 0: +32.0 C (high = +71.0 C, crit = +91.0 C)
Core 2: +32.0 C (high = +71.0 C, crit = +91.0 C)
Core 6: +29.0 C (high = +71.0 C, crit = +91.0 C)
Core 8: +30.0 C (high = +71.0 C, crit = +91.0 C)
Core 12: +31.0 C (high = +71.0 C, crit = +91.0 C)

root@as5835-54x:/home/admin#
root@as5835-54x:/home/admin# show platform fan
Drawer LED FAN Speed Direction Presence Status Timestamp


FanTray1 green FAN-1F N/A N/A Not Present N/A 20230207 07:35:34
FanTray1 green FAN-1R N/A N/A Not Present N/A 20230207 07:35:34
FanTray2 green FAN-2F 100% exhaust Present OK 20230207 07:35:34
FanTray2 green FAN-2R 100% exhaust Present OK 20230207 07:35:34
FanTray3 green FAN-3F 100% exhaust Present OK 20230207 07:35:34
FanTray3 green FAN-3R 100% exhaust Present OK 20230207 07:35:34
FanTray4 green FAN-4F 100% exhaust Present OK 20230207 07:35:34
FanTray4 green FAN-4R 100% exhaust Present OK 20230207 07:35:34
FanTray5 green FAN-5F 100% exhaust Present OK 20230207 07:35:34
FanTray5 green FAN-5R 100% exhaust Present OK 20230207 07:35:34
N/A off PSU-1 FAN-1 0% Present Not OK 20230207 07:35:36
N/A green PSU-2 FAN-1 17% exhaust Present OK 20230207 07:35:36

root@as5835-54x:~# show platform firmware status
Chassis Module Component Version Description


5835-54X-O-AC-F N/A MB CPLD1 2 Mainboard CPLD 1
MB CPLD2 2 Mainboard CPLD 2
MB CPLD3 2 Mainboard CPLD 3
FAN CPLD 1 Fan board CPLD
CPU CPLD 1 CPU CPLD
BIOS v37.0b.01.02 Basic Input/Output System
root@as5835-54x:~#

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)

@jostar-yang jostar-yang requested a review from lguohan as a code owner February 7, 2023 08:04
@jostar-yang jostar-yang mentioned this pull request Feb 7, 2023
4 tasks
@jostar-yang
Copy link
Contributor Author

Modify sys_led in pddf-device.json. Refer to below PDDF framework PR.
#13475
#13476
#13477
#13474

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jostar-yang
Copy link
Contributor Author

We create new PR for this one. Close the old PR.
#24028

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.

5 participants