diff options
author | Jes Sorensen | 2015-05-13 19:21:57 +0200 |
---|---|---|
committer | Greg Kroah-Hartman | 2015-05-24 22:28:53 +0200 |
commit | d3368a58776bfdd7ca22db35ea83567e9657b6e5 (patch) | |
tree | 44904c40ed63d6e7065f0f291654e3ad5948a1a6 /drivers/staging/unisys/visorbus/visorchipset.c | |
parent | staging: unisys: Remove unused visorchipset_save_message() (diff) | |
download | kernel-qcow2-linux-d3368a58776bfdd7ca22db35ea83567e9657b6e5.tar.gz kernel-qcow2-linux-d3368a58776bfdd7ca22db35ea83567e9657b6e5.tar.xz kernel-qcow2-linux-d3368a58776bfdd7ca22db35ea83567e9657b6e5.zip |
staging: unisys: visorchipset_init(): Simplify initial checks
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Benjamin Romer <benjamin.romer@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/unisys/visorbus/visorchipset.c')
-rw-r--r-- | drivers/staging/unisys/visorbus/visorchipset.c | 40 |
1 files changed, 17 insertions, 23 deletions
diff --git a/drivers/staging/unisys/visorbus/visorchipset.c b/drivers/staging/unisys/visorbus/visorchipset.c index d88f11fa4258..88a2b5e52323 100644 --- a/drivers/staging/unisys/visorbus/visorchipset.c +++ b/drivers/staging/unisys/visorbus/visorchipset.c @@ -1474,15 +1474,14 @@ initialize_controlvm_payload(void) /* Send ACTION=online for DEVPATH=/sys/devices/platform/visorchipset. * Returns CONTROLVM_RESP_xxx code. */ -int +static int visorchipset_chipset_ready(void) { kobject_uevent(&visorchipset_platform_device.dev.kobj, KOBJ_ONLINE); return CONTROLVM_RESP_SUCCESS; } -EXPORT_SYMBOL_GPL(visorchipset_chipset_ready); -int +static int visorchipset_chipset_selftest(void) { char env_selftest[20]; @@ -1493,18 +1492,16 @@ visorchipset_chipset_selftest(void) envp); return CONTROLVM_RESP_SUCCESS; } -EXPORT_SYMBOL_GPL(visorchipset_chipset_selftest); /* Send ACTION=offline for DEVPATH=/sys/devices/platform/visorchipset. * Returns CONTROLVM_RESP_xxx code. */ -int +static int visorchipset_chipset_notready(void) { kobject_uevent(&visorchipset_platform_device.dev.kobj, KOBJ_OFFLINE); return CONTROLVM_RESP_SUCCESS; } -EXPORT_SYMBOL_GPL(visorchipset_chipset_notready); static void chipset_ready(struct controlvm_message_header *msg_hdr) @@ -2099,14 +2096,13 @@ device_destroy_response(u32 bus_no, u32 dev_no, int response) device_responder(CONTROLVM_DEVICE_DESTROY, bus_no, dev_no, response); } -void +static void visorchipset_device_pause_response(u32 bus_no, u32 dev_no, int response) { device_changestate_responder(CONTROLVM_DEVICE_CHANGESTATE, bus_no, dev_no, response, segment_state_standby); } -EXPORT_SYMBOL_GPL(visorchipset_device_pause_response); static void device_resume_response(u32 bus_no, u32 dev_no, int response) @@ -2327,28 +2323,26 @@ visorchipset_init(struct acpi_device *acpi_device) { int rc = 0; u64 addr; + int tmp_sz = sizeof(struct spar_controlvm_channel_protocol); + uuid_le uuid = SPAR_CONTROLVM_CHANNEL_PROTOCOL_UUID; + + addr = controlvm_get_channel_address(); + if (!addr) + return -ENODEV; memset(&busdev_notifiers, 0, sizeof(busdev_notifiers)); memset(&controlvm_payload_info, 0, sizeof(controlvm_payload_info)); memset(&livedump_info, 0, sizeof(livedump_info)); atomic_set(&livedump_info.buffers_in_use, 0); - addr = controlvm_get_channel_address(); - if (addr) { - int tmp_sz = sizeof(struct spar_controlvm_channel_protocol); - uuid_le uuid = SPAR_CONTROLVM_CHANNEL_PROTOCOL_UUID; - controlvm_channel = - visorchannel_create_with_lock(addr, tmp_sz, - GFP_KERNEL, uuid); - if (SPAR_CONTROLVM_CHANNEL_OK_CLIENT( - visorchannel_get_header(controlvm_channel))) { - initialize_controlvm_payload(); - } else { - visorchannel_destroy(controlvm_channel); - controlvm_channel = NULL; - return -ENODEV; - } + controlvm_channel = visorchannel_create_with_lock(addr, tmp_sz, + GFP_KERNEL, uuid); + if (SPAR_CONTROLVM_CHANNEL_OK_CLIENT( + visorchannel_get_header(controlvm_channel))) { + initialize_controlvm_payload(); } else { + visorchannel_destroy(controlvm_channel); + controlvm_channel = NULL; return -ENODEV; } |