wifi: carl9170: mac80211: document spatial multiplexing power save handler

Replace the bare TODO comment in the SMPS configuration handler
with documentation explaining why the driver accepts but does not
act on SMPS mode changes.

The AR9170 advertises SM_PS disabled (both chains always active)
in its HT capabilities.  While mac80211 may still send SMPS
configuration requests, implementing static or dynamic SMPS would
require firmware support for per-chain enable/disable that the
AR9170 firmware (v1.9.9) does not provide.

Signed-off-by: Masi Osmani <mas-i@hotmail.de>
Acked-by: Christian Lamparter <chunkeey@gmail.com>
Link: https://patch.msgid.link/AM7PPF5613FA0B6B76223A73FFC756A19A99444A@AM7PPF5613FA0B6.EURP251.PROD.OUTLOOK.COM
Signed-off-by: Jeff Johnson <jeff.johnson@oss.qualcomm.com>
This commit is contained in:
Masi Osmani 2026-03-12 11:37:58 +01:00 committed by Jeff Johnson
parent d6850f1984
commit a711bf00b8

View File

@ -908,7 +908,13 @@ static int carl9170_op_config(struct ieee80211_hw *hw, int radio_idx, u32 change
}
if (changed & IEEE80211_CONF_CHANGE_SMPS) {
/* TODO */
/*
* We advertise SM_PS disabled (all chains active).
* mac80211 may still request mode changes, which we
* accept but only support OFF (both chains active).
* Static/dynamic SMPS would require firmware support
* for chain control that the AR9170 does not provide.
*/
err = 0;
}