On 03.12.23 20:33, Yury Norov wrote: > The function opencodes find_and_set_bit() with a for_each() loop. Use > it, and make the whole function a simple almost one-liner. > > While here, drop explicit initialization of *idx, because it's already > initialized by the caller in case of ENOLINK, or set properly with > ->wr_tx_mask, if nothing is found, in case of EBUSY. > > CC: Tony Lu <tonylu@xxxxxxxxxxxxxxxxx> > CC: Alexandra Winter <wintera@xxxxxxxxxxxxx> > Signed-off-by: Yury Norov <yury.norov@xxxxxxxxx> > --- Reviewed-by: Alexandra Winter <wintera@xxxxxxxxxxxxx> Thanks a lot for the great helper function! I guess the top-level maintainers will figure out, how this series best finds its way upstream.