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

2017-03-27 Thread Luis R. Rodriguez
On Fri, Mar 17, 2017 at 06:29:36PM +0100, Luis R. Rodriguez wrote: > On Thu, Mar 16, 2017 at 11:03:52PM +0900, Greg KH wrote: > > On Thu, Mar 16, 2017 at 02:55:09PM +0100, Rafał Miłecki wrote: > > > Luis: would you ack this patch now I followed your guidance? > > > > It's up to Luis now :) > > I'

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

2017-03-17 Thread Luis R. Rodriguez
On Thu, Mar 16, 2017 at 11:03:52PM +0900, Greg KH wrote: > On Thu, Mar 16, 2017 at 02:55:09PM +0100, Rafał Miłecki wrote: > > Luis: would you ack this patch now I followed your guidance? > > It's up to Luis now :) I'm reviewing now! Luis

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

2017-03-16 Thread Greg KH
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 > > > > So far we got only one function for loading firmware asynchronously: > > request_firmware_nowait. It didn't allow much customization of firmware > > loadin

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

2017-03-16 Thread Rafał Miłecki
On 02/23/2017 07:30 PM, 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. Moreover this bool also control

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

2017-03-16 Thread Greg KH
On Thu, Mar 16, 2017 at 10:57:00AM +0100, Rafał Miłecki wrote: > On 23 February 2017 at 19:30, 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 > >

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

2017-03-16 Thread Rafał Miłecki
On 23 February 2017 at 19:30, 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. Moreover this > b

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

2017-02-23 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. Resolve this