On Mon, 4 Sep 2023 12:56:34 +0800 <[email protected]> wrote: > +void * > +sssnic_workq_consume(struct sssnic_workq *workq, uint16_t num_entries, > + uint16_t *ci) > +{ > + void *e; > + uint16_t current_ci; > + > + if (workq->idle_entries + num_entries > workq->num_entries) > + return NULL; > + > + current_ci = sssnic_workq_ci_get(workq); > + e = (void *)sssnic_workq_entry_get(workq, current_ci);
Minor nit: cast to void * is not normally necessary in C. Compiler will do it for you.

