Skip to content

ledmatrix: Limit max brightness, and do custom brightness control #49

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

Merged
merged 1 commit into from
Jun 27, 2023

Conversation

JohnAZoidberg
Copy link
Member

The new hardware is extremely bright, so we need to limit the brightness.
We limit it so that the max power consumption is slightly under 500mA. This limit is done by using the scaling factor in the LED controller.

This also has the advantage that we have more brightness levels available for each LED because we can use the full range (previously much of the range would be too bright to use, even with the first hardware revision).

But since we can't use the scaling factor for brightness control anymore, we have to do the brightness calculation ourselves.

Base automatically changed from update-deps to main June 27, 2023 08:07
The new hardware is extremely bright, so we need to limit the
brightness.
We limit it so that the max power consumption is slightly under 500mA.
This limit is done by using the scaling factor in the LED controller.

This also has the advantage that we have more brightness levels
available for each LED because we can use the full range (previously
much of the range would be too bright to use, even with the first
hardware revision).

But since we can't use the scaling factor for brightness control
anymore, we have to do the brightness calculation ourselves.

Signed-off-by: Daniel Schaefer <[email protected]>
@JohnAZoidberg JohnAZoidberg merged commit 818691b into main Jun 27, 2023
@JohnAZoidberg JohnAZoidberg deleted the max-brightness branch June 27, 2023 13:01
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.

1 participant