net80211 / drivers: remove public use of ieee80211_node_incref()

ieee80211_node_incref() is the FreeBSD implementation of
ieee80211_ref_node().  Not being interested in the node returned
it was used as a shortcut in 3 drivers (ath, uath, wpi).
Replace the call with the public KPI of ieee80211_ref_node() and
ignore the result.
This leaves us with the single internal call going
ieee80211_ref_node() -> ieee80211_node_incref() and that should
help increasing portability but also limiting the places to trace
for node reference operations.

Sponsored by:	The FreeBSD Foundation
MFC after:	4 weeks
This commit is contained in:
Bjoern A. Zeeb 2023-10-02 14:44:29 +00:00
parent db195a523f
commit f156cd892b
3 changed files with 3 additions and 3 deletions

View file

@ -288,7 +288,7 @@ ath_txfrag_setup(struct ath_softc *sc, ath_bufhead *frags,
ath_txfrag_cleanup(sc, frags, ni);
break;
}
ieee80211_node_incref(ni);
(void) ieee80211_ref_node(ni);
TAILQ_INSERT_TAIL(frags, bf, bf_list);
}
ATH_TXBUF_UNLOCK(sc);

View file

@ -1657,7 +1657,7 @@ uath_txfrag_setup(struct uath_softc *sc, uath_datahead *frags,
uath_txfrag_cleanup(sc, frags, ni);
break;
}
ieee80211_node_incref(ni);
(void) ieee80211_ref_node(ni);
STAILQ_INSERT_TAIL(frags, bf, next);
}

View file

@ -2768,7 +2768,7 @@ wpi_cmd2(struct wpi_softc *sc, struct wpi_buf *buf)
ring->pending = 0;
sc->sc_update_tx_ring(sc, ring);
} else
ieee80211_node_incref(data->ni);
(void) ieee80211_ref_node(data->ni);
end: DPRINTF(sc, WPI_DEBUG_TRACE, error ? TRACE_STR_END_ERR : TRACE_STR_END,
__func__);