Return-path: Received: from mail.neratec.com ([46.140.151.2]:11161 "EHLO mail.neratec.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751145AbaEUNz6 convert rfc822-to-8bit (ORCPT ); Wed, 21 May 2014 09:55:58 -0400 Message-ID: <537CB066.1070803@neratec.com> (sfid-20140521_155601_688116_B4892EB2) Date: Wed, 21 May 2014 15:55:50 +0200 From: Zefir Kurtisi MIME-Version: 1.0 To: Julian Calaby , Rajkumar Manoharan CC: Jim Davis , Stephen Rothwell , linux-next , linux-kernel , linville , linux-wireless , ath9k-devel@lists.ath9k.org, netdev Subject: Re: [ath9k-devel] randconfig build error with next-20140519, in drivers/net/wireless/ath/ath9k/dfs.c References: <20140521110143.GA2375@qca.qualcomm.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Sender: linux-wireless-owner@vger.kernel.org List-ID: Hi, sorry, I posted a related patch today ('[PATCH] ath9k: fix build error with disabled debug'), but seems did not manage to set the correct in-reply-to field. On 05/21/2014 02:12 PM, Julian Calaby wrote: > Hi Rajkumar, > > On Wed, May 21, 2014 at 9:01 PM, Rajkumar Manoharan > wrote: >> On Mon, May 19, 2014 at 09:31:16AM -0700, Jim Davis wrote: >>> Building with the attached random configuration file, >>> >>> In file included from drivers/net/wireless/ath/ath9k/eeprom.h:22:0, >>> from drivers/net/wireless/ath/ath9k/hw.h:27, >>> from drivers/net/wireless/ath/ath9k/dfs.c:18: >>> drivers/net/wireless/ath/ath9k/dfs.c: In function ‘ath9k_dfs_process_phyerr’: >>> drivers/net/wireless/ath/ath9k/dfs.c:185:14: error: ‘struct ath_softc’ >>> has no member named ‘debug’ >>> pe.ts - sc->debug.stats.dfs_stats.last_ts); >>> ^ >>> drivers/net/wireless/ath/ath9k/../ath.h:273:41: note: in definition of >>> macro ‘ath_dbg’ >>> ath_printk(KERN_DEBUG, common, fmt, ##__VA_ARGS__); \ >>> ^ >>> drivers/net/wireless/ath/ath9k/dfs.c:186:5: error: ‘struct ath_softc’ >>> has no member named ‘debug’ >>> sc->debug.stats.dfs_stats.last_ts = pe.ts; >>> ^ >>> make[5]: *** [drivers/net/wireless/ath/ath9k/dfs.o] Error 1 >> >> Please select CONFIG_ATH9K_DEBUGFS. > > The point of randconfig builds is to ensure that there it's really > unlikely that there are any config combinations that don't build, even > if they're utterly insane. > > You should really change the ath9k Kconfig rules or move or add ifdefs > to prevent this from being possible. > > Thanks, >