Paths

Table of Contentst

Differential D48239

ath_hal_ar9300: quick refactor of tx/rx chain handling
ClosedPublic
Actions

Authored by adrian on Dec 29 2024, 5:28 AM.

Tags

Referenced Files

	F164587205: D48239.diff
	Sun, Aug 2, 5:34 AM

	Unknown Object (File)
	Sun, Jul 26, 6:08 AM

	Unknown Object (File)
	Sat, Jul 18, 1:00 PM

	Unknown Object (File)
	Sat, Jul 18, 8:18 AM

	Unknown Object (File)
	Jul 1 2026, 12:28 PM

	Unknown Object (File)
	Jul 1 2026, 12:13 PM

	Unknown Object (File)
	Jun 30 2026, 6:37 AM

	Unknown Object (File)
	Jun 30 2026, 3:40 AM

Subscribers

Details

Reviewers

Group Reviewers

Commits

rG18fabd338ce7: ath_hal_ar9300: quick refactor of tx/rx chain handling

Summary

Use a temporary variable, make the lines shorter.

Diff Detail

Repository

rG FreeBSD src repository

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 61382
Build 58266: arc lint + arc unit

Event Timeline

adrian created this revision.Dec 29 2024, 5:28 AM

Herald added a subscriber: imp. · View Herald TranscriptDec 29 2024, 5:28 AM

adrian requested review of this revision.Dec 29 2024, 5:28 AM

adrian added a parent revision: D48210: iwm: add sku_cap_mimo_disable; populate sku_cap_11n_enable.Dec 29 2024, 5:28 AM

Harbormaster completed remote builds in B61382: Diff 148508.Dec 29 2024, 5:28 AM

adrian added a child revision: D48240: ath_hal_ar9300: implement the TX/RX chainmask override for AR9300 HAL.Dec 29 2024, 5:29 AM

adrian added a reviewer: wireless.Dec 29 2024, 5:32 AM

adrian added a project: wireless.

emaste accepted this revision as: emaste.Dec 29 2024, 3:04 PM

This revision is now accepted and ready to land.Dec 29 2024, 3:04 PM

Perhaps p_cap matching the style apparent in D48240 though? Seems to be a mix of underscore and camelcase names in here.

oh yeah, ar9300_freebsd.c is a mess between the AR5416 camelcase and AR9300 "We wanted to land this in upstream linux so we renamed everything" linux case. :)

Closed by commit rG18fabd338ce7: ath_hal_ar9300: quick refactor of tx/rx chain handling (authored by adrian). · Explain WhyDec 31 2024, 1:36 AM

This revision was automatically updated to reflect the committed changes.

adrian added a commit: rG18fabd338ce7: ath_hal_ar9300: quick refactor of tx/rx chain handling.

adrian edited parent revisions, added: D48118: rtwn: set the maximum AMPDU size correctly for RTL8812AU/RTL8821AU; removed: D48210: iwm: add sku_cap_mimo_disable; populate sku_cap_11n_enable.Jan 9 2025, 5:07 PM

Revision Contents
Changeset List

Path

Size

sys/

contrib/

dev/

ath/

ath_hal/

ar9300/

ar9300_freebsd.c

7 lines

Diff 148508

sys/contrib/dev/ath/ath_hal/ar9300/ar9300_freebsd.c

Loading...