mbox series

[0/5,SRU,I/J/OEM-5.13/OEM-5.14] Add basic support of MT7922

Message ID 20220117161918.7324-1-aaron.ma@canonical.com
Headers show
Series Add basic support of MT7922 | expand

Message

Aaron Ma Jan. 17, 2022, 4:19 p.m. UTC
BugLink: https://bugs.launchpad.net/bugs/1958151

[Impact]
No driver is loaded for MT7922, and fw is missing too.

[Fix]
Add basic support of MT7922, and add ID of AMD RZ616/RZ608.
Enable 160Mhz channel.

[Test]
Verified on RZ616, it works fine, suspend OK.
iperf test result looks fine.
Bluetooth works fine.

[Where problems could occur]
It may break MT7921 wifi support which shares mt7921e driver.
Almost the code is specific for MT7922, checking security fw is based on
MT7922 fw.

Aaron Ma (1):
  Bluetooth: btusb: Add support for Foxconn MT7922A

Deren Wu (3):
  mt76: mt7921: Add mt7922 support
  mt76: mt7921: add support for PCIe ID 0x0608/0x0616
  mt76: mt7921: introduce 160 MHz channel bandwidth support

mark-yw.chen (1):
  Bluetooth: btusb: Add protocol for MediaTek bluetooth devices(MT7922)

 drivers/bluetooth/btusb.c                     |  9 +++
 .../net/wireless/mediatek/mt76/mt76_connac.h  |  7 +-
 .../wireless/mediatek/mt76/mt7921/eeprom.c    |  1 +
 .../net/wireless/mediatek/mt76/mt7921/init.c  |  4 ++
 .../net/wireless/mediatek/mt76/mt7921/main.c  | 12 ++++
 .../net/wireless/mediatek/mt76/mt7921/mcu.c   | 71 +++++++++++++++++--
 .../wireless/mediatek/mt76/mt7921/mt7921.h    |  3 +
 .../net/wireless/mediatek/mt76/mt7921/pci.c   |  5 ++
 8 files changed, 107 insertions(+), 5 deletions(-)

Comments

Tim Gardner Jan. 17, 2022, 5:19 p.m. UTC | #1
Acked-by: Tim Gardner <tim.gardner@canonical.com>

On 1/17/22 9:19 AM, Aaron Ma wrote:
> BugLink: https://bugs.launchpad.net/bugs/1958151
> 
> [Impact]
> No driver is loaded for MT7922, and fw is missing too.
> 
> [Fix]
> Add basic support of MT7922, and add ID of AMD RZ616/RZ608.
> Enable 160Mhz channel.
> 
> [Test]
> Verified on RZ616, it works fine, suspend OK.
> iperf test result looks fine.
> Bluetooth works fine.
> 
> [Where problems could occur]
> It may break MT7921 wifi support which shares mt7921e driver.
> Almost the code is specific for MT7922, checking security fw is based on
> MT7922 fw.
> 
> Aaron Ma (1):
>    Bluetooth: btusb: Add support for Foxconn MT7922A
> 
> Deren Wu (3):
>    mt76: mt7921: Add mt7922 support
>    mt76: mt7921: add support for PCIe ID 0x0608/0x0616
>    mt76: mt7921: introduce 160 MHz channel bandwidth support
> 
> mark-yw.chen (1):
>    Bluetooth: btusb: Add protocol for MediaTek bluetooth devices(MT7922)
> 
>   drivers/bluetooth/btusb.c                     |  9 +++
>   .../net/wireless/mediatek/mt76/mt76_connac.h  |  7 +-
>   .../wireless/mediatek/mt76/mt7921/eeprom.c    |  1 +
>   .../net/wireless/mediatek/mt76/mt7921/init.c  |  4 ++
>   .../net/wireless/mediatek/mt76/mt7921/main.c  | 12 ++++
>   .../net/wireless/mediatek/mt76/mt7921/mcu.c   | 71 +++++++++++++++++--
>   .../wireless/mediatek/mt76/mt7921/mt7921.h    |  3 +
>   .../net/wireless/mediatek/mt76/mt7921/pci.c   |  5 ++
>   8 files changed, 107 insertions(+), 5 deletions(-)
>
Timo Aaltonen Jan. 19, 2022, 11:42 a.m. UTC | #2
On 17.1.2022 18.19, Aaron Ma wrote:
> BugLink: https://bugs.launchpad.net/bugs/1958151
> 
> [Impact]
> No driver is loaded for MT7922, and fw is missing too.
> 
> [Fix]
> Add basic support of MT7922, and add ID of AMD RZ616/RZ608.
> Enable 160Mhz channel.
> 
> [Test]
> Verified on RZ616, it works fine, suspend OK.
> iperf test result looks fine.
> Bluetooth works fine.
> 
> [Where problems could occur]
> It may break MT7921 wifi support which shares mt7921e driver.
> Almost the code is specific for MT7922, checking security fw is based on
> MT7922 fw.
> 
> Aaron Ma (1):
>    Bluetooth: btusb: Add support for Foxconn MT7922A
> 
> Deren Wu (3):
>    mt76: mt7921: Add mt7922 support
>    mt76: mt7921: add support for PCIe ID 0x0608/0x0616
>    mt76: mt7921: introduce 160 MHz channel bandwidth support
> 
> mark-yw.chen (1):
>    Bluetooth: btusb: Add protocol for MediaTek bluetooth devices(MT7922)
> 
>   drivers/bluetooth/btusb.c                     |  9 +++
>   .../net/wireless/mediatek/mt76/mt76_connac.h  |  7 +-
>   .../wireless/mediatek/mt76/mt7921/eeprom.c    |  1 +
>   .../net/wireless/mediatek/mt76/mt7921/init.c  |  4 ++
>   .../net/wireless/mediatek/mt76/mt7921/main.c  | 12 ++++
>   .../net/wireless/mediatek/mt76/mt7921/mcu.c   | 71 +++++++++++++++++--
>   .../wireless/mediatek/mt76/mt7921/mt7921.h    |  3 +
>   .../net/wireless/mediatek/mt76/mt7921/pci.c   |  5 ++
>   8 files changed, 107 insertions(+), 5 deletions(-)
> 

applied to oem-5.14, thanks

patches 4 & 5 came via stable already
Timo Aaltonen Jan. 19, 2022, 6:50 p.m. UTC | #3
On 17.1.2022 18.19, Aaron Ma wrote:
> BugLink: https://bugs.launchpad.net/bugs/1958151
> 
> [Impact]
> No driver is loaded for MT7922, and fw is missing too.
> 
> [Fix]
> Add basic support of MT7922, and add ID of AMD RZ616/RZ608.
> Enable 160Mhz channel.
> 
> [Test]
> Verified on RZ616, it works fine, suspend OK.
> iperf test result looks fine.
> Bluetooth works fine.
> 
> [Where problems could occur]
> It may break MT7921 wifi support which shares mt7921e driver.
> Almost the code is specific for MT7922, checking security fw is based on
> MT7922 fw.
> 
> Aaron Ma (1):
>    Bluetooth: btusb: Add support for Foxconn MT7922A
> 
> Deren Wu (3):
>    mt76: mt7921: Add mt7922 support
>    mt76: mt7921: add support for PCIe ID 0x0608/0x0616
>    mt76: mt7921: introduce 160 MHz channel bandwidth support
> 
> mark-yw.chen (1):
>    Bluetooth: btusb: Add protocol for MediaTek bluetooth devices(MT7922)
> 
>   drivers/bluetooth/btusb.c                     |  9 +++
>   .../net/wireless/mediatek/mt76/mt76_connac.h  |  7 +-
>   .../wireless/mediatek/mt76/mt7921/eeprom.c    |  1 +
>   .../net/wireless/mediatek/mt76/mt7921/init.c  |  4 ++
>   .../net/wireless/mediatek/mt76/mt7921/main.c  | 12 ++++
>   .../net/wireless/mediatek/mt76/mt7921/mcu.c   | 71 +++++++++++++++++--
>   .../wireless/mediatek/mt76/mt7921/mt7921.h    |  3 +
>   .../net/wireless/mediatek/mt76/mt7921/pci.c   |  5 ++
>   8 files changed, 107 insertions(+), 5 deletions(-)
> 

The third commit doesn't apply to oem-5.13, is this series really needed 
there? It'll migrate to 5.14 next cycle.
Aaron Ma Jan. 20, 2022, 2:13 a.m. UTC | #4
Hi Timo,

Patch 3 is backported for 5.13, it's different from 5.14.
I tried the patch with subject [I/OEM-5.13][PATCH 3/5] successfully.

Also I think we can drop it for oem-5.13.

Aaron

On 1/20/22 02:50, Timo Aaltonen wrote:
> 
> The third commit doesn't apply to oem-5.13, is this series really needed there? It'll migrate to 5.14 next cycle.
Timo Aaltonen Jan. 20, 2022, 5:42 a.m. UTC | #5
On 20.1.2022 4.13, Aaron Ma wrote:
> Hi Timo,
> 
> Patch 3 is backported for 5.13, it's different from 5.14.
> I tried the patch with subject [I/OEM-5.13][PATCH 3/5] successfully.
> 
> Also I think we can drop it for oem-5.13.

Ok good, in that case I'll NAK it.
Stefan Bader Jan. 25, 2022, 8:31 a.m. UTC | #6
On 17.01.22 17:19, Aaron Ma wrote:
> BugLink: https://bugs.launchpad.net/bugs/1958151
> 
> [Impact]
> No driver is loaded for MT7922, and fw is missing too.
> 
> [Fix]
> Add basic support of MT7922, and add ID of AMD RZ616/RZ608.
> Enable 160Mhz channel.
> 
> [Test]
> Verified on RZ616, it works fine, suspend OK.
> iperf test result looks fine.
> Bluetooth works fine.
> 
> [Where problems could occur]
> It may break MT7921 wifi support which shares mt7921e driver.
> Almost the code is specific for MT7922, checking security fw is based on
> MT7922 fw.
> 
> Aaron Ma (1):
>    Bluetooth: btusb: Add support for Foxconn MT7922A
> 
> Deren Wu (3):
>    mt76: mt7921: Add mt7922 support
>    mt76: mt7921: add support for PCIe ID 0x0608/0x0616
>    mt76: mt7921: introduce 160 MHz channel bandwidth support
> 
> mark-yw.chen (1):
>    Bluetooth: btusb: Add protocol for MediaTek bluetooth devices(MT7922)
> 
>   drivers/bluetooth/btusb.c                     |  9 +++
>   .../net/wireless/mediatek/mt76/mt76_connac.h  |  7 +-
>   .../wireless/mediatek/mt76/mt7921/eeprom.c    |  1 +
>   .../net/wireless/mediatek/mt76/mt7921/init.c  |  4 ++
>   .../net/wireless/mediatek/mt76/mt7921/main.c  | 12 ++++
>   .../net/wireless/mediatek/mt76/mt7921/mcu.c   | 71 +++++++++++++++++--
>   .../wireless/mediatek/mt76/mt7921/mt7921.h    |  3 +
>   .../net/wireless/mediatek/mt76/mt7921/pci.c   |  5 ++
>   8 files changed, 107 insertions(+), 5 deletions(-)
> 
Acked-by: Stefan Bader <stefan.bader@canonical.com>
Stefan Bader Jan. 26, 2022, 3:52 p.m. UTC | #7
On 17.01.22 17:19, Aaron Ma wrote:
> BugLink: https://bugs.launchpad.net/bugs/1958151
> 
> [Impact]
> No driver is loaded for MT7922, and fw is missing too.
> 
> [Fix]
> Add basic support of MT7922, and add ID of AMD RZ616/RZ608.
> Enable 160Mhz channel.
> 
> [Test]
> Verified on RZ616, it works fine, suspend OK.
> iperf test result looks fine.
> Bluetooth works fine.
> 
> [Where problems could occur]
> It may break MT7921 wifi support which shares mt7921e driver.
> Almost the code is specific for MT7922, checking security fw is based on
> MT7922 fw.
> 
> Aaron Ma (1):
>    Bluetooth: btusb: Add support for Foxconn MT7922A
> 
> Deren Wu (3):
>    mt76: mt7921: Add mt7922 support
>    mt76: mt7921: add support for PCIe ID 0x0608/0x0616
>    mt76: mt7921: introduce 160 MHz channel bandwidth support
> 
> mark-yw.chen (1):
>    Bluetooth: btusb: Add protocol for MediaTek bluetooth devices(MT7922)
> 
>   drivers/bluetooth/btusb.c                     |  9 +++
>   .../net/wireless/mediatek/mt76/mt76_connac.h  |  7 +-
>   .../wireless/mediatek/mt76/mt7921/eeprom.c    |  1 +
>   .../net/wireless/mediatek/mt76/mt7921/init.c  |  4 ++
>   .../net/wireless/mediatek/mt76/mt7921/main.c  | 12 ++++
>   .../net/wireless/mediatek/mt76/mt7921/mcu.c   | 71 +++++++++++++++++--
>   .../wireless/mediatek/mt76/mt7921/mt7921.h    |  3 +
>   .../net/wireless/mediatek/mt76/mt7921/pci.c   |  5 ++
>   8 files changed, 107 insertions(+), 5 deletions(-)
> 

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

-Stefan
Paolo Pisati Feb. 11, 2022, 2:28 p.m. UTC | #8
On Tue, Jan 18, 2022 at 12:19:12AM +0800, Aaron Ma wrote:
> BugLink: https://bugs.launchpad.net/bugs/1958151

This patchset doesn't apply to Jammy HEAD, could you rebase and resend?

Thank you.
Stefan Bader Feb. 11, 2022, 2:57 p.m. UTC | #9
On 11.02.22 15:28, Paolo Pisati wrote:
> On Tue, Jan 18, 2022 at 12:19:12AM +0800, Aaron Ma wrote:
>> BugLink: https://bugs.launchpad.net/bugs/1958151
> 
> This patchset doesn't apply to Jammy HEAD, could you rebase and resend?
> 
> Thank you.

I would treat this as NACK for Jammy. All other series did complete. So when 
sending again, it only has to be for Jammy.

-Stefan