> +     if (queue >= ARRAY_SIZE(wl->qos_params)) {
> +             /* Queue not available or don't support setting
> +              * params on this queue. Return success to not
> +              * confuse mac80211. */
> +             return 0;
> +     }

Is that really necessary? If so, I'll probably remove it when doing the
QoS rework, it shouldn't be, imho you should return an error if mac80211
tries to configure a queue >= hw.queues.

johannes

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Bcm43xx-dev mailing list
Bcm43xx-dev@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/bcm43xx-dev

Reply via email to