From: Stanislaw Gruszka <sgruszka@redhat.com>
To: "Alex A. Mihaylov" <minimumlaw@gmail.com>
Cc: linux-wireless@vger.kernel.org, users@rt2x00.serialmonkey.com
Subject: Re: RFC: RT3070 TX Power troubles
Date: Tue, 12 Mar 2013 13:44:42 +0100 [thread overview]
Message-ID: <20130312124441.GA2775@redhat.com> (raw)
In-Reply-To: <CALK7YaggBFjUwv_Pm_Zs_mtt6dRCFkHkukaeZKM2Y0tBwhs_gA@mail.gmail.com>
On Tue, Mar 12, 2013 at 03:59:09PM +0400, Alex A. Mihaylov wrote:
> 2013/3/12 Stanislaw Gruszka <sgruszka@redhat.com>:
> > On Sat, Mar 09, 2013 at 10:05:48AM +0400, Alex A. Mihaylov wrote:
> >> 2013/3/8 Stanislaw Gruszka <sgruszka@redhat.com>:
> > However I just realized that we probably do not set RF_R49 on vendor
> > driver for chips other than 5392. I thought we set RF_R49 in function
> > AsicGetAutoAgcOffset() for all chipsets handled by driver, but seems
> > I missed IS_RT5392(pAd) branch.
>
> Yes. I found code for modify RF_R49 and RF_R50 in
> AsicGetAutoArgcOffset() in vendor driver (called by AsicAdjustTxPower,
> called by MlmePeriodicExec). Name of called function
> RT30xxReadRFRegister(pAd, RF_R49, &RFValue) and
> RT30xxWriteRFRegister(pAd, RF_R49, RFValue) make me stuppid, and I
> can't see if (IS_RT5392(pAd)) above.
>
> But RT3070 based module WN8020 start work correctly with this modification.
I think I'm fine with the changes, but maybe they should go
to rt2800_config_txpower().
However, I'm still not sure if Ralink vendor driver initialize RF_R49
register. Perhaps it achieve good power characteristic on 3070 using
other registers?
> > Also could you please elaborate how did you measure TX power?
>
> I use ZX47-40-S+ power detector from http://www.minicircuits.com/
> connected to WN8020 antenna output, and 1K load at output of power
> detector connected to osciloscope.
> Direct pdf datasheet link http://217.34.103.131/pdfs/ZX47-40+.pdf
WOW :-)
> Os X with Ralink driver show about 580mV on 1K load (about 15dBm)
> regardless txpower setup in Ralink wireless utility. Vanilla 3.8.2
> driver show 860mV (about 4dBm) if tx power set to 20dBm with iwconfig
> wlan0 txpower 20 and 980mV(0dBm) if tx power set to zero with iwconfig
> wlan0 txpower 0. Patched driver show about 580mV (15dBm) after setup
> 20dBm txpower with iwconfig wlan0 txpower 20, and about 980mV(0dBm) if
> tx power set to zero with iwconfig wlan0 txpower 0. I send
> screenshorts from osciloscpe directly for you some later.
I don't know how to convert mV to dBm, but I believe you :-)
> Unfortunately, no resons to init RF49 in vanilla wireless code. At
> this moment I got critical analyse for my code, and still thinking.
Not sure if I understand that ...
> > BTW: please cc rt2x00 related patches/emails to users@rt2x00.serialmonkey.com
> > (I already do this for this email). List is moderated to non-subscribers,
> > so would be good if you subscribe there.
>
> Can you send me some instructions directly? I can't find this mail list.
http://rt2x00.serialmonkey.com/mailman/listinfo/users_rt2x00.serialmonkey.com
Thanks
Stanislaw
prev parent reply other threads:[~2013-03-12 12:44 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-05 16:00 RFC: RT3070 TX Power troubles Alex A. Mihaylov
2013-03-08 16:21 ` Stanislaw Gruszka
2013-03-09 6:05 ` Alex A. Mihaylov
2013-03-12 9:42 ` Stanislaw Gruszka
2013-03-12 11:59 ` Alex A. Mihaylov
2013-03-12 12:44 ` Stanislaw Gruszka [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20130312124441.GA2775@redhat.com \
--to=sgruszka@redhat.com \
--cc=linux-wireless@vger.kernel.org \
--cc=minimumlaw@gmail.com \
--cc=users@rt2x00.serialmonkey.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).