[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