Thread (10 messages) 10 messages, 3 authors, 2018-12-10

Re: [PATCH 1/2] Input: omap-keypad: Fix keyboard debounce configuration

From: Pavel Machek <hidden>
Date: 2018-12-09 11:56:54
Also in: linux-omap, lkml

On Sun 2018-12-02 17:29:32, Tony Lindgren wrote:
I noticed that the Android v3.0.8 kernel on droid4 is using different
keypad values from the mainline kernel and does not have issues with
keys occasionally being stuck until pressed again. Turns out there was
an earlier patch posted to fix this as "Input: omap-keypad: errata i689:
Correct debounce time", but it was never reposted to fix use macros
for timing calculations.

This updated version is using macros, and also fixes the use of the
input clock rate to use 32768KiHz instead of 32000KiHz. And we want to
use the known good Android kernel values of 3 and 6 instead of 2 and 6
in the earlier patch.

Cc: Axel Haslam <redacted>
Cc: Felipe Balbi <redacted>
Cc: Illia Smyrnov <redacted>
Cc: Marcel Partap <redacted>
Cc: Merlijn Wajer <redacted>
Cc: Michael Scott <redacted>
Cc: NeKit <redacted>
Cc: Pavel Machek <redacted>
Cc: Sebastian Reichel <sre@kernel.org>
Reported-by: Pavel Machek <redacted>
Signed-off-by: Tony Lindgren <tony@atomide.com>
So, if I press key "." quickly on console in
4.18.0-rc4-88970-gf075a2c-dirty it starts repeating. I could reproduce
it 3 times from tree tries. Then I decided to test your patch, and so
I booted 4.20-rc5+ using kexec. Turns out, I can't reproduce that
repeat there... I tried more than 10 times. But that's before your
patch.

So -- bad news for you -- seems like someone -- maybe you -- already
fixed this. (But that does not mean the patch is bad idea).

Thanks,
									Pavel

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help