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 <linux@xxxxxxxxxxxxxxxx>
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 <linux@xxxxxxxxxxxxxxxx>
Signed-off-by: John W. Linville <linville@xxxxxxxxxxxxx>
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 <linux@xxxxxxxxxxxxxxxx>
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 <linux@xxxxxxxxxxxxxxxx>
Signed-off-by: John W. Linville <linville@xxxxxxxxxxxxx>
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@xxxxxxxxx
http://github.com/jmahler
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
