On Thu, Mar 16, 2017 at 02:55:09PM +0100, Rafał Miłecki wrote: > On 02/23/2017 07:30 PM, Rafał Miłecki wrote: > > From: Rafał Miłecki <rafal@xxxxxxxxxx> > > > > So far we got only one function for loading firmware asynchronously: > > request_firmware_nowait. It didn't allow much customization of firmware > > loading process - there is only one bool uevent argument. Moreover this > > bool also controls user helper in an unclear way. > > > > Resolve this problem by adding one internally shared function that > > allows specifying any flags manually. > > > > This implementation: > > 1) Allows keeping old request_firmware_nowait API unchanged > > 2) Doesn't require adjusting / rewriting current drivers > > 3) Minimizes risk of regressions > > 4) Adds new function for drivers that need more control over loading a > > firmware. > > > > The new function takes options struct pointer as an argument to make > > further improvements possible (without any big reworks). > > > > Signed-off-by: Rafał Miłecki <rafal@xxxxxxxxxx> > > --- > > V3: Don't expose all FW_OPT_* flags. > > As Luis noted we want a struct so add struct firmware_opts for real > > flexibility. > > Thank you Luis for your review! > > > > Ming/Luis/Greg: assuming this gets a positive review, could someone of you pick > > this patchset? > > Hi Ming, > > It seems you changed e-mail address somewhere between V3 and V4 of this > patchset. Could you review/comment/ack this, please? Ming isn't the firmware maintainer anymore :( > Luis: would you ack this patch now I followed your guidance? It's up to Luis now :) greg k-h
