staging: unisys: visorbus: update comment
authorDavid Kershner <david.kershner@unisys.com>
Tue, 18 Apr 2017 20:55:05 +0000 (16:55 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 28 Apr 2017 09:49:52 +0000 (11:49 +0200)
Comment was outdated and did not reflect what was actually happening.
Update the comment to reflect reality.

Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: Tim Sell <timothy.sell@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/unisys/visorbus/visorchipset.c

index 34b3170..5cbc69f 100644 (file)
@@ -1237,18 +1237,11 @@ parahotplug_process_message(struct controlvm_message *inmsg)
        if (!req)
                return -ENOMEM;
 
+       /*
+        * For enable messages, just respond with success right away, we don't
+        * need to wait to see if the enable was successful.
+        */
        if (inmsg->cmd.device_change_state.state.active) {
-               /*
-                * For enable messages, just respond with success
-                * right away. This is a bit of a hack, but there are
-                * issues with the early enable messages we get (with
-                * either the udev script not detecting that the device
-                * is up, or not getting called at all). Fortunately
-                * the messages that get lost don't matter anyway, as
-                *
-                * devices are automatically enabled at
-                * initialization.
-                */
                err = parahotplug_request_kickoff(req);
                if (err)
                        goto err_respond;