Page MenuHomeFreeBSD
Differential D49196

ath: fix 20MHz HT short-gi logic.
ClosedPublic
Actions

Authored by adrian on Sat, Mar 1, 10:36 PM.
Tags
Referenced Files
F111889844: D49196.diff
Sun, Mar 9, 9:16 PM
Unknown Object (File)
Thu, Mar 6, 7:20 AM
Unknown Object (File)
Wed, Mar 5, 11:03 PM
Unknown Object (File)
Mon, Mar 3, 8:49 PM
Unknown Object (File)
Mon, Mar 3, 8:49 PM
Unknown Object (File)
Mon, Mar 3, 8:49 PM
Unknown Object (File)
Mon, Mar 3, 8:49 PM
Unknown Object (File)
Mon, Mar 3, 6:08 PM
View All 22 Files
Subscribers
bz
imp

Details

Reviewers
bz
Group Reviewers
wireless
Commits
rG50ec1e7af002: ath: fix 20MHz HT short-gi logic.
Summary

This was broken in an earlier commit

(ca389486a9599768e0ba69dca13c208020623083).

PR: kern/285096

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 62718
Build 59602: arc lint + arc unit

Event Timeline

adrian created this revision.Sat, Mar 1, 10:36 PM
Herald added a subscriber: imp. · View Herald TranscriptSat, Mar 1, 10:36 PM
adrian requested review of this revision.Sat, Mar 1, 10:36 PM
Harbormaster completed remote builds in B62716: Diff 151726.Sat, Mar 1, 10:36 PM
adrian added a parent revision: D49190: rtwn: add support for AES-GCM-128 in the supported software cipher set.Sat, Mar 1, 10:37 PM
adrian added a reviewer: wireless.
adrian added a project: wireless.
adrian added a child revision: D49197: ath_rate_sample: fix setting HT rates.Sat, Mar 1, 10:41 PM
bz added a subscriber: bz.Sat, Mar 1, 10:54 PM

Commit message:

Fixes: <hash>

Seems we both missed it in main ...

adrian updated this revision to Diff 151728.Sat, Mar 1, 10:56 PM

update commit message

Harbormaster completed remote builds in B62718: Diff 151728.Sat, Mar 1, 10:56 PM
bz added a comment.Sat, Mar 1, 11:03 PM

This is unrelated to the PR mentioned btw as the PR is about stable/14 which is correct. This is a pure fix for main so adding a commit to the PR will be confusing if you ask me.

bz accepted this revision.Sat, Mar 1, 11:05 PM
In D49196#1122010, @adrian wrote:

update commit message

I cannot really see that (arc does not update it) but if you manage to have it correctly locally just commit it.

This revision is now accepted and ready to land.Sat, Mar 1, 11:05 PM
Closed by commit rG50ec1e7af002: ath: fix 20MHz HT short-gi logic. (authored by adrian). · Explain WhySat, Mar 1, 11:33 PM
This revision was automatically updated to reflect the committed changes.
adrian added a commit: rG50ec1e7af002: ath: fix 20MHz HT short-gi logic..

Revision Contents
Changeset List

PathSize
sys/
dev/
ath/
2 lines

Diff 151728

View Options

sys/dev/ath/if_ath_tx_ht.c

Loading...