Re: [PATCH REGRESSION] Revert "ath10k: add quiet mode support for QCA6174/QCA9377"

From: Rajkumar Manoharan
Date: Wed Nov 07 2018 - 16:30:26 EST


On 2018-11-07 10:56, Brian Norris wrote:
This reverts commit cfb353c0dc058bc1619cc226d3cbbda1f360bdd3.

WCN3990 firmware does not yet implement this feature, and so it crashes
like this:

fatal error received: err_qdi.c:456:EX:wlan_process:1:WLAN RT:207a:PC=b001b4f0

This feature can be re-implemented with a proper service bitmap or other
feature-discovery mechanism in the future. But it should not break
working boards.

Brian,

The change "ath10k: add quiet mode support for QCA6174/QCA9377" was merged even
before full WCN3990 device support was added in ath10k. How come it could be regression
for WCN3990. I know both are sharing same WMI-TLV interface but reverting this
will break QCA6174/QCA9377. no?

I would prefer to handle this within WMI callback or upper layer.

-Rajkumar