Commit 800da5fb authored by David Kershner's avatar David Kershner Committed by Greg Kroah-Hartman

staging: unisys: visorbus: Clean up vmcall address function.

The function vmcall address needed to be cleaned up. The structure
vmcall_controlvm_addr was not needed so it was removed and was replaced
with vmcall_io_controlvm_addr_params since it needs to be allocated on the
heap for DMA access.

With the structure removed and the fields as local variables, it helped
clean up the formatting of the function.
Signed-off-by: default avatarDavid Kershner <david.kershner@unisys.com>
Reviewed-by: default avatarTim Sell <timothy.sell@unisys.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 496c8902
...@@ -86,12 +86,6 @@ struct vmcall_io_controlvm_addr_params { ...@@ -86,12 +86,6 @@ struct vmcall_io_controlvm_addr_params {
u8 unused[4]; u8 unused[4];
} __packed; } __packed;
struct vmcall_controlvm_addr {
struct vmcall_io_controlvm_addr_params params;
int err;
u64 physaddr;
};
struct visorchipset_device { struct visorchipset_device {
struct acpi_device *acpi_device; struct acpi_device *acpi_device;
unsigned long poll_jiffies; unsigned long poll_jiffies;
...@@ -109,7 +103,7 @@ struct visorchipset_device { ...@@ -109,7 +103,7 @@ struct visorchipset_device {
*/ */
struct controlvm_message controlvm_pending_msg; struct controlvm_message controlvm_pending_msg;
bool controlvm_pending_msg_valid; bool controlvm_pending_msg_valid;
struct vmcall_controlvm_addr controlvm_addr; struct vmcall_io_controlvm_addr_params controlvm_params;
}; };
static struct visorchipset_device *chipset_dev; static struct visorchipset_device *chipset_dev;
...@@ -1341,15 +1335,16 @@ static int unisys_vmcall(unsigned long tuple, unsigned long param) ...@@ -1341,15 +1335,16 @@ static int unisys_vmcall(unsigned long tuple, unsigned long param)
static unsigned int issue_vmcall_io_controlvm_addr(u64 *control_addr, static unsigned int issue_vmcall_io_controlvm_addr(u64 *control_addr,
u32 *control_bytes) u32 *control_bytes)
{ {
chipset_dev->controlvm_addr.physaddr = virt_to_phys( u64 physaddr;
&chipset_dev->controlvm_addr.params); int err;
chipset_dev->controlvm_addr.err = unisys_vmcall(VMCALL_CONTROLVM_ADDR,
chipset_dev->controlvm_addr.physaddr); physaddr = virt_to_phys(&chipset_dev->controlvm_params);
if (chipset_dev->controlvm_addr.err) err = unisys_vmcall(VMCALL_CONTROLVM_ADDR, physaddr);
return chipset_dev->controlvm_addr.err; if (err)
return err;
*control_addr = chipset_dev->controlvm_addr.params.address;
*control_bytes = chipset_dev->controlvm_addr.params.channel_bytes; *control_addr = chipset_dev->controlvm_params.address;
*control_bytes = chipset_dev->controlvm_params.channel_bytes;
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment