Paths

Table of Contentst

Differential D48102

rtwn: add VHT20/VHT40/VHT80 bandwidth configuration for transmit.
ClosedPublic
Actions

Authored by adrian on Dec 16 2024, 2:36 AM.

Tags

Referenced Files

	F162280565: D48102.diff
	Sat, Jul 11, 4:10 PM

	Unknown Object (File)
	Wed, Jul 1, 2:37 AM

	Unknown Object (File)
	Tue, Jun 30, 5:24 PM

	Unknown Object (File)
	Tue, Jun 30, 3:54 AM

	Unknown Object (File)
	Mon, Jun 29, 11:18 AM

	Unknown Object (File)
	Sat, Jun 27, 12:36 PM

	Unknown Object (File)
	Thu, Jun 25, 10:49 PM

	Unknown Object (File)
	Wed, Jun 24, 8:56 PM

Subscribers

Details

Reviewers

None

Group Reviewers

Commits

rG82182587bcc3: rtwn: add VHT20/VHT40/VHT80 bandwidth configuration for transmit.

Summary

Add a separate function and path for VHT 20/40/80MHz bandwidth
transmission.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 61440
Build 58324: arc lint + arc unit

Event Timeline

adrian created this revision.Dec 16 2024, 2:36 AM

Herald added a subscriber: imp. · View Herald TranscriptDec 16 2024, 2:36 AM

adrian requested review of this revision.Dec 16 2024, 2:36 AM

Harbormaster completed remote builds in B61191: Diff 148004.Dec 16 2024, 2:36 AM

adrian added a parent revision: D48101: net80211: add helper functions for VHT transmit.Dec 16 2024, 2:36 AM

adrian added a reviewer: wireless.

adrian added a project: wireless.

adrian added a child revision: D48103: rtwn: announce VHT support for RTL8812AU/RTL8821AU..Dec 16 2024, 4:18 AM

Looks reasonable.

sys/dev/rtwn/rtl8812a/r12a_tx.c
106	Redundant empty line?

feedback from fuz, cleanup, comment

Harbormaster completed remote builds in B61440: Diff 148630.Dec 31 2024, 9:40 PM

I'll wait with this one as D48101 will likely change the KPI.

update!

Harbormaster completed remote builds in B61473: Diff 148711.Jan 3 2025, 12:45 AM

rebase

Harbormaster completed remote builds in B61577: Diff 148937.Jan 8 2025, 6:24 PM

emaste added a subscriber: emaste.Jan 8 2025, 7:36 PM

emaste added inline comments.

sys/dev/rtwn/rtl8812a/r12a_tx.c
73	Out of curiosity is there a reason someone would assume otherwise? I don't have all of the context but I would assume `...tx_set_vht_bw` is called for vht.

This revision was not accepted when it landed; it landed in state Needs Review.Jan 9 2025, 1:00 AM

Closed by commit rG82182587bcc3: rtwn: add VHT20/VHT40/VHT80 bandwidth configuration for transmit. (authored by adrian). · Explain Why

This revision was automatically updated to reflect the committed changes.

adrian added a commit: rG82182587bcc3: rtwn: add VHT20/VHT40/VHT80 bandwidth configuration for transmit..

Revision Contents
Changeset List

Path

Size

sys/

dev/

rtwn/

rtl8812a/

41 lines

Diff 148630

sys/dev/rtwn/rtl8812a/r12a_tx.c

Loading...