Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759639AbZCSSsY (ORCPT ); Thu, 19 Mar 2009 14:48:24 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1756570AbZCSSsN (ORCPT ); Thu, 19 Mar 2009 14:48:13 -0400 Received: from acsinet12.oracle.com ([141.146.126.234]:55697 "EHLO acsinet12.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756453AbZCSSsM (ORCPT ); Thu, 19 Mar 2009 14:48:12 -0400 Message-ID: <49C293CC.6020009@oracle.com> Date: Thu, 19 Mar 2009 11:49:48 -0700 From: Randy Dunlap Organization: Oracle Linux Engineering User-Agent: Thunderbird 2.0.0.6 (X11/20070801) MIME-Version: 1.0 To: Michael Buesch CC: Randy Dunlap , Stephen Rothwell , linux-next@vger.kernel.org, LKML , "linux-wireless@vger.kernel.org" Subject: Re: linux-next: Tree for March 19 (wireless/p54) References: <20090319221024.5e2ad6e5.sfr@canb.auug.org.au> <49C280BE.8090106@oracle.com> <200903191850.17825.mb@bu3sch.de> In-Reply-To: <200903191850.17825.mb@bu3sch.de> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-Source-IP: acsmt704.oracle.com [141.146.40.82] X-Auth-Type: Internal IP X-CT-RefId: str=0001.0A090208.49C29347.02B7:SCFSTAT928724,ss=1,fgs=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1244 Lines: 38 Michael Buesch wrote: > On Thursday 19 March 2009 18:28:30 Randy Dunlap wrote: >> Stephen Rothwell wrote: >>> Hi all, >>> >>> Changes since 20090318: >> >> ERROR: "led_classdev_register" [drivers/net/wireless/p54/p54common.ko] undefined! >> ERROR: "led_classdev_unregister" [drivers/net/wireless/p54/p54common.ko] undefined! >> >> One possible option (fix) is to have CONFIG_MAC80211_LEDS also select >> LEDS_CLASS (that would build the missing functions). > > Does p54 select CONFIG_MAC80211_LEDS? If that's the case, I think it should No, it doesn't. p54common.c just has a large block of code that is surrounded by #ifdef CONFIG_MAC80211_LEDS ... #endif so we could argue that it's just purely a p54 bug. > probably do it the other way around. p54 should build its LED code if CONFIG_MAC80211_LEDS > is available. Otherwise it should not. > > This way you don't enter the SELECT dependency hell and still have automagic configuration. > > Take a look at b43 for an example. -- ~Randy -- 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/