UseDeprecatedKeyboardDriver should be TRUE on most OS'es since the new keyboard
driver is not supported on them.
The simplest check for which OS'es it works on and doesn't seems to be seeing if
xf86OSKbdPreInit is a stub that just returns FALSE or actually inits the function
pointers in the OS'es directory under xc/programs/Xserver/hw/xfree86/os-support
A quick peek seems to show these implementations do have kbd driver support:
and these do not:
The fix proposed at release-wranglers was to change the default to YES and just
set NO for the platforms we know support it, at least for this release.
After thinking about this one some more, I think it might be better to keep the
default as NO, and explicitly enable the deprecated driver on systems that do no
yet support the new kbd driver. This will help remind people that it is being
deprecated and additional support is needed.
Created attachment 626 [details] [review]
Patch to enable deprecated keyboard driver
This patch sets UseDeprecatedKeyboardDriver to YES for those platforms that
doesn't support the new driver. It keeps NO as the default.
Created attachment 628 [details] [review]
Patch that doesn't break FreeBSD, add define for DGUX also.
The patch looks okay to me. Alan?
Looks good to me too - not sure we got all the platforms, but I don't even know
that we've ever put together a complete list of platforms that the Xorg server
Patch committed. I may not cover all platforms, but it does cover the platforms
that have a subdirectory in os-support. If I've missed some platforms, we can
fix that in 6.8.1, and anyway it's just a matter of setting the #define in host.def.