From: Felix Fietkau Date: Sat, 30 Apr 2016 10:52:29 +0700 Subject: [ath9k-devel] [PATCH] ath9k: Fix symbol overlap window for half/quarter channels In-Reply-To: <1461935194-19632-1-git-send-email-helmut.schaa@googlemail.com> References: <1461935194-19632-1-git-send-email-helmut.schaa@googlemail.com> Message-ID: <57242BFD.4060605@openwrt.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: ath9k-devel@lists.ath9k.org On 2016-04-29 20:06, Helmut Schaa wrote: > Since commit cd6cfd7311a385144a2f9c74f692ae2df3ae033f > "ath9k: do not set half/quarter channel flags in AR_PHY_MODE" the > condition "rfMode & (AR_PHY_MODE_QUARTER | AR_PHY_MODE_HALF)" would > never evaluate to true. > > Fix this by using the available IS_CHAN_HALF_RATE and IS_CHAN_QUARTER_RATE > marcros instead. > > Signed-off-by: Helmut Schaa > Cc: Felix Fietkau > --- > Just stumbled over that piece of code while looking into TX99, so > this is only compile-tested. > > Felix, can you please confirm if this is correct or if removing > the whole block would be better? This patch looks good to me. Acked-by: Felix Fietkau