hconf(4): Do not fetch report before writing new usage values back.

[wulf]

Summary

There is a report that reading of surface/button switch feature report
causes SYN1B7D touchpad malfunction. As specs does not require it to
be readable assume that report usages have default value on attach and
last written value during operation.
While here, do not apply default usage values on resume.

Test Plan

Test dev.hmt.*.* sysctls before and after suspend/resume

[wulf]

Fix stupid bug leading to setting of old usage value instead of new one.

[alvisen_gmail.com]

hconf(4) says dev.hmt where I think it should say dev.hconf:

diff --git a/share/man/man4/hconf.4 b/share/man/man4/hconf.4
index 4d595e9f1c6..a2b50dbb8de 100644

• a/share/man/man4/hconf.4

+++ b/share/man/man4/hconf.4
@@ -61,11 +61,11 @@ Debug parameter is available as
.Xr loader 8
tunable as well.
.Bl -tag -width indent
-.It Va dev.hmt.*.input_mode
+.It Va dev.hconf.*.input_mode
HID device input mode: 0 = mouse, 3 = touchpad.
-.It Va dev.hmt.*.surface_switch
+.It Va dev.hconf.*.surface_switch
Enable / disable switch for surface: 1 = on, 0 = off.
-.It Va dev.hmt.*.buttons_switch
+.It Va dev.hconf.*.buttons_switch
Enable / disable switch for buttons: 1 = on, 0 = off.
.It Va hw.hid.hconf.debug
Debug output level, where 0 is debugging disabled and larger values increase

[wulf]

Fix manpage

[avg]

LGTM. Thank you!
I did not realize that hardware can be this quirky.