Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S964870AbaKNAHx (ORCPT ); Thu, 13 Nov 2014 19:07:53 -0500 Received: from mail-pa0-f46.google.com ([209.85.220.46]:44777 "EHLO mail-pa0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753044AbaKNAHv (ORCPT ); Thu, 13 Nov 2014 19:07:51 -0500 Date: Thu, 13 Nov 2014 16:07:47 -0800 From: Jeremiah Mahler To: Oleksij Rempel Cc: Jeremiah Mahler , linux-kernel@vger.kernel.org, ath9k-devel@qca.qualcomm.com, linville@tuxdriver.com, linux-wireless@vger.kernel.org, ath9k-devel@venema.h4ckr.net, netdev@vger.kernel.org Subject: linux-next: ath9k: build failure, ath_cmn_process_fft() redefinition Message-ID: <20141114000747.GA6865@hudson.localdomain> Mail-Followup-To: Jeremiah Mahler , Oleksij Rempel , linux-kernel@vger.kernel.org, ath9k-devel@qca.qualcomm.com, linville@tuxdriver.com, linux-wireless@vger.kernel.org, ath9k-devel@lists.ath9k.org, netdev@vger.kernel.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit 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 In version 20141113 of the linux-next kernel, if it is compiled with CONFIG_ATH9K_DEBUGFS unset, an error about ath_cmn_process_fft() being redefined will be produced. make ... LD [M] drivers/net/wireless/ath/ath9k/ath9k_hw.o CC [M] drivers/net/wireless/ath/ath9k/common-spectral.o CC lib/debug_locks.o CC lib/random32.o drivers/net/wireless/ath/ath9k/common-spectral.c:40:5: error: redefinition of ‘ath_cmn_process_fft’ int ath_cmn_process_fft(struct ath_spec_scan_priv *spec_priv, struct ieee80211_hdr *hdr, ^ In file included from drivers/net/wireless/ath/ath9k/common.h:27:0, from drivers/net/wireless/ath/ath9k/ath9k.h:27, from drivers/net/wireless/ath/ath9k/common-spectral.c:18: drivers/net/wireless/ath/ath9k/common-spectral.h:146:19: note: previous definition of ‘ath_cmn_process_fft’ was here static inline int ath_cmn_process_fft(struct ath_spec_scan_priv *spec_priv, ^ scripts/Makefile.build:257: recipe for target 'drivers/net/wireless/ath/ath9k/common-spectral.o' failed make[5]: *** [drivers/net/wireless/ath/ath9k/common-spectral.o] Error 1 scripts/Makefile.build:402: recipe for target 'drivers/net/wireless/ath/ath9k' failed make[4]: *** [drivers/net/wireless/ath/ath9k] Error 2 scripts/Makefile.build:402: recipe for target 'drivers/net/wireless/ath' failed make[3]: *** [drivers/net/wireless/ath] Error 2 scripts/Makefile.build:402: recipe for target 'drivers/net/wireless' failed make[2]: *** [drivers/net/wireless] Error 2 scripts/Makefile.build:402: recipe for target 'drivers/net' failed make[1]: *** [drivers/net] Error 2 Makefile:953: recipe for target 'drivers' failed make: *** [drivers] Error 2 make: *** Waiting for unfinished jobs.... CC lib/bust_spinlocks.o ... Bisecting the kernel found that the following patch was the cause. commit 67dc74f15f147b9f88702de2952d2951e3e000ec Author: Oleksij Rempel Date: Thu Nov 6 08:53:30 2014 +0100 ath9k: move spectral.* to common-spectral.* and rename exports from ath9k_spectral_* to ath9k_cmn_spectral_* Signed-off-by: Oleksij Rempel Signed-off-by: John W. Linville This patch mostly consists of renaming functions and moving code but there was a functional change to the Makefile. common-spectral.h uses CONFIG_ATH9K_DEBUGFS to conditionally provide a prototype of ath_cmn_process_fft() when set or to define it as a noop when it is unset. The Makefile was changed so that CONFIG_ATH9K_DEBUGFS no longer applied to common-spectral and this will result in two definitions of ath_cmn_process_fft(). > --- a/drivers/net/wireless/ath/ath9k/Makefile > +++ b/drivers/net/wireless/ath/ath9k/Makefile > @@ -16,8 +16,7 @@ ath9k-$(CONFIG_ATH9K_DFS_CERTIFIED) += dfs.o > ath9k-$(CONFIG_ATH9K_TX99) += tx99.o > ath9k-$(CONFIG_ATH9K_WOW) += wow.o > > -ath9k-$(CONFIG_ATH9K_DEBUGFS) += debug.o \ > - spectral.o > +ath9k-$(CONFIG_ATH9K_DEBUGFS) += debug.o > > ath9k-$(CONFIG_ATH9K_STATION_STATISTICS) += debug_sta.o > > @@ -59,7 +58,8 @@ obj-$(CONFIG_ATH9K_COMMON) += ath9k_common.o > ath9k_common-y:= common.o \ > common-init.o \ > common-beacon.o \ > - common-debug.o > + common-debug.o \ > + common-spectral.o Reverting the patch solves one error, but then a new one is produced. make ... MODPOST 185 modules CC arch/x86/boot/edd.o VOFFSET arch/x86/boot/voffset.h ERROR: "ath9k_cmn_spectral_scan_trigger" [drivers/net/wireless/ath/ath9k/ath9k.ko] undefined! scripts/Makefile.modpost:90: recipe for target '__modpost' failed make[1]: *** [__modpost] Error 1 ... This error is caused by the patch before it. commit f00a422cc81ef665f5098c0bc43cb0c616e55a9b Author: Oleksij Rempel Date: Thu Nov 6 08:53:29 2014 +0100 ath9k: move ath9k_spectral_scan_ from main.c to spectral.c Now we should be ready to make this code common. Signed-off-by: Oleksij Rempel Signed-off-by: John W. Linville Since the code was moved from main.c to spectral.c, it is now involved with CONFIG_ATH9K_DEBUGFS, which causes it to break. Reverting both the above patches resolves the build errors. -- Jeremiah Mahler jmmahler@gmail.com http://github.com/jmahler -- 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/