util: added another note to monitor window

through_hole firmware variant renamed to universal

All model Fs: double the threshold offset used.

Changing default debounce type for all keyboards, to sym_pk

CAPSENSE_CAL_BINS bumped up to '5' for all model F keyboards

Added locklights support

keyboards/xwhatsit/sneakyrobb/beam104/through_hole: change default threshold offset

my keymap: add solenoid haptic keys

First attempt at implementing displaywriter support

quantum/debounce/sym_pk: code review: cleaner code

Co-Authored-By: Nick Brassel <nick@tzarc.org>

quantum/debounce/sym_pk: Modifications for code readability according to code review (2)

quantum/debounce/sym_pk: Modifications for code readability according to code review

quantum/debounce/sym_pk: delete comments and rename functions following code review

Apply suggestions from code review

Co-Authored-By: Ryan <fauxpark@gmail.com>

quantum/debounce: Added sym_pk debounce algorithm

quantum/debounce/eager_pr and eager_pk: modifications for code readability according to code review. (2)

quantum/debounce/eager_pr and eager_pk: modifications for code readability according to code review.

quantum/debounce: Fix custom wrapping timers in eager_pr and eager_pk debounce algorithms

Please see the below simulated sequence of events:
Column A is the 16-bit value returned by read_timer();
Column B is the value returned by custom_wrap_timer_read();
Column C is the original code: (timer_read() % MAX_DEBOUNCE)

    A,     B,     C
65530,    19,    30
65531,    20,    31
65532,    21,    32
65533,    22,    33
65534,    23,    34
65535,    24,    35
    0     25,     0
    1,    26,     1
    2,    27,     2
    3,    28,     3
    4,    29,     4
    5,    30,     5

read_timer() wraps about every 1.09 seconds, and so debouncing might
fail at these times without this commit.

tmk_core/common: Improve code generation for TIMER_DIFF* macros

Because of integer promotion the compiler is having a hard time generating
efficient code to calculate TIMER_DIFF* macros in some situations.
In the below example, the return value is "int", and this is causing the
trouble.

Example C code:

int __attribute__ ((noinline)) test(uint8_t current_timer, uint8_t start_timer)
{
    return TIMER_DIFF_8(current_timer, start_timer);
}

BEFORE: (with -Os)

00004c40 <test>:
    4c40:       28 2f           mov     r18, r24
    4c42:       30 e0           ldi     r19, 0x00       ; 0
    4c44:       46 2f           mov     r20, r22
    4c46:       50 e0           ldi     r21, 0x00       ; 0
    4c48:       86 17           cp      r24, r22
    4c4a:       20 f0           brcs    .+8             ; 0x4c54 <test+0x14>
    4c4c:       c9 01           movw    r24, r18
    4c4e:       84 1b           sub     r24, r20
    4c50:       95 0b           sbc     r25, r21
    4c52:       08 95           ret
    4c54:       c9 01           movw    r24, r18
    4c56:       84 1b           sub     r24, r20
    4c58:       95 0b           sbc     r25, r21
    4c5a:       93 95           inc     r25
    4c5c:       08 95           ret

AFTER: (with -Os)

00004c40 <test>:
    4c40:       86 1b           sub     r24, r22
    4c42:       90 e0           ldi     r25, 0x00       ; 0
    4c44:       08 95           ret

Note: the example is showing -Os but improvements can be seen at all optimization levels,
including -O0. We never use -O0, but I tested it to make sure that no extra code is
generated in that case.OA

tmk_core/common: Fixing TIMER_DIFF macro to calculate difference correctly after the timer wraps.

Let's go through an example, using the following macro:

If the first timer read is 0xe4 and the second one is 0x32, the timer wrapped.
If the timer would have had more bits, it's new value would have been 0x132,
and the correct difference in time is 0x132 - 0xe4 = 0x4e

old code TIMER_DIFF_8(0x32, 0xe4) = 0xff - 0xe4 + 0x32 = 0x4d, which is wrong.
new code TIMER_DIFF_8(0x32, 0xe4) = 0xff + 1 - 0xe4 + 0x32 = 0x4e, which is correct.

This also gives a chance for a smart compiler to optimize the code using normal
integer overflow.

For example on AVR, the following C code:
uint8_t __attribute__ ((noinline)) test(uint8_t current_timer, uint8_t start_timer)
{
    return TIMER_DIFF_8(current_timer, start_timer);
}
With the original code, it gets translated to the following list of instructions:
00004c6e <test>:
    4c6e:       98 2f           mov     r25, r24
    4c70:       86 1b           sub     r24, r22
    4c72:       96 17           cp      r25, r22
    4c74:       08 f4           brcc    .+2             ; 0x4c78 <test+0xa>
    4c76:       81 50           subi    r24, 0x01       ; 1
    4c78:       08 95           ret
But with this commit, it gets translated to a single instruction:
00004c40 <test>:
    4c40:       86 1b           sub     r24, r22
    4c42:       08 95           ret

This unfortunately doesn't always work so nicely, for example the following C code:
int __attribute__ ((noinline)) test(uint8_t current_timer, uint8_t start_timer)
{
    return TIMER_DIFF_8(current_timer, start_timer);
}
(Note: return type changed to int)
With the original code it gets translated to:
00004c6e <test>:
    4c6e:       28 2f           mov     r18, r24
    4c70:       30 e0           ldi     r19, 0x00       ; 0
    4c72:       46 2f           mov     r20, r22
    4c74:       50 e0           ldi     r21, 0x00       ; 0
    4c76:       86 17           cp      r24, r22
    4c78:       20 f0           brcs    .+8             ; 0x4c82 <test+0x14>
    4c7a:       c9 01           movw    r24, r18
    4c7c:       84 1b           sub     r24, r20
    4c7e:       95 0b           sbc     r25, r21
    4c80:       08 95           ret
    4c82:       c9 01           movw    r24, r18
    4c84:       84 1b           sub     r24, r20
    4c86:       95 0b           sbc     r25, r21
    4c88:       81 50           subi    r24, 0x01       ; 1
    4c8a:       9f 4f           sbci    r25, 0xFF       ; 255
    4c8c:       08 95           ret
Wth this commit it gets translated to:
00004c40 <test>:
    4c40:       28 2f           mov     r18, r24
    4c42:       30 e0           ldi     r19, 0x00       ; 0
    4c44:       46 2f           mov     r20, r22
    4c46:       50 e0           ldi     r21, 0x00       ; 0
    4c48:       86 17           cp      r24, r22
    4c4a:       20 f0           brcs    .+8             ; 0x4c54 <test+0x14>
    4c4c:       c9 01           movw    r24, r18
    4c4e:       84 1b           sub     r24, r20
    4c50:       95 0b           sbc     r25, r21
    4c52:       08 95           ret
    4c54:       c9 01           movw    r24, r18
    4c56:       84 1b           sub     r24, r20
    4c58:       95 0b           sbc     r25, r21
    4c5a:       93 95           inc     r25
    4c5c:       08 95           ret
There is not much performance improvement in this case, however at least with this
commit it functions correctly.

Note: The following commit will improve compiler output for the latter example.

brand_new_model_f: fixing typo in product descriptor string

Added some (so far unused) code to initialise unused pins to a known value by enabling the pull-up resistors.

TH xwhatsit: enable internal pull-ups on rows. This will be necessary on some future versions of TH controller

Be explicit about the value of the OE pin. OE is active low.

bugfix to f122 iso layouts

util: added keyboard defs for at wcass and original xwhatsit controller variants

keyboards/xwhatsit/ibm/fat/wcass: added

added: keyboards/xwhatsit/ibm/fat/xwhatsit

xwhatsit ibm model F AT: fix up key positions

xwhatsit util: make it a little more resistant to garbage left from previous util run

xwhatsit util: bugfix for recently introduced bug

xwhatsit util: fix code that handles minimum window size

xwhatsit keyboards: fix >16cols 8rows support in keypress monitor

xwhatsit keyboards: bugfix of through hole with model F keyboards

xwhatsit model f keyboards with through hole controller: turn off promicro leds

xwhatsit ibm F AT: bugfixes

xwhatsit ibm fAT: renamed layout to LAYOUT_all, and tweaked sizes of ctrl keys

keyboards: xwhatsit: ibm: fAT: added

brand_new_model_f/f??: added readme

xwhatsit/brand_new_model_f/f??: make wcass the default controller

kbd defs added for ibm 4978

xwhatsit/ibm/4978/through_hole: added (untested)

xwhatsit util: improve monitor layouts

xwhatsit keyboards: add missing license headers to source files

xwhatsit keyboards: generate_layout.py: make generated file more reproducable

xwhatsit keyboards: implement row and dac tester

xwhatsit util: re-enable keyboard from column tester + refactor

xwhatsit keyboards: refactor macros from the top of matrix.c to make sure they are included in every .c file

xwhatsit keyboards: improve logic to detect > 16 physical cols

xwhatsit keyboards: implemented a form of anti-interference mechanism

xwhatsit keyboards: updated keyboard defs for util

keyboards/xwhatsit/ibm/f107: renamed LAYOUT to LAYOUT_102key_ansi, and added 107-key layout

xwhatsit keyboards: fix typo for macro to select through hole pcb

keyboards/xwhatsit/ibm/f122/wcass: added

keyboards/xwhatsit/ibm/f122/through_hole: added

keyboards/xwhatsit/ibm/f122/xwhatsit: added

f107: fix: typically long zero keys have the sense pad on the right side

xwhatsit keyboards: added some commented out options

keyboards/xwhatsit/brand_new_model_f/f62/through_hole and keyboards/xwhatsit/brand_new_model_f/f77/through_hole: added

keyboards/xwhatsit/brand_new_model_f/f62/xwhatsit and keyboards/xwhatsit/brand_new_model_f/f77/xwhatsit: added

keyboards/xwhatsit/ibm/f62/through_hole: added

keyboards/xwhatsit/ibm/f50/through_hole: added

keyboards/xwhatsit/ibm/f107/through_hole: added

keyboards/xwhatsit/ibm/3277_split_66key/through_hole: added

keyboards/xwhatsit/ibm/3277_78key/through_hole: added

keyboards/xwhatsit/ibm/3277_66key/through_hole: added

xwhatsit keyboards: moved info.json out of the controller folders

xwhatsit keyboards: move keymaps out of the controller folders

drivers/haptic: fix compilation issue, when haptic is enabled, but solenoid isn't

haptic: improve what happens if solenoid is enabled without erasing eeprom

Haptic and solenoid cleanup https://github.com/qmk/qmk_firmware/pull/9700

xwhatsit keyboards running non-through-hole controller: bugfix to the setup of solenoid pins

xwhatsit keyboards: attempt to enable solenoid. To be tested by people with solenoid.

xwhatsit keyboards: added comment on expansion header pinout

keyboards/xwhatsit/sneakyrobb/beam104/through_hole: minor fixes

xwhatsit: updated keyboard defs

first attempt at beam104 keyboard

xwhatsit: through hole: implement 8-rows

xwhatsit comm: report through hole correctly

xwhatsit keyboards: bugfix: correctly detect unassigned keys on layer 0

added my layout

updated util .gitignore

added xwhatsit/ibm/5251/through_hole

xwhatsit util: updated keyboard defs

added xwhatsit/ibm/3101_3727_3278_87key/through_hole

xwhatsit: fix some readme typos

xwhatsit util: make sure name translation works on signal level monitor too.

xwhatsit util: fix some colors, when it's used on a system with non-standard colors.

xwhatsit util: fix some scaling issues

xwhatsit: f77: bugfix to the right arrow key

xwhatsit gui: column tester: bugfix

xwhatsit gui: window title update for column tester

xwhatsit util: main window: when a subwindow is open correctly disable all of the main window (1)

xwhatsit: util: removed currently unused buttons

xwhatsit gui: added column tester

xwhatsit firmware: added column tester

xwhatsit util: main window: make it resizable

xwhatsit util: add matrix signal level monitor

xwhatsit: updated the image links for some keyboards

xwhatsit use 4 times the threshold offset for through_hole controller