[PATCH RESEND 5/5] xfree86: use NewInputDeviceRequest for xorg.conf devices too
Daniel Stone
daniel at fooishbar.org
Thu Oct 20 06:17:57 PDT 2011
Hi,
On Thu, Oct 20, 2011 at 01:05:32PM +1000, Peter Hutterer wrote:
> diff --git a/hw/xfree86/common/xf86Config.c b/hw/xfree86/common/xf86Config.c
> index 96e98c1..cb4be42 100644
> --- a/hw/xfree86/common/xf86Config.c
> +++ b/hw/xfree86/common/xf86Config.c
> @@ -1196,6 +1196,10 @@ checkCoreInputDevices(serverLayoutPtr servlayoutp, Bool implicitLayout)
> if (foundPointer) {
> Pointer->options = xf86AddNewOption(Pointer->options,
> "CorePointer", "on");
> + Pointer->options = xf86AddNewOption(Pointer->options,
> + "driver", confInput->inp_driver);
> + Pointer->options = xf86AddNewOption(Pointer->options,
> + "identifier", confInput->inp_identifier);
> servlayoutp->inputs = addDevice(servlayoutp->inputs, Pointer);
> }
> }
> @@ -1286,6 +1290,10 @@ checkCoreInputDevices(serverLayoutPtr servlayoutp, Bool implicitLayout)
> if (foundKeyboard) {
> Keyboard->options = xf86AddNewOption(Keyboard->options,
> "CoreKeyboard", "on");
> + Keyboard->options = xf86AddNewOption(Keyboard->options,
> + "driver", confInput->inp_driver);
> + Keyboard->options = xf86AddNewOption(Keyboard->options,
> + "identifier", confInput->inp_identifier);
> servlayoutp->inputs = addDevice(servlayoutp->inputs, Keyboard);
> }
> }
Adding "_source" / "server/conf" or similar might be nice. Even though
no-one really uses it.
> @@ -842,20 +827,8 @@ InitInput(int argc, char **argv)
>
> /* Initialize all configured input devices */
> for (pInfo = xf86ConfigLayout.inputs; pInfo && *pInfo; pInfo++) {
> - InputInfoPtr dup;
> - /* Replace obsolete keyboard driver with kbd */
> - if (!xf86NameCmp((*pInfo)->driver, "keyboard")) {
> - strcpy((*pInfo)->driver, "kbd");
> - }
It'd probably be polite to keep this logic around in NIDR.
Otherwise:
Reviewed-by: Daniel Stone <daniel at fooishbar.org>
Cheers,
Daniel
More information about the xorg-devel
mailing list