Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote: > The parameter order for clamp is supposed to be clamp(value, low, high). > When we write it in this order it's equivalent to > min(head->plcp[3] & 0x7f, 75) which works in this context where the min > is zero. But it's not a correct use of the API. > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > Acked-by: Christian Lamparter <chunkeey@xxxxxxxxx> Patch applied to wireless-drivers-next.git, thanks. dc76bb1e12e9 carl9170: clean up a clamp() call -- https://patchwork.kernel.org/patch/10817541/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
