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

led_service: add DARWIN48V platform support #263

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

adamcalabrigo
Copy link
Contributor

Description

Adds initial led_service support for DARWIN48V platform.

Note that this depends on the changes in #257.

Test Plan

FBOSS OSS build passes with CentOS 9 and Linux kernel 6.4.

Validated on DARWIN48V hardware.

data_corral_service loads correctly:

# systemctl status data_corral_service
● data_corral_service.service - Start data_corral_service
     Loaded: loaded (/etc/systemd/system/data_corral_service.service; enabled; preset: disabled)
     Active: active (running) since Tue 2024-10-08 00:39:22 UTC; 2h 9min ago
   Main PID: 4893 (run_data_corral)
      Tasks: 34 (limit: 200901)
     Memory: 4.5M
        CPU: 773ms
     CGroup: /system.slice/data_corral_service.service
             ├─4893 /bin/bash /opt/fboss/bin/run_data_corral_service.sh
             └─4895 /opt/fboss/bin/data_corral_service -config_file /opt/fboss/share/platform_configs/led_manager.json

LEDs are initialized correctly:

I1008 02:54:29.428475  5242 PlatformNameLib.cpp:71] Platform name read from cache: DARWIN48V
I1008 02:54:29.428544  5242 ConfigLib.cpp:48] Using config file: /opt/fboss/share/platform_configs/led_manager.json
I1008 02:54:29.428772  5243 FruPresenceExplorer.cpp:25] Detecting presence of FRUs
I1008 02:54:29.429607  5242 ThriftServer.cpp:836] Using thread manager (resource pools not enabled) on address/port 5971: runtime: thriftFlagNotSet, >
I1008 02:54:29.429641  5243 FruPresenceExplorer.cpp:40] Detected that FAN1 is present
I1008 02:54:29.430628  5243 FruPresenceExplorer.cpp:40] Detected that FAN2 is present
I1008 02:54:29.431602  5243 FruPresenceExplorer.cpp:40] Detected that FAN3 is present
I1008 02:54:29.432599  5243 FruPresenceExplorer.cpp:40] Detected that FAN4 is present
I1008 02:54:29.433598  5243 FruPresenceExplorer.cpp:40] Detected that FAN5 is present
I1008 02:54:29.433636  5243 FruPresenceExplorer.cpp:40] Detected that FAN6 is present
I1008 02:54:29.433667  5243 FruPresenceExplorer.cpp:40] Detected that PSU1 is present
I1008 02:54:29.433696  5243 LedManager.cpp:45] Programming PSU LED with true
I1008 02:54:29.433760  5243 LedManager.cpp:80] Wrote 1 to file /sys/class/leds/psu_led:green:status/brightness
I1008 02:54:29.433803  5243 LedManager.cpp:80] Wrote 0 to file /sys/class/leds/psu_led:red:status/brightness
I1008 02:54:29.433817  5243 LedManager.cpp:55] Programmed PSU LED with presence true
I1008 02:54:29.433829  5243 LedManager.cpp:45] Programming FAN LED with true
I1008 02:54:29.433868  5243 LedManager.cpp:80] Wrote 1 to file /sys/class/leds/fan_led:green:status/brightness
I1008 02:54:29.433908  5243 LedManager.cpp:80] Wrote 0 to file /sys/class/leds/fan_led:red:status/brightness
I1008 02:54:29.433921  5243 LedManager.cpp:55] Programmed FAN LED with presence true
I1008 02:54:29.433931  5243 LedManager.cpp:25] Programming system LED with true
I1008 02:54:29.433967  5243 LedManager.cpp:80] Wrote 1 to file /sys/class/leds/sys_led:green:status/brightness
I1008 02:54:29.434007  5243 LedManager.cpp:80] Wrote 0 to file /sys/class/leds/sys_led:red:status/brightness
I1008 02:54:29.434019  5243 LedManager.cpp:34] Programmed system LED with true

@facebook-github-bot
Copy link
Contributor

@joancaneus has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@adamcalabrigo has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@adamcalabrigo has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@adamcalabrigo has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@joancaneus has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot pushed a commit that referenced this pull request Dec 4, 2024
Summary:
# Description
Combines #257, #258 and  #263.

#263 fails builds due to a dependencies found in #257 and  #258.

Pull Request resolved: #293

Reviewed By: somasun, mikechoifb

Differential Revision: D65801211

Pulled By: joancaneus

fbshipit-source-id: 2724e1073e2369fe802c4d144555ce93e95e6c6f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants