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

Refactor AVR backlight #8041

Merged
merged 1 commit into from
Mar 1, 2020
Merged

Conversation

zvecr
Copy link
Member

@zvecr zvecr commented Jan 29, 2020

Description

Would advise diff ignore whitespace due to the clang-format stuff on all the ifdefs.

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.
  • 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).

@zvecr zvecr added the core label Jan 29, 2020
@zvecr zvecr force-pushed the feature/backlight_avr_tidy branch 2 times, most recently from be71873 to ac6a78a Compare January 31, 2020 00:23
@zvecr zvecr marked this pull request as ready for review January 31, 2020 00:23
@zvecr zvecr requested a review from a team January 31, 2020 00:24
Copy link
Contributor

@Duckle29 Duckle29 left a comment

Choose a reason for hiding this comment

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

#pragma weak

Code looks good, though I don't have any hardware to test it on.

Copy link
Member

@drashna drashna left a comment

Choose a reason for hiding this comment

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

Looks like a pretty simple logic change, especially if you hide whitespace changes @Duckle29. :)

@drashna
Copy link
Member

drashna commented Feb 2, 2020

quefrency rev2 is erroring out, due to change. But I suspect that's the intended behavior, due to this code change.

@fauxpark
Copy link
Member

fauxpark commented Feb 9, 2020

@nooges can you confirm the backlight pin used for the Quefrency rev2?

@nooges
Copy link
Member

nooges commented Feb 10, 2020

@fauxpark It's B5, w/B6 for the Caps Lock

@zvecr zvecr force-pushed the feature/backlight_avr_tidy branch from ac6a78a to ff62e04 Compare February 13, 2020 19:37
@zvecr zvecr force-pushed the feature/backlight_avr_tidy branch from ff62e04 to 1b06acb Compare February 29, 2020 23:18
@zvecr zvecr merged commit 88356c8 into qmk:master Mar 1, 2020
Shinichi-Ohki added a commit to Shinichi-Ohki/qmk_firmware that referenced this pull request Mar 3, 2020
* 'master' of https://github.com/qmk/qmk_firmware: (37 commits)
  Update Hungarian keymap and add sendstring LUT (qmk#8220)
  Remove "ugly hack in usb_main.c" comments (qmk#8296)
  Update encoder functions for Iris VIA keymap (qmk#8295)
  Reduce PROGMEM usage for sendstring LUT (qmk#8109)
  [Docs] Update ISP Flashing guide (qmk#8149)
  Rewrite the Bathroom Epiphanies Frosty Flake matrix and LED handling (qmk#8243)
  Add onekey keymap for testing reset to bootloader. (qmk#8288)
  Get the direction right on the S75 encoder (qmk#8287)
  Prune out pure software pwm && custom driver && remove wrapping BACKLIGHT_PIN (qmk#8041)
  Make a fix to savage65 and tmov2 for via (qmk#8286)
  format code according to conventions [skip ci]
  Short term fix for conflicting types for 'tfp_printf' (qmk#8157)
  Fix recent clang-format breaking quantum.c (qmk#8282)
  format code according to conventions [skip ci]
  Remove duplicate BRTG case (qmk#8277)
  Clean up includes for glcdfont headers (qmk#7745)
  Fix the Breaking Changes doc again
  [Docs] translated 'feature_tap_dance.md' to japanese. (qmk#8137)
  PWM DMA based RGB Underglow for STM32 (qmk#7928)
  Add VIA support to Prime_M. Clean up all files (qmk#8247)
  ...
HokieGeek pushed a commit to HokieGeek/qmk_firmware that referenced this pull request Mar 5, 2020
c0psrul3 pushed a commit to c0psrul3/qmk_firmware that referenced this pull request Mar 23, 2020
sowbug pushed a commit to sowbug/qmk_firmware that referenced this pull request Apr 2, 2020
kylekuj pushed a commit to kylekuj/qmk_firmware that referenced this pull request Apr 21, 2020
@zvecr zvecr deleted the feature/backlight_avr_tidy branch April 28, 2020 01:09
drashna pushed a commit to zsa/qmk_firmware that referenced this pull request May 24, 2020
drashna pushed a commit to zsa/qmk_firmware that referenced this pull request May 24, 2020
fdidron pushed a commit to zsa/qmk_firmware that referenced this pull request Jun 12, 2020
jakeisnt pushed a commit to jakeisnt/qmk_firmware that referenced this pull request Aug 20, 2020
BorisTestov pushed a commit to BorisTestov/qmk_firmware that referenced this pull request May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants