Re: kernel compile fails without AH_SUPPORT_AR5416



Hi,

Can you please try this patch?
I can't commit it until STABLE is unfrozen after 7.2-RELEASE is cut.

Sam Leffler wrote:
Bruce Simpson wrote:
Hi,

Looks like I'm late to the party. I was responsible for committing these
ath(4) changes to RELENG_7.
I can't remember if I tested the kernel compile without the
AH_SUPPORT_AR5416 option or not, I have been so incredibly busy.
...
ru had a change to fix this but decided not to; can't say why.
Otherwise there is a better way to fix this which I alluded to in
previous mail--use the config-generated #define that is generated for
the "ath_hal" device.


thanks,
BMS
Index: UPDATING
===================================================================
--- UPDATING (revision 191718)
+++ UPDATING (working copy)
@@ -8,6 +8,11 @@
/usr/ports/UPDATING. Please read that file before running
portupgrade.

+20090505:
+ The kernel compile-time option AH_SUPPORT_AR5416 has been
+ removed; it is now enabled by default as if_ath.c depends on it
+ in order to build.
+
20090504:
FreeBSD 7.2-RELEASE

Index: sys/arm/conf/AVILA
===================================================================
--- sys/arm/conf/AVILA (revision 191718)
+++ sys/arm/conf/AVILA (working copy)
@@ -133,7 +133,6 @@
#device wlan_tkip # 802.11 TKIP support
device ath # Atheros pci/cardbus NIC's
device ath_hal # Atheros HAL (Hardware Access Layer)
-options AH_SUPPORT_AR5416 # enable AR5416 tx/rx descriptors
device ath_rate_sample # SampleRate tx rate control for ath
options ATH_DEBUG

Index: sys/sparc64/conf/GENERIC
===================================================================
--- sys/sparc64/conf/GENERIC (revision 191718)
+++ sys/sparc64/conf/GENERIC (working copy)
@@ -191,7 +191,6 @@
device wlan_scan_sta # 802.11 STA mode scanning
device ath # Atheros pci/cardbus NIC's
device ath_hal # Atheros HAL (Hardware Access Layer)
-options AH_SUPPORT_AR5416 # enable AR5416 tx/rx descriptors
device ath_rate_sample # SampleRate tx rate control for ath

# Pseudo devices.
Index: sys/conf/options
===================================================================
--- sys/conf/options (revision 191718)
+++ sys/conf/options (working copy)
@@ -731,8 +731,6 @@
ATH_TX99_DIAG opt_ath.h

# options for the Atheros hal
-AH_SUPPORT_AR5416 opt_ah.h
-
AH_DEBUG opt_ah.h
AH_ASSERT opt_ah.h
AH_DEBUG_ALQ opt_ah.h
Index: sys/modules/ath/Makefile
===================================================================
--- sys/modules/ath/Makefile (revision 191718)
+++ sys/modules/ath/Makefile (working copy)
@@ -76,8 +76,9 @@
#
# AR5416, AR9160 support; these are 11n parts but only really
# supported (right now) operating in legacy mode. Note enabling
-# this support requires defining AH_SUPPORT_AR5416 in opt_ah.h
+# this support requires defining AH_SUPPORT_AR5416
# so the 11n tx/rx descriptor format is handled.
+# This support is now enabled by default.
#
# NB: 9160 depends on 5416 but 5416 does not require 9160
#
@@ -106,7 +107,4 @@

CFLAGS+= -I. -I${.CURDIR}/../../dev/ath -I${.CURDIR}/../../dev/ath/ath_hal

-opt_ah.h:
- echo '#define AH_SUPPORT_AR5416 1' > $@
-
.include <bsd.kmod.mk>
Index: sys/dev/ath/ath_hal/ah_desc.h
===================================================================
--- sys/dev/ath/ath_hal/ah_desc.h (revision 191718)
+++ sys/dev/ath/ath_hal/ah_desc.h (working copy)
@@ -20,8 +20,12 @@
#ifndef _DEV_ATH_DESC_H
#define _DEV_ATH_DESC_H

-#include "opt_ah.h" /* NB: required for AH_SUPPORT_AR5416 */
+#include "opt_ah.h"

+#ifndef AH_SUPPORT_AR5416
+#define AH_SUPPORT_AR5416 /* always support AR5416 */
+#endif
+
/*
* Transmit descriptor status. This structure is filled
* in only after the tx descriptor process method finds a
Index: sys/dev/ath/if_ath.c
===================================================================
--- sys/dev/ath/if_ath.c (revision 191718)
+++ sys/dev/ath/if_ath.c (working copy)
@@ -3399,7 +3399,7 @@
rix = rs->rs_rate;
sc->sc_rx_th.wr_rate = sc->sc_hwmap[rix].ieeerate;
sc->sc_rx_th.wr_flags = sc->sc_hwmap[rix].rxflags;
-#if HAL_ABI_VERSION >= 0x07050400
+#if HAL_ABI_VERSION >= 0x07050400 && defined(AH_SUPPORT_AR5416)
if (sc->sc_curchan.channelFlags & CHANNEL_HT) {
/*
* For HT operation we must specify the channel
Index: sys/i386/conf/GENERIC
===================================================================
--- sys/i386/conf/GENERIC (revision 191718)
+++ sys/i386/conf/GENERIC (working copy)
@@ -255,7 +255,6 @@
device an # Aironet 4500/4800 802.11 wireless NICs.
device ath # Atheros pci/cardbus NIC's
device ath_hal # Atheros HAL (Hardware Access Layer)
-options AH_SUPPORT_AR5416 # enable AR5416 tx/rx descriptors
device ath_rate_sample # SampleRate tx rate control for ath
device awi # BayStack 660 and others
device ral # Ralink Technology RT2500 wireless NICs.
Index: sys/amd64/conf/GENERIC
===================================================================
--- sys/amd64/conf/GENERIC (revision 191718)
+++ sys/amd64/conf/GENERIC (working copy)
@@ -242,7 +242,6 @@
device an # Aironet 4500/4800 802.11 wireless NICs.
device ath # Atheros pci/cardbus NIC's
device ath_hal # Atheros HAL (Hardware Access Layer)
-options AH_SUPPORT_AR5416 # enable AR5416 tx/rx descriptors
device ath_rate_sample # SampleRate tx rate control for ath
device awi # BayStack 660 and others
device ral # Ralink Technology RT2500 wireless NICs.
_______________________________________________
freebsd-stable@xxxxxxxxxxx mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-stable
To unsubscribe, send any mail to "freebsd-stable-unsubscribe@xxxxxxxxxxx"

Relevant Pages

  • Re: kernel compile fails without AH_SUPPORT_AR5416
    ... I can't commit it until STABLE is unfrozen after 7.2-RELEASE is cut. ... # supported operating in legacy mode. ... +# this support requires defining AH_SUPPORT_AR5416 ... Transmit descriptor status. ...
    (freebsd-current)
  • cvs-src summaries misposted
    ... about the fact that I wouldn't be able to post the cvs-src summaries. ... Søren Schmidt added support to the ATA code for detaching PCI ATA ... As a follow-up to last week's commit of `ICC support for kernel building`, ... imported OpenSSL 0.9.7d, which includes the fix, to -CURRENT. ...
    (freebsd-current)
  • [BUG,2.6.29-rc7,s390] System goes into endless loop during boot or logon
    ... commit ca109491f612aab5c8152207631c0444f63da97f ... # Core Netfilter Configuration ... # IPVS transport protocol load balancing support ... # Device Drivers ...
    (Linux-Kernel)
  • Re: commit "radeonfb: Fix resume from D3Cold on some platforms" breaks resume from RAM on PowerBook
    ... On resume the pulsing led indicating sleep just changes to solid white ... Reverting this commit on top of the current Linus' tree fixes resume. ... # Processor support ... # Enable WiMAX to see the WiMAX drivers ...
    (Linux-Kernel)
  • [git pull] drm
    ... Kernel Modesetting core code (jbarnes you can go to drinking island ... Intel i915 support for kernel modesetting ... commit aa5966296675a5092505f68d72563d5939a92353 ... drm: Add a debug node for vblank state. ...
    (Linux-Kernel)