This patch implements synthetic keyboard driver for FreeBSD on Hyper-V Gen2. In Gen2 mode, the legacy keyboard does not work.
Details
Details
Diff Detail
Diff Detail
- Lint
Lint Skipped - Unit
Tests Skipped
Event Timeline
Comment Actions
Looks pretty good to me. Some minor code style suggestion.
| sys/dev/hyperv/input/hv_kbdc.c | ||
|---|---|---|
| 68–70 | Better use C style comment, or '#if 0 ... #endif' | |
| 108–119 | May be we should use more consistent style of comment? /* * */ or your preference: /** * */ | |
| 308 | I'd prefer for (;;) | |
| 332–343 | We don't indent switch-case: switch () {
case XXX:
aaaa;
case YYY:
bbbb;
default:
cccc;
} | |