upstream/opnsense-src
1
0
Fork 0
mirror of https://github.com/opnsense/src.git synced 2026-06-06 15:22:34 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions
opnsense-src/sys/dev/ath
History
Dimitry Andric 36d486cc2e Fix enum warning in ath_hal's ar9002 
This fixes a clang 19 warning:

    sys/dev/ath/ath_hal/ar9002/ar9285_btcoex.c:57:32: error: comparison of different enumeration types ('HAL_BOOL' and 'HAL_ANT_SETTING') [-Werror,-Wenum-compare]
       57 |             (AH5212(ah)->ah_diversity != HAL_ANT_VARIABLE)) {
          |              ~~~~~~~~~~~~~~~~~~~~~~~~ ^  ~~~~~~~~~~~~~~~~

The `ah_diversity` field of `struct ath_hal_5212` is of type `HAL_BOOL`,
not the enum type `HAL_ANT_SETTING`. In other code, `ah_diversity` is
set to `AH_TRUE` whenever the related field `ah_antControl` is set to
`HAL_ANT_VARIABLE`.

It is not entirely clear to me what the intended statement is here: the
test as it is written now compares the enum value 0 to `ah_diversity`,
so in effect it enables the following block whenever `ah_diversity` is
`AH_TRUE`. Write it like that, to avoid the compiler warning.

MFC after:	3 days
2024-10-23 20:26:35 +02:00
..
ath_dfs/null sys: Remove $FreeBSD$: one-line .c pattern 2023-08-16 11:54:36 -06:00
ath_hal Fix enum warning in ath_hal's ar9002 2024-10-23 20:26:35 +02:00
ath_rate ath: Handle errors from copyout() in ath_rate_fetch_node_stats() 2023-12-25 21:04:00 -05:00
ah_osdep.c sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
ah_osdep.h sys: Automated cleanup of cdefs and other formatting 2023-11-26 22:24:00 -07:00
ah_osdep_ar5210.c sys: Automated cleanup of cdefs and other formatting 2023-11-26 22:24:00 -07:00
ah_osdep_ar5211.c sys: Automated cleanup of cdefs and other formatting 2023-11-26 22:24:00 -07:00
ah_osdep_ar5212.c sys: Automated cleanup of cdefs and other formatting 2023-11-26 22:24:00 -07:00
ah_osdep_ar5416.c sys: Automated cleanup of cdefs and other formatting 2023-11-26 22:24:00 -07:00
ah_osdep_ar9300.c sys: Automated cleanup of cdefs and other formatting 2023-11-26 22:24:00 -07:00
if_ath.c ath: Revert "Use NET_TASK_INIT() and NET_GROUPTASK_INIT() for drivers that process" 2023-12-01 00:48:55 +00:00
if_ath_alq.c sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_alq.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_beacon.c ath(4): Fix a typo in a source code comment 2023-11-23 09:57:28 +01:00
if_ath_beacon.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_btcoex.c sys: Remove $FreeBSD$: one-line .c pattern 2023-08-16 11:54:36 -06:00
if_ath_btcoex.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_btcoex_mci.c sys: Remove $FreeBSD$: one-line .c pattern 2023-08-16 11:54:36 -06:00
if_ath_btcoex_mci.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_debug.c sys: Remove $FreeBSD$: one-line .c pattern 2023-08-16 11:54:36 -06:00
if_ath_debug.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_descdma.c sys: Remove $FreeBSD$: one-line .c pattern 2023-08-16 11:54:36 -06:00
if_ath_descdma.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_dfs.c sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_drv.c sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_ioctl.c sys: Remove $FreeBSD$: one-line .c pattern 2023-08-16 11:54:36 -06:00
if_ath_ioctl.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_keycache.c net80211: migrate the group/unicast key check into inline functions 2024-07-15 11:45:30 -07:00
if_ath_keycache.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_led.c sys: Remove $FreeBSD$: one-line .c pattern 2023-08-16 11:54:36 -06:00
if_ath_led.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_lna_div.c ath(4): Stop checking for failures from malloc(M_WAITOK) 2024-09-03 18:25:24 +08:00
if_ath_lna_div.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_misc.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_pci.c ath(4): Stop checking for failures from malloc(M_WAITOK) 2024-09-03 18:25:24 +08:00
if_ath_pci_devlist.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_rate.c sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_rx.c Revert "[ath] Attempt to fix epoch handling." 2023-12-01 00:48:55 +00:00
if_ath_rx.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_rx_edma.c Revert "[ath] Attempt to fix epoch handling." 2023-12-01 00:48:55 +00:00
if_ath_rx_edma.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_spectral.c ath(4): Stop checking for failures from malloc(M_WAITOK) 2024-09-03 18:25:24 +08:00
if_ath_spectral.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_sysctl.c sys: Remove $FreeBSD$: one-line .c pattern 2023-08-16 11:54:36 -06:00
if_ath_sysctl.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_tdma.c sys: Remove $FreeBSD$: one-line .c pattern 2023-08-16 11:54:36 -06:00
if_ath_tdma.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_tsf.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_tx.c net80211 / drivers: remove public use of ieee80211_node_incref() 2023-10-02 14:52:57 +00:00
if_ath_tx.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_tx_edma.c sys: Remove $FreeBSD$: one-line .c pattern 2023-08-16 11:54:36 -06:00
if_ath_tx_edma.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_ath_tx_ht.c sys: Remove $FreeBSD$: one-line .c pattern 2023-08-16 11:54:36 -06:00
if_ath_tx_ht.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_athdfs.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_athioctl.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_athrate.h sys: Remove $FreeBSD$: two-line .h pattern 2023-08-16 11:54:11 -06:00
if_athvar.h ath(4): always enable 11n 2024-02-22 22:34:17 +00:00