On Mon, Nov 01, 2010 at 10:59:39PM -0200, Herton Ronaldo Krzesinski wrote: > Current 8187B initialization misses anaparam registers restore after > 8187 reset. This causes ANAPARAM register to stay zeroed out (ANAPARAM2 > kept its value on my tests). To avoid this, call rtl8187_set_anaparam > right after chip reset (to be on the safe side, as it makes sure we > restore all ANAPARAM registers). > Hello, Herton. Thank you very much for these patches. I am in the process of testing them right now. The first thing I've noticed is a drop in the signal level by 10dBm when using this last patch. Is this something we should be concerned with? I will do some tests with the distance to the Access Point and send my results later. Regards, Cascardo. > Signed-off-by: Herton Ronaldo Krzesinski <herton@xxxxxxxxxxxxxxx> > Acked-by: Larry Finger <Larry.Finger@xxxxxxxxxxxx> > Cc: seno <senada@xxxxxxxxxxx> > --- > drivers/net/wireless/rtl818x/rtl8187_dev.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/drivers/net/wireless/rtl818x/rtl8187_dev.c b/drivers/net/wireless/rtl818x/rtl8187_dev.c > index 4448647..eeee244 100644 > --- a/drivers/net/wireless/rtl818x/rtl8187_dev.c > +++ b/drivers/net/wireless/rtl818x/rtl8187_dev.c > @@ -771,6 +771,8 @@ static int rtl8187b_init_hw(struct ieee80211_hw *dev) > if (res) > return res; > > + rtl8187_set_anaparam(priv, true); > + > /* BRSR (Basic Rate Set Register) on 8187B looks to be the same as > * RESP_RATE on 8187L in Realtek sources: each bit should be each > * one of the 12 rates, all are enabled */ > -- > 1.7.3.2 > > -- > 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
Attachment:
signature.asc
Description: Digital signature
