Commit 3f0c4044 authored Nov 15, 2018 by Raghavendra Rao Ananta Committed by Gerrit - the friendly Code Review server Nov 27, 2018

HID: uhid: Remove the redundant reference counting for open/close



Currently, the uhid driver manages the reference count for the number
of clients active for this uhid device. However, the reference count
is already managed by the hid-core. As a result, during uhid_open for
example, we never get the UHID_OPEN event in the upper layers as the
reference count is incremented twice for each open(). Hence, let the
reference counting be done only by the hid-core and not by the uhid
driver.

Change-Id: I473239db0f373d9188fc379653cedd305d2a552b
Signed-off-by: Raghavendra Rao Ananta <rananta@codeaurora.org>

parent 0d2cc45e

drivers/hid/uhid.c

+2 −9

Original line number	Diff line number	Diff line
		@@ -142,21 +142,14 @@ static void uhid_hid_stop(struct hid_device *hid)
		static int uhid_hid_open(struct hid_device *hid)
		{
		struct uhid_device *uhid = hid->driver_data;
		int retval = 0;

		if (!hid->ll_open_count++) {
		retval = uhid_queue_event(uhid, UHID_OPEN);
		if (retval)
		hid->ll_open_count--;
		}
		return retval;
		return uhid_queue_event(uhid, UHID_OPEN);
		}

		static void uhid_hid_close(struct hid_device *hid)
		{
		struct uhid_device *uhid = hid->driver_data;

		if (!--hid->ll_open_count)
		uhid_queue_event(uhid, UHID_CLOSE);
		}