From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mika Westerberg Subject: Re: [PATCH v2] leds: Add user LED driver for NIC78bx device Date: Mon, 31 Oct 2016 10:47:04 +0200 Message-ID: <20161031084704.GI23812@lahna.fi.intel.com> References: <1477566173-124969-1-git-send-email-hui.chun.ong@ni.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mga11.intel.com ([192.55.52.93]:20029 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1763068AbcJaIzx (ORCPT ); Mon, 31 Oct 2016 04:55:53 -0400 Content-Disposition: inline In-Reply-To: Sender: linux-leds-owner@vger.kernel.org List-Id: linux-leds@vger.kernel.org To: Jacek Anaszewski Cc: Hui Chun Ong , rpurdie@rpsys.net, linux-leds@vger.kernel.org, jonathan.hearn@ni.com, julia.cartwright@ni.com, Brad Mouring , "Rafael J. Wysocki" On Fri, Oct 28, 2016 at 12:08:57PM +0200, Jacek Anaszewski wrote: > > +static struct platform_driver led_driver = { > > + .probe = ni78bx_add, > > + .remove = ni78bx_remove, > > + .driver = { > > + .name = KBUILD_MODNAME, > > + .acpi_match_table = ACPI_PTR(led_device_ids), > > Please CC ACPI maintainers always if you include acpi.h > Cc Rafael and Mika. >>From ACPI perspective this driver looks fine to me now.