Add gpiokeys driver
ClosedPublic
Actions

Authored by gonzo on May 9 2016, 8:50 PM.

Details

Reviewers: None
Commits: rS299475: Add gpiokeys driver

Summary

Add gpiokeys driver that implements functional subset of gpiokeys device-tree bindings:

https://www.kernel.org/doc/Documentation/devicetree/bindings/input/gpio-keys.txt

Driver maps linux scancodes for most common keys to FreeBSD scancodes and also extends spec by introducing freebsd,code property to specify FreeBSD-native scancodes.

Test Plan

Use 4DCAPE-43T with vendor-provided DTB to test keys.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

gonzo updated this revision to Diff 16087.May 9 2016, 8:50 PM

gonzo retitled this revision from to Add gpiokeys driver.

gonzo updated this object.

gonzo edited the test plan for this revision. (Show Details)

gonzo set the repository for this revision to rS FreeBSD src repository - subversion.

gonzo added a subscriber: ARM.

Herald added subscribers: loos, imp. · View Herald TranscriptMay 9 2016, 8:50 PM

gonzo added a parent revision: D6277: Pass node to the gpio_pin_get_by_ofw_XXX functions.May 9 2016, 8:56 PM

jmcneill added a subscriber: jmcneill.May 9 2016, 9:04 PM

jmcneill added inline comments.

sys/dev/gpio/gpiokeys.c
61	#define<tab>
273	This will always be zero -- I would change to `(hz*AUTOREPEAT_DELAY)/1000`
277	Same as above.

mmel added a subscriber: mmel.May 10 2016, 5:13 AM

mmel added inline comments.

sys/dev/gpio/gpiokeys.c
239	Magic constant ?
349	ihmo, callout_drain() must be used here.
619	Whats about #define GPIO_ASSERT_LOCKED(_key) mtx_assert(&(_key)->mtx, MA_OWNED) and GPIO_ASSERT_LOCKED(sc);
643	Lock is missing here -> gpiokey_get_key() must be called with lock.

Add handler for "debounce-interval" key property
Replace static gpiokeys_sc with dynamic one which allows us have more than one gpiokeys instance
Fix locking
Fix arithmetic expressions
style(9) fixes