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

	F166697656: D48102.id148630.diff
	Sat, Aug 15, 5:01 PM

	F166693083: D48102.id.diff
	Sat, Aug 15, 3:47 PM

	F166683476: D48102.id148711.diff
	Sat, Aug 15, 1:00 PM

	F166650672: D48102.id148937.diff
	Sat, Aug 15, 5:27 AM

	F166625281: D48102.diff
	Fri, Aug 14, 11:34 PM

	Unknown Object (File)
	Thu, Aug 13, 9:46 PM

	Unknown Object (File)
	Wed, Aug 12, 6:59 AM

	Unknown Object (File)
	Tue, Aug 11, 8:25 AM

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 61191
Build 58075: 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
96	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/

31 lines

Diff 148004

sys/dev/rtwn/rtl8812a/r12a_tx.c

Loading...