On 4 October 2010 14:34, Vasanthakumar Thiagarajan <vasanth@xxxxxxxxxxx> wrote: >> + >> + /* prevent wraparound */ >> + if (ah->cc.cycles & BIT(31)) >> + clear = true; > > This does not look right, previous if should take care of > any wrap around. I was just in this area of the FreeBSD ath HAL code last week. I wonder why you don't just poll the values periodically and store them in a history array. Would you need to handle wrap-around if you're already polling the values every 100ms? Adrian -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
