mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-09-05 02:19:51 +00:00
ath5k: Use new function to stop beacon queue
* Since we only use ath5k_hw_stop_tx_dma to stop the beacon queue, introduce a new function ath5k_hw_stop_beacon_queue so that we can use that instead and have better control. In the future we can add more beacon queue specific stuff there (maybe tweak beacon timers or something), for now just call ath5k_hw_stop_tx_dma. * Also since we don't call ath5k_hw_stop_rx/tx_dma from outside dma.c, make them static. Signed-off-by: Nick Kossifidis <mickflemm@gmail.com> Signed-off-by: John W. Linville <linville@tuxdriver.com>
This commit is contained in:
parent
e8325ed874
commit
14fae2d4b6
3 changed files with 25 additions and 6 deletions
|
@ -1166,11 +1166,10 @@ void ath5k_hw_set_clockrate(struct ath5k_hw *ah);
|
|||
|
||||
/* DMA Related Functions */
|
||||
void ath5k_hw_start_rx_dma(struct ath5k_hw *ah);
|
||||
int ath5k_hw_stop_rx_dma(struct ath5k_hw *ah);
|
||||
u32 ath5k_hw_get_rxdp(struct ath5k_hw *ah);
|
||||
int ath5k_hw_set_rxdp(struct ath5k_hw *ah, u32 phys_addr);
|
||||
int ath5k_hw_start_tx_dma(struct ath5k_hw *ah, unsigned int queue);
|
||||
int ath5k_hw_stop_tx_dma(struct ath5k_hw *ah, unsigned int queue);
|
||||
int ath5k_hw_stop_beacon_queue(struct ath5k_hw *ah, unsigned int queue);
|
||||
u32 ath5k_hw_get_txdp(struct ath5k_hw *ah, unsigned int queue);
|
||||
int ath5k_hw_set_txdp(struct ath5k_hw *ah, unsigned int queue,
|
||||
u32 phys_addr);
|
||||
|
|
|
@ -1922,7 +1922,7 @@ ath5k_beacon_send(struct ath5k_softc *sc)
|
|||
* This should never fail since we check above that no frames
|
||||
* are still pending on the queue.
|
||||
*/
|
||||
if (unlikely(ath5k_hw_stop_tx_dma(ah, sc->bhalq))) {
|
||||
if (unlikely(ath5k_hw_stop_beacon_queue(ah, sc->bhalq))) {
|
||||
ATH5K_WARN(sc, "beacon queue %u didn't start/stop ?\n", sc->bhalq);
|
||||
/* NB: hw still stops DMA, so proceed */
|
||||
}
|
||||
|
@ -2091,7 +2091,7 @@ ath5k_beacon_config(struct ath5k_softc *sc)
|
|||
} else
|
||||
ath5k_beacon_update_timers(sc, -1);
|
||||
} else {
|
||||
ath5k_hw_stop_tx_dma(sc->ah, sc->bhalq);
|
||||
ath5k_hw_stop_beacon_queue(sc->ah, sc->bhalq);
|
||||
}
|
||||
|
||||
ath5k_hw_set_imr(ah, sc->imask);
|
||||
|
|
|
@ -58,7 +58,7 @@ void ath5k_hw_start_rx_dma(struct ath5k_hw *ah)
|
|||
*
|
||||
* @ah: The &struct ath5k_hw
|
||||
*/
|
||||
int ath5k_hw_stop_rx_dma(struct ath5k_hw *ah)
|
||||
static int ath5k_hw_stop_rx_dma(struct ath5k_hw *ah)
|
||||
{
|
||||
unsigned int i;
|
||||
|
||||
|
@ -188,7 +188,7 @@ int ath5k_hw_start_tx_dma(struct ath5k_hw *ah, unsigned int queue)
|
|||
* -EINVAL if queue number is out of range or inactive.
|
||||
*
|
||||
*/
|
||||
int ath5k_hw_stop_tx_dma(struct ath5k_hw *ah, unsigned int queue)
|
||||
static int ath5k_hw_stop_tx_dma(struct ath5k_hw *ah, unsigned int queue)
|
||||
{
|
||||
unsigned int i = 40;
|
||||
u32 tx_queue, pending;
|
||||
|
@ -320,6 +320,26 @@ int ath5k_hw_stop_tx_dma(struct ath5k_hw *ah, unsigned int queue)
|
|||
return 0;
|
||||
}
|
||||
|
||||
/**
|
||||
* ath5k_hw_stop_beacon_queue - Stop beacon queue
|
||||
*
|
||||
* @ah The &struct ath5k_hw
|
||||
* @queue The queue number
|
||||
*
|
||||
* Returns -EIO if queue didn't stop
|
||||
*/
|
||||
int ath5k_hw_stop_beacon_queue(struct ath5k_hw *ah, unsigned int queue)
|
||||
{
|
||||
int ret;
|
||||
ret = ath5k_hw_stop_tx_dma(ah, queue);
|
||||
if (ret) {
|
||||
ATH5K_DBG(ah->ah_sc, ATH5K_DEBUG_DMA,
|
||||
"beacon queue didn't stop !\n");
|
||||
return -EIO;
|
||||
}
|
||||
return 0;
|
||||
}
|
||||
|
||||
/**
|
||||
* ath5k_hw_get_txdp - Get TX Descriptor's address for a specific queue
|
||||
*
|
||||
|
|
Loading…
Reference in a new issue