On Tue, 2022-05-24 at 14:33 +0300, viktor.barna@xxxxxxxxxx wrote:
>
> + if (!ieee80211_hw_check(cl_hw->hw, AMPDU_AGGREGATION) || !cl_hw->conf->ci_agg_tx)
> + return -ENOTSUPP;
> +
> + if (!cl_txq_is_agg_available(cl_hw)) {
> + cl_dbg_warn(cl_hw, "No free aggregation queue for sta_idx [%u] tid [%u]\n",
> + cl_sta->sta_idx, tid);
> + return -1;
>
That'd be -EPERM, which is really not what you want to return - please
use symbolic errors throughout. Maybe -ENOSPC in this case.
johannes
