Send users mailing list submissions to
[email protected]
To subscribe or unsubscribe via the World Wide Web, visit
http://rt2x00.serialmonkey.com/mailman/listinfo/users_rt2x00.serialmonkey.com
or, via email, send a message with subject or body 'help' to
[email protected]
You can reach the person managing the list at
[email protected]
When replying, please edit your Subject line so it is more specific
than "Re: Contents of users digest..."
Today's Topics:
1. Re: [PATCH 00/19] rt2x00: add experimental support for RT3593
(Alex)
2. Re: [PATCH] rt2x00: cleanup indentation in rt2800.h (Paul Menzel)
3. Re: [PATCH] rt2x00: cleanup indentation in rt2800.h (Gabor Juhos)
4. AVM Fritz Stick N v2 has troubles loading the driver
([email protected])
5. [PATCH v2] rt2x00: rt2800lib: fix RF registers for
RT5390/RT5392 (Kevin Lo)
6. [PATCH v3] rt2x00: rt2800lib: remove duplicate rf_vals for
RF3053 (Kevin Lo)
7. Re: [PATCH v3] rt2x00: rt2800lib: remove duplicate rf_vals
for RF3053 (Gabor Juhos)
8. Re: rt2800usb heavu CPU Usage (John Balian)
----------------------------------------------------------------------
Message: 1
Date: Sat, 12 Oct 2013 05:53:08 +0000 (UTC)
From: Alex <[email protected]>
To: [email protected]
Subject: Re: [rt2x00-users] [PATCH 00/19] rt2x00: add experimental
support for RT3593
Message-ID: <[email protected]>
Content-Type: text/plain; charset=us-ascii
Hello guys,
I am far from the kernel development, but how can I test this?
For now I am using this package: https://aur.archlinux.org/packages/rt3593/
with lots of patches found in the internet.
This driver works well except one bug, which is explained here:
https://bugzilla.gnome.org/show_bug.cgi?id=700695
Best,
Alex
------------------------------
Message: 2
Date: Sat, 12 Oct 2013 11:41:16 +0200
From: Paul Menzel <[email protected]>
To: Gabor Juhos <[email protected]>
Cc: [email protected], [email protected]
Subject: Re: [rt2x00-users] [PATCH] rt2x00: cleanup indentation in
rt2800.h
Message-ID: <1381570876.22335.13.camel@mattotaupa>
Content-Type: text/plain; charset="utf-8"
Am Freitag, den 11.10.2013, 12:50 +0200 schrieb Gabor Juhos:
> Adjust whitespaces to move badly aligned constants
> to the right column.
Did you use a tool for that or did you do it manually?
[?]
Thanks,
Paul
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL:
<http://rt2x00.serialmonkey.com/pipermail/users_rt2x00.serialmonkey.com/attachments/20131012/79120105/attachment-0001.bin>
------------------------------
Message: 3
Date: Sat, 12 Oct 2013 12:15:28 +0200
From: Gabor Juhos <[email protected]>
To: Paul Menzel <[email protected]>
Cc: [email protected], [email protected]
Subject: Re: [rt2x00-users] [PATCH] rt2x00: cleanup indentation in
rt2800.h
Message-ID: <[email protected]>
Content-Type: text/plain; charset=UTF-8
2013.10.12. 11:41 keltez?ssel, Paul Menzel ?rta:
> Am Freitag, den 11.10.2013, 12:50 +0200 schrieb Gabor Juhos:
>> Adjust whitespaces to move badly aligned constants
>> to the right column.
>
> Did you use a tool for that or did you do it manually?
I did that manually.
-Gabor
------------------------------
Message: 4
Date: Sat, 12 Oct 2013 13:49:50 +0200 (CEST)
From: [email protected]
To: [email protected]
Subject: [rt2x00-users] AVM Fritz Stick N v2 has troubles loading the
driver
Message-ID:
<trinity-bc30fadd-a53f-4544-9419-e84fac1cf3ef-1381578590883@msvc023>
Content-Type: text/plain; charset=UTF-8
I recently tried to get my AVM Fritz Stick N v2 working under Kubuntu
13.04 with a new 3.12 kernel candidate from the mainline ppa after i
heard it was supported in this driver.
Here some diagnostic logs:
lsusb
Bus 001 Device 004: ID 057c:8501 AVM GmbH
uname -a
Linux greyback 3.12.0-031200rc2-generic #201309231935 SMP Mon Sep 23 23:37:12
UTC 2013 x86_64 x86_64 x86_64 GNU/Linux
Error from dmesg:
[ 23.011160] ieee80211 phy0: rt2x00_set_rt: Info - RT chipset 5592, rev 0222
detected
[ 23.048498] ieee80211 phy0: rt2x00usb_vendor_request: Error - Vendor Request
0x09 failed for offset 0x0000 with error -32
[ 23.048539] ieee80211 phy0: rt2x00lib_probe_dev: Error - Failed to allocate
device
[ 23.048586] rt2800usb: probe of 1-1.3:1.0 failed with error -32
[ 23.048641] usbcore: registered new interface driver rt2800usb
It is detected , driver modules are loaded according to lsmod.
What is going south here?
Regards
Sam
------------------------------
Message: 5
Date: Sat, 12 Oct 2013 23:25:23 +0800
From: Kevin Lo <[email protected]>
To: John Linville <[email protected]>
Cc: [email protected], [email protected]
Subject: [rt2x00-users] [PATCH v2] rt2x00: rt2800lib: fix RF registers
for RT5390/RT5392
Message-ID: <[email protected]>
Content-Type: text/plain; charset=us-ascii
Update rf registers to use the same values that the MediaTek/Ralink
reference driver DPO_RT5572_LinuxSTA_2.6.1.3_20121022 uses.
References:
RF5390RegTable in chips/rt5390.c
RF5392RegTable in chips/rt5390.c
Tested on TP-Link TL-WN727N and D-Link DWA-140 Rev.b3 usb wifi dongles.
Signed-off-by: Kevin Lo <[email protected]>
Acked-by: Stanislaw Gruszka <[email protected]>
---
Changes since v1:
- the previous patch was malformed
- add Stanislaw's Acked-by tag
---
diff --git a/drivers/net/wireless/rt2x00/rt2800lib.c
b/drivers/net/wireless/rt2x00/rt2800lib.c
index a114cab..373efde 100644
--- a/drivers/net/wireless/rt2x00/rt2800lib.c
+++ b/drivers/net/wireless/rt2x00/rt2800lib.c
@@ -6476,7 +6476,7 @@ static void rt2800_init_rfcsr_5390(struct rt2x00_dev
*rt2x00dev)
rt2800_rfcsr_write(rt2x00dev, 28, 0x00);
rt2800_rfcsr_write(rt2x00dev, 29, 0x10);
- rt2800_rfcsr_write(rt2x00dev, 30, 0x00);
+ rt2800_rfcsr_write(rt2x00dev, 30, 0x10);
rt2800_rfcsr_write(rt2x00dev, 31, 0x80);
rt2800_rfcsr_write(rt2x00dev, 32, 0x80);
rt2800_rfcsr_write(rt2x00dev, 33, 0x00);
@@ -6514,7 +6514,7 @@ static void rt2800_init_rfcsr_5390(struct rt2x00_dev
*rt2x00dev)
rt2800_rfcsr_write(rt2x00dev, 56, 0x22);
rt2800_rfcsr_write(rt2x00dev, 57, 0x80);
rt2800_rfcsr_write(rt2x00dev, 58, 0x7f);
- rt2800_rfcsr_write(rt2x00dev, 59, 0x63);
+ rt2800_rfcsr_write(rt2x00dev, 59, 0x8f);
rt2800_rfcsr_write(rt2x00dev, 60, 0x45);
if (rt2x00_rt_rev_gte(rt2x00dev, RT5390, REV_RT5390F))
@@ -6534,7 +6534,6 @@ static void rt2800_init_rfcsr_5392(struct rt2x00_dev
*rt2x00dev)
rt2800_rf_init_calibration(rt2x00dev, 2);
rt2800_rfcsr_write(rt2x00dev, 1, 0x17);
- rt2800_rfcsr_write(rt2x00dev, 2, 0x80);
rt2800_rfcsr_write(rt2x00dev, 3, 0x88);
rt2800_rfcsr_write(rt2x00dev, 5, 0x10);
rt2800_rfcsr_write(rt2x00dev, 6, 0xe0);
------------------------------
Message: 6
Date: Sat, 12 Oct 2013 23:27:35 +0800
From: Kevin Lo <[email protected]>
To: John Linville <[email protected]>
Cc: [email protected], [email protected]
Subject: [rt2x00-users] [PATCH v3] rt2x00: rt2800lib: remove duplicate
rf_vals for RF3053
Message-ID: <[email protected]>
Content-Type: text/plain; charset=us-ascii
We already have rf_vals_3x with same values. Hence rf_vals_3053 is removed
in this patch.
Signed-off-by: Kevin Lo <[email protected]>
Acked-by: Paul Menzel <[email protected]>
Acked-by: Stanislaw Gruszka <[email protected]>
---
Changes since v1:
- update comment of rf_vals_3x to indicate that it also supports RF3053
- add Paul's Acked-by tag
---
Changes since v2:
- the previous patch was malformed
- add Stanislaw's Acked-by tag
---
diff --git a/drivers/net/wireless/rt2x00/rt2800lib.c
b/drivers/net/wireless/rt2x00/rt2800lib.c
index a114cab..c755309 100644
--- a/drivers/net/wireless/rt2x00/rt2800lib.c
+++ b/drivers/net/wireless/rt2x00/rt2800lib.c
@@ -7248,7 +7248,7 @@ static const struct rf_channel rf_vals[] = {
/*
* RF value list for rt3xxx
- * Supports: 2.4 GHz (all) & 5.2 GHz (RF3052)
+ * Supports: 2.4 GHz (all) & 5.2 GHz (RF3052 & RF3053)
*/
static const struct rf_channel rf_vals_3x[] = {
{1, 241, 2, 2 },
@@ -7444,72 +7444,6 @@ static const struct rf_channel rf_vals_5592_xtal40[] = {
{196, 83, 0, 12, 1},
};
-static const struct rf_channel rf_vals_3053[] = {
- /* Channel, N, R, K */
- {1, 241, 2, 2},
- {2, 241, 2, 7},
- {3, 242, 2, 2},
- {4, 242, 2, 7},
- {5, 243, 2, 2},
- {6, 243, 2, 7},
- {7, 244, 2, 2},
- {8, 244, 2, 7},
- {9, 245, 2, 2},
- {10, 245, 2, 7},
- {11, 246, 2, 2},
- {12, 246, 2, 7},
- {13, 247, 2, 2},
- {14, 248, 2, 4},
-
- {36, 0x56, 0, 4},
- {38, 0x56, 0, 6},
- {40, 0x56, 0, 8},
- {44, 0x57, 0, 0},
- {46, 0x57, 0, 2},
- {48, 0x57, 0, 4},
- {52, 0x57, 0, 8},
- {54, 0x57, 0, 10},
- {56, 0x58, 0, 0},
- {60, 0x58, 0, 4},
- {62, 0x58, 0, 6},
- {64, 0x58, 0, 8},
-
- {100, 0x5B, 0, 8},
- {102, 0x5B, 0, 10},
- {104, 0x5C, 0, 0},
- {108, 0x5C, 0, 4},
- {110, 0x5C, 0, 6},
- {112, 0x5C, 0, 8},
-
- /* NOTE: Channel 114 has been removed intentionally.
- * The EEPROM contains no TX power values for that,
- * and it is disabled in the vendor driver as well.
- */
-
- {116, 0x5D, 0, 0},
- {118, 0x5D, 0, 2},
- {120, 0x5D, 0, 4},
- {124, 0x5D, 0, 8},
- {126, 0x5D, 0, 10},
- {128, 0x5E, 0, 0},
- {132, 0x5E, 0, 4},
- {134, 0x5E, 0, 6},
- {136, 0x5E, 0, 8},
- {140, 0x5F, 0, 0},
-
- {149, 0x5F, 0, 9},
- {151, 0x5F, 0, 11},
- {153, 0x60, 0, 1},
- {157, 0x60, 0, 5},
- {159, 0x60, 0, 7},
- {161, 0x60, 0, 9},
- {165, 0x61, 0, 1},
- {167, 0x61, 0, 3},
- {169, 0x61, 0, 5},
- {171, 0x61, 0, 7},
- {173, 0x61, 0, 9},
-};
-
static int rt2800_probe_hw_mode(struct rt2x00_dev *rt2x00dev)
{
struct hw_mode_spec *spec = &rt2x00dev->spec;
@@ -7599,14 +7533,11 @@ static int rt2800_probe_hw_mode(struct rt2x00_dev
*rt2x00dev)
rt2x00_rf(rt2x00dev, RF5392)) {
spec->num_channels = 14;
spec->channels = rf_vals_3x;
- } else if (rt2x00_rf(rt2x00dev, RF3052)) {
+ } else if (rt2x00_rf(rt2x00dev, RF3052) ||
+ rt2x00_rf(rt2x00dev, RF3053)) {
spec->supported_bands |= SUPPORT_BAND_5GHZ;
spec->num_channels = ARRAY_SIZE(rf_vals_3x);
spec->channels = rf_vals_3x;
- } else if (rt2x00_rf(rt2x00dev, RF3053)) {
- spec->supported_bands |= SUPPORT_BAND_5GHZ;
- spec->num_channels = ARRAY_SIZE(rf_vals_3053);
- spec->channels = rf_vals_3053;
} else if (rt2x00_rf(rt2x00dev, RF5592)) {
spec->supported_bands |= SUPPORT_BAND_5GHZ;
------------------------------
Message: 7
Date: Sat, 12 Oct 2013 18:20:29 +0200
From: Gabor Juhos <[email protected]>
To: Kevin Lo <[email protected]>
Cc: [email protected], [email protected]
Subject: Re: [rt2x00-users] [PATCH v3] rt2x00: rt2800lib: remove
duplicate rf_vals for RF3053
Message-ID: <[email protected]>
Content-Type: text/plain; charset=ISO-8859-1
2013.10.12. 17:27 keltez?ssel, Kevin Lo ?rta:
> We already have rf_vals_3x with same values. Hence rf_vals_3053 is removed
> in this patch.
>
> Signed-off-by: Kevin Lo <[email protected]>
> Acked-by: Paul Menzel <[email protected]>
> Acked-by: Stanislaw Gruszka <[email protected]>
> ---
> Changes since v1:
> - update comment of rf_vals_3x to indicate that it also supports RF3053
> - add Paul's Acked-by tag
> ---
> Changes since v2:
> - the previous patch was malformed
> - add Stanislaw's Acked-by tag
> ---
>
> diff --git a/drivers/net/wireless/rt2x00/rt2800lib.c
> b/drivers/net/wireless/rt2x00/rt2800lib.c
<...>
> struct hw_mode_spec *spec = &rt2x00dev->spec;
> @@ -7599,14 +7533,11 @@ static int rt2800_probe_hw_mode(struct rt2x00_dev
> *rt2x00dev)
> rt2x00_rf(rt2x00dev, RF5392)) {
> spec->num_channels = 14;
> spec->channels = rf_vals_3x;
> - } else if (rt2x00_rf(rt2x00dev, RF3052)) {
> + } else if (rt2x00_rf(rt2x00dev, RF3052) ||
> + rt2x00_rf(rt2x00dev, RF3053)) {
checpatch.pl says this:
ERROR: trailing whitespace
#127: FILE: drivers/net/wireless/rt2x00/rt2800lib.c:7537:
+^I^I rt2x00_rf(rt2x00dev, RF3053)) { $
total: 1 errors, 0 warnings, 96 lines checked
Otherwise the patch is ok.
-Gabor
------------------------------
Message: 8
Date: Wed, 9 Oct 2013 10:39:57 -0400
From: John Balian <[email protected]>
To: Helmut Schaa <[email protected]>
Cc: rt2x00 Users List <[email protected]>
Subject: Re: [rt2x00-users] rt2800usb heavu CPU Usage
Message-ID:
<CALPEBBPOnoZRYbXhm0nv7+5Gfpm9Pvh59DP11SCfzf3=a2k...@mail.gmail.com>
Content-Type: text/plain; charset="iso-8859-1"
Helmut,
Thank you for your reply.
I tried the patch, but it does not help with the CPU usage. I've also
confirmed that the original rt2x00queue_insert_l2pad never needs to call
the memmove() for the payload_length with the data I am transmitting, so
that probably explains why this patch doesn't help in this case.
Do you know of any other patches or fixes that might help with CPU usage?
John
(Sorry for the type-o in the subject line!)
-----------------------------------------------
John Balian
[email protected]
On Wed, Oct 9, 2013 at 5:41 AM, Helmut Schaa <[email protected]>wrote:
> Hi,
>
> [CC [email protected]]
>
> On Tue, Oct 8, 2013 at 7:11 PM, John Balian <[email protected]> wrote:
> > I realize this thread is probably ancient history...
> >
> >
> http://rt2x00.serialmonkey.com/pipermail/users_rt2x00.serialmonkey.com/2011-September/004201.html
> >
> > but I'm having trouble with the rt2800usb driver consuming almost 100%
> cpu
> > while transmitting lots of data. Did you ever come up with a solution
> for
> > this type of problem? Do you have any tips on what I could try to
> improve
> > CPU usage?
>
> You can find a current version of the patch in the openwrt repo [1].
> Run some tests and let us know if it helps in your case as well.
>
> Helmut
>
> [1]
> http://nbd.name/gitweb.cgi?p=openwrt.git;a=blob;f=package/kernel/mac80211/patches/606-rt2x00_no_realign.patch;h=9608df2e20894dafc76623262dbf606e9c4c20f8;hb=HEAD
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://rt2x00.serialmonkey.com/pipermail/users_rt2x00.serialmonkey.com/attachments/20131009/fcfe1421/attachment-0001.html>
------------------------------
Subject: Digest Footer
_______________________________________________
users mailing list
[email protected]
http://rt2x00.serialmonkey.com/mailman/listinfo/users_rt2x00.serialmonkey.com
------------------------------
End of users Digest, Vol 56, Issue 16
*************************************