On Wed, Nov 25, 2015 at 07:19:48AM +1000, Peter Hutterer wrote: > On Tue, Nov 24, 2015 at 12:46:44PM +0500, Alexandr Shadchin wrote: > > Signed-off-by: Alexandr Shadchin <alexandr.shadc...@gmail.com> > > do you have follow-up patches that require these changes? I'm ok merging > this set if it makes follow-ups easier, but a patchset just to remove > functionality seems a bit superfluous. > > Cheers, > Peter >
Not yet. I found these changes in my repository. I considered that it makes sense, xorg-server 1.10 was released in 2011. evdev and synaptics: xorg-server >= 1.12 > > --- > > configure.ac | 2 +- > > src/kbd.c | 83 > > ++++++++++-------------------------------------------------- > > 2 files changed, 14 insertions(+), 71 deletions(-) > > > > diff --git a/configure.ac b/configure.ac > > index 6fd6576..993d5a8 100644 > > --- a/configure.ac > > +++ b/configure.ac > > @@ -44,7 +44,7 @@ XORG_MACROS_VERSION(1.8) > > XORG_DEFAULT_OPTIONS > > > > # Obtain compiler/linker options from server and required extensions > > -PKG_CHECK_MODULES(XORG, [xorg-server >= 1.4] xproto inputproto) > > +PKG_CHECK_MODULES(XORG, [xorg-server >= 1.10] xproto inputproto) > > > > # Define a configure option for an alternate input module directory > > AC_ARG_WITH(xorg-module-dir, > > diff --git a/src/kbd.c b/src/kbd.c > > index 1ce4ee6..e378dcc 100644 > > --- a/src/kbd.c > > +++ b/src/kbd.c > > @@ -53,11 +53,7 @@ > > /* Used to know when the first DEVICE_ON after a DEVICE_INIT is called */ > > #define INITFLAG (1U << 31) > > > > -#if GET_ABI_MAJOR(ABI_XINPUT_VERSION) < 12 > > -static InputInfoPtr KbdPreInit(InputDriverPtr drv, IDevPtr dev, int flags); > > -#else > > static int KbdPreInit(InputDriverPtr drv, InputInfoPtr pInfo, int flags); > > -#endif > > static int KbdProc(DeviceIntPtr device, int what); > > static void KbdCtrl(DeviceIntPtr device, KeybdCtrl *ctrl); > > static void KbdBell(int percent, DeviceIntPtr dev, pointer ctrl, int > > unused); > > @@ -132,43 +128,8 @@ _X_EXPORT XF86ModuleData kbdModuleData = { > > NULL > > }; > > > > -#if GET_ABI_MAJOR(ABI_XINPUT_VERSION) < 12 > > -static int > > -NewKbdPreInit(InputDriverPtr drv, InputInfoPtr pInfo, int flags); > > - > > -static InputInfoPtr > > -KbdPreInit(InputDriverPtr drv, IDevPtr dev, int flags) > > -{ > > - InputInfoPtr pInfo; > > - > > - if (!(pInfo = xf86AllocateInput(drv, 0))) > > - return NULL; > > - > > - pInfo->name = dev->identifier; > > - pInfo->flags = XI86_KEYBOARD_CAPABLE; > > - pInfo->conversion_proc = NULL; > > - pInfo->reverse_conversion_proc = NULL; > > - pInfo->private_flags = 0; > > - pInfo->always_core_feedback = NULL; > > - pInfo->conf_idev = dev; > > - pInfo->close_proc = NULL; > > - > > - if (NewKbdPreInit(drv, pInfo, flags) == Success) > > - { > > - pInfo->flags |= XI86_CONFIGURED; > > - return pInfo; > > - } > > - > > - xf86DeleteInput(pInfo, 0); > > - return NULL; > > -} > > - > > -static int > > -NewKbdPreInit(InputDriverPtr drv, InputInfoPtr pInfo, int flags) > > -#else > > static int > > KbdPreInit(InputDriverPtr drv, InputInfoPtr pInfo, int flags) > > -#endif > > { > > KbdDevPtr pKbd; > > char *s; > > @@ -189,11 +150,7 @@ KbdPreInit(InputDriverPtr drv, InputInfoPtr pInfo, int > > flags) > > pInfo->dev = NULL; > > > > defaults = kbdDefaults; > > - xf86CollectInputOptions(pInfo, defaults > > -#if GET_ABI_MAJOR(ABI_XINPUT_VERSION) < 12 > > - , NULL > > -#endif > > - ); > > + xf86CollectInputOptions(pInfo, defaults); > > xf86ProcessCommonOptions(pInfo, pInfo->options); > > > > if (!(pKbd = calloc(sizeof(KbdDevRec), 1))) { > > @@ -335,6 +292,7 @@ KbdProc(DeviceIntPtr device, int what) > > > > InputInfoPtr pInfo = device->public.devicePrivate; > > KbdDevPtr pKbd = (KbdDevPtr) pInfo->private; > > + XkbRMLVOSet rmlvo; > > KeySymsRec keySyms; > > CARD8 modMap[MAP_LENGTH]; > > int ret; > > @@ -348,23 +306,19 @@ KbdProc(DeviceIntPtr device, int what) > > pKbd->KbdGetMapping(pInfo, &keySyms, modMap); > > > > device->public.on = FALSE; > > -#if GET_ABI_MAJOR(ABI_XINPUT_VERSION) >= 5 > > + rmlvo.rules = xkb_rules; > > + rmlvo.model = xkb_model; > > + rmlvo.layout = xkb_layout; > > + rmlvo.variant = xkb_variant; > > + rmlvo.options = xkb_options; > > + > > + if (!InitKeyboardDeviceStruct(device, &rmlvo, KbdBell, KbdCtrl)) > > { > > - XkbRMLVOSet rmlvo; > > - rmlvo.rules = xkb_rules; > > - rmlvo.model = xkb_model; > > - rmlvo.layout = xkb_layout; > > - rmlvo.variant = xkb_variant; > > - rmlvo.options = xkb_options; > > - > > - if (!InitKeyboardDeviceStruct(device, &rmlvo, KbdBell, > > KbdCtrl)) > > - { > > - xf86Msg(X_ERROR, "%s: Keyboard initialization failed. > > This " > > - "could be a missing or incorrect setup of " > > - "xkeyboard-config.\n", device->name); > > + xf86Msg(X_ERROR, "%s: Keyboard initialization failed. This " > > + "could be a missing or incorrect setup of " > > + "xkeyboard-config.\n", device->name); > > > > - return BadValue; > > - } > > + return BadValue; > > } > > # ifdef XI_PROP_DEVICE_NODE > > { > > @@ -381,17 +335,6 @@ KbdProc(DeviceIntPtr device, int what) > > } > > } > > # endif /* XI_PROP_DEVICE_NODE */ > > -#else > > - { > > - XkbComponentNamesRec xkbnames; > > - memset(&xkbnames, 0, sizeof(xkbnames)); > > - XkbSetRulesDflts(xkb_rules, xkb_model, xkb_layout, > > - xkb_variant, xkb_options); > > - XkbInitKeyboardDeviceStruct(device, &xkbnames, &keySyms, > > - modMap, KbdBell, > > - (KbdCtrlProcPtr)KbdCtrl); > > - } > > -#endif /* XINPUT ABI 5*/ > > InitKBD(pInfo, TRUE); > > break; > > case DEVICE_ON: > > -- > > 2.6.3 > > -- Alexandr Shadchin _______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel