On 5 February 2012 23:10, Rajkumar Manoharan <rmanohar@xxxxxxxxxxxxxxxx> wrote: >> I'd like to suggest we hold off on committing anything that really >> changes the ANI behaviour without figuring out exactly what's going on >> underneath the hood. I think it's worthwhile filing a bugzilla report >> and stuffing that patch against the bug though, so it's not "lost". >> [snip] > To be very precise, there are two types of weak signal detection logic in our > baseband - > > 1) normal weak signal detection logic - involves thresholds m1_thresh, > m2_thresh, m2_Count_thr > 2) low weak signal detection logic - involves thresholds m1_thresh_low, > m2_thresh_low, m2_count_thr_low > > Type 2 is intended to handle low weak signals and internally baseband logic does > averaging over larger time windows compared to type 1. In current ANI code, > both types 1) and 2) are clubbed together and called as "weak signal detection". > > What we've found in extensive testing is that disabling type 1 and type 2 form of > weak signal detection logic together causes "beacon miss". Disabling both types > 1 and type 2 together is definitely harmful to our receiver as it cannot receive > even medium strength signals. Disabling type 2 alone is completely safe which > is what we're doing in this change. > > For disabling type 2 logic alone there is a register field named > "use_self_corr_low". So instead of maxing out threshold values of 3-4 registers > we simply set use_self_corr_low=0 saving register writes. Hepe this could > clarify your question. This does, yes. I'll run this by the baseband guys just to see which chips this applies to and I'll get back to you. How'd you determine that it was these weak signal detection parameters? I've been thinking about writing an ANI (well, "baseband") visualisation tool for FreeBSD that just watches the error rates and a few other registers and plots things. That way we could watch how effective ANI is being over time. Thanks very much for chasing this up! 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
