On Mon, Jan 12, 2015 at 01:55:28PM +0200, Alexander Shishkin wrote: > Peter Chen <peter.c...@freescale.com> writes: > > > + mutex_lock(&udc_lock); > > + list_for_each_entry(udc, &udc_list, list) > > + if (udc->gadget == gadget) > > + goto found; > > + mutex_unlock(&udc_lock); > > Not entirely specific to this patchset, but this search pattern is > copied verbatim in udc-core.c like 5 times. Maybe it makes sense to make > it a separate function?
yeah, might be better to do that as patch 1 in this series. -- balbi
signature.asc
Description: Digital signature
