firmware loader: fix comments on request_firmware_nowait

request_firmware_nowait is allowed to be called in atomic
context now if @gfp is GFP_ATOMIC, so fix the obsolete
comments and states which situations are suitable for using
it.

Signed-off-by: Ming Lei <ming.lei@canonical.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

1 files changed, 7 insertions, 3 deletions

diff --git a/drivers/base/firmware_class.c b/drivers/base/firmware_class.c
index 7d3a83bb1318..a47266ccfc60 100644
--- a/drivers/base/firmware_class.c
+++ b/drivers/base/firmware_class.c
@@ -846,9 +846,13 @@ static void request_firmware_work_func(struct work_struct *work)
  *
  *      Caller must hold the reference count of @device.
  *
- *      Asynchronous variant of request_firmware() for user contexts where
- *      it is not possible to sleep for long time. It can't be called
- *      in atomic contexts.
+ *      Asynchronous variant of request_firmware() for user contexts:
+ *              - sleep for as small periods as possible since it may
+ *              increase kernel boot time of built-in device drivers
+ *              requesting firmware in their ->probe() methods, if
+ *              @gfp is GFP_KERNEL.
+ *
+ *              - can't sleep at all if @gfp is GFP_ATOMIC.
  **/
 int
 request_firmware_nowait(