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

Allow backlight duty cycle limit #10260

Merged
merged 1 commit into from
Oct 23, 2020
Merged

Conversation

tzarc
Copy link
Member

@tzarc tzarc commented Sep 7, 2020

Description

Needed to be able to apply a limit to the backlighting duty cycle -- breadboard testing of using QMK's backlight code to control an LCD backlight showed it was pulling 80mA, but in circuit with SMT components it showed it was capable of pulling 400mA per LCD.

Artificially limiting the duty cycle allows controlling max brightness instead of messing about with current limiting circuits, much like the RGB limit constants we already have.

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout/userspace (addition or update)
  • Documentation

Checklist

  • My code follows the code style of this project: C, Python
  • I have read the PR Checklist document and have made the appropriate changes.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).

@tzarc
Copy link
Member Author

tzarc commented Sep 7, 2020

Pending other backlight drivers, documentation.

@tzarc tzarc force-pushed the backlight-duty-cycle-limit branch from 8d1b858 to 38fb196 Compare October 5, 2020 00:04
@tzarc tzarc marked this pull request as ready for review October 5, 2020 00:09
@tzarc tzarc added hacktoberfest-accepted All the good issues to tackle during Hacktoberfest! and removed in progress hacktoberfest-accepted All the good issues to tackle during Hacktoberfest! labels Oct 5, 2020
@drashna drashna requested a review from a team October 7, 2020 21:53
@tzarc tzarc force-pushed the backlight-duty-cycle-limit branch from 734afad to 8d7d959 Compare October 23, 2020 03:41
@tzarc tzarc force-pushed the backlight-duty-cycle-limit branch from 8d7d959 to 12cefd6 Compare October 23, 2020 03:42
@tzarc tzarc merged commit 5720202 into qmk:develop Oct 23, 2020
@tzarc tzarc deleted the backlight-duty-cycle-limit branch October 23, 2020 04:58
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