diff options
author | Bruno Randolf <bruno@thinktube.com> | 2008-03-05 04:36:05 -0500 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2008-03-07 16:07:50 -0500 |
commit | e14296cabac60806606b4ebc1a83ee4697876346 (patch) | |
tree | 98522cb54c0f43d3803dfcec9747757e8827d2b2 /drivers/net/wireless/ath5k | |
parent | b47f407bef0d5349dacf65cd3560a976609d4b45 (diff) |
ath5k: add notes about rx timestamp
add comments about the fact that we don't know when exactly the atheros
hardware takes the RX timestamp.
drivers/net/wireless/ath5k/base.c: Changes-licensed-under: 3-Clause-BSD
Signed-off-by: Bruno Randolf <bruno@thinktube.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless/ath5k')
-rw-r--r-- | drivers/net/wireless/ath5k/base.c | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/drivers/net/wireless/ath5k/base.c b/drivers/net/wireless/ath5k/base.c index 73e7e613a3c9..5eacc7cdd9e3 100644 --- a/drivers/net/wireless/ath5k/base.c +++ b/drivers/net/wireless/ath5k/base.c | |||
@@ -1844,6 +1844,17 @@ accept: | |||
1844 | * 15bit only. that means TSF extension has to be done within | 1844 | * 15bit only. that means TSF extension has to be done within |
1845 | * 32768usec (about 32ms). it might be necessary to move this to | 1845 | * 32768usec (about 32ms). it might be necessary to move this to |
1846 | * the interrupt handler, like it is done in madwifi. | 1846 | * the interrupt handler, like it is done in madwifi. |
1847 | * | ||
1848 | * Unfortunately we don't know when the hardware takes the rx | ||
1849 | * timestamp (beginning of phy frame, data frame, end of rx?). | ||
1850 | * The only thing we know is that it is hardware specific... | ||
1851 | * On AR5213 it seems the rx timestamp is at the end of the | ||
1852 | * frame, but i'm not sure. | ||
1853 | * | ||
1854 | * NOTE: mac80211 defines mactime at the beginning of the first | ||
1855 | * data symbol. Since we don't have any time references it's | ||
1856 | * impossible to comply to that. This affects IBSS merge only | ||
1857 | * right now, so it's not too bad... | ||
1847 | */ | 1858 | */ |
1848 | rxs.mactime = ath5k_extend_tsf(sc->ah, rs.rs_tstamp); | 1859 | rxs.mactime = ath5k_extend_tsf(sc->ah, rs.rs_tstamp); |
1849 | rxs.flag |= RX_FLAG_TSFT; | 1860 | rxs.flag |= RX_FLAG_TSFT; |