Force Feedback Logitech G25/G25/Driving Force Pro

Force Feedback Logitech G25/G25/Driving Force Pro

[Peter]

Hi,

i'd like to ask if someone could add the id's of the logitech wheels to
drivers/hid/hid-lgff.c, so that the wheels autocenter can be
enabled/disabled by userland programs like wine. There has been
discussion over this before:
http://www.mail-archive.com/linux-input@atrey.karlin.mff.cuni.cz/msg01426.html

Line 72:
        { 0x046d, 0xc294, ff_wheel },
has to be copied twice with 0xc294 changed to 0xc298 (DFP) and 0xc299
(G25/G27), because after switching to native mode the devices report
different ids:
        { 0x046d, 0xc298, ff_wheel },
        { 0x046d, 0xc299, ff_wheel },

That's it. 
I managed to get my G27 fully working in LFS (http://www.lfs.net) with
this small modification and some additional fixes to wine. The G25's
Force Feedback, all 4 axis and 20some buttons work like a charm.

Thx in advance, Peter

Re: Force Feedback Logitech G25/G25/Driving Force Pro

[Olivier Michel]

I fully support this request, as it would be very useful to me as well and
probably to many other users (I am struggling with kernel patches to get my G27
race wheel work with force feedback)

[PATCH] G25_support

[Simon Wood]

From: Peter <peter@slim-one.de>

Small patch to add support for the G25/G27 by adding USB ID's
as suggested by Peter.

Boots but otherwise untested as I don't have hardware, .debs for
kernel (2.6.38) here if want to test/run Ubuntu/Debian:
http://www.mungewell.org/Logitech_Wii_Wheel/

Reported-by: Peter <peter@slim-one.de>
Signed-off-by: Simon Wood <simon@mungewell.org>
---
 drivers/hid/hid-lgff.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/drivers/hid/hid-lgff.c b/drivers/hid/hid-lgff.c
index 90d0ef2..f099079 100644
--- a/drivers/hid/hid-lgff.c
+++ b/drivers/hid/hid-lgff.c
@@ -73,6 +73,8 @@ static const struct dev_type devices[] = {
 	{ 0x046d, 0xc293, ff_joystick },
 	{ 0x046d, 0xc294, ff_wheel },
 	{ 0x046d, 0xc295, ff_joystick },
+	{ 0x046d, 0xc298, ff_wheel },
+	{ 0x046d, 0xc299, ff_wheel },
 	{ 0x046d, 0xca03, ff_wheel },
 };
 
-- 
1.7.0.4

Re: [PATCH] G25_support

[Jiri Kosina]

On Mon, 21 Mar 2011, Simon Wood wrote:

> From: Peter <peter@slim-one.de>
> 
> Small patch to add support for the G25/G27 by adding USB ID's
> as suggested by Peter.
> 
> Boots but otherwise untested as I don't have hardware, .debs for
> kernel (2.6.38) here if want to test/run Ubuntu/Debian:
> http://www.mungewell.org/Logitech_Wii_Wheel/
> 
> Reported-by: Peter <peter@slim-one.de>
> Signed-off-by: Simon Wood <simon@mungewell.org>
> ---
>  drivers/hid/hid-lgff.c |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/hid/hid-lgff.c b/drivers/hid/hid-lgff.c
> index 90d0ef2..f099079 100644
> --- a/drivers/hid/hid-lgff.c
> +++ b/drivers/hid/hid-lgff.c
> @@ -73,6 +73,8 @@ static const struct dev_type devices[] = {
>  	{ 0x046d, 0xc293, ff_joystick },
>  	{ 0x046d, 0xc294, ff_wheel },
>  	{ 0x046d, 0xc295, ff_joystick },
> +	{ 0x046d, 0xc298, ff_wheel },
> +	{ 0x046d, 0xc299, ff_wheel },
>  	{ 0x046d, 0xca03, ff_wheel },
>  };

You are right that missing entries in this struct for devices which have 
LG_FF flag set doesn't make too much sense.

Adding Michal and Chritophe to CC, as they should have the HW to test.

-- 
Jiri Kosina
SUSE Labs, Novell Inc.

Re: [PATCH] G25_support

[Jiri Kosina]

On Mon, 21 Mar 2011, Simon Wood wrote:

> From: Peter <peter@slim-one.de>

Could you please provide full name here, so that I could apply the patch 
with proper authorship attribution?

Thanks.

> 
> Small patch to add support for the G25/G27 by adding USB ID's
> as suggested by Peter.
> 
> Boots but otherwise untested as I don't have hardware, .debs for
> kernel (2.6.38) here if want to test/run Ubuntu/Debian:
> http://www.mungewell.org/Logitech_Wii_Wheel/
> 
> Reported-by: Peter <peter@slim-one.de>
> Signed-off-by: Simon Wood <simon@mungewell.org>
> ---
>  drivers/hid/hid-lgff.c |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/hid/hid-lgff.c b/drivers/hid/hid-lgff.c
> index 90d0ef2..f099079 100644
> --- a/drivers/hid/hid-lgff.c
> +++ b/drivers/hid/hid-lgff.c
> @@ -73,6 +73,8 @@ static const struct dev_type devices[] = {
>  	{ 0x046d, 0xc293, ff_joystick },
>  	{ 0x046d, 0xc294, ff_wheel },
>  	{ 0x046d, 0xc295, ff_joystick },
> +	{ 0x046d, 0xc298, ff_wheel },
> +	{ 0x046d, 0xc299, ff_wheel },
>  	{ 0x046d, 0xca03, ff_wheel },
>  };

-- 
Jiri Kosina
SUSE Labs, Novell Inc.

[PATCH] G27_support

[Peter]

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

Hi,

here's a small patch to add support for the Logitech G27 wheel, since
the prior patch only added FF support for the Driving Force Pro and G25.
The patch contains the changes from the G25 and DFP, too, since they
were not in linux/kernel/git/jikos/hid.git.
I tested the changes with wine/LFS and got full support for all axes and
buttons.

Regards, Peter

[-- Attachment #2: G27.patch --]
[-- Type: text/x-patch, Size: 2412 bytes --]

diff --git a/drivers/hid/hid-core.c b/drivers/hid/hid-core.c
index c3d6626..4e33886 100644
--- a/drivers/hid/hid-core.c
+++ b/drivers/hid/hid-core.c
@@ -1405,6 +1405,7 @@ static const struct hid_device_id hid_have_special_driver[] = {
 	{ HID_USB_DEVICE(USB_VENDOR_ID_LOGITECH, USB_DEVICE_ID_LOGITECH_MOMO_WHEEL2) },
 	{ HID_USB_DEVICE(USB_VENDOR_ID_LOGITECH, USB_DEVICE_ID_LOGITECH_DFP_WHEEL) },
 	{ HID_USB_DEVICE(USB_VENDOR_ID_LOGITECH, USB_DEVICE_ID_LOGITECH_G25_WHEEL) },
+	{ HID_USB_DEVICE(USB_VENDOR_ID_LOGITECH, USB_DEVICE_ID_LOGITECH_G27_WHEEL) },
 	{ HID_USB_DEVICE(USB_VENDOR_ID_LOGITECH, USB_DEVICE_ID_LOGITECH_WII_WHEEL) },
 	{ HID_USB_DEVICE(USB_VENDOR_ID_LOGITECH, USB_DEVICE_ID_LOGITECH_RUMBLEPAD2) },
 	{ HID_USB_DEVICE(USB_VENDOR_ID_LOGITECH, USB_DEVICE_ID_SPACETRAVELLER) },
diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h
index d485894..7db251c 100644
--- a/drivers/hid/hid-ids.h
+++ b/drivers/hid/hid-ids.h
@@ -394,6 +394,7 @@
 #define USB_DEVICE_ID_LOGITECH_MOMO_WHEEL	0xc295
 #define USB_DEVICE_ID_LOGITECH_DFP_WHEEL	0xc298
 #define USB_DEVICE_ID_LOGITECH_G25_WHEEL	0xc299
+#define USB_DEVICE_ID_LOGITECH_G27_WHEEL	0xc29b
 #define USB_DEVICE_ID_LOGITECH_WII_WHEEL	0xc29c
 #define USB_DEVICE_ID_LOGITECH_ELITE_KBD	0xc30a
 #define USB_DEVICE_ID_S510_RECEIVER	0xc50c
diff --git a/drivers/hid/hid-lg.c b/drivers/hid/hid-lg.c
index 3da9040..21f205f 100644
--- a/drivers/hid/hid-lg.c
+++ b/drivers/hid/hid-lg.c
@@ -377,6 +377,8 @@ static const struct hid_device_id lg_devices[] = {
 		.driver_data = LG_FF },
 	{ HID_USB_DEVICE(USB_VENDOR_ID_LOGITECH, USB_DEVICE_ID_LOGITECH_G25_WHEEL),
 		.driver_data = LG_FF },
+	{ HID_USB_DEVICE(USB_VENDOR_ID_LOGITECH, USB_DEVICE_ID_LOGITECH_G27_WHEEL),
+		.driver_data = LG_FF },
 	{ HID_USB_DEVICE(USB_VENDOR_ID_LOGITECH, USB_DEVICE_ID_LOGITECH_DFP_WHEEL),
 		.driver_data = LG_FF },
 	{ HID_USB_DEVICE(USB_VENDOR_ID_LOGITECH, USB_DEVICE_ID_LOGITECH_WII_WHEEL),
diff --git a/drivers/hid/hid-lgff.c b/drivers/hid/hid-lgff.c
index 90d0ef2..f901349 100644
--- a/drivers/hid/hid-lgff.c
+++ b/drivers/hid/hid-lgff.c
@@ -72,6 +72,9 @@ static const struct dev_type devices[] = {
 	{ 0x046d, 0xc287, ff_joystick_ac },
 	{ 0x046d, 0xc293, ff_joystick },
 	{ 0x046d, 0xc294, ff_wheel },
+	{ 0x046d, 0xc298, ff_wheel },
+	{ 0x046d, 0xc299, ff_wheel },
+	{ 0x046d, 0xc29b, ff_wheel },
 	{ 0x046d, 0xc295, ff_joystick },
 	{ 0x046d, 0xca03, ff_wheel },
 };

Re: [PATCH] G27_support

[Jiri Kosina]

On Mon, 2 May 2011, Peter wrote:

> Hi,
> 
> here's a small patch to add support for the Logitech G27 wheel, since
> the prior patch only added FF support for the Driving Force Pro and G25.
> The patch contains the changes from the G25 and DFP, too, since they
> were not in linux/kernel/git/jikos/hid.git.
> I tested the changes with wine/LFS and got full support for all axes and
> buttons.

Thanks a lot for the patch.

Could you please send it altogether with proper From: and Signed-off-by: 
lines, so that I could apply it?

Thanks,

-- 
Jiri Kosina
SUSE Labs