Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757408AbbGTW0s (ORCPT ); Mon, 20 Jul 2015 18:26:48 -0400 Received: from mail-pa0-f53.google.com ([209.85.220.53]:34947 "EHLO mail-pa0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752319AbbGTW0q (ORCPT ); Mon, 20 Jul 2015 18:26:46 -0400 Date: Mon, 20 Jul 2015 15:26:40 -0700 From: Dmitry Torokhov To: Greg KH Cc: Yann Cantin , linux-input@vger.kernel.org, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, linux-api@vger.kernel.org, jkosina@suse.cz Subject: Re: [RFC ebeam PATCH 2/2] input: misc: New USB eBeam input driver Message-ID: <20150720222640.GD30252@dtor-ws> References: <1437426199-29866-1-git-send-email-yann.cantin@laposte.net> <1437426199-29866-3-git-send-email-yann.cantin@laposte.net> <20150720215956.GA21796@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150720215956.GA21796@kroah.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1111 Lines: 30 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: > > Signed-off-by: Yann Cantin > > > + > > + /* 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 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/