2013.10.04. 16:38 keltezéssel, Stanislaw Gruszka írta: > On Thu, Oct 03, 2013 at 08:00:42PM +0200, Gabor Juhos wrote: >> Update the rt2800_get_default_vgc function to use the same VGC >> values that the DPO_RT5572_LinuxSTA_2.6.1.3_20121022 reference >> driver uses. >> >> References: >> RT35xx_ChipAGCAdjust in chips/rt35xx.c >> RT3593_R66_MID_LOW_SENS_GET macro in include/chip/rt3593.h >> RT3593_R66_NON_MID_LOW_SEMS_GET macro in include/chips/rt3593.h >> >> Signed-off-by: Gabor Juhos <juhosg@xxxxxxxxxxx> > > Acked-by: Stanislaw Gruszka <stf_xl@xxxxx> > >> @@ -4422,6 +4423,8 @@ static u8 rt2800_get_default_vgc(struct rt2x00_dev *rt2x00dev) >> } else { /* 5GHZ band */ >> if (rt2x00_rt(rt2x00dev, RT3572)) >> vgc = 0x22 + (rt2x00dev->lna_gain * 5) / 3; > > Looks RT3572 values should be also changed ? According to: > > if (pAd->CommonCfg.BBPCurrentBW == BW_20) > { > R66 = 0x32 + (lanGain*5) / 3; > if (Rssi > RSSI_FOR_MID_LOW_SENSIBILITY) > R66 += 0x10; > } > else > { > R66 = 0x3A + (lanGain*5)/3; > if (Rssi > RSSI_FOR_MID_LOW_SENSIBILITY) > R66 += 0x10; > } Yes. I even created a patch for that, but I forgot to add that into the series. Will send that separately. -Gabor -- 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