Module Name: src Committed By: christos Date: Sat Mar 30 14:14:31 UTC 2013
Modified Files: src/sys/dev/ic: athnvar.h src/sys/dev/pci: if_iwnreg.h src/sys/dev/usb: if_athn_usb.c if_athn_usb.h if_otus.c src/sys/net80211: ieee80211.h Log Message: Use only one set of EDCA variables To generate a diff of this commit: cvs rdiff -u -r1.1 -r1.2 src/sys/dev/ic/athnvar.h cvs rdiff -u -r1.10 -r1.11 src/sys/dev/pci/if_iwnreg.h cvs rdiff -u -r1.2 -r1.3 src/sys/dev/usb/if_athn_usb.c cvs rdiff -u -r1.1 -r1.2 src/sys/dev/usb/if_athn_usb.h cvs rdiff -u -r1.23 -r1.24 src/sys/dev/usb/if_otus.c cvs rdiff -u -r1.25 -r1.26 src/sys/net80211/ieee80211.h Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/sys/dev/ic/athnvar.h diff -u src/sys/dev/ic/athnvar.h:1.1 src/sys/dev/ic/athnvar.h:1.2 --- src/sys/dev/ic/athnvar.h:1.1 Fri Mar 29 22:53:02 2013 +++ src/sys/dev/ic/athnvar.h Sat Mar 30 10:14:30 2013 @@ -1,4 +1,4 @@ -/* $NetBSD: athnvar.h,v 1.1 2013/03/30 02:53:02 christos Exp $ */ +/* $NetBSD: athnvar.h,v 1.2 2013/03/30 14:14:30 christos Exp $ */ /* $OpenBSD: athnvar.h,v 1.33 2012/10/20 09:54:20 stsp Exp $ */ /*- @@ -254,11 +254,11 @@ struct athn_serdes { #define ATHN_QID_COUNT 8 /* Map Access Category to Tx queue Id. */ -static const uint8_t athn_ac2qid[EDCA_NUM_AC] = { - ATHN_QID_AC_BE, /* EDCA_AC_BE */ - ATHN_QID_AC_BK, /* EDCA_AC_BK */ - ATHN_QID_AC_VI, /* EDCA_AC_VI */ - ATHN_QID_AC_VO /* EDCA_AC_VO */ +static const uint8_t athn_ac2qid[WME_NUM_AC] = { + ATHN_QID_AC_BE, /* WME_AC_BE */ + ATHN_QID_AC_BK, /* WME_AC_BK */ + ATHN_QID_AC_VI, /* WME_AC_VI */ + ATHN_QID_AC_VO /* WME_AC_VO */ }; static const uint8_t athn_5ghz_chans[] = { Index: src/sys/dev/pci/if_iwnreg.h diff -u src/sys/dev/pci/if_iwnreg.h:1.10 src/sys/dev/pci/if_iwnreg.h:1.11 --- src/sys/dev/pci/if_iwnreg.h:1.10 Sat Oct 8 05:15:08 2011 +++ src/sys/dev/pci/if_iwnreg.h Sat Mar 30 10:14:31 2013 @@ -1,4 +1,4 @@ -/* $NetBSD: if_iwnreg.h,v 1.10 2011/10/08 09:15:08 elric Exp $ */ +/* $NetBSD: if_iwnreg.h,v 1.11 2013/03/30 14:14:31 christos Exp $ */ /* $OpenBSD: if_iwnreg.h,v 1.40 2010/05/05 19:41:57 damien Exp $ */ /*- @@ -19,7 +19,6 @@ */ /* XXX Added for NetBSD */ -#define EDCA_NUM_AC 4 #define IEEE80211_TKIP_MICLEN 8 #define IWN_TX_RING_COUNT 256 @@ -536,7 +535,7 @@ struct iwn_edca_params { uint8_t aifsn; uint8_t reserved; uint16_t txoplimit; - } __packed ac[EDCA_NUM_AC]; + } __packed ac[WME_NUM_AC]; } __packed; /* Structure for command IWN_CMD_TIMING. */ @@ -687,7 +686,7 @@ struct iwn_cmd_link_quality { uint8_t mimo; uint8_t antmsk_1stream; uint8_t antmsk_2stream; - uint8_t ridx[EDCA_NUM_AC]; + uint8_t ridx[WME_NUM_AC]; uint16_t ampdu_limit; uint8_t ampdu_threshold; uint8_t ampdu_max; Index: src/sys/dev/usb/if_athn_usb.c diff -u src/sys/dev/usb/if_athn_usb.c:1.2 src/sys/dev/usb/if_athn_usb.c:1.3 --- src/sys/dev/usb/if_athn_usb.c:1.2 Fri Mar 29 23:12:39 2013 +++ src/sys/dev/usb/if_athn_usb.c Sat Mar 30 10:14:31 2013 @@ -1,4 +1,4 @@ -/* $NetBSD: if_athn_usb.c,v 1.2 2013/03/30 03:12:39 christos Exp $ */ +/* $NetBSD: if_athn_usb.c,v 1.3 2013/03/30 14:14:31 christos Exp $ */ /* $OpenBSD: if_athn_usb.c,v 1.12 2013/01/14 09:50:31 jsing Exp $ */ /*- @@ -22,7 +22,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: if_athn_usb.c,v 1.2 2013/03/30 03:12:39 christos Exp $"); +__KERNEL_RCSID(0, "$NetBSD: if_athn_usb.c,v 1.3 2013/03/30 14:14:31 christos Exp $"); #ifdef _KERNEL_OPT #include "opt_inet.h" @@ -134,7 +134,7 @@ Static void athn_usb_rx_enable(struct at Static void athn_usb_rx_frame(struct athn_usb_softc *, struct mbuf *); Static void athn_usb_rx_radiotap(struct athn_softc *, struct mbuf *, struct ar_rx_status *); -Static void athn_usb_rx_wmi_ctrl(struct athn_usb_softc *, uint8_t *, int); +Static void athn_usb_rx_wmi_ctrl(struct athn_usb_softc *, uint8_t *, size_t); Static void athn_usb_rxeof(usbd_xfer_handle, usbd_private_handle, usbd_status); Static void athn_usb_start(struct ifnet *); @@ -813,7 +813,7 @@ athn_usb_load_firmware(struct athn_usb_s name = "athn-ar9271"; /* Read firmware image from the filesystem. */ - if ((error = firmware_open("if_athn", name, &fwh)) != 0) { + if ((error = firmware_open("if_athn_usb", name, &fwh)) != 0) { aprint_error_dev(sc->sc_dev, "failed to open firmware file %s (%d)\n", name, error); return error; @@ -938,19 +938,19 @@ athn_usb_htc_setup(struct athn_usb_softc if (error != 0) return error; error = athn_usb_htc_connect_svc(usc, AR_SVC_WMI_DATA_BE, - AR_PIPE_TX_DATA, AR_PIPE_RX_DATA, &usc->usc_ep_data[EDCA_AC_BE]); + AR_PIPE_TX_DATA, AR_PIPE_RX_DATA, &usc->usc_ep_data[WME_AC_BE]); if (error != 0) return error; error = athn_usb_htc_connect_svc(usc, AR_SVC_WMI_DATA_BK, - AR_PIPE_TX_DATA, AR_PIPE_RX_DATA, &usc->usc_ep_data[EDCA_AC_BK]); + AR_PIPE_TX_DATA, AR_PIPE_RX_DATA, &usc->usc_ep_data[WME_AC_BK]); if (error != 0) return error; error = athn_usb_htc_connect_svc(usc, AR_SVC_WMI_DATA_VI, - AR_PIPE_TX_DATA, AR_PIPE_RX_DATA, &usc->usc_ep_data[EDCA_AC_VI]); + AR_PIPE_TX_DATA, AR_PIPE_RX_DATA, &usc->usc_ep_data[WME_AC_VI]); if (error != 0) return error; error = athn_usb_htc_connect_svc(usc, AR_SVC_WMI_DATA_VO, - AR_PIPE_TX_DATA, AR_PIPE_RX_DATA, &usc->usc_ep_data[EDCA_AC_VO]); + AR_PIPE_TX_DATA, AR_PIPE_RX_DATA, &usc->usc_ep_data[WME_AC_VO]); if (error != 0) return error; @@ -1833,7 +1833,7 @@ athn_usb_swba(struct athn_usb_softc *usc #endif Static void -athn_usb_rx_wmi_ctrl(struct athn_usb_softc *usc, uint8_t *buf, int len) +athn_usb_rx_wmi_ctrl(struct athn_usb_softc *usc, uint8_t *buf, size_t len) { #ifdef ATHN_DEBUG struct ar_wmi_evt_txrate *txrate; @@ -1846,7 +1846,7 @@ athn_usb_rx_wmi_ctrl(struct athn_usb_sof DPRINTFN(DBG_FN, usc, "\n"); - if (__predict_false(len < (int)sizeof(*wmi))) + if (__predict_false(len < sizeof(*wmi))) return; wmi = (struct ar_wmi_cmd_hdr *)buf; cmd_id = be16toh(wmi->cmd_id); @@ -2293,7 +2293,7 @@ athn_usb_tx(struct athn_softc *sc, struc #endif /* notyet_edca */ { tid = 0; - qid = EDCA_AC_BE; + qid = WME_AC_BE; } /* XXX Change radiotap Tx header for USB (no txrate). */ Index: src/sys/dev/usb/if_athn_usb.h diff -u src/sys/dev/usb/if_athn_usb.h:1.1 src/sys/dev/usb/if_athn_usb.h:1.2 --- src/sys/dev/usb/if_athn_usb.h:1.1 Fri Mar 29 22:53:02 2013 +++ src/sys/dev/usb/if_athn_usb.h Sat Mar 30 10:14:31 2013 @@ -1,4 +1,4 @@ -/* $NetBSD: if_athn_usb.h,v 1.1 2013/03/30 02:53:02 christos Exp $ */ +/* $NetBSD: if_athn_usb.h,v 1.2 2013/03/30 14:14:31 christos Exp $ */ /* $OpenBSD: if_athn_usb.h,v 1.3 2012/11/10 14:35:06 mikeb Exp $ */ /*- @@ -486,7 +486,7 @@ struct athn_usb_softc { uint8_t usc_ep_cab; uint8_t usc_ep_uapsd; uint8_t usc_ep_mgmt; - uint8_t usc_ep_data[EDCA_NUM_AC]; + uint8_t usc_ep_data[WME_NUM_AC]; void (*usc_node_cleanup)(struct ieee80211_node *); }; Index: src/sys/dev/usb/if_otus.c diff -u src/sys/dev/usb/if_otus.c:1.23 src/sys/dev/usb/if_otus.c:1.24 --- src/sys/dev/usb/if_otus.c:1.23 Tue Jan 29 08:54:26 2013 +++ src/sys/dev/usb/if_otus.c Sat Mar 30 10:14:31 2013 @@ -1,4 +1,4 @@ -/* $NetBSD: if_otus.c,v 1.23 2013/01/29 13:54:26 christos Exp $ */ +/* $NetBSD: if_otus.c,v 1.24 2013/03/30 14:14:31 christos Exp $ */ /* $OpenBSD: if_otus.c,v 1.18 2010/08/27 17:08:00 jsg Exp $ */ /*- @@ -23,7 +23,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: if_otus.c,v 1.23 2013/01/29 13:54:26 christos Exp $"); +__KERNEL_RCSID(0, "$NetBSD: if_otus.c,v 1.24 2013/03/30 14:14:31 christos Exp $"); #include <sys/param.h> #include <sys/sockio.h> @@ -1972,12 +1972,12 @@ otus_tx(struct otus_softc *sc, struct mb qid = ieee80211_up_to_ac(ic, qos & IEEE80211_QOS_TID); } else { qos = 0; - qid = EDCA_AC_BE; + qid = WME_AC_BE; } #else hasqos = 0; qos = 0; - qid = EDCA_AC_BE; + qid = WME_AC_BE; #endif /* Pickup a rate index. */ @@ -2382,38 +2382,38 @@ otus_updateedca_cb_locked(struct otus_so /* Set CWmin/CWmax values. */ otus_write(sc, AR_MAC_REG_AC0_CW, - EXP2(edca[EDCA_AC_BE].ac_ecwmax) << 16 | - EXP2(edca[EDCA_AC_BE].ac_ecwmin)); + EXP2(edca[WME_AC_BE].ac_ecwmax) << 16 | + EXP2(edca[WME_AC_BE].ac_ecwmin)); otus_write(sc, AR_MAC_REG_AC1_CW, - EXP2(edca[EDCA_AC_BK].ac_ecwmax) << 16 | - EXP2(edca[EDCA_AC_BK].ac_ecwmin)); + EXP2(edca[WME_AC_BK].ac_ecwmax) << 16 | + EXP2(edca[WME_AC_BK].ac_ecwmin)); otus_write(sc, AR_MAC_REG_AC2_CW, - EXP2(edca[EDCA_AC_VI].ac_ecwmax) << 16 | - EXP2(edca[EDCA_AC_VI].ac_ecwmin)); + EXP2(edca[WME_AC_VI].ac_ecwmax) << 16 | + EXP2(edca[WME_AC_VI].ac_ecwmin)); otus_write(sc, AR_MAC_REG_AC3_CW, - EXP2(edca[EDCA_AC_VO].ac_ecwmax) << 16 | - EXP2(edca[EDCA_AC_VO].ac_ecwmin)); + EXP2(edca[WME_AC_VO].ac_ecwmax) << 16 | + EXP2(edca[WME_AC_VO].ac_ecwmin)); otus_write(sc, AR_MAC_REG_AC4_CW, /* Special TXQ. */ - EXP2(edca[EDCA_AC_VO].ac_ecwmax) << 16 | - EXP2(edca[EDCA_AC_VO].ac_ecwmin)); + EXP2(edca[WME_AC_VO].ac_ecwmax) << 16 | + EXP2(edca[WME_AC_VO].ac_ecwmin)); /* Set AIFSN values. */ otus_write(sc, AR_MAC_REG_AC1_AC0_AIFS, - AIFS(edca[EDCA_AC_VI].ac_aifsn) << 24 | - AIFS(edca[EDCA_AC_BK].ac_aifsn) << 12 | - AIFS(edca[EDCA_AC_BE].ac_aifsn)); + AIFS(edca[WME_AC_VI].ac_aifsn) << 24 | + AIFS(edca[WME_AC_BK].ac_aifsn) << 12 | + AIFS(edca[WME_AC_BE].ac_aifsn)); otus_write(sc, AR_MAC_REG_AC3_AC2_AIFS, - AIFS(edca[EDCA_AC_VO].ac_aifsn) << 16 | /* Special TXQ. */ - AIFS(edca[EDCA_AC_VO].ac_aifsn) << 4 | - AIFS(edca[EDCA_AC_VI].ac_aifsn) >> 8); + AIFS(edca[WME_AC_VO].ac_aifsn) << 16 | /* Special TXQ. */ + AIFS(edca[WME_AC_VO].ac_aifsn) << 4 | + AIFS(edca[WME_AC_VI].ac_aifsn) >> 8); /* Set TXOP limit. */ otus_write(sc, AR_MAC_REG_AC1_AC0_TXOP, - edca[EDCA_AC_BK].ac_txoplimit << 16 | - edca[EDCA_AC_BE].ac_txoplimit); + edca[WME_AC_BK].ac_txoplimit << 16 | + edca[WME_AC_BE].ac_txoplimit); otus_write(sc, AR_MAC_REG_AC3_AC2_TXOP, - edca[EDCA_AC_VO].ac_txoplimit << 16 | - edca[EDCA_AC_VI].ac_txoplimit); + edca[WME_AC_VO].ac_txoplimit << 16 | + edca[WME_AC_VI].ac_txoplimit); #undef AIFS #undef EXP2 Index: src/sys/net80211/ieee80211.h diff -u src/sys/net80211/ieee80211.h:1.25 src/sys/net80211/ieee80211.h:1.26 --- src/sys/net80211/ieee80211.h:1.25 Fri Mar 29 23:25:47 2013 +++ src/sys/net80211/ieee80211.h Sat Mar 30 10:14:31 2013 @@ -1,4 +1,4 @@ -/* $NetBSD: ieee80211.h,v 1.25 2013/03/30 03:25:47 christos Exp $ */ +/* $NetBSD: ieee80211.h,v 1.26 2013/03/30 14:14:31 christos Exp $ */ /*- * Copyright (c) 2001 Atsushi Onoe * Copyright (c) 2002-2005 Sam Leffler, Errno Consulting @@ -203,17 +203,6 @@ struct ieee80211_qosframe_addr4 { #define IEEE80211_QOS_TID 0x000f /* - * EDCA Access Categories. - */ -enum ieee80211_edca_ac { - EDCA_AC_BK = 1, /* Background */ - EDCA_AC_BE = 0, /* Best Effort */ - EDCA_AC_VI = 2, /* Video */ - EDCA_AC_VO = 3 /* Voice */ -}; -#define EDCA_NUM_AC 4 - -/* * WME/802.11e information element. */ struct ieee80211_wme_info { @@ -263,7 +252,14 @@ struct ieee80211_wme_acparams { u_int16_t acp_txop; } __packed; -#define WME_NUM_AC 4 /* 4 AC categories */ +/* WME stream classes */ +enum ieee80211_wme_ac { + WME_AC_BE = 0, /* best effort */ + WME_AC_BK = 1, /* background */ + WME_AC_VI = 2, /* video */ + WME_AC_VO = 3, /* voice */ +}; +#define WME_NUM_AC 4 /* 4 AC categories */ #define WME_PARAM_ACI 0x60 /* Mask for ACI field */ #define WME_PARAM_ACI_S 5 /* Shift for ACI field */ @@ -581,12 +577,6 @@ struct ieee80211_country_ie { #define WME_PARAM_OUI_SUBTYPE 0x01 #define WME_VERSION 1 -/* WME stream classes */ -#define WME_AC_BE 0 /* best effort */ -#define WME_AC_BK 1 /* background */ -#define WME_AC_VI 2 /* video */ -#define WME_AC_VO 3 /* voice */ - /* * AUTH management packets *