Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754334AbaFWSXj (ORCPT ); Mon, 23 Jun 2014 14:23:39 -0400 Received: from out1-smtp.messagingengine.com ([66.111.4.25]:39163 "EHLO out1-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754171AbaFWSXh (ORCPT ); Mon, 23 Jun 2014 14:23:37 -0400 X-Sasl-enc: u2W8LQ3l0o3gHew7Al38pDcrRvJc6Hz5JHm8RRYCaCQu 1403547816 Date: Mon, 23 Jun 2014 14:23:24 -0400 From: Greg KH To: Janne Kanniainen Cc: johan@kernel.org, jkosina@suse.cz, cooloney@gmail.com, linux-kernel@vger.kernel.org, linux-leds@vger.kernel.org, linux-usb@vger.kernel.org, linux-input@vger.kernel.org Subject: Re: [PATCH v10] leds: USB: HID: Add support for MSI GT683R led panels Message-ID: <20140623182324.GA30831@kroah.com> References: <20140623161723.GA20421@kroah.com> <1403543808-8228-1-git-send-email-janne.kanniainen@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1403543808-8228-1-git-send-email-janne.kanniainen@gmail.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Jun 23, 2014 at 08:16:48PM +0300, Janne Kanniainen wrote: > + ret = sysfs_create_group(&led->hdev->dev.kobj, >683r_attribute_group); > + if (ret) { > + hid_err(hdev, "failed to create sysfs attributes\n"); > + goto fail; > + } No, you need to set the attribute group _before_ you call led_classdev_register, as that is where the device will be created in sysfs. Surely the other led drivers already do this? I'm almost afraid to go look... You also have to document your sysfs file in Documentation/ABI/ thanks, greg k-h -- 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/