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

keychron/c2_pro/ansi/white: Fix column 19 in the custom matrix #21805

Merged
merged 1 commit into from
Aug 23, 2023

Commits on Aug 22, 2023

  1. keychron/c2_pro/ansi/white: Fix column 19 in the custom matrix

    Although `keychron/c2_pro/ansi/rgb` and `keychron/c2_pro/ansi/white` use
    the same custom matrix code, the matrix layouts are slightly different;
    in particular, only the `keychron/c2_pro/ansi/white` board actually uses
    column 19.  However, the handling of column 19 in the custom matrix code
    was broken, therefore that column did not work.
    
    Looks like the custom matrix code assumes that `SHIFT_COL_END` refers to
    the last column connected to the shift register, and not to the column
    past that; so the value of `SHIFT_COL_END` needs to be changed from 19
    to 18 (columns 11...18 are connected to the shift register, and column
    19 is connected to the C14 pin).
    
    Also the code which was determining `SIZE_T` and `UNSELECT_ALL_COL` had
    an off-by-one bug when counting the required number of bits (again due
    to the confusion on the `SHIFT_COL_END` meaning); this had been fixed
    too (the actual behavior of that part of the code did not change,
    because both the old and the new version select the 8 bit variant).
    sigprof committed Aug 22, 2023
    Configuration menu
    Copy the full SHA
    c1dd16e View commit details
    Browse the repository at this point in the history