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

keyboards: Add support for Sofle Choc #6

Open
wants to merge 85 commits into
base: develop
Choose a base branch
from
Open

keyboards: Add support for Sofle Choc #6

wants to merge 85 commits into from

Conversation

t-8ch
Copy link
Collaborator

@t-8ch t-8ch commented Jul 30, 2023

This PR is to review and test my changes for the Sofle Choc upstreaming process.

@brianlow @duese are you fine with me squashing all your work into this commit together?
There is authorship attribution in the commit message.

Copy link
Owner

@brianlow brianlow left a comment

Choose a reason for hiding this comment

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

LGTM. Squash ok with me. How do plan to get this into PR? Force push the choc2 branch?

@t-8ch
Copy link
Collaborator Author

t-8ch commented Jul 31, 2023

Force push the choc2 branch?

Yes.

But before that somebody needs to test it on a physical device.

@duese
Copy link
Collaborator

duese commented Jul 31, 2023

I can test it tonight.

@duese
Copy link
Collaborator

duese commented Aug 8, 2023

Hey. Finally had time to test this PR. Squashing is fine for me too.
Unfortunately something isn't right. After flashing the two halves, the OLED display on the left half shows some crippled artefacts as shown in the attached photo.
signal-2023-08-08-072629_002

My Macbook is up to date, but while flashing I get the following output:

> qmk flash -kb sofle_choc -km default
Ψ Compiling keymap with gmake --jobs=1 sofle_choc:default:flash


QMK Firmware 0.21.6
Making sofle_choc with keymap default and target flash

avr-gcc (Homebrew AVR GCC 8.5.0_1) 8.5.0
Copyright (C) 2018 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Size before:
   text	  data	   bss	   dec	   hex	filename
      0	 24764	     0	 24764	  60bc	sofle_choc_default.hex

Copying sofle_choc_default.hex to qmk_firmware folder                                               [OK]
Checking file size of sofle_choc_default.hex                                                        [OK]
 * The firmware size is fine - 24764/28672 (86%, 3908 bytes free)
Flashing for bootloader: caterina
Waiting for USB serial port - reset your controller now (Ctrl+C to cancel).....
Device /dev/tty.usbmodem14201 has appeared; assuming it is the controller.
Waiting for /dev/tty.usbmodem14201 to become writable.
avrdude warning: mosi is deprecated, will be removed in v8.0, use sdo [/usr/local/etc/avrdude.conf:404]
avrdude warning: miso is deprecated, will be removed in v8.0, use sdi [/usr/local/etc/avrdude.conf:405]
avrdude warning: mosi is deprecated, will be removed in v8.0, use sdo [/usr/local/etc/avrdude.conf:433]
avrdude warning: miso is deprecated, will be removed in v8.0, use sdi [/usr/local/etc/avrdude.conf:434]
avrdude warning: mosi is deprecated, will be removed in v8.0, use sdo [/usr/local/etc/avrdude.conf:466]
avrdude warning: miso is deprecated, will be removed in v8.0, use sdi [/usr/local/etc/avrdude.conf:467]
avrdude warning: mosi is deprecated, will be removed in v8.0, use sdo [/usr/local/etc/avrdude.conf:488]
avrdude warning: miso is deprecated, will be removed in v8.0, use sdi [/usr/local/etc/avrdude.conf:489]
avrdude warning: mosi is deprecated, will be removed in v8.0, use sdo [/usr/local/etc/avrdude.conf:552]
avrdude warning: miso is deprecated, will be removed in v8.0, use sdi [/usr/local/etc/avrdude.conf:553]
avrdude warning: mosi is deprecated, will be removed in v8.0, use sdo [/usr/local/etc/avrdude.conf:572]
avrdude warning: miso is deprecated, will be removed in v8.0, use sdi [/usr/local/etc/avrdude.conf:573]
avrdude warning: mosi is deprecated, will be removed in v8.0, use sdo [/usr/local/etc/avrdude.conf:591]
avrdude warning: miso is deprecated, will be removed in v8.0, use sdi [/usr/local/etc/avrdude.conf:592]
avrdude warning: mosi is deprecated, will be removed in v8.0, use sdo [/usr/local/etc/avrdude.conf:622]
avrdude warning: miso is deprecated, will be removed in v8.0, use sdi [/usr/local/etc/avrdude.conf:623]
avrdude warning: mosi is deprecated, will be removed in v8.0, use sdo [/usr/local/etc/avrdude.conf:676]
avrdude warning: miso is deprecated, will be removed in v8.0, use sdi [/usr/local/etc/avrdude.conf:677]
avrdude warning: miso is deprecated, will be removed in v8.0, use sdi [/usr/local/etc/avrdude.conf:760]
avrdude warning: mosi is deprecated, will be removed in v8.0, use sdo [/usr/local/etc/avrdude.conf:762]
avrdude warning: miso is deprecated, will be removed in v8.0, use sdi [/usr/local/etc/avrdude.conf:771]
avrdude warning: mosi is deprecated, will be removed in v8.0, use sdo [/usr/local/etc/avrdude.conf:773]
avrdude warning: miso is deprecated, will be removed in v8.0, use sdi [/usr/local/etc/avrdude.conf:783]
avrdude warning: mosi is deprecated, will be removed in v8.0, use sdo [/usr/local/etc/avrdude.conf:785]
avrdude warning: miso is deprecated, will be removed in v8.0, use sdi [/usr/local/etc/avrdude.conf:796]
avrdude warning: mosi is deprecated, will be removed in v8.0, use sdo [/usr/local/etc/avrdude.conf:798]
avrdude warning: mosi is deprecated, will be removed in v8.0, use sdo [/usr/local/etc/avrdude.conf:808]
avrdude warning: miso is deprecated, will be removed in v8.0, use sdi [/usr/local/etc/avrdude.conf:809]
avrdude warning: miso is deprecated, will be removed in v8.0, use sdi [/usr/local/etc/avrdude.conf:831]
avrdude warning: mosi is deprecated, will be removed in v8.0, use sdo [/usr/local/etc/avrdude.conf:833]
avrdude warning: miso is deprecated, will be removed in v8.0, use sdi [/usr/local/etc/avrdude.conf:855]
avrdude warning: mosi is deprecated, will be removed in v8.0, use sdo [/usr/local/etc/avrdude.conf:857]
avrdude warning: mosi is deprecated, will be removed in v8.0, use sdo [/usr/local/etc/avrdude.conf:1433]
avrdude warning: miso is deprecated, will be removed in v8.0, use sdi [/usr/local/etc/avrdude.conf:1434]
avrdude warning: mosi is deprecated, will be removed in v8.0, use sdo [/usr/local/etc/avrdude.conf:1455]
avrdude warning: miso is deprecated, will be removed in v8.0, use sdi [/usr/local/etc/avrdude.conf:1456]
avrdude warning: mosi is deprecated, will be removed in v8.0, use sdo [/usr/local/etc/avrdude.conf:1469]
avrdude warning: miso is deprecated, will be removed in v8.0, use sdi [/usr/local/etc/avrdude.conf:1470]
avrdude warning: mosi is deprecated, will be removed in v8.0, use sdo [/usr/local/etc/avrdude.conf:1483]
avrdude warning: miso is deprecated, will be removed in v8.0, use sdi [/usr/local/etc/avrdude.conf:1484]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:1708]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:3199]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:3388]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:3577]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:3769]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:3961]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:4154]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:4340]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:4664]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:4883]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:5083]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:5439]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:5651]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:5905]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:6139]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:7104]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:7301]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:7498]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:7755]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:7982]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:8209]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:8438]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:8635]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:8821]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:9008]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:9196]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:9384]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:9725]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:9923]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:10107]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:10390]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:10574]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:10757]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:10943]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:11133]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:11336]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:11778]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:11971]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:12164]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:12480]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:12663]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:12854]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:13045]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:13249]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:13451]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:13452]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:13638]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:13639]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:13827]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:13828]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:14015]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:14016]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:14203]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:14204]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:14387]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:14609]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:14829]
avrdude warning: has_pdi is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:15108]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:15236]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:15325]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:15425]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:15585]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:15643]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:15755]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:15887]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:15987]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:16264]
avrdude warning: is_avr32 is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:16265]
avrdude warning: has_debugwire is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:16289]
avrdude warning: has_tpi is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:16487]
avrdude warning: has_jtag is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:16644]
avrdude warning: has_updi is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:16709]
avrdude warning: has_updi is deprecated, will be removed in v8.0, use prog_modes [/usr/local/etc/avrdude.conf:18001]
avrdude: AVR device initialized and ready to accept instructions
avrdude: device signature = 0x1e9587 (probably m32u4)
avrdude: Note: flash memory has been specified, an erase cycle will be performed.
         To disable this feature, specify the -D option.
avrdude: erasing chip

avrdude: processing -U flash:w:.build/sofle_choc_default.hex:i
avrdude: reading input file .build/sofle_choc_default.hex for flash
         with 24764 bytes in 1 section within [0, 0x60bb]
         using 194 pages and 68 pad bytes
avrdude: writing 24764 bytes flash ...
Writing | ################################################## | 100% 1.90 s 
avrdude: 24764 bytes of flash written
avrdude: verifying flash memory against .build/sofle_choc_default.hex
Reading | ################################################## | 100% 0.20 s 
avrdude: 24764 bytes of flash verified

avrdude done.  Thank you.

I'm not sure whether the warnings can be ignored.

qmk doctor does not complain:

> qmk doctor
Ψ QMK Doctor is checking your environment.
Ψ CLI version: 1.1.2
Ψ QMK home: /Users/duese/dev/qmk
Ψ Detected macOS 13.5 (Intel).
Ψ Git branch: sofle_choc
Ψ Repo version: 0.21.6
Ψ - Latest sofle_choc: 2023-07-30 17:24:39 +0200 (115b82511e) -- keyboards: Add support for Sofle Choc
Ψ - Latest upstream/master: 2023-08-07 03:13:20 +1000 (d52bafade3) -- Fix `binepad/bn009` and `durgod/dgk6x/hades_ansi` keymaps (#21701)
Ψ - Latest upstream/develop: 2023-08-06 17:13:52 +0000 (2e67945033) -- Merge remote-tracking branch 'origin/master' into develop
Ψ - Common ancestor with upstream/master: 2023-07-30 00:22:39 -0400 (14e14e9ab8) -- Correct "less than" to "up to" in squeezing_avr?id=layers (#21639)
Ψ - Common ancestor with upstream/develop: 2023-07-30 04:23:13 +0000 (fa4d51dab7) -- Merge remote-tracking branch 'origin/master' into develop
Ψ CLI installed in virtualenv.
Ψ All dependencies are installed.
Ψ Found arm-none-eabi-gcc version 8.3.1
Ψ Found avr-gcc version 8.5.0
Ψ Found avrdude version 7.2
Ψ Found dfu-programmer version 1.1.0
Ψ Found dfu-util version 0.11
Ψ Submodules are up to date.
Ψ Submodule status:
Ψ - lib/chibios: 2023-04-15 13:48:04 +0000 --  (11edb1610)
Ψ - lib/chibios-contrib: 2023-07-17 11:39:05 +0200 --  (da78eb37)
Ψ - lib/googletest: 2021-06-11 06:37:43 -0700 --  (e2239ee6)
Ψ - lib/lufa: 2022-08-26 12:09:55 +1000 --  (549b97320)
Ψ - lib/vusb: 2022-06-13 09:18:17 +1000 --  (819dbc1)
Ψ - lib/printf: 2022-06-29 23:59:58 +0300 --  (c2e3b4e)
Ψ - lib/pico-sdk: 2023-02-12 20:19:37 +0100 --  (a3398d8)
Ψ - lib/lvgl: 2022-04-11 04:44:53 -0600 --  (e19410f8)
Ψ QMK is ready to go

Maybe I can investigate later.

@t-8ch
Copy link
Collaborator Author

t-8ch commented Aug 8, 2023

The warnings during flashing seem unrelated.

I'll take a look at the display, did it work with the original PR?

Are keypresses, LEDs working?

@duese
Copy link
Collaborator

duese commented Aug 8, 2023

Are keypresses, LEDs working?

Regular keypresses without modifier keys are working normal. Shift modifier works too. It seems the layer shifting modifiers don't work as shown on in the default layout depicted in the build guide. E.g. when pressing any layer modifier in combination with "asdfg" I get "åß∂ƒ©" instead.

I can only see the default red pulsing. Due to the disfunctional layer modifiers I can't switch the animation and/or LED color.

@t-8ch
Copy link
Collaborator Author

t-8ch commented Aug 8, 2023

The layers being gone is intentional, as the maintainers requested a simpler keymap by default.
I guess this means the LEDs should default to something more conservative.

@duese
Copy link
Collaborator

duese commented Aug 8, 2023

The layers being gone is intentional, as the maintainers requested a simpler keymap by default.

Do you really think this is what Drashna means with "pristine/simple"? Even the corne keyboard has layers in its default keymap. And the default map has Drashna in its copyright message.

But to get this board merged I would be ok with that, but then I would appreciate it if you move the previous default map to something like "old default", "brians default" or "creators default". What good is it having a RGB board if you can't change anything RBG related?

@duese
Copy link
Collaborator

duese commented Aug 8, 2023

If one can't change the animation you should then also comment out all other animations except for the one used in config.h

@t-8ch
Copy link
Collaborator Author

t-8ch commented Aug 8, 2023

Do you really think this is what Drashna means with "pristine/simple"?

This was my impression, yes.

I would appreciate it if you move the previous default map to something like "old default", "brians default" or "creators default"

I would prefer to keep this out of the initial PR. Let's focus on the minimal amount of stuff to make it useful and get it merged.
Currently there is already an issue with reviewer capacity additional work needed.
Any additional code will only make the process slower and more work for everybody.

The original layout is a nice example but it's much easier to add it after the initial merge.

also comment out all other animations except for the one used in config.h

I'll delete them completely for the first merge.
They are not lost and can be readded as part of brians keymap after the board support has landed.

@duese
Copy link
Collaborator

duese commented Aug 8, 2023

Agreed.

I'll delete them completely for the first merge.

Do that and I'll approve.

@duese
Copy link
Collaborator

duese commented Aug 8, 2023

And did you look into the the crippeled OLED display?

@t-8ch
Copy link
Collaborator Author

t-8ch commented Aug 8, 2023

I didn't have time yet to look at the OLED. Will do so later.
Just to be sure: The OLED does work correctly on the original upstreaming PR?

@duese
Copy link
Collaborator

duese commented Aug 8, 2023

Just to be sure: The OLED does work correctly on the original upstreaming PR?

What upstreaming PR do mean? I checked out your PR without any modifications, compiled and flashed it to my Sofle Choc.

@t-8ch
Copy link
Collaborator Author

t-8ch commented Aug 8, 2023

qmk#16736

@duese
Copy link
Collaborator

duese commented Aug 8, 2023

qmk#16736

Had to merge upstream/develop (cf5626e) into choc2 to get that PR to compile but then I could flash it and it worked.
No issues with the OLED display there.

@github-actions github-actions bot added documentation Improvements or additions to documentation core cli python labels Aug 9, 2023
@t-8ch
Copy link
Collaborator Author

t-8ch commented Aug 9, 2023

@duese Thanks for testing! Care to try again?

@github-actions github-actions bot removed documentation Improvements or additions to documentation core cli python labels Aug 9, 2023
qmk-bot and others added 22 commits August 23, 2023 09:13
Bumps [tj-actions/changed-files](https://github.com/tj-actions/changed-files) from 37 to 38.
- [Release notes](https://github.com/tj-actions/changed-files/releases)
- [Changelog](https://github.com/tj-actions/changed-files/blob/main/HISTORY.md)
- [Commits](tj-actions/changed-files@v37...v38)

---
updated-dependencies:
- dependency-name: tj-actions/changed-files
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
GCC 13 needs the same workaround as GCC 12.
To avoid having to maintain an ever-growing list of broken versions
apply the workaround wherever it can be used.

If at some point a fixed version of GCC is released the workaround can
be disabled for those fixed versions.

See qmk#17064
* Move RGBLight animations to data driven, 0-9

* Move RGBLight animations to data driven, A

* Move RGBLight animations to data driven, B

* Move RGBLight animations to data driven, C

* Move RGBLight animations to data driven, D

* Move RGBLight animations to data driven, E

* Move RGBLight animations to data driven, F

* Move RGBLight animations to data driven, G

* Move RGBLight animations to data driven, H

* Move RGBLight animations to data driven, handwired

* Move RGBLight animations to data driven, I

* Move RGBLight animations to data driven, J

* Move RGBLight animations to data driven, K

* Move RGBLight animations to data driven, L

* Move RGBLight animations to data driven, M

* Move RGBLight animations to data driven, N

* Move RGBLight animations to data driven, O

* Move RGBLight animations to data driven, P

* Move RGBLight animations to data driven, Q

* Move RGBLight animations to data driven, R

* Move RGBLight animations to data driven, S

* Move RGBLight animations to data driven, T

* Move RGBLight animations to data driven, U

* Move RGBLight animations to data driven, V

* Move RGBLight animations to data driven, W

* Move RGBLight animations to data driven, X

* Move RGBLight animations to data driven, Y

* Move RGBLight animations to data driven, Z

* Fix incorrect placement

* Fix build failures and mismatches
* Rework info.json reference

* Apply suggestions from code review

Co-authored-by: Sergey Vlasov <[email protected]>

* Update backlight.pins description

* Add rgblight.driver and lowercase other driver names

* Remove `split.main` and `split.matrix_grid` for now as they are not working yet

---------

Co-authored-by: Sergey Vlasov <[email protected]>
* Partially revert qmk#19780

* Finish

* Get teensy 3.5/3.6 board files too

* fix lint issue

* Revert "[Bug] Restore usb suspend wakeup delay (qmk#21676)"

This reverts commit e8e989f.

* Apply suggestions from code review

Co-authored-by: Joel Challis <[email protected]>

---------

Co-authored-by: Joel Challis <[email protected]>
Co-authored-by: James Young <[email protected]>
Co-authored-by: Ryan <[email protected]>
Co-authored-by: honorless <[email protected]>
Co-developed-by: Brian Low <[email protected]>
Co-developed-by: duese <[email protected]>
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.