From: David Kershner <david.kershner@unisys.com> Date: Tue, 18 Apr 2017 20:55:05 +0000 (-0400) Subject: staging: unisys: visorbus: update comment X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=d02bde9d37bc00376727a2b422ee95323bbcd6d6;p=linux.git staging: unisys: visorbus: update comment 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> --- diff --git a/drivers/staging/unisys/visorbus/visorchipset.c b/drivers/staging/unisys/visorbus/visorchipset.c index 34b31706f4113..5cbc69ff33818 100644 --- a/drivers/staging/unisys/visorbus/visorchipset.c +++ b/drivers/staging/unisys/visorbus/visorchipset.c @@ -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;