Re: [PATCH 2/3] Add support for switching from HCI->HID mode for Dell BT adapters.

[Mario Limonciello <mario_limonciello@dell.com>]

[-- Attachment #1.1: Type: text/plain, Size: 427 bytes --]

Hi Marcel:

Marcel Holtmann wrote:
> Hi Mario,
>
> did you ever tested this patch with --enable-debug compilation? No code
> before variable declaration. Please fix that.
>
> Regards
>
> Marcel
>   
No, I didn't realize I was supposed to test with --enable-debug.  I've
done so now and fixed up this patch.  See attached.

Regards
-- 
Mario Limonciello
*Dell | Linux Engineering*
mario_limonciello@dell.com

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1.2: 0002-Add-support-for-switching-from-HCI-HID-mode-for-Del.patch --]
[-- Type: text/x-patch; name="0002-Add-support-for-switching-from-HCI-HID-mode-for-Del.patch", Size: 1120 bytes --]

From 424c70338db22122e78e53c767c4224d2c6f2a29 Mon Sep 17 00:00:00 2001
From: Mario Limonciello <Mario_Limonciello@Dell.com>
Date: Fri, 15 May 2009 16:54:39 -0500
Subject: [PATCH] Add support for switching from HCI->HID mode for Dell BT adapters.
 This allows Dell BT adapters to be able to be switched back to HID
 mode if users desire to do so, such as on shutdown.

---
 tools/hid2hci.c |   11 ++++++++++-
 1 files changed, 10 insertions(+), 1 deletions(-)

diff --git a/tools/hid2hci.c b/tools/hid2hci.c
index 7692de3..bf03739 100644
--- a/tools/hid2hci.c
+++ b/tools/hid2hci.c
@@ -207,11 +207,20 @@ static int switch_logitech(struct device_info *devinfo)
 
 static int switch_dell(struct device_info *devinfo)
 {
-	char report[] = { 0x7f, 0x13, 0x00, 0x00 };
+	char report[] = { 0x7f, 0x00, 0x00, 0x00 };
 
 	struct usb_dev_handle *handle;
 	int err;
 
+	switch(devinfo->mode) {
+		case HCI:
+			report[1] = 0x13;
+			break;
+		case HID:
+			report[1] = 0x14;
+			break;
+	}
+
 	handle = usb_open(devinfo->dev);
 	if (handle) {
 		usb_claim_interface(handle, 0);
-- 
1.6.0.4


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 260 bytes --]