Re: [PATCH v2 0/4] Add ff-memless-next and make hid-lg4ff use it

2014-02-24 Thread Elias Vanderstuyft
On Mon, Feb 24, 2014 at 1:58 AM, Michal Malý wrote: > On Monday 24 of February 2014 02:32:27 Anssi Hannula wrote: >> >> I think we should extend the current ff-memless instead of duplicating >> its functionality (even on a "for now" basis). >> >> Having looked at ff-memless-next briefly, it seems

Re: [PATCH] input: ff-memless: don't schedule already playing effect to play again

2014-03-02 Thread Elias Vanderstuyft
On Sun, Mar 2, 2014 at 12:35 PM, Felix Rueegg wrote: > When an effect with zero replay length, zero replay delay > and zero envelope attack length is uploaded, it is played and then scheduled > to play > again one timer tick later. This triggers a warning (URB submitted while > active) in combina

Fwd: [PATCH] input: ff-memless: don't schedule already playing effect to play again

2014-03-02 Thread Elias Vanderstuyft
-- Forwarded message -- From: Michal Malý Date: Sun, Mar 2, 2014 at 2:29 PM Subject: Re: [PATCH] input: ff-memless: don't schedule already playing effect to play again To: Elias Vanderstuyft On Sunday 02 of March 2014 14:17:58 you wrote: > On Sun, Mar 2, 2014 at 12:35 P

Re: PATCH hid: Implement mode switching on Logitech gaming wheels accordingly to the documentation

2014-07-31 Thread Elias Vanderstuyft
la Vibration : PID 0xCA04 Behaviour didn't deviate from normal, which is as expected because these devices don't use the native versus compatibility mode, according to the documentation. So I consider this patch successful for these devices: Tested-by: Elias Vanderstuyft Elias -- To unsu

Re: [PATCH v3 23/24] hid: Port hid-lg4ff to ff-memless-next

2014-04-26 Thread Elias Vanderstuyft
default: > + dbg_hid("Unsupported effect command"); > + return -EINVAL; > } > - return 0; > } > > /* Sends default autocentering command compatible with > @@ -610,7 +633,7 @@ int lg4ff_init(struct hid_device *hid) > for

Re: [PATCH v4 01/24] input: Add ff-memless-next module

2014-05-14 Thread Elias Vanderstuyft
On Wed, May 14, 2014 at 10:35 AM, Michal Malý wrote: > Hi Dmitry, > > thank you for reviewing this. > > On Tuesday 13 of May 2014 23:38:06 Dmitry Torokhov wrote: >> On Sat, Apr 26, 2014 at 05:02:00PM +0200, Michal Malý wrote: >> > + >> > +/** DEFINITION OF TERMS >> > + * >> > + * Combined effect -

[PATCH] HID: lg2ff: add rumble magnitude clamping quirk

2014-04-08 Thread Elias Vanderstuyft
The same thing appears to happen in the Windows Logitech driver, except the max clamping bound is not 0xfd, but 0xfe. Experimentally, I proved this to be wrong. Tested-by: Hendrik Iben Signed-off-by: Elias Vanderstuyft Cc: Edgar Simo-Serra Cc: Michal Malý Cc: linux-in...@vger.kerne

Re: [PATCH] HID: lg2ff: add rumble magnitude clamping quirk

2014-04-09 Thread Elias Vanderstuyft
Since this patch haven't been applied yet, I withdraw this patch for the following reason: It will get in as a part of the move to ff-memless-next. Elias -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo

Re: [PATCH 12/24] Port hid-lgff to ff-memless-next

2014-04-20 Thread Elias Vanderstuyft
On Wed, Apr 9, 2014 at 1:24 PM, Michal Malý wrote: > Port hid-lgff to ff-memless-next > > Signed-off-by: Michal Malý > --- > drivers/hid/Kconfig| 2 +- > drivers/hid/hid-lgff.c | 63 > ++ > 2 files changed, 44 insertions(+), 21 deletions(-) >

Re: [PATCH 0/24] Introduce ff-memless-next as an improved replacement for ff-memless

2014-04-20 Thread Elias Vanderstuyft
On Sun, Apr 20, 2014 at 7:27 PM, wrote: > Hi all, > I got a chance to build this series of patches and test with the > controllers I have (*). Without specific instructions I wasn't sure > exactly what to test, but it seems to be OK and the devices > rumbled/wobbled appropriately, > Simon > > tes

Re: [PATCH 13/24] Port hid-lg3ff to ff-memless-next

2014-04-20 Thread Elias Vanderstuyft
On Wed, Apr 9, 2014 at 1:24 PM, Michal Malý wrote: > Port hid-lg3ff to ff-memless-next > > Signed-off-by: Michal Malý > --- > drivers/hid/Kconfig | 2 +- > drivers/hid/hid-lg3ff.c | 56 > +++-- > 2 files changed, 37 insertions(+), 21 deletions(-)

Re: [PATCH 0/24] Introduce ff-memless-next as an improved replacement for ff-memless

2014-04-21 Thread Elias Vanderstuyft
On Mon, Apr 21, 2014 at 12:29 AM, wrote: > >> Did the WiiWheel have working FF_CONSTANT before this patchset? >> It would be weird if yes, because lg4ff is also used for MOMO-Black >> which works fine. > > Yes, I think that the force requested by fftest was just too weak to > actually move the wh

[PATCH v2 1/1] Input: don't modify the id of ioctl-provided ff effect on upload failure

2014-03-29 Thread Elias Vanderstuyft
s been discussed on: http://www.mail-archive.com/linux-input@vger.kernel.org/msg08513.html ("ff-core effect id handling in case of a failed effect upload") Suggested-by: Dmitry Torokhov Signed-off-by: Elias Vanderstuyft Cc: Anssi Hannula Cc: Michal Malý Cc: linux-in...@vger.kernel.org

Re: [PATCH v4 01/24] input: Add ff-memless-next module

2014-05-20 Thread Elias Vanderstuyft
On Tue, May 20, 2014 at 10:58 PM, Michal Malý wrote: > On Tuesday 20 of May 2014 16:16:12 si...@mungewell.org wrote: >> Regarding the question of emulated vs. real effects, can we extend the API >> so that applications can know which effects are really supported, and >> enable/disable emulation so

Re: [PATCH v4 01/24] input: Add ff-memless-next module

2014-05-21 Thread Elias Vanderstuyft
On Wed, May 21, 2014 at 4:13 AM, Michal Malý wrote: > On Tuesday 20 of May 2014 18:17:51 Roland Bosa wrote: >> In any case, the USB traffic should be decoupled from the app. Any force >> updates should only change state in the ff-memless[-next] driver. Any >> change there should trickle down to a

Re: [PATCH v4 01/24] input: Add ff-memless-next module

2014-05-21 Thread Elias Vanderstuyft
On Wed, May 21, 2014 at 4:13 AM, Michal Malý wrote: > On Tuesday 20 of May 2014 18:17:51 Roland Bosa wrote: >> >> The file format of an IFR is probably easily deducible. There's a lot of >> textual clues to parameters and the values are also written out in >> string form. >> >> I don't have a FEdi

Re: [PATCH v4 01/24] input: Add ff-memless-next module

2014-05-21 Thread Elias Vanderstuyft
On Wed, May 21, 2014 at 12:17 PM, Nestor Lopez Casado wrote: > Elias, Simon, Michal, > > It is unfortunate that we didn't get back to you in 2013 when you > asked for help in reverse engineering, oftentimes we have conflicting > priorities and a particular request may be left laying around for so

Re: [PATCH v4 01/24] input: Add ff-memless-next module

2014-05-21 Thread Elias Vanderstuyft
On Wed, May 21, 2014 at 3:35 PM, Elias Vanderstuyft wrote: > On Wed, May 21, 2014 at 4:13 AM, Michal Malý > wrote: >> Proper decoupling of the userspace and driver is the only important thing >> that >> is missing from the current code. > > Also dynamic updating

Re: [PATCH v4 01/24] input: Add ff-memless-next module

2014-05-21 Thread Elias Vanderstuyft
On Tue, May 20, 2014 at 11:38 PM, Roland Bosa wrote: > On 05/20/2014 12:00 PM, Michal Malı wrote: > There's a healthy amount of > code in the Windows driver that you would call 'quirks' which deals with > deciding how to allocate multiple springs and dampers to a single slot. But 2 Spring effects

Re: [PATCH v4 01/24] input: Add ff-memless-next module

2014-05-22 Thread Elias Vanderstuyft
On Tue, May 20, 2014 at 11:26 PM, Elias Vanderstuyft wrote: > On Tue, May 20, 2014 at 10:58 PM, Michal Malý > wrote: >> On Tuesday 20 of May 2014 16:16:12 si...@mungewell.org wrote: >>> Regarding the question of emulated vs. real effects, can we extend the API >>>

Re: [PATCH 0/2] HID: Force feedback support for the Logitech G920 Wheel

2015-11-27 Thread Elias Vanderstuyft
644 drivers/hid/hid-logitech-hidpp-base.c > create mode 100644 drivers/hid/hid-logitech-hidpp-base.h > create mode 100644 drivers/hid/hid-logitech-hidpp-ff.c > create mode 100644 drivers/hid/hid-logitech-hidpp-ff.h > delete mode 100644 drivers/hid/hid-logitech-hidpp.c > > --

Re: [Patch-V2 1/6] INPUT: xpad: Add minimal support for Logitech G920 Wheel

2015-12-13 Thread Elias Vanderstuyft
On Thu, Dec 10, 2015 at 6:08 PM, Benjamin Tissoires wrote: > On Dec 09 2015 or thereabouts, Dmitry Torokhov wrote: >> On Wed, Dec 9, 2015 at 5:23 PM, Dmitry Torokhov >> wrote: >> > On Thu, Nov 19, 2015 at 10:31 AM, Dmitry Torokhov >> > wrote: >> >> On Thu, Nov 19, 2015 at 02:50:51PM +0100, Jiri

Re: [PATCH 00/12] HID: hid-lg, hid-lg4ff: Mostly cleanup patches

2015-03-29 Thread Elias Vanderstuyft
which made use of effects with nonzero attack times, thus the ff-memless timer's timeout got modified often), and then unplugging the device while in use. Instead of crashing, the kernel behaved like it should, no panics or lockups were found. Tested with to following devices: - Logit

Re: [PATCH 0/4] HID: Improve handling of multimode Logitech handling wheels

2015-02-09 Thread Elias Vanderstuyft
tions(+), 76 deletions(-) > > -- > 2.2.2 > Applied cleanly on kernel 3.17.8, and tested with: - Logitech Formula Vibration Wheel - Logitech MOMO (Black) Wheel Everything keeps working like it was before, which is correct because these wheels are no multimode wheels. Tested-by: E

Re: [PATCH v2 0/4] HID: hid-lg4ff: Improve handling of Logitech multimode gaming wheels

2015-02-16 Thread Elias Vanderstuyft
t; 2.3.0 > Applied cleanly on kernel 3.18.6, and retested with: - Logitech Formula Vibration Wheel - Logitech MOMO (Black) Wheel Everything keeps working like it was before, which is correct because these wheels are no multimode wheels. Tested-by: Elias Vanderstuyft Thanks, Elias -- To unsubsc

Re: [PATCH-V2 1/1] HID: Force feedback support for the Logitech G920

2016-04-08 Thread Elias Vanderstuyft
On Thu, Jan 28, 2016 at 2:33 PM, Jiri Kosina wrote: > On Tue, 26 Jan 2016, Simon Wood wrote: > >> On Sun, January 10, 2016 4:25 pm, Edwin Velds wrote: >> > This patch implements force feedback support for the Logitech >> > G920 Driving Force Racing Wheel. It is a generic implementation >> > of fea

Re: [PATCH 2/2] Input: uinput: Sanity check on ff_effects_max and EV_FF

2015-11-05 Thread Elias Vanderstuyft
Hi Dmitry, Excuse me for the long delay. On Thu, Oct 15, 2015 at 2:52 AM, Dmitry Torokhov wrote: > On Thu, Sep 17, 2015 at 07:29:48PM +0200, Elias Vanderstuyft wrote: >> Currently the user can specify a non-zero value for ff_effects_max, >> without setting the EV_FF bit. >&

Re: [PATCH v3] Input: uinput - add new UINPUT_DEV_SETUP and UI_ABS_SETUP ioctl

2015-11-08 Thread Elias Vanderstuyft
Hi, On Tue, Aug 25, 2015 at 5:12 PM, Benjamin Tissoires wrote: > diff --git a/include/uapi/linux/uinput.h b/include/uapi/linux/uinput.h > index 013c9d8..ef6c9f5 100644 > --- a/include/uapi/linux/uinput.h > +++ b/include/uapi/linux/uinput.h > @@ -20,6 +20,11 @@ > * Author: Aristeu Sergio Rozansk

[PATCH v2 2/2] Input: uinput: Sanity check on ff_effects_max and EV_FF

2015-11-08 Thread Elias Vanderstuyft
elpful message and return -EINVAL in case the check fails. Signed-off-by: Elias Vanderstuyft --- Changes in v2: - Rebase on pending patches from David Herrmann and Benjamin Tissoires: - v3 Input: uinput - add new UINPUT_DEV_SETUP and UI_ABS_SETUP ioctl - Input: uinput - rework ABS vali

Re: [PATCH 5/6] HID: Add vendor specific usage pages for Logitech G920

2015-11-10 Thread Elias Vanderstuyft
these pages to the 'ignore' list. >> >> Reported-by: Elias Vanderstuyft >> Signed-off-by: Simon Wood >> --- >> drivers/hid/hid-input.c | 2 +- >> include/linux/hid.h | 2 ++ >> 2 files changed, 3 insertions(+), 1 deletion(-) >> >&g

Re: [PATCH v3] Input: uinput - add new UINPUT_DEV_SETUP and UI_ABS_SETUP ioctl

2015-11-10 Thread Elias Vanderstuyft
On Mon, Nov 9, 2015 at 8:50 AM, Benjamin Tissoires wrote: > > - Original Message - >> From: "Elias Vanderstuyft" >> To: "Benjamin Tissoires" >> Cc: "Dmitry Torokhov" , "David Herrmann" >> , "Peter Hutterer&

Re: [PATCH] Input: input.h: Fix EVIOCSFF macro inconsistency by using _IOW()

2015-09-26 Thread Elias Vanderstuyft
On Thu, Sep 17, 2015 at 7:26 PM, Elias Vanderstuyft wrote: > Just like the EVIOCSABS(abs) macro, use the more compact > _IOW(..., type) instead of _IOC(_IOC_WRITE, ..., sizeof(type)) > for the EVIOCSFF macro. > > Signed-off-by: Elias Vanderstuyft > --- > include/uapi/linu

[PATCH] Input: input.h: Fix EVIOCSFF macro inconsistency by using _IOW()

2015-09-17 Thread Elias Vanderstuyft
Just like the EVIOCSABS(abs) macro, use the more compact _IOW(..., type) instead of _IOC(_IOC_WRITE, ..., sizeof(type)) for the EVIOCSFF macro. Signed-off-by: Elias Vanderstuyft --- include/uapi/linux/input.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/uapi/linux

[PATCH 2/2] Input: uinput: Sanity check on ff_effects_max and EV_FF

2015-09-17 Thread Elias Vanderstuyft
g a check in uinput_create_device() and omitting setup of ff-core infrastructure silently in case the check fails, perform the check early in uinput_setup_device(), and print a helpful message and return -EINVAL in case the check fails. Signed-off-by: Elias Vanderstuyft --- drivers/input/m

[PATCH 1/2] Input: Document and check on implicitly defined FF_MAX_EFFECTS

2015-09-17 Thread Elias Vanderstuyft
r exceed FF_GAIN. Define FF_MAX_EFFECTS as FF_GAIN and check on this limit in ff-core. Signed-off-by: Elias Vanderstuyft --- drivers/input/ff-core.c| 5 + include/uapi/linux/input.h | 8 2 files changed, 13 insertions(+) diff --git a/drivers/input/ff-core.c b/drivers/input

[PATCH 0/2] Input: Improve handling of ff max_effects

2015-09-17 Thread Elias Vanderstuyft
This is a patch-set to improve the handling of max_effects in ff-core and uinput. Elias Vanderstuyft (2): Input: Document and check on implicitly defined FF_MAX_EFFECTS Input: uinput: Sanity check on ff_effects_max and EV_FF drivers/input/ff-core.c | 5 + drivers/input/misc