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 existing SPI ws2812 implementations #7694

Merged
merged 1 commit into from
Apr 6, 2020

Conversation

zvecr
Copy link
Member

@zvecr zvecr commented Dec 21, 2019

Description

Now that we have the SPI based ws2812 driver migrated to core, this PR aims to remove the existing implementations and migrate to the core provided ones.

Tagging @awkannan as maintainer

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 keyboard label Dec 21, 2019
@zvecr zvecr requested a review from a team December 21, 2019 18:21
@zvecr zvecr mentioned this pull request Dec 22, 2019
13 tasks
@zvecr zvecr force-pushed the feature/ws2812_spi_board_refactor branch 2 times, most recently from bfb5366 to 261e1b2 Compare December 31, 2019 17:46
@zvecr zvecr marked this pull request as ready for review December 31, 2019 17:47
@zvecr
Copy link
Member Author

zvecr commented Dec 31, 2019

Rebased now #7674 has been merged

Edit: Travis build failures are due to audio/backlight support being assumed in community layouts.

@awkannan
Copy link
Contributor

awkannan commented Jan 2, 2020

I'll test this tomorrow! Looks pretty good to me but will double check.

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.

LGTM

@drashna drashna requested a review from a team January 2, 2020 19:32
@zvecr zvecr force-pushed the feature/ws2812_spi_board_refactor branch from 261e1b2 to 5617c89 Compare January 4, 2020 05:37
@zvecr zvecr force-pushed the feature/ws2812_spi_board_refactor branch from 5617c89 to f695ed5 Compare January 26, 2020 23:25
@zvecr zvecr force-pushed the feature/ws2812_spi_board_refactor branch from e3d8efb to fc76b0e Compare February 4, 2020 23:37
@drashna drashna requested a review from a team February 5, 2020 03:41
@zvecr zvecr force-pushed the feature/ws2812_spi_board_refactor branch from fc76b0e to 8a93ac3 Compare March 1, 2020 19:32
@zvecr zvecr force-pushed the feature/ws2812_spi_board_refactor branch from 8a93ac3 to feecd45 Compare April 6, 2020 20:19
@zvecr zvecr merged commit 6c8e205 into qmk:master Apr 6, 2020
@zvecr zvecr mentioned this pull request Apr 7, 2020
13 tasks
HokieGeek pushed a commit to HokieGeek/qmk_firmware that referenced this pull request Apr 10, 2020
violet-fish pushed a commit to violet-fish/qmk_firmware that referenced this pull request Apr 12, 2020
Quarren42 pushed a commit to Quarren42/qmk_firmware that referenced this pull request Apr 15, 2020
kylekuj pushed a commit to kylekuj/qmk_firmware that referenced this pull request Apr 21, 2020
@zvecr zvecr deleted the feature/ws2812_spi_board_refactor branch April 28, 2020 01:05
bitherder pushed a commit to bitherder/qmk_firmware that referenced this pull request May 15, 2020
sowbug pushed a commit to sowbug/qmk_firmware that referenced this pull request May 24, 2020
turky pushed a commit to turky/qmk_firmware that referenced this pull request Jun 13, 2020
jakobaa pushed a commit to jakobaa/qmk_firmware that referenced this pull request Jul 7, 2020
thorstenweber83 pushed a commit to thorstenweber83/qmk_firmware that referenced this pull request Sep 2, 2020
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.

4 participants