From mboxrd@z Thu Jan 1 00:00:00 1970 From: Randy Dunlap Subject: Re: [PATCH -next] hid: fix hid-steelseries kconfig/build Date: Wed, 01 May 2013 12:39:57 -0700 Message-ID: <51816F8D.80607@infradead.org> References: <20130501183734.7ad1efca2d06e75432edabbd@canb.auug.org.au> <51816271.3040400@infradead.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: Received: from casper.infradead.org ([85.118.1.10]:56202 "EHLO casper.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757081Ab3EATk1 (ORCPT ); Wed, 1 May 2013 15:40:27 -0400 In-Reply-To: Sender: linux-next-owner@vger.kernel.org List-ID: To: simon@mungewell.org Cc: Stephen Rothwell , linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, Jiri Kosina , "linux-input@vger.kernel.org" On 05/01/13 12:27, simon@mungewell.org wrote: > Hi Randy and all, > Seems like you found a problem... but the relevant sections of > 'hid-steelseries.c' already have > -- > #if defined(CONFIG_LEDS_CLASS) || defined(CONFIG_LEDS_CLASS_MODULE) which should be replaced with: #include #if IS_ENABLED(LEDS_CLASS) > ... > #endif > -- > > Shouldn't this prevent the module having calls to register/unregister if > the LED_CLASS is not enabled? Please read the patch description. CONFIG_LEDS_CLASS=m but CONFIG_HID_STEELSEREIS=y. A builtin driver cannot make calls to a modular driver if the modular driver is not loaded. Is there a way to require that the modular driver is loaded for hid-steelseries? Maybe so, but I don't know. > Does forcing a 'depends on LED_CLASS' in Kconfig prevent the > hid-steelseries module being built on systems without LEDs, or is this > simply a way to ensure that the LED_CLASS module gets loaded first? It prevents the driver from being built. I think that a satisfactory solution would be this small change: depends on LEDS_CLASS || LEDS_CLASS=n That makes HID_STEELSERIES depend on LEDS_CLASS when it is enabled but still allows the driver to build when LEDS_CLASS=n. Makes sense? Oh, and please don't top-post. Thanks. > > Simon. > >> From: Randy Dunlap >> >> Fix hid-steelseries build by making it depends on LEDS_CLASS. >> Build errors happen when LEDS_CLASS=m and HID_STEELSERIES=y. >> >> drivers/built-in.o: In function `steelseries_srws1_remove': >> hid-steelseries.c:(.text+0x3b97a1): undefined reference to >> `led_classdev_unregister' >> drivers/built-in.o: In function `steelseries_srws1_probe': >> hid-steelseries.c:(.text+0x3b9c51): undefined reference to >> `led_classdev_register' >> hid-steelseries.c:(.text+0x3b9ce5): undefined reference to >> `led_classdev_register' >> hid-steelseries.c:(.text+0x3b9d4b): undefined reference to >> `led_classdev_unregister' >> >> Signed-off-by: Randy Dunlap >> --- >> drivers/hid/Kconfig | 1 + >> 1 file changed, 1 insertion(+) >> >> --- linux-next-20130501.orig/drivers/hid/Kconfig >> +++ linux-next-20130501/drivers/hid/Kconfig >> @@ -610,6 +610,7 @@ config HID_SPEEDLINK >> config HID_STEELSERIES >> tristate "Steelseries SRW-S1 steering wheel support" >> depends on HID >> + depends on LEDS_CLASS >> ---help--- >> Support for Steelseries SRW-S1 steering wheel >> >> -- -- ~Randy