Re: New Force Feedback device support - GreenAsia 0x12

["Łukasz Lubojański" <lukasz@lubojanski.info>]

[-- Attachment #1: Type: text/plain, Size: 1624 bytes --]

On Thu, Dec 4, 2008 at 9:35 PM, Łukasz Lubojański
<lukasz@lubojanski.info> wrote:
> 2008/11/29 Jiri Kosina <jkosina@suse.cz>:
>> On Fri, 28 Nov 2008, Łukasz Lubojański wrote:
>>
>>> > It seems the protocol resembles more the hid-lg2ff one. The differences
>>> > are the additional 0xfa 0xfe 0x0 report sent to the device, and the
>>> > missing 0xf3 stop command.
>>> Yep - different reports are send in case of Pantherlord and GreenAsia
>>> 0x12 - It could be implemented in it but it will require checking what
>>> hardware is used and send different reports.
>>
>> OK, so as the reports are not really identical, and in the future we might
>> discover that there are many more other Greenasia devices which require a
>> slightly different handling as well, I would rather prefer to have it as a
>> separate driver, to avoid additions of here-and-there device-specific
>> quirks to random places in the code. That's exactly what we are trying to
>> avoid with the HID bus approach in the first place.
>>
>> So I think separate driver is fine.
>>
>> Thanks to both of you.
>>
>> --
>> Jiri Kosina
>> SUSE Labs
>
> Hi,
>
> Here is new version of the GreenAsia patch - I hope this time
> everything will be OK. It is based on the Pantherlord.
>
> Sorry to take so long but I have problems with the 2.6.28 (2.6.28-rc6
> was not loading my driver and 2.6.28-rc7 is crashing when IO APIC is
> enabled). Anyway I done it and I'm waiting for your feedback :D
>
> regards
> Lukasz Lubojanski
>

And one more time - because I have generated and attached wrong file previously.

This time should be OK.

regards
Lukasz Lubojanski

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: greenasia.diff --]
[-- Type: text/x-diff; name=greenasia.diff, Size: 8523 bytes --]

Signed-off-by: Lukasz Lubojanski <lukasz@lubojanski.info>
diff --git a/drivers/hid/Kconfig b/drivers/hid/Kconfig
index b4fd8ca..45f25ef 100644
--- a/drivers/hid/Kconfig
+++ b/drivers/hid/Kconfig
@@ -217,6 +217,24 @@ config PANTHERLORD_FF
 	  Say Y here if you have a PantherLord/GreenAsia based game controller
 	  or adapter and want to enable force feedback support for it.
 
+config HID_GREENASIA
+	tristate "GreenAsia (Product ID 0x12) based device support" if EMBEDDED
+	depends on USB_HID
+	default y
+	---help---
+	  Support for GreenAsia (Product ID 0x12) based device support
+	  (like MANTA Warior MM816 and SpeedLink Strike2 SL-6635).
+
+config GREENASIA_FF
+	bool "GreenAsia (Product ID 0x12) force feedback support"
+	depends on HID_GREENASIA
+	select INPUT_FF_MEMLESS
+	---help---
+	  Say Y here if you have a GreenAsia (Product ID 0x12) based game controller
+	  or adapter and want to enable force feedback support for it.
+
+
+
 config HID_PETALYNX
 	tristate "Petalynx" if EMBEDDED
 	depends on USB_HID
diff --git a/drivers/hid/Makefile b/drivers/hid/Makefile
index b09e43e..20662b7 100644
--- a/drivers/hid/Makefile
+++ b/drivers/hid/Makefile
@@ -33,6 +33,7 @@ obj-$(CONFIG_HID_GYRATION)	+= hid-gyration.o
 obj-$(CONFIG_HID_LOGITECH)	+= hid-logitech.o
 obj-$(CONFIG_HID_MICROSOFT)	+= hid-microsoft.o
 obj-$(CONFIG_HID_MONTEREY)	+= hid-monterey.o
+obj-$(CONFIG_HID_GREENASIA)	+= hid-ga.o
 obj-$(CONFIG_HID_PANTHERLORD)	+= hid-pl.o
 obj-$(CONFIG_HID_PETALYNX)	+= hid-petalynx.o
 obj-$(CONFIG_HID_SAMSUNG)	+= hid-samsung.o
diff --git a/drivers/hid/hid-core.c b/drivers/hid/hid-core.c
index 40df3e1..f2184cf 100644
--- a/drivers/hid/hid-core.c
+++ b/drivers/hid/hid-core.c
@@ -1269,6 +1269,7 @@ static const struct hid_device_id hid_blacklist[] = {
 	{ HID_USB_DEVICE(USB_VENDOR_ID_GAMERON, USB_DEVICE_ID_GAMERON_DUAL_PSX_ADAPTOR) },
 	{ HID_USB_DEVICE(USB_VENDOR_ID_GAMERON, USB_DEVICE_ID_GAMERON_DUAL_PCS_ADAPTOR) },
 	{ HID_USB_DEVICE(USB_VENDOR_ID_GREENASIA, 0x0003) },
+	{ HID_USB_DEVICE(USB_VENDOR_ID_GREENASIA, 0x0012) },
 	{ HID_USB_DEVICE(USB_VENDOR_ID_GYRATION, USB_DEVICE_ID_GYRATION_REMOTE) },
 	{ HID_USB_DEVICE(USB_VENDOR_ID_GYRATION, USB_DEVICE_ID_GYRATION_REMOTE_2) },
 	{ HID_USB_DEVICE(USB_VENDOR_ID_LABTEC, USB_DEVICE_ID_LABTEC_WIRELESS_KEYBOARD) },
diff --git a/drivers/hid/hid-dummy.c b/drivers/hid/hid-dummy.c
index e148f86..d2b1b37 100644
--- a/drivers/hid/hid-dummy.c
+++ b/drivers/hid/hid-dummy.c
@@ -43,6 +43,9 @@ static int __init hid_dummy_init(void)
 #ifdef CONFIG_HID_MONTEREY_MODULE
 	HID_COMPAT_CALL_DRIVER(monterey);
 #endif
+#ifdef CONFIG_HID_GREENASIA_MODULE
+	HID_COMPAT_CALL_DRIVER(greenasia);
+#endif
 #ifdef CONFIG_HID_PANTHERLORD_MODULE
 	HID_COMPAT_CALL_DRIVER(pantherlord);
 #endif
diff --git a/drivers/hid/hid-ga.c b/drivers/hid/hid-ga.c
new file mode 100644
index 0000000..e50f3f5
--- /dev/null
+++ b/drivers/hid/hid-ga.c
@@ -0,0 +1,223 @@
+/*
+ *  Force feedback support for GreenAsia (Product ID 0x12) based devices
+ *
+ *  The devices are distributed under various names and the same USB device ID
+ *  can be used in many game controllers.
+ *
+ *
+ *  0e8f:0012 "GreenAsia Inc.    USB Joystick     "
+ *   - tested with MANTA Warior MM816 and SpeedLink Strike2 SL-6635.
+ *
+ *  Copyright (c) 2008 Lukasz Lubojanski <lukasz@lubojanski.info>
+ */
+
+/*
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
+ */
+
+
+#define DEBUG
+
+#define debug(format, arg...) pr_debug("hid-ga: " format "\n" , ## arg)
+
+#include <linux/input.h>
+#include <linux/usb.h>
+#include <linux/hid.h>
+
+#include "hid-ids.h"
+
+#ifdef CONFIG_GREENASIA_FF
+#include "usbhid/usbhid.h"
+
+
+static const signed short ff_rumble[] = {
+	FF_RUMBLE,
+	-1
+};
+
+
+struct gaff_device {
+	struct hid_report *report;
+};
+
+static int hid_gaff_play(struct input_dev *dev, void *data,
+			 struct ff_effect *effect)
+{
+	struct hid_device *hid = input_get_drvdata(dev);
+	struct gaff_device *gaff = data;
+	int left, right;
+
+	left = effect->u.rumble.strong_magnitude;
+	right = effect->u.rumble.weak_magnitude;
+
+	debug("called with 0x%04x 0x%04x", left, right);
+
+	left = left * 0xfe / 0xffff;
+	right = right * 0xfe / 0xffff;
+
+	gaff->report->field[0]->value[0] = 0x51;
+	gaff->report->field[0]->value[1] = 0x0;
+	gaff->report->field[0]->value[2] = right;
+	gaff->report->field[0]->value[3] = 0;
+	gaff->report->field[0]->value[4] = left;
+	gaff->report->field[0]->value[5] = 0;
+	debug("running with 0x%02x 0x%02x", left, right);
+	usbhid_submit_report(hid, gaff->report, USB_DIR_OUT);
+
+	gaff->report->field[0]->value[0] = 0xfa;
+	gaff->report->field[0]->value[1] = 0xfe;
+	gaff->report->field[0]->value[2] = 0x0;
+	gaff->report->field[0]->value[4] = 0x0;
+
+	usbhid_submit_report(hid, gaff->report, USB_DIR_OUT);
+
+	return 0;
+}
+
+static int gaff_init(struct hid_device *hid)
+{
+	struct gaff_device *gaff;
+	struct hid_report *report;
+	struct hid_input *hidinput;
+	struct list_head *report_list =
+			&hid->report_enum[HID_OUTPUT_REPORT].report_list;
+	struct list_head *report_ptr = report_list;
+	struct input_dev *dev;
+	int error;
+
+	if (list_empty(report_list)) {
+		dev_err(&hid->dev, "no output reports found\n");
+		return -ENODEV;
+	}
+
+	list_for_each_entry(hidinput, &hid->inputs, list) {
+
+		report_ptr = report_ptr->next;
+
+		if (report_ptr == report_list) {
+			dev_err(&hid->dev, "required output report is "
+					"missing\n");
+			return -ENODEV;
+		}
+
+		report = list_entry(report_ptr, struct hid_report, list);
+		if (report->maxfield < 1) {
+			dev_err(&hid->dev, "no fields in the report\n");
+			return -ENODEV;
+		}
+
+		if (report->field[0]->report_count < 4) {
+			dev_err(&hid->dev, "not enough values in the field\n");
+			return -ENODEV;
+		}
+
+		gaff = kzalloc(sizeof(struct gaff_device), GFP_KERNEL);
+		if (!gaff)
+			return -ENOMEM;
+
+		dev = hidinput->input;
+
+		set_bit(FF_RUMBLE, dev->ffbit);
+
+		error = input_ff_create_memless(dev, gaff, hid_gaff_play);
+		if (error) {
+			kfree(gaff);
+			return error;
+		}
+
+		gaff->report = report;
+		gaff->report->field[0]->value[0] = 0x51;
+		gaff->report->field[0]->value[1] = 0x00;
+		gaff->report->field[0]->value[2] = 0x00;
+		gaff->report->field[0]->value[3] = 0x00;
+		usbhid_submit_report(hid, gaff->report, USB_DIR_OUT);
+
+		gaff->report->field[0]->value[0] = 0xfa;
+		gaff->report->field[0]->value[1] = 0xfe;
+
+		usbhid_submit_report(hid, gaff->report, USB_DIR_OUT);
+	}
+
+
+	dev_info(&hid->dev, "hid-ga: Force Feedback for GreenAsia 0x12"
+	       " devices by Lukasz Lubojanski <lukasz@lubojanski.info>\n");
+
+	return 0;
+}
+#else
+static inline int gaff_init(struct hid_device *hid)
+{
+	dev_info(&hid->dev, "hid-ga: GreenAsia 0x12 "
+		" devices by Lukasz Lubojanski <lukasz@lubojanski.info>\n");
+
+	return 0;
+}
+#endif
+
+static int ga_probe(struct hid_device *hdev, const struct hid_device_id *id)
+{
+	int ret;
+
+	debug("Greenasia HID hardware probe...");
+
+	if (id->driver_data)
+		hdev->quirks |= HID_QUIRK_MULTI_INPUT;
+
+	ret = hid_parse(hdev);
+	if (ret) {
+		dev_err(&hdev->dev, "parse failed\n");
+		goto err;
+	}
+
+	ret = hid_hw_start(hdev, HID_CONNECT_DEFAULT & ~HID_CONNECT_FF);
+	if (ret) {
+		dev_err(&hdev->dev, "hw start failed\n");
+		goto err;
+	}
+
+	gaff_init(hdev);
+
+	return 0;
+err:
+	return ret;
+}
+
+static const struct hid_device_id ga_devices[] = {
+	{ HID_USB_DEVICE(USB_VENDOR_ID_GREENASIA, 0x0012),  }, /* GreenAsia Inc. USB Joystick */
+	{ }
+};
+MODULE_DEVICE_TABLE(hid, ga_devices);
+
+static struct hid_driver ga_driver = {
+	.name = "greenasia",
+	.id_table = ga_devices,
+	.probe = ga_probe,
+};
+
+static int ga_init(void)
+{
+	return hid_register_driver(&ga_driver);
+}
+
+static void ga_exit(void)
+{
+	hid_unregister_driver(&ga_driver);
+}
+
+module_init(ga_init);
+module_exit(ga_exit);
+MODULE_LICENSE("GPL");
+
+HID_COMPAT_LOAD_DRIVER(greenasia);