[SRU,K,0/4] Mediatek WLAN RZ616(MT7922) SAR table control

Message ID	20221121055537.3796-1-aaron.ma@canonical.com
Headers	show Return-Path: <kernel-team-bounces@lists.ubuntu.com> From: Aaron Ma <aaron.ma@canonical.com> To: kernel-team@lists.ubuntu.com Subject: [SRU][K][PATCH 0/4] Mediatek WLAN RZ616(MT7922) SAR table control Date: Mon, 21 Nov 2022 13:55:33 +0800 Message-Id: <20221121055537.3796-1-aaron.ma@canonical.com> MIME-Version: 1.0 Precedence: list Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Errors-To: kernel-team-bounces@lists.ubuntu.com Sender: "kernel-team" <kernel-team-bounces@lists.ubuntu.com>
Series	Mediatek WLAN RZ616(MT7922) SAR table control \| expand [SRU,K,0/4] Mediatek WLAN RZ616(MT7922) SAR table control [K,1/4] mt76: add 6 GHz band support in mt76_sar_freq_ranges [K,2/4] mt76: mt7921: introduce ACPI SAR support [K,3/4] mt76: mt7921: introduce ACPI SAR config in tx power [K,4/4] wifi: mt76: mt7921: fix use after free in mt7921_acpi_read()

Message ID

20221121055537.3796-1-aaron.ma@canonical.com

Headers

From: Aaron Ma <aaron.ma@canonical.com>
To: kernel-team@lists.ubuntu.com
Subject: [SRU][K][PATCH 0/4] Mediatek WLAN RZ616(MT7922) SAR table control
Date: Mon, 21 Nov 2022 13:55:33 +0800
Message-Id: <20221121055537.3796-1-aaron.ma@canonical.com>
MIME-Version: 1.0
Precedence: list
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
Errors-To: kernel-team-bounces@lists.ubuntu.com
Sender: "kernel-team" <kernel-team-bounces@lists.ubuntu.com>

Series

Mediatek WLAN RZ616(MT7922) SAR table control | expand

Message

Aaron Ma Nov. 21, 2022, 5:55 a.m. UTC

BugLink: https://bugs.launchpad.net/bugs/1997200

[Impact]
In order to make sure the amount of RF energy being absorbed by our
bodies is safe according to the FCC’s guidelines, products must undergo
and pass SAR testing.

[Fix]
Add ACPI SAR table control to pass the testing.

[Test]
the unit is 0.5dBm in following:

Without the SAR table control:
$ sudo cat /sys/kernel/debug/ieee80211/phy0/mt76/txpower_sku | grep -i
user
CCK (user) : N.A N.A N.A N.A
OFDM (user) : 40 40 40 40 40 40 40 40
HT20 (user) : 40 40 40 40 40 40 40 40
HT40 (user) : 40 40 40 40 40 40 40 40 40
VHT20 (user) : 40 40 40 40 40 40 40 40 40 40 0 0
VHT40 (user) : 40 40 40 40 40 40 40 40 40 40 0 0
VHT80 (user) : 40 40 40 40 40 40 40 40 40 40 0 0
VHT160 (user) : 40 40 40 40 40 40 40 40 40 40 0 0
HE26 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
HE52 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
HE106 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
HE242 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
HE484 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
HE996 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
HE996x2 (user) : 40 40 40 40 40 40 40 40 40 40 40 40

After enabled SAR table control:
$ sudo cat /sys/kernel/debug/ieee80211/phy0/mt76/txpower_sku | grep -i
user
CCK (user) : N.A N.A N.A N.A
OFDM (user) : 26 26 26 26 26 26 26 26
HT20 (user) : 26 26 26 26 26 26 26 26
HT40 (user) : 26 26 26 26 26 26 26 26 26
VHT20 (user) : 26 26 26 26 26 26 26 26 26 26 0 0
VHT40 (user) : 26 26 26 26 26 26 26 26 26 26 0 0
VHT80 (user) : 26 26 26 26 26 26 26 26 26 26 0 0
VHT160 (user) : 26 26 26 26 26 26 26 26 26 26 0 0
HE26 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
HE52 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
HE106 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
HE242 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
HE484 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
HE996 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
HE996x2 (user) : 26 26 26 26 26 26 26 26 26 26 26 26

Also done stress test with iperf, all works fine.

[Where problems could occur]
It may break mt7922 driver.

These commits are already in oem-6.0 and unstable.

Dan Carpenter (1):
  wifi: mt76: mt7921: fix use after free in mt7921_acpi_read()

Deren Wu (3):
  mt76: add 6 GHz band support in mt76_sar_freq_ranges
  mt76: mt7921: introduce ACPI SAR support
  mt76: mt7921: introduce ACPI SAR config in tx power

 drivers/net/wireless/mediatek/mt76/mac80211.c |   6 +
 .../wireless/mediatek/mt76/mt7921/Makefile    |   1 +
 .../wireless/mediatek/mt76/mt7921/acpi_sar.c  | 282 ++++++++++++++++++
 .../wireless/mediatek/mt76/mt7921/acpi_sar.h  |  93 ++++++
 .../net/wireless/mediatek/mt76/mt7921/init.c  |   5 +-
 .../net/wireless/mediatek/mt76/mt7921/main.c  |  31 +-
 .../wireless/mediatek/mt76/mt7921/mt7921.h    |  22 ++
 7 files changed, 429 insertions(+), 11 deletions(-)
 create mode 100644 drivers/net/wireless/mediatek/mt76/mt7921/acpi_sar.c
 create mode 100644 drivers/net/wireless/mediatek/mt76/mt7921/acpi_sar.h

Comments

Tim Gardner Nov. 22, 2022, 7:20 p.m. UTC | #1

On 11/20/22 10:55 PM, Aaron Ma wrote:
> BugLink: https://bugs.launchpad.net/bugs/1997200
> 
> [Impact]
> In order to make sure the amount of RF energy being absorbed by our
> bodies is safe according to the FCC’s guidelines, products must undergo
> and pass SAR testing.
> 
> [Fix]
> Add ACPI SAR table control to pass the testing.
> 
> [Test]
> the unit is 0.5dBm in following:
> 
> Without the SAR table control:
> $ sudo cat /sys/kernel/debug/ieee80211/phy0/mt76/txpower_sku | grep -i
> user
> CCK (user) : N.A N.A N.A N.A
> OFDM (user) : 40 40 40 40 40 40 40 40
> HT20 (user) : 40 40 40 40 40 40 40 40
> HT40 (user) : 40 40 40 40 40 40 40 40 40
> VHT20 (user) : 40 40 40 40 40 40 40 40 40 40 0 0
> VHT40 (user) : 40 40 40 40 40 40 40 40 40 40 0 0
> VHT80 (user) : 40 40 40 40 40 40 40 40 40 40 0 0
> VHT160 (user) : 40 40 40 40 40 40 40 40 40 40 0 0
> HE26 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
> HE52 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
> HE106 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
> HE242 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
> HE484 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
> HE996 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
> HE996x2 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
> 
> After enabled SAR table control:
> $ sudo cat /sys/kernel/debug/ieee80211/phy0/mt76/txpower_sku | grep -i
> user
> CCK (user) : N.A N.A N.A N.A
> OFDM (user) : 26 26 26 26 26 26 26 26
> HT20 (user) : 26 26 26 26 26 26 26 26
> HT40 (user) : 26 26 26 26 26 26 26 26 26
> VHT20 (user) : 26 26 26 26 26 26 26 26 26 26 0 0
> VHT40 (user) : 26 26 26 26 26 26 26 26 26 26 0 0
> VHT80 (user) : 26 26 26 26 26 26 26 26 26 26 0 0
> VHT160 (user) : 26 26 26 26 26 26 26 26 26 26 0 0
> HE26 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
> HE52 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
> HE106 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
> HE242 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
> HE484 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
> HE996 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
> HE996x2 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
> 
> Also done stress test with iperf, all works fine.
> 
> [Where problems could occur]
> It may break mt7922 driver.
> 
> These commits are already in oem-6.0 and unstable.
> 
> Dan Carpenter (1):
>    wifi: mt76: mt7921: fix use after free in mt7921_acpi_read()
> 
> Deren Wu (3):
>    mt76: add 6 GHz band support in mt76_sar_freq_ranges
>    mt76: mt7921: introduce ACPI SAR support
>    mt76: mt7921: introduce ACPI SAR config in tx power
> 
>   drivers/net/wireless/mediatek/mt76/mac80211.c |   6 +
>   .../wireless/mediatek/mt76/mt7921/Makefile    |   1 +
>   .../wireless/mediatek/mt76/mt7921/acpi_sar.c  | 282 ++++++++++++++++++
>   .../wireless/mediatek/mt76/mt7921/acpi_sar.h  |  93 ++++++
>   .../net/wireless/mediatek/mt76/mt7921/init.c  |   5 +-
>   .../net/wireless/mediatek/mt76/mt7921/main.c  |  31 +-
>   .../wireless/mediatek/mt76/mt7921/mt7921.h    |  22 ++
>   7 files changed, 429 insertions(+), 11 deletions(-)
>   create mode 100644 drivers/net/wireless/mediatek/mt76/mt7921/acpi_sar.c
>   create mode 100644 drivers/net/wireless/mediatek/mt76/mt7921/acpi_sar.h
> 
Acked-by: Tim Gardner <tim.gardner@canonical.com>

Khalid Elmously Nov. 23, 2022, 1:47 a.m. UTC | #2

Acked-by: Khalid Elmously <khalid.elmously@canonical.com>


On 2022-11-21 13:55:33 , Aaron Ma wrote:
> BugLink: https://bugs.launchpad.net/bugs/1997200
> 
> [Impact]
> In order to make sure the amount of RF energy being absorbed by our
> bodies is safe according to the FCC’s guidelines, products must undergo
> and pass SAR testing.
> 
> [Fix]
> Add ACPI SAR table control to pass the testing.
> 
> [Test]
> the unit is 0.5dBm in following:
> 
> Without the SAR table control:
> $ sudo cat /sys/kernel/debug/ieee80211/phy0/mt76/txpower_sku | grep -i
> user
> CCK (user) : N.A N.A N.A N.A
> OFDM (user) : 40 40 40 40 40 40 40 40
> HT20 (user) : 40 40 40 40 40 40 40 40
> HT40 (user) : 40 40 40 40 40 40 40 40 40
> VHT20 (user) : 40 40 40 40 40 40 40 40 40 40 0 0
> VHT40 (user) : 40 40 40 40 40 40 40 40 40 40 0 0
> VHT80 (user) : 40 40 40 40 40 40 40 40 40 40 0 0
> VHT160 (user) : 40 40 40 40 40 40 40 40 40 40 0 0
> HE26 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
> HE52 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
> HE106 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
> HE242 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
> HE484 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
> HE996 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
> HE996x2 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
> 
> After enabled SAR table control:
> $ sudo cat /sys/kernel/debug/ieee80211/phy0/mt76/txpower_sku | grep -i
> user
> CCK (user) : N.A N.A N.A N.A
> OFDM (user) : 26 26 26 26 26 26 26 26
> HT20 (user) : 26 26 26 26 26 26 26 26
> HT40 (user) : 26 26 26 26 26 26 26 26 26
> VHT20 (user) : 26 26 26 26 26 26 26 26 26 26 0 0
> VHT40 (user) : 26 26 26 26 26 26 26 26 26 26 0 0
> VHT80 (user) : 26 26 26 26 26 26 26 26 26 26 0 0
> VHT160 (user) : 26 26 26 26 26 26 26 26 26 26 0 0
> HE26 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
> HE52 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
> HE106 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
> HE242 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
> HE484 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
> HE996 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
> HE996x2 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
> 
> Also done stress test with iperf, all works fine.
> 
> [Where problems could occur]
> It may break mt7922 driver.
> 
> These commits are already in oem-6.0 and unstable.
> 
> Dan Carpenter (1):
>   wifi: mt76: mt7921: fix use after free in mt7921_acpi_read()
> 
> Deren Wu (3):
>   mt76: add 6 GHz band support in mt76_sar_freq_ranges
>   mt76: mt7921: introduce ACPI SAR support
>   mt76: mt7921: introduce ACPI SAR config in tx power
> 
>  drivers/net/wireless/mediatek/mt76/mac80211.c |   6 +
>  .../wireless/mediatek/mt76/mt7921/Makefile    |   1 +
>  .../wireless/mediatek/mt76/mt7921/acpi_sar.c  | 282 ++++++++++++++++++
>  .../wireless/mediatek/mt76/mt7921/acpi_sar.h  |  93 ++++++
>  .../net/wireless/mediatek/mt76/mt7921/init.c  |   5 +-
>  .../net/wireless/mediatek/mt76/mt7921/main.c  |  31 +-
>  .../wireless/mediatek/mt76/mt7921/mt7921.h    |  22 ++
>  7 files changed, 429 insertions(+), 11 deletions(-)
>  create mode 100644 drivers/net/wireless/mediatek/mt76/mt7921/acpi_sar.c
>  create mode 100644 drivers/net/wireless/mediatek/mt76/mt7921/acpi_sar.h
> 
> -- 
> 2.34.1
> 
> 
> -- 
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team

Stefan Bader Nov. 23, 2022, 9:06 a.m. UTC | #3

On 21.11.22 06:55, Aaron Ma wrote:
> BugLink: https://bugs.launchpad.net/bugs/1997200
> 
> [Impact]
> In order to make sure the amount of RF energy being absorbed by our
> bodies is safe according to the FCC’s guidelines, products must undergo
> and pass SAR testing.
> 
> [Fix]
> Add ACPI SAR table control to pass the testing.
> 
> [Test]
> the unit is 0.5dBm in following:
> 
> Without the SAR table control:
> $ sudo cat /sys/kernel/debug/ieee80211/phy0/mt76/txpower_sku | grep -i
> user
> CCK (user) : N.A N.A N.A N.A
> OFDM (user) : 40 40 40 40 40 40 40 40
> HT20 (user) : 40 40 40 40 40 40 40 40
> HT40 (user) : 40 40 40 40 40 40 40 40 40
> VHT20 (user) : 40 40 40 40 40 40 40 40 40 40 0 0
> VHT40 (user) : 40 40 40 40 40 40 40 40 40 40 0 0
> VHT80 (user) : 40 40 40 40 40 40 40 40 40 40 0 0
> VHT160 (user) : 40 40 40 40 40 40 40 40 40 40 0 0
> HE26 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
> HE52 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
> HE106 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
> HE242 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
> HE484 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
> HE996 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
> HE996x2 (user) : 40 40 40 40 40 40 40 40 40 40 40 40
> 
> After enabled SAR table control:
> $ sudo cat /sys/kernel/debug/ieee80211/phy0/mt76/txpower_sku | grep -i
> user
> CCK (user) : N.A N.A N.A N.A
> OFDM (user) : 26 26 26 26 26 26 26 26
> HT20 (user) : 26 26 26 26 26 26 26 26
> HT40 (user) : 26 26 26 26 26 26 26 26 26
> VHT20 (user) : 26 26 26 26 26 26 26 26 26 26 0 0
> VHT40 (user) : 26 26 26 26 26 26 26 26 26 26 0 0
> VHT80 (user) : 26 26 26 26 26 26 26 26 26 26 0 0
> VHT160 (user) : 26 26 26 26 26 26 26 26 26 26 0 0
> HE26 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
> HE52 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
> HE106 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
> HE242 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
> HE484 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
> HE996 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
> HE996x2 (user) : 26 26 26 26 26 26 26 26 26 26 26 26
> 
> Also done stress test with iperf, all works fine.
> 
> [Where problems could occur]
> It may break mt7922 driver.
> 
> These commits are already in oem-6.0 and unstable.
> 
> Dan Carpenter (1):
>    wifi: mt76: mt7921: fix use after free in mt7921_acpi_read()
> 
> Deren Wu (3):
>    mt76: add 6 GHz band support in mt76_sar_freq_ranges
>    mt76: mt7921: introduce ACPI SAR support
>    mt76: mt7921: introduce ACPI SAR config in tx power
> 
>   drivers/net/wireless/mediatek/mt76/mac80211.c |   6 +
>   .../wireless/mediatek/mt76/mt7921/Makefile    |   1 +
>   .../wireless/mediatek/mt76/mt7921/acpi_sar.c  | 282 ++++++++++++++++++
>   .../wireless/mediatek/mt76/mt7921/acpi_sar.h  |  93 ++++++
>   .../net/wireless/mediatek/mt76/mt7921/init.c  |   5 +-
>   .../net/wireless/mediatek/mt76/mt7921/main.c  |  31 +-
>   .../wireless/mediatek/mt76/mt7921/mt7921.h    |  22 ++
>   7 files changed, 429 insertions(+), 11 deletions(-)
>   create mode 100644 drivers/net/wireless/mediatek/mt76/mt7921/acpi_sar.c
>   create mode 100644 drivers/net/wireless/mediatek/mt76/mt7921/acpi_sar.h
> 

Applied to kinetic:linux/master-next. Thanks.

-Stefan