-
Notifications
You must be signed in to change notification settings - Fork 84
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
darp8 keyboard debounce too lax resulting in multiple chharacters #509
Comments
@leviport mentioned system76/ec#405 (review) that they could nnoot tell the difference between two different debounce settings? Is it possible that this is another Clevo change since yoour engineering samples? Also it seems that some letters are more susceptible to repeats such as "o" |
Indeed it seems that "o" and keys pressed before/after are repeated most often:
|
I doubt it's a hardware difference. Keyboard debounce is a highly subjective setting. When we had it set to 5ms, I didn't notice any repeats. Changing it to 10ms fixed it for most. Typing styles vary widely across different individuals. Are you comfortable building and flashing EC on your machine? I'd be curious whether changing it back to 15ms is sufficient for you. Since 5ms doesn't cause repeated keystrokes for my typing style, the testing I can do myself is quite limited. |
15ms is better,, but i'll try 20 diff --git a/src/board/system76/common/kbscan.c b/src/board/system76/common/kbscan.c
index 4d3b650..a5ef0a7 100644
--- a/src/board/system76/common/kbscan.c
+++ b/src/board/system76/common/kbscan.c
@@ -21,7 +21,7 @@
#endif // KM_NKEY
// Debounce time in milliseconds
-#define DEBOUNCE_DELAY 10
+#define DEBOUNCE_DELAY 15
// Deselect all columns for reading
#define KBSCAN_MATRIX_NONE 0xFF
|
20ms seems to do the trick, I can type over 10 lines with no repeats. Could y'all be convinced to up the value to 20ms? |
I found that a debounce time of 10ms was causing very frequent twice-registered keys with my typing style. After setting the debounce time to 20ms I find that no repeated characters occur when typing 10 line of text. Ref: system76/firmware-open#509 Signed-off-by: Jordan Speicher <jordan@jspeicher.com>
I hhave the same problem with darp7, since I have bought this machine. I thought it's a haardware issue, so I didn't open the issue. Please System76, fix this problem by setting debounce even to 30 ms. |
@ShabazzKing have you tried 20ms? We don't want to go too high, or quick presses will get ignored. |
No, because I'm not able to compile firmware byy myself, I don't have enough expperience. I can only compile userspace programs, like VLC media player or Strawberry. But of course, I suspect, that 20 ms will do the work. |
I've been using 20ms for some time now and while I do get repeated characters, they are infrequent enough as to not be annoying. Maybe a couple per day. I find it incredible that anyone could use this hardware with the settings at 10ms. I understand there are differences in typing style but even if I'm careful and slow when typing I get a lot of repeated characters. |
We're finding that debounce time is highly subjective. 15ms was originally the value, but we had reports of skipped keystrokes, which is why we went to 5ms. Then 5ms was too short for many, and 10ms was the new value. Having tried the whole range myself, I don't seem to have issues with any of them. I only got repeats when I moved the debounce time down to 2ms, and I didn't get skipped keys until I moved it above 30ms (I think I tried 45 or so, but I forget). If there's no perfect default that we can settle on, we may have to make this an end-user configurable setting. |
This is absolutely the right answer, if doable. Thehre can be a sane default while still allowing useres to set their own tweaks. As you can see, I would also benefit from having this as a customizable setting. |
I agree that making it user configurable is the best solution. Use case: I needed to use my wife's laptop (same model) and immediately I had lots of repeated chars. So many that it was difficult to do what I needed which was simple terminal commands to upgrade packages and things. No way am I going to flash firmware to use her machine for a few minutes! |
Is there any pprogress in this issue? |
Keys are sommetimes repeated when typing normally. Typing in this issue is intentionally noot coorrected.
Steps to reproduce
Type a bunch of text and eventually kkeys will be repeated.
Expected behavior
OOne press of the key will result in one character typed.
Actual behavior
Sometimes two characters are typed.
Additional info
Probably related to #471
The text was updated successfully, but these errors were encountered: