Re: [PATCH 1/2] driver core: firmware loader: don't cache FW_ACTION_NOHOTPLUG firmware

2013-05-27 Thread anish singh
On Mon, May 27, 2013 at 6:52 PM, Ming Lei wrote: > On Mon, May 27, 2013 at 8:40 PM, anish singh > wrote: >> On Mon, May 27, 2013 at 5:53 PM, Takashi Iwai wrote: >>> At Mon, 27 May 2013 17:26:22 +0530, >>> anish singh wrote: On Mon, May 27, 2013 at 4:00 PM, Ming Lei wrote: > Gener

Re: [PATCH 1/2] driver core: firmware loader: don't cache FW_ACTION_NOHOTPLUG firmware

2013-05-27 Thread Ming Lei
On Mon, May 27, 2013 at 8:40 PM, anish singh wrote: > On Mon, May 27, 2013 at 5:53 PM, Takashi Iwai wrote: >> At Mon, 27 May 2013 17:26:22 +0530, >> anish singh wrote: >>> >>> On Mon, May 27, 2013 at 4:00 PM, Ming Lei wrote: >>> > Generally there are only two drivers which don't need uevent to >

Re: [PATCH 1/2] driver core: firmware loader: don't cache FW_ACTION_NOHOTPLUG firmware

2013-05-27 Thread anish singh
On Mon, May 27, 2013 at 5:53 PM, Takashi Iwai wrote: > At Mon, 27 May 2013 17:26:22 +0530, > anish singh wrote: >> >> On Mon, May 27, 2013 at 4:00 PM, Ming Lei wrote: >> > Generally there are only two drivers which don't need uevent to >> > handle firmware loading, so don't cache these firmwares

Re: [PATCH 1/2] driver core: firmware loader: don't cache FW_ACTION_NOHOTPLUG firmware

2013-05-27 Thread Takashi Iwai
At Mon, 27 May 2013 18:30:03 +0800, Ming Lei wrote: > > Generally there are only two drivers which don't need uevent to > handle firmware loading, so don't cache these firmwares during > suspend for these drivers since doing that may block firmware > loading forever. > > Both the two drivers are

Re: [PATCH 1/2] driver core: firmware loader: don't cache FW_ACTION_NOHOTPLUG firmware

2013-05-27 Thread Takashi Iwai
At Mon, 27 May 2013 17:26:22 +0530, anish singh wrote: > > On Mon, May 27, 2013 at 4:00 PM, Ming Lei wrote: > > Generally there are only two drivers which don't need uevent to > > handle firmware loading, so don't cache these firmwares during > Sorry but this statement confuses me i.e. "drivers w

Re: [PATCH 1/2] driver core: firmware loader: don't cache FW_ACTION_NOHOTPLUG firmware

2013-05-27 Thread anish singh
On Mon, May 27, 2013 at 4:00 PM, Ming Lei wrote: > Generally there are only two drivers which don't need uevent to > handle firmware loading, so don't cache these firmwares during Sorry but this statement confuses me i.e. "drivers which don't need uevent to handle firmware loading". Does this mean

[PATCH 1/2] driver core: firmware loader: don't cache FW_ACTION_NOHOTPLUG firmware

2013-05-27 Thread Ming Lei
Generally there are only two drivers which don't need uevent to handle firmware loading, so don't cache these firmwares during suspend for these drivers since doing that may block firmware loading forever. Both the two drivers are involved in private firmware images, so they don't hit in direct lo