src/sys/dev/adb, branch releng/12.4

MFC r355796-r355797, r355799: kbd: defaults for get_fkeystr/diag

2019-12-22T17:15:48Z

The genkbd version of these remains exposed for stable branches, but keyboard drivers that just want to use the defaults can simply not provide their own. There shouldn't be any unset in the wild. r355796: kbd: provide default implementations of get_fkeystr/diag Most keyboard drivers are using the genkbd implementations as it is; formally use them for any that aren't set. r355797: chrome_kb: remove default get_fkeystr/diag implementations This file was missed in r355796, but no harm would have come from this. r355799: kbd: patch linker set methods, too This is needed after r355796. Some double-registration of kbd drivers needs to be sorted out, then this sysinit will simply add these drivers into the normal list and kill off any other bits in the driver that are aware of the linker set, for simplicity.

MFC r355794: keyboard switch definitions: standardize on c99 initializers

2019-12-22T16:04:12Z

sys/dev: further adoption of SPDX licensing ID tags.

2017-11-27T14:52:40Z

Mainly focus on files that use BSD 2-Clause license, however the tool I was using misidentified many licenses so this was mostly a manual - error prone - task. The Software Package Data Exchange (SPDX) group provides a specification to make it easier for automated tools to detect and summarize well known opensource licenses. We are gradually adopting the specification, noting that the tags are considered only advisory and do not, in any way, superceed or replace the license texts.

Force the setting of bit 7 in the sysmouse packet byte 1 to be unsigned.

2017-01-30T02:32:33Z

Clang complains about the shift of (1 << 7) into a int8_t changing the value: warning: implicit conversion from 'int' to 'int8_t' (aka 'signed char') changes value from 128 to -128 [-Wconstant-conversion] Squash this warning by forcing clang to see it as an unsigned bit. This seems odd, given that it's still a conversion of 128->-128, but I'm guessing the explicit unsigned attribute notifies clang that sign really doesn't matter in this case. Reported by: Mark Millard MFC after: 2 weeks

sys: use our roundup2/rounddown2() macros when param.h is available.

2016-04-21T19:57:40Z

rounddown2 tends to produce longer lines than the original code and when the code has a high indentation level it was not really advantageous to do the replacement. This tries to strike a balance between readability using the macros and flexibility of having the expressions, so not everything is converted.

sys/dev: use our nitems() macro when it is avaliable through param.h.

2016-04-19T23:37:24Z

No functional change, only trivial cases are done in this sweep, Drivers that can get further enhancements will be done independently. Discussed in: freebsd-current

No need to check if devd is running before posting an event.

2014-06-21T00:53:56Z

Properly sort the arguments to mtx_init(9).

2014-01-23T01:09:33Z

PR: misc/186020 Submitted by: alfred MFC after: 1 week

Return NOKEY instead of 0 if there are no more key presses queued. This

2013-10-29T00:53:17Z

worked by accident if and only if akbd was part of a kbdmux (which it always was in practice). MFC after: 1 week

Fix powerpc/161045. ams_poll() needs to return that any data is available, not

2013-09-28T02:13:59Z

just a new packet. PR: powerpc/161045 Approved by: re(marius) MFC after: 1 week