linux-kernel - Re: [PATCH v4 00/24] input: Introduce ff-memless-next as an improved replacement for ff-memless

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <996a1b32fb688eeafed959bd545d9301.squirrel@mungewell.org>
Date:	Tue, 29 Apr 2014 11:59:02 -0400
From:	simon@...gewell.org
To:	"Michal MalÃ½" 
	<madcatxster@...oid-pointer.net>
Cc:	linux-input@...r.kernel.org, linux-kernel@...r.kernel.org,
	dmitry.torokhov@...il.com, jkosina@...e.cz, elias.vds@...il.com,
	anssi.hannula@....fi, simon@...gewell.org,
	"Michal MalÃ½" 
	<madcatxster@...oid-pointer.net>
Subject: Re: [PATCH v4 00/24] input: Introduce ff-memless-next as an improved
 replacement for ff-memless

> This patch series:
> 1) Adds "ff-memless-next" module [1]
> 2) Ports all hardware-specific drivers to MLNX's API [2-23]
> 3) Removes FFML and replaces it with MLNX [24]
>
> Signed-off-by: Michal MalÃ½ <madcatxster@...oid-pointer.net>
>
> v4:
>  - Add a summary of changes between MLNX and FFML to the last patch
>  - Remove a stale empty line in hid-sony.c
>  - Add "Tested-by: Elias Vanderstuyft <elias.vds@...il.com>" to hid-lg4ff
> patch.
>
> v3:
>  - Rebase against latest linux-next. Fixes conflict in hid-sony.c and
>    max8997_haptic.c
>  - Updated documentation in ff-memless-next.h. The documentation now
> describes
>    parameters of the callback function and specifically mentions that
>    HW-specific drivers must not keep a reference to mlnx_effect_command
> struct
>    to which a pointer is passed in the callback function.
>  - Fix a minor brace inconsistency in hid-lgff
>  I believe that all concerns regarding v2 have been resolved as false
> alarms.
>
> v2:
>  - Add missing msecs to jiffies conversion in ff-memless-next
>  - lgff: Properly convert force on Y axis from MLNX to device range
>          Support periodic effects for "joystick_ac" device class
>  - lg3ff: Properly convert forces from MLNX to device range
>  - Very minor coding style issues fixed

Hi all,
I'd confirm that I build v2 and tested on a number of devices (1), and it
appears to work OK.

The only slight hiccup was with an older version (Xubuntu 12.10)
'ffcfstress' application which did not correctly detect the CF
capabilities of my gaming wheel(s). This is believed to be a fault with
the application not using correct bit-field testing and appears to have
been fixed on later versions (Xubuntu 13.10).

I also built v4, but have not yet had time/access to all the devices
(other than DS4) to test.

Cheers,
Simon

Tested-by: Simon Wood <simon@...gewell.org>

(1) Devices:
Logitech Momo Red, Momo Black, DFGT, WiiWheel, G27
Sony DS3-SA, DS4, Intec 3rd Party PS3 controller
Nintendo Wii Remote


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/