Commit 3ef8a2ee authored by Oleksandr Andrushchenko's avatar Oleksandr Andrushchenko Committed by Dmitry Torokhov
Browse files

Input: xen-kbdfront - allow better run-time configuration 



It is now only possible to control if multi-touch virtual device
is created or not (via the corresponding XenStore entries),
but keyboard and pointer devices are always created.
In some cases this is not desirable. For example, if virtual
keyboard device is exposed to Android then the latter won't
automatically show on-screen keyboard as it expects that a
physical keyboard device can be used for typing.

Utilize keyboard and pointer device XenStore feature fields to
configure which virtual devices are created:
 - set "feature-disable-keyboard" to 1 if no keyboard device
   needs to be created
 - set "feature-disable-pointer" to 1 if no pointer device
   needs to be created
Keep old behavior by default.

Signed-off-by: default avatarOleksandr Andrushchenko <oleksandr_andrushchenko@epam.com>
Suggested-by: default avatarAndrii Chepurnyi <andrii_chepurnyi@epam.com>
Tested-by: default avatarAndrii Chepurnyi <andrii_chepurnyi@epam.com>
Reviewed-by: default avatarJuergen Gross <jgross@suse.com>
Signed-off-by: default avatarDmitry Torokhov <dmitry.torokhov@gmail.com>
parent ce6f7d08

drivers/input/misc/xen-kbdfront.c

+106 −71
Original line number Diff line number Diff line
@@ -63,6 +63,9 @@ static void xenkbd_disconnect_backend(struct xenkbd_info *); 
static void xenkbd_handle_motion_event(struct xenkbd_info *info,

				       struct xenkbd_motion *motion)

{

	if (unlikely(!info->ptr))

		return;



	input_report_rel(info->ptr, REL_X, motion->rel_x);

	input_report_rel(info->ptr, REL_Y, motion->rel_y);

	if (motion->rel_z)
@@ -73,6 +76,9 @@ static void xenkbd_handle_motion_event(struct xenkbd_info *info, 
static void xenkbd_handle_position_event(struct xenkbd_info *info,

					 struct xenkbd_position *pos)

{

	if (unlikely(!info->ptr))

		return;



	input_report_abs(info->ptr, ABS_X, pos->abs_x);

	input_report_abs(info->ptr, ABS_Y, pos->abs_y);

	if (pos->rel_z)
@@ -97,6 +103,9 @@ static void xenkbd_handle_key_event(struct xenkbd_info *info, 
		return;

	}



	if (unlikely(!dev))

		return;



	input_event(dev, EV_KEY, key->keycode, value);

	input_sync(dev);

}
@@ -192,7 +201,7 @@ static int xenkbd_probe(struct xenbus_device *dev, 
				  const struct xenbus_device_id *id)

{

	int ret, i;

	unsigned int abs, touch;

	bool with_mtouch, with_kbd, with_ptr;

	struct xenkbd_info *info;

	struct input_dev *kbd, *ptr, *mtouch;
@@ -211,36 +220,30 @@ static int xenkbd_probe(struct xenbus_device *dev, 
	if (!info->page)

		goto error_nomem;



	/* Set input abs params to match backend screen res */

	abs = xenbus_read_unsigned(dev->otherend,

				   XENKBD_FIELD_FEAT_ABS_POINTER, 0);

	ptr_size[KPARAM_X] = xenbus_read_unsigned(dev->otherend,

						  XENKBD_FIELD_WIDTH,

						  ptr_size[KPARAM_X]);

	ptr_size[KPARAM_Y] = xenbus_read_unsigned(dev->otherend,

						  XENKBD_FIELD_HEIGHT,

						  ptr_size[KPARAM_Y]);

	if (abs) {

		ret = xenbus_write(XBT_NIL, dev->nodename,

				   XENKBD_FIELD_REQ_ABS_POINTER, "1");

		if (ret) {

			pr_warn("xenkbd: can't request abs-pointer\n");

			abs = 0;

		}

	}

	/*

	 * The below are reverse logic, e.g. if the feature is set, then

	 * do not expose the corresponding virtual device.

	 */

	with_kbd = !xenbus_read_unsigned(dev->otherend,

					 XENKBD_FIELD_FEAT_DSBL_KEYBRD, 0);



	touch = xenbus_read_unsigned(dev->otherend,

	with_ptr = !xenbus_read_unsigned(dev->otherend,

					 XENKBD_FIELD_FEAT_DSBL_POINTER, 0);



	/* Direct logic: if set, then create multi-touch device. */

	with_mtouch = xenbus_read_unsigned(dev->otherend,

					   XENKBD_FIELD_FEAT_MTOUCH, 0);

	if (touch) {

	if (with_mtouch) {

		ret = xenbus_write(XBT_NIL, dev->nodename,

				   XENKBD_FIELD_REQ_MTOUCH, "1");

		if (ret) {

			pr_warn("xenkbd: can't request multi-touch");

			touch = 0;

			with_mtouch = 0;

		}

	}



	/* keyboard */

	if (with_kbd) {

		kbd = input_allocate_device();

		if (!kbd)

			goto error_nomem;
@@ -259,12 +262,35 @@ static int xenkbd_probe(struct xenbus_device *dev, 
		ret = input_register_device(kbd);

		if (ret) {

			input_free_device(kbd);

		xenbus_dev_fatal(dev, ret, "input_register_device(kbd)");

			xenbus_dev_fatal(dev, ret,

					 "input_register_device(kbd)");

			goto error;

		}

		info->kbd = kbd;

	}



	/* pointing device */

	if (with_ptr) {

		unsigned int abs;



		/* Set input abs params to match backend screen res */

		abs = xenbus_read_unsigned(dev->otherend,

					   XENKBD_FIELD_FEAT_ABS_POINTER, 0);

		ptr_size[KPARAM_X] = xenbus_read_unsigned(dev->otherend,

							  XENKBD_FIELD_WIDTH,

							  ptr_size[KPARAM_X]);

		ptr_size[KPARAM_Y] = xenbus_read_unsigned(dev->otherend,

							  XENKBD_FIELD_HEIGHT,

							  ptr_size[KPARAM_Y]);

		if (abs) {

			ret = xenbus_write(XBT_NIL, dev->nodename,

					   XENKBD_FIELD_REQ_ABS_POINTER, "1");

			if (ret) {

				pr_warn("xenkbd: can't request abs-pointer\n");

				abs = 0;

			}

		}



		ptr = input_allocate_device();

		if (!ptr)

			goto error_nomem;
@@ -276,8 +302,10 @@ static int xenkbd_probe(struct xenbus_device *dev, 


		if (abs) {

			__set_bit(EV_ABS, ptr->evbit);

		input_set_abs_params(ptr, ABS_X, 0, ptr_size[KPARAM_X], 0, 0);

		input_set_abs_params(ptr, ABS_Y, 0, ptr_size[KPARAM_Y], 0, 0);

			input_set_abs_params(ptr, ABS_X, 0,

					     ptr_size[KPARAM_X], 0, 0);

			input_set_abs_params(ptr, ABS_Y, 0,

					     ptr_size[KPARAM_Y], 0, 0);

		} else {

			input_set_capability(ptr, EV_REL, REL_X);

			input_set_capability(ptr, EV_REL, REL_Y);
@@ -291,13 +319,15 @@ static int xenkbd_probe(struct xenbus_device *dev, 
		ret = input_register_device(ptr);

		if (ret) {

			input_free_device(ptr);

		xenbus_dev_fatal(dev, ret, "input_register_device(ptr)");

			xenbus_dev_fatal(dev, ret,

					 "input_register_device(ptr)");

			goto error;

		}

		info->ptr = ptr;

	}



	/* multi-touch device */

	if (touch) {

	if (with_mtouch) {

		int num_cont, width, height;



		mtouch = input_allocate_device();
@@ -346,6 +376,11 @@ static int xenkbd_probe(struct xenbus_device *dev, 
		info->mtouch = mtouch;

	}



	if (!(with_kbd || with_ptr || with_mtouch)) {

		ret = -ENXIO;

		goto error;

	}



	ret = xenkbd_connect_backend(dev, info);

	if (ret < 0)

		goto error;

CRA Git | Maintained and supported by SUSTech CRA and CCSE