On Fri, Mar 19, 2010 at 07:09:36PM +0900, Bruno Randolf wrote: > +++ b/drivers/net/wireless/ath/ath5k/phy.c > + if (time_is_before_eq_jiffies(ah->ah_cal_next_full)) { > + ah->ah_cal_next_full = jiffies + > + msecs_to_jiffies(ATH5K_TUNE_CALIBRATION_INTERVAL_FULL); > + ah->ah_cal_mask = AR5K_FULL_CALIBRATION; > + tasklet_schedule(&ah->ah_sc->calib); > + }
I didn't see it but we probably want to initialize ah_cal_next_full somewhere? Timer wrap-around will probably happen before we get here so it won't matter much, but might as well be explicit. -- Bob Copeland %% www.bobcopeland.com _______________________________________________ ath5k-devel mailing list ath5k-devel@lists.ath5k.org https://lists.ath5k.org/mailman/listinfo/ath5k-devel