diff options
| author | David Kershner <david.kershner@unisys.com> | 2016-11-21 12:15:44 -0500 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2016-11-23 10:01:56 +0100 |
| commit | 1d7f5522036c28e7cc809531de2354eb52b9a623 (patch) | |
| tree | 36e2b377dea34422957d7fe3f346c4e385aa5b78 /drivers/staging/unisys | |
| parent | 9ed146cdb99651d0ae4498ae658654625f6bade7 (diff) | |
| download | op-kernel-dev-1d7f5522036c28e7cc809531de2354eb52b9a623.zip op-kernel-dev-1d7f5522036c28e7cc809531de2354eb52b9a623.tar.gz | |
staging: unisys: visorbus: controlvm_respond_chipset_init add error handling
Now that visorchannel_signalinsert returns a Linux error code, we need
to convert controlvm_respond_chipset_init to return an error instead of
a void.
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reported-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/unisys')
| -rw-r--r-- | drivers/staging/unisys/visorbus/visorchipset.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/drivers/staging/unisys/visorbus/visorchipset.c b/drivers/staging/unisys/visorbus/visorchipset.c index 97d40d0..e982a46 100644 --- a/drivers/staging/unisys/visorbus/visorchipset.c +++ b/drivers/staging/unisys/visorbus/visorchipset.c @@ -498,7 +498,7 @@ controlvm_init_response(struct controlvm_message *msg, } } -static void +static int controlvm_respond_chipset_init(struct controlvm_message_header *msg_hdr, int response, enum ultra_chipset_feature features) @@ -507,10 +507,8 @@ controlvm_respond_chipset_init(struct controlvm_message_header *msg_hdr, controlvm_init_response(&outmsg, msg_hdr, response); outmsg.cmd.init_chipset.features = features; - if (visorchannel_signalinsert(controlvm_channel, - CONTROLVM_QUEUE_REQUEST, &outmsg)) { - return; - } + return visorchannel_signalinsert(controlvm_channel, + CONTROLVM_QUEUE_REQUEST, &outmsg); } static void |
