[PATCH] xfree86: a missing input driver is not an error.
Dan Nicholson
dbn.lists at gmail.com
Thu Apr 29 18:19:52 PDT 2010
On Wed, Apr 28, 2010 at 4:51 PM, Peter Hutterer
<peter.hutterer at who-t.net> wrote:
> We call NIDR on all devices that make it through the config backend.
> Including some that have no driver assigned to them (/dev/input/mouse0 for
> example). Those ones then simply get ignored by NIDR, but this should not be
> noted as an error in the log file.
>
> X_INFO is sufficient, and it may just prevent some bugreports.
>
> Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
> ---
> hw/xfree86/common/xf86Xinput.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/hw/xfree86/common/xf86Xinput.c b/hw/xfree86/common/xf86Xinput.c
> index dba3370..a1a5527 100644
> --- a/hw/xfree86/common/xf86Xinput.c
> +++ b/hw/xfree86/common/xf86Xinput.c
> @@ -825,7 +825,7 @@ NewInputDeviceRequest (InputOption *options, InputAttributes *attrs,
> }
>
> if (!idev->driver || !idev->identifier) {
> - xf86Msg(X_ERROR, "No input driver/identifier specified (ignoring)\n");
> + xf86Msg(X_INFO, "No input driver/identifier specified (ignoring)\n");
> rval = BadRequest;
> goto unwind;
> }
Yep. I wanted to audit all the "fail" pathways from calling NIDR, but
I haven't gotten to it. This is one of them, though.
Reviewed-by: Dan Nicholson <dbn.lists at gmail.com>
More information about the xorg-devel
mailing list