Hello Few days ago I sent email to the list with this text: > Hello > I look code of ff-utils (fftest.c) and found that this program > checks axis information for FF effects. For example, if kernel > provides an ABS_X (also supported ABS_Y and ABS_WHEEL) via > EVIOCGBIT(EV_FF, ...), then fftest shows "Axes query: Axis X". But I > cannot found this feature in any existing driver. Moreover, > documentation does not contain any information about it. So, is it > really neccessary feature (I think yes)? > I am ready to create autocenter support patch for Logitech Force 3D > Pro (046d:c286). But all tested by me Logitech Joysticks with FF has > support for autocentering in same way. May be it is good idea to add > this feature for all joysticks in hid-lgff.c? If some joystick do not > support it (users reported), then it is possible to exclude > autocentering for this joystick later. > Best regards, > Sergey Belyashov But no answer I recieve. So I write 3 incompatible patches: 1. adds autocentering support for Logitech force 3D Pro 2. adds autocentering support for all Logitech Joysticks and wheels 3. same as 2, but also adds information about applicable axes to the EVIOCGBIT(EV_FF, ...) result and to FF documentation (discussion required). Only one patch may be applied. Best regards, Sergey Belyashov PATCH N1 --------------------------------------------------------- --- orig/drivers/hid/hid-lgff.c 2009-05-08 22:20:37.000000000 +0400 +++ new/drivers/hid/hid-lgff.c 2009-05-08 22:20:26.000000000 +0400 @@ -50,6 +50,12 @@ -1 }; +static const signed short ff_joystick_ac[] = { + FF_CONSTANT, + FF_AUTOCENTER, + -1 +}; + static const signed short ff_wheel[] = { FF_CONSTANT, FF_AUTOCENTER, @@ -60,7 +66,7 @@ { 0x046d, 0xc211, ff_rumble }, { 0x046d, 0xc219, ff_rumble }, { 0x046d, 0xc283, ff_joystick }, - { 0x046d, 0xc286, ff_joystick }, + { 0x046d, 0xc286, ff_joystick_ac }, { 0x046d, 0xc294, ff_wheel }, { 0x046d, 0xc295, ff_joystick }, { 0x046d, 0xca03, ff_wheel }, PATCH N2 --------------------------------------------------------- --- orig/drivers/hid/hid-lgff.c 2009-05-08 22:20:37.000000000 +0400 +++ new/drivers/hid/hid-lgff.c 2009-05-08 22:22:15.000000000 +0400 @@ -47,11 +47,6 @@ static const signed short ff_joystick[] = { FF_CONSTANT, - -1 -}; - -static const signed short ff_wheel[] = { - FF_CONSTANT, FF_AUTOCENTER, -1 }; @@ -61,9 +56,9 @@ { 0x046d, 0xc219, ff_rumble }, { 0x046d, 0xc283, ff_joystick }, { 0x046d, 0xc286, ff_joystick }, - { 0x046d, 0xc294, ff_wheel }, + { 0x046d, 0xc294, ff_joystick }, { 0x046d, 0xc295, ff_joystick }, - { 0x046d, 0xca03, ff_wheel }, + { 0x046d, 0xca03, ff_joystick }, }; static int hid_lgff_play(struct input_dev *dev, void *data, struct ff_effect *effect) PATCH N3 --------------------------------------------------------- --- orig/Documentation/input/ff.txt 2009-05-08 23:06:05.000000000 +0400 +++ new/Documentation/input/ff.txt 2009-05-08 23:09:29.000000000 +0400 @@ -72,6 +72,9 @@ - FF_INERTIA can simulate inertia - FF_GAIN gain is adjustable - FF_AUTOCENTER autocenter is adjustable +- ABS_X effects applicable for axis X +- ABS_Y effects applicable for axis Y +- ABS_WHEEL effects applicable for wheel Note: In most cases you should use FF_PERIODIC instead of FF_RUMBLE. All devices that support FF_RUMBLE support FF_PERIODIC (square, triangle, --- orig/drivers/hid/hid-lgff.c 2009-05-08 22:20:37.000000000 +0400 +++ new/drivers/hid/hid-lgff.c 2009-05-08 22:55:56.000000000 +0400 @@ -47,12 +47,16 @@ static const signed short ff_joystick[] = { FF_CONSTANT, + FF_AUTOCENTER, + ABS_X, + ABS_Y, -1 }; static const signed short ff_wheel[] = { FF_CONSTANT, FF_AUTOCENTER, + ABS_X, -1 }; ======================================================================================== Signed-off-by: Sergey Belyashov <Sergey.Belyashov@xxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html