* Keep an open count in the JoystickProtocolHandler.

* Only cancel the update thread if the last open instance is closed. This fixes
  that other applications using the same joystick would stop getting updated
  values as soon as one application was closed.


git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@41905 a95241bf-73f2-0310-859d-f6bbb57e9c96

src/add-ons/kernel/drivers/input/usb_hid/JoystickProtocolHandler.cpp

@@ -31,6 +31,7 @@ JoystickProtocolHandler::JoystickProtocolHandler(HIDReport &report)
 	fButtonCount(0),
 	fMaxButton(0),
 	fButtons(NULL),
+	fOpenCount(0),
 	fUpdateThread(-1)
 {
 	mutex_init(&fUpdateLock, "joystick update lock");
@@ -101,7 +102,7 @@ JoystickProtocolHandler::JoystickProtocolHandler(HIDReport &report)
 
 	fCurrentValues.initialize(fAxisCount, 0, fMaxButton);
 
-	TRACE("joystick device with %lu buttons\n", buttonCount);
+	TRACE("joystick device with %lu buttons\n", fButtonCount);
 	TRACE("report id: %u\n", report.ID());
 }
 
@@ -190,6 +191,9 @@ JoystickProtocolHandler::Open(uint32 flags, uint32 *cookie)
 			resume_thread(fUpdateThread);
 	}
 
+	if (result == B_OK)
+		fOpenCount++;
+
 	mutex_unlock(&fUpdateLock);
 	if (result != B_OK)
 		return result;
@@ -203,7 +207,8 @@ JoystickProtocolHandler::Close(uint32 *cookie)
 {
 	status_t result = mutex_lock(&fUpdateLock);
 	if (result == B_OK) {
-		fUpdateThread = -1;
+		if (--fOpenCount == 0)
+			fUpdateThread = -1;
 		mutex_unlock(&fUpdateLock);
 	}
 

src/add-ons/kernel/drivers/input/usb_hid/JoystickProtocolHandler.h

@@ -55,6 +55,7 @@ private:
 			joystick_module_info 	fJoystickModuleInfo;
 			variable_joystick		fCurrentValues;
 
+			uint32					fOpenCount;
 			mutex					fUpdateLock;
 			thread_id				fUpdateThread;
 };