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

pillar: ledmanager: Fix user LED for phyBAORD-Pollux #3681

Merged
merged 1 commit into from
Dec 17, 2023

Conversation

rene
Copy link
Contributor

@rene rene commented Dec 15, 2023

Commit e1538b5 updated the u-boot for phyBOARD-Pollux device. In the new device tree the user LEDs were renamed from user-led? to just led?. This commit updates the LED name on ledmanager.

Commit e1538b5 updated the u-boot for
phyBOARD-Pollux device. In the new device tree the user LEDs were
renamed from user-led? to just led?. This commit updates the LED name on
ledmanager.

Signed-off-by: Renê de Souza Pinto <rene@renesp.com.br>
Copy link

codecov bot commented Dec 15, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (68b0c39) 19.60% compared to head (b2025a3) 19.60%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3681   +/-   ##
=======================================
  Coverage   19.60%   19.60%           
=======================================
  Files         232      232           
  Lines       50770    50770           
=======================================
+ Hits         9953     9954    +1     
+ Misses      40094    40093    -1     
  Partials      723      723           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@eriknordmark eriknordmark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@eriknordmark eriknordmark merged commit d31b84a into lf-edge:master Dec 17, 2023
28 of 30 checks passed
@rene rene deleted the pollux-led branch December 18, 2023 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants