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

[minipack3n] Fix LED Mapping: Rename MCB_LED_PORT_66_LED_1 to _65_LED_2 #347

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

brandonchuang
Copy link
Contributor

Description:

  1. The original pmUnitScopedName "MCB_LED_PORT_66_LED_1" was incorrect.
  2. The correct port LED mapping corresponds to port-65-led-2.
  3. Renamed MCB_LED_PORT_66_LED_1 to MCB_LED_PORT_65_LED_2 to reflect the correct mapping.

Test Plan:

  1. Run platform_manager with the updated configuration.
  2. Verify that the correct sysfs files are created:
    ls /sys/class/leds/port65_led2:*
    Test Log: 20250306_Port65_Led2_Test.txt
  3. Execute Port65_Led2_Test.py to ensure the LED color changes as expected.
    Test Log: Port65_Led2_Test.py.txt

1) The original pmUnitScopedName "MCB_LED_PORT_66_LED_1" was incorrect.
2) The correct port LED mapping corresponds to port-65-led-2.
3) Renamed MCB_LED_PORT_66_LED_1 to MCB_LED_PORT_65_LED_2 to reflect the correct mapping.

1) Run platform_manager with the updated configuration.
2) Verify that the correct sysfs files are created:
    ls /sys/class/leds/port65_led2\:*
    Test Log: [20250306_Port65_Led2_Test.txt](https://github.com/user-attachments/files/19104721/20250306_Port65_Led2_Test.txt)

3) Execute Port65_Led2_Test.py to ensure the LED color changes as expected.
   Test Log: [Port65_Led2_Test.py.txt](https://github.com/user-attachments/files/19104799/Port65_Led2_Test.py.txt)
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