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

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

Open
wants to merge 23 commits into
base: master
Choose a base branch
from

Conversation

jostar-yang
Copy link
Contributor

Signed-off-by: jostar-yang jostar_yang@edge-core.com

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)

Signed-off-by: jostar-yang <jostar_yang@edge-core.com>
@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

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.

4 participants