ath79: wndr3700 series: fix wifi range & throughput
authorChristian Lamparter <chunkeey@gmail.com>
Sun, 7 Jun 2020 20:57:46 +0000 (22:57 +0200)
committerRISCi_ATOM <bob@bobcall.me>
Wed, 24 Jun 2020 14:15:10 +0000 (10:15 -0400)
This patch adds ar71xx's GPIO setup for the 2.4GHz and 5GHz antennae
demultiplexer:

| 158         /* 2.4 GHz uses the first fixed antenna group (1, 0, 1, 0) */
| 159         ap9x_pci_setup_wmac_gpio(0, (0xf << 6), (0xa << 6));
| 160
| 161         /* 5 GHz uses the second fixed antenna group (0, 1, 1, 0) */
| 162         ap9x_pci_setup_wmac_gpio(1, (0xf << 6), (0x6 << 6));

This should restore the range and throughput of the 2.4GHz radio
on all the derived wndr3700 variants and versions with the AR7161 SoC.
A special case is the 5GHz radio. The original wndr3700(v1) will
benefit from this change. However the wndr3700v2 and later revisions
were unaffected by the missing bits, as there is no demultiplexer
present in the later designs.

This patch uses gpio-hogs within the device-tree for all
wndr3700/wndr3800/wndrmac variants.

Notes:

Based on the PCB pictures, the WNDR3700(v1) really had eight
independent antennae. Four antennae for each radio and all of
those were printed on the circut board.

The WNDR3700v2 and later have just six antennae. Four of those
are printed on the circuit board and serve the 2.4GHz radio.
Whereas the remaining two are special 5GHz Rayspan Patch Antennae
which are directly connected to the 5GHz radio.

Hannu Nyman dug pretty deep and unearthed a treasure of information
regarding the history of how these values came to be in the OpenWrt
archives: <https://dev.archive.openwrt.org/ticket/6533.html>.

Mark Mentovai came across the fixed antenna group when he was looking
into the driver:

    fixed_antenna_group 1, (0, 1, 0, 1)
    fixed_antenna_group 2, (0, 1, 1, 0)
    fixed_antenna_group 3, (1, 0, 0, 1)
    fixed_antenna_group 4, (1, 0, 1, 0)

Fixes: FS#3088

Reported-by: Luca Bensi
Reported-by: Maciej Mazur
Reported-by: Hannu Nyman <hannu.nyman@iki.fi>
Debugged-by: Hannu Nyman <hannu.nyman@iki.fi>
Signed-off-by: Christian Lamparter <chunkeey@gmail.com>
(cherry picked from commit 61307544d1f1ab81a2eb3a200164456c59308d81)

target/linux/ath79/dts/ar7161_netgear_wndr3700.dts
target/linux/ath79/dts/ar7161_netgear_wndr3700.dtsi

index ac6f2362ecb224ca3d18541429a317c20acbb083..6bca300cceb1ef9580ab208dd390ef10797e8718 100644 (file)
                read-only;
        };
 };
+
+&ath9k1 {
+       /* The original WNDR3700(v1) variant have four antennae dedicated
+        * to the 5GHz radio as well. Again, two antennae are available for
+        * each chain to switch between. The following configuration is the
+        * default setting which taken from the vendor's wifi
+        * code for that radio.
+        *
+        * All possible options [GPIO6,GPIO7,GPIO8,GPIO9]:
+        *      [0,1,0,1], [0,1,1,0], [1,0,0,1], [1,0,1,0]
+        */
+       antenna-demux {
+               gpio-hog;
+               line-name = "fixed antenna group 2";
+               gpios = <6 GPIO_ACTIVE_LOW>,
+                       <7 GPIO_ACTIVE_HIGH>,
+                       <8 GPIO_ACTIVE_HIGH>,
+                       <9 GPIO_ACTIVE_LOW>;
+               output-high;
+       };
+};
index 779584e342e276eded79c9542a700efeed0aa572..8f701341a989fa5366949743e055a4eaf3e53d76 100644 (file)
                qca,no-eeprom;
                #gpio-cells = <2>;
                gpio-controller;
+
+               /* all WNDR3700 variants have four antennae dedicated
+                * to the 2.4GHz radio. Two antennae are available for
+                * each chain. The following configuration is the
+                * default setting which taken from the vendor's wifi
+                * code for that radio.
+                *
+                * All possible options [GPIO6,GPIO7,GPIO8,GPIO9]:
+                *      [0,1,0,1], [0,1,1,0], [1,0,0,1], [1,0,1,0]
+                */
+               antenna-demux {
+                       gpio-hog;
+                       line-name = "fixed antenna group 1";
+                       gpios = <6 GPIO_ACTIVE_LOW>,
+                               <7 GPIO_ACTIVE_HIGH>,
+                               <8 GPIO_ACTIVE_LOW>,
+                               <9 GPIO_ACTIVE_HIGH>;
+                       output-high;
+               };
        };
 
        ath9k1: wifi@0,12 {