> >> +int qtnf_cmd_start_cac(const struct qtnf_vif *vif,
> >> + const struct cfg80211_chan_def *chdef,
> >> + u32 cac_time_ms)
> >> +{
> >> + struct qtnf_bus *bus = vif->mac->bus;
> >> + struct sk_buff *cmd_skb;
> >> + struct qlink_cmd_start_cac *cmd;
> >> + int ret;
> >> + u16 res_code;
> >> +
> >> + cmd_skb = qtnf_cmd_alloc_new_cmdskb(vif->mac->macid, vif->vifid,
> >> + QLINK_CMD_START_CAC,
> >> + sizeof(*cmd));
> >> + if (unlikely(!cmd_skb))
> >> + return -ENOMEM;
> >
> > The unlikely() looks pretty useless here. likely() and unlikely() are
> > supposed to be used in cases where it matters and the improvment can be
> > measured, I doubt that's the case here.
Noted once again. Though it is not easy to get rid of some habits sometimes :)
Regards,
Sergey
