On Mon, Jul 20, 2015 at 02:59:56PM -0700, Greg KH wrote:

On Mon, Jul 20, 2015 at 11:03:19PM +0200, Yann Cantin wrote:

quoted

Signed-off-by: Yann Cantin <redacted>

quoted

+
+	/* sysfs setup */
+	err = sysfs_create_group(&intf->dev.kobj, &ebeam_attr_group);

Ick, you just added the sysfs files to the USB device, not your input
device, are you sure you tested this?

And there should be a race-free way to add an attribute group to an
input device, as this is, you are adding them to the device _after_ it
is created, so userspace will not see them at creation time, causing a
race.

No, there are no driver-specific attributed on input devices themselves,
they belong to the actual hardware devices. The input devices only
export standard attributes applicable to every and all input devices
in the system.

Thanks.

-- 
Dmitry