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.

Sign up for GitHub

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

Jump to bottom

drivers: led: lp50xx: number of LEDs not checked during init #86101

Closed
ivanwagner opened this issue Feb 20, 2025 · 1 comment
Closed

drivers: led: lp50xx: number of LEDs not checked during init #86101

ivanwagner opened this issue Feb 20, 2025 · 1 comment
Assignees
@Mani-Sadhasivam @simonguinot
Labels
area: LED Label to identify LED subsystem Backport Backport PR and backport failure issues bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug

Comments

@ivanwagner
Copy link

ivanwagner commented Feb 20, 2025 •
edited
Loading

Describe the bug

This bug refers to a PR done by @simonguinot which fixes this issue and was merged on main: #85607 which is the original PR

It would be the case to backport it to v4.0. #85912

To Reproduce
It is not necessary to reproduce it is quite evident that the amount of LEDs is not checked and @simonguinot fixes the issue.

Expected behavior
An error should be returned when the number of LEDs exceeds the maximum allowed.

Impact

Logs and console output

Environment (please complete the following information):

Additional context
@ivanwagner ivanwagner added the bug The issue is a bug, or the PR is fixing a bug label Feb 20, 2025
@ivanwagner ivanwagner mentioned this issue Feb 20, 2025
Merged
@henrikbrixandersen henrikbrixandersen added area: LED Label to identify LED subsystem Backport Backport PR and backport failure issues labels Feb 21, 2025
@fabiobaltieri fabiobaltieri added the priority: low Low impact/importance bug label Feb 25, 2025
@simonguinot
Copy link
Collaborator

Closed by #85607.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Mani-Sadhasivam Mani-Sadhasivam

@simonguinot simonguinot

Labels
area: LED Label to identify LED subsystem Backport Backport PR and backport failure issues bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@fabiobaltieri @ivanwagner @henrikbrixandersen @Mani-Sadhasivam @simonguinot