Messages in this thread | | | Date | Thu, 16 Mar 2017 23:03:52 +0900 | From | Greg KH <> | Subject | Re: [PATCH V4 1/2] firmware: add more flexible request_firmware_async function |
| |
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@milecki.pl> > > > > 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@milecki.pl> > > --- > > 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
| |