Re: [PATCH V5 1/2] firmware: add more flexible request_firmware_async function

2017-08-10 Thread Luis R. Rodriguez
On Thu, Aug 03, 2017 at 05:55:18AM +, Coelho, Luciano wrote: > On Thu, 2017-08-03 at 08:23 +0300, Kalle Valo wrote: > > "Luis R. Rodriguez" writes: > > > > > > +int request_firmware_nowait(struct module *module, bool uevent, > > > > + const char

Re: [PATCH V5 1/2] firmware: add more flexible request_firmware_async function

2017-08-10 Thread Luis R. Rodriguez
On Thu, Aug 03, 2017 at 05:55:18AM +, Coelho, Luciano wrote: > On Thu, 2017-08-03 at 08:23 +0300, Kalle Valo wrote: > > "Luis R. Rodriguez" writes: > > > > > > +int request_firmware_nowait(struct module *module, bool uevent, > > > > + const char *name, struct device

Re: [PATCH V5 1/2] firmware: add more flexible request_firmware_async function

2017-08-10 Thread Luis R. Rodriguez
On Thu, Aug 03, 2017 at 08:23:00AM +0300, Kalle Valo wrote: > "Luis R. Rodriguez" writes: > > >> +int request_firmware_nowait(struct module *module, bool uevent, > >> + const char *name, struct device *device, gfp_t gfp, > >> + void

Re: [PATCH V5 1/2] firmware: add more flexible request_firmware_async function

2017-08-10 Thread Luis R. Rodriguez
On Thu, Aug 03, 2017 at 08:23:00AM +0300, Kalle Valo wrote: > "Luis R. Rodriguez" writes: > > >> +int request_firmware_nowait(struct module *module, bool uevent, > >> + const char *name, struct device *device, gfp_t gfp, > >> + void *context, > >> +

Re: [PATCH V5 1/2] firmware: add more flexible request_firmware_async function

2017-08-03 Thread Luis R. Rodriguez
On Thu, Aug 03, 2017 at 05:55:18AM +, Coelho, Luciano wrote: > On Thu, 2017-08-03 at 08:23 +0300, Kalle Valo wrote: > > "Luis R. Rodriguez" writes: > > > > > > +int request_firmware_nowait(struct module *module, bool uevent, > > > > + const char

Re: [PATCH V5 1/2] firmware: add more flexible request_firmware_async function

2017-08-03 Thread Luis R. Rodriguez
On Thu, Aug 03, 2017 at 05:55:18AM +, Coelho, Luciano wrote: > On Thu, 2017-08-03 at 08:23 +0300, Kalle Valo wrote: > > "Luis R. Rodriguez" writes: > > > > > > +int request_firmware_nowait(struct module *module, bool uevent, > > > > + const char *name, struct device

Re: [PATCH V5 1/2] firmware: add more flexible request_firmware_async function

2017-08-02 Thread Coelho, Luciano
On Thu, 2017-08-03 at 08:23 +0300, Kalle Valo wrote: > "Luis R. Rodriguez" writes: > > > > +int request_firmware_nowait(struct module *module, bool uevent, > > > + const char *name, struct device *device, gfp_t gfp, > > > + void

Re: [PATCH V5 1/2] firmware: add more flexible request_firmware_async function

2017-08-02 Thread Coelho, Luciano
On Thu, 2017-08-03 at 08:23 +0300, Kalle Valo wrote: > "Luis R. Rodriguez" writes: > > > > +int request_firmware_nowait(struct module *module, bool uevent, > > > + const char *name, struct device *device, gfp_t gfp, > > > + void *context, > > > +

Re: [PATCH V5 1/2] firmware: add more flexible request_firmware_async function

2017-08-02 Thread Kalle Valo
"Luis R. Rodriguez" writes: >> +int request_firmware_nowait(struct module *module, bool uevent, >> +const char *name, struct device *device, gfp_t gfp, >> +void *context, >> +void (*cont)(const struct

Re: [PATCH V5 1/2] firmware: add more flexible request_firmware_async function

2017-08-02 Thread Kalle Valo
"Luis R. Rodriguez" writes: >> +int request_firmware_nowait(struct module *module, bool uevent, >> +const char *name, struct device *device, gfp_t gfp, >> +void *context, >> +void (*cont)(const struct firmware *fw, void >>

Re: [PATCH V5 1/2] firmware: add more flexible request_firmware_async function

2017-08-02 Thread Luis R. Rodriguez
On Mon, Jul 31, 2017 at 05:09:44PM +0200, Rafał Miłecki wrote: > From: Rafał Miłecki > > 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

Re: [PATCH V5 1/2] firmware: add more flexible request_firmware_async function

2017-08-02 Thread Luis R. Rodriguez
On Mon, Jul 31, 2017 at 05:09:44PM +0200, Rafał Miłecki wrote: > From: Rafał Miłecki > > 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.

Re: [PATCH V5 1/2] firmware: add more flexible request_firmware_async function

2017-07-31 Thread kbuild test robot
Hi Rafał, [auto build test WARNING on driver-core/driver-core-testing] [also build test WARNING on v4.13-rc3 next-20170731] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH V5 1/2] firmware: add more flexible request_firmware_async function

2017-07-31 Thread kbuild test robot
Hi Rafał, [auto build test WARNING on driver-core/driver-core-testing] [also build test WARNING on v4.13-rc3 next-20170731] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

[PATCH V5 1/2] firmware: add more flexible request_firmware_async function

2017-07-31 Thread Rafał Miłecki
From: Rafał Miłecki 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

[PATCH V5 1/2] firmware: add more flexible request_firmware_async function

2017-07-31 Thread Rafał Miłecki
From: Rafał Miłecki 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. Some drivers