Skip to content

Commit

Permalink
Quefrency rev1: Fix default VIA layout options (qmk#9618)
Browse files Browse the repository at this point in the history
Apparently VIA allocates bits in the layout options field from the
lowest bit, but starting from the **last** option defined in the JSON
file.  So the default value 0x06 was actually trying to set the value
`3` (`0b11`) for the second-to-last option ("Right Shift"), which had
only 3 values defined, and the attempt to set an undefined option value
caused the VIA app to hang with a black window.

Fix the default layout options so that it works as intended (the
"Macropad" and "65% Column" options are set).
  • Loading branch information
sigprof committed Jul 1, 2020
1 parent 92839f8 commit d4dc2a5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion keyboards/keebio/quefrency/rev1/config.h
Original file line number Diff line number Diff line change
Expand Up @@ -57,4 +57,4 @@ along with this program. If not, see <http:https://www.gnu.org/licenses/>.
#define RGBLED_NUM 16 // Number of LEDs

// Set 65% column (option 1) and Macro (option 2) on by default
#define VIA_EEPROM_LAYOUT_OPTIONS_DEFAULT 0x06
#define VIA_EEPROM_LAYOUT_OPTIONS_DEFAULT 0x60

0 comments on commit d4dc2a5

Please sign in to comment.