Commit daaa8cc3 authored by Greg Kroah-Hartman's avatar Greg Kroah-Hartman

Staging: hv: osd.h: fix GUID reference problem

As GUID was a typedef, it hid the fact that we were passing it
a 2 variables in functions.  This fixes this up by passing it
as a pointer, as it should be.

Cc: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent caf26a31
...@@ -284,8 +284,8 @@ VmbusChannelProcessOffer( ...@@ -284,8 +284,8 @@ VmbusChannelProcessOffer(
/* Start the process of binding this offer to the driver */ /* Start the process of binding this offer to the driver */
/* We need to set the DeviceObject field before calling VmbusChildDeviceAdd() */ /* We need to set the DeviceObject field before calling VmbusChildDeviceAdd() */
newChannel->DeviceObject = VmbusChildDeviceCreate( newChannel->DeviceObject = VmbusChildDeviceCreate(
newChannel->OfferMsg.Offer.InterfaceType, &newChannel->OfferMsg.Offer.InterfaceType,
newChannel->OfferMsg.Offer.InterfaceInstance, &newChannel->OfferMsg.Offer.InterfaceInstance,
newChannel); newChannel);
DPRINT_DBG(VMBUS, "child device object allocated - %p", newChannel->DeviceObject); DPRINT_DBG(VMBUS, "child device object allocated - %p", newChannel->DeviceObject);
......
...@@ -231,8 +231,8 @@ Description: ...@@ -231,8 +231,8 @@ Description:
--*/ --*/
struct hv_device *VmbusChildDeviceCreate(struct hv_guid DeviceType, struct hv_device *VmbusChildDeviceCreate(struct hv_guid *DeviceType,
struct hv_guid DeviceInstance, struct hv_guid *DeviceInstance,
void *Context) void *Context)
{ {
VMBUS_DRIVER_OBJECT* vmbusDriver = (VMBUS_DRIVER_OBJECT*)gDriver; VMBUS_DRIVER_OBJECT* vmbusDriver = (VMBUS_DRIVER_OBJECT*)gDriver;
......
...@@ -103,8 +103,8 @@ extern struct VMBUS_CONNECTION gVmbusConnection; ...@@ -103,8 +103,8 @@ extern struct VMBUS_CONNECTION gVmbusConnection;
/* General vmbus interface */ /* General vmbus interface */
struct hv_device *VmbusChildDeviceCreate(struct hv_guid deviceType, struct hv_device *VmbusChildDeviceCreate(struct hv_guid *deviceType,
struct hv_guid deviceInstance, struct hv_guid *deviceInstance,
void *context); void *context);
int VmbusChildDeviceAdd(struct hv_device *Device); int VmbusChildDeviceAdd(struct hv_device *Device);
......
...@@ -76,7 +76,7 @@ typedef int (*PFN_ON_ISR)(struct hv_driver *drv); ...@@ -76,7 +76,7 @@ typedef int (*PFN_ON_ISR)(struct hv_driver *drv);
typedef void (*PFN_ON_DPC)(struct hv_driver *drv); typedef void (*PFN_ON_DPC)(struct hv_driver *drv);
typedef void (*PFN_GET_CHANNEL_OFFERS)(void); typedef void (*PFN_GET_CHANNEL_OFFERS)(void);
typedef struct hv_device *(*PFN_ON_CHILDDEVICE_CREATE)(struct hv_guid DeviceType, struct hv_guid DeviceInstance, void *Context); typedef struct hv_device *(*PFN_ON_CHILDDEVICE_CREATE)(struct hv_guid *DeviceType, struct hv_guid *DeviceInstance, void *Context);
typedef void (*PFN_ON_CHILDDEVICE_DESTROY)(struct hv_device *Device); typedef void (*PFN_ON_CHILDDEVICE_DESTROY)(struct hv_device *Device);
typedef int (*PFN_ON_CHILDDEVICE_ADD)(struct hv_device *RootDevice, struct hv_device *ChildDevice); typedef int (*PFN_ON_CHILDDEVICE_ADD)(struct hv_device *RootDevice, struct hv_device *ChildDevice);
typedef void (*PFN_ON_CHILDDEVICE_REMOVE)(struct hv_device *Device); typedef void (*PFN_ON_CHILDDEVICE_REMOVE)(struct hv_device *Device);
......
...@@ -75,7 +75,7 @@ static irqreturn_t vmbus_isr(int irq, void* dev_id); ...@@ -75,7 +75,7 @@ static irqreturn_t vmbus_isr(int irq, void* dev_id);
static void vmbus_device_release(struct device *device); static void vmbus_device_release(struct device *device);
static void vmbus_bus_release(struct device *device); static void vmbus_bus_release(struct device *device);
static struct hv_device *vmbus_child_device_create(struct hv_guid type, struct hv_guid instance, void* context); static struct hv_device *vmbus_child_device_create(struct hv_guid *type, struct hv_guid *instance, void *context);
static void vmbus_child_device_destroy(struct hv_device *device_obj); static void vmbus_child_device_destroy(struct hv_device *device_obj);
static int vmbus_child_device_register(struct hv_device *root_device_obj, struct hv_device *child_device_obj); static int vmbus_child_device_register(struct hv_device *root_device_obj, struct hv_device *child_device_obj);
static void vmbus_child_device_unregister(struct hv_device *child_device_obj); static void vmbus_child_device_unregister(struct hv_device *child_device_obj);
...@@ -565,7 +565,9 @@ Name: vmbus_child_device_create() ...@@ -565,7 +565,9 @@ Name: vmbus_child_device_create()
Desc: Creates and registers a new child device on the vmbus. Desc: Creates and registers a new child device on the vmbus.
--*/ --*/
static struct hv_device *vmbus_child_device_create(struct hv_guid type, struct hv_guid instance, void* context) static struct hv_device *vmbus_child_device_create(struct hv_guid *type,
struct hv_guid *instance,
void *context)
{ {
struct device_context *child_device_ctx; struct device_context *child_device_ctx;
struct hv_device *child_device_obj; struct hv_device *child_device_obj;
...@@ -586,14 +588,14 @@ static struct hv_device *vmbus_child_device_create(struct hv_guid type, struct h ...@@ -586,14 +588,14 @@ static struct hv_device *vmbus_child_device_create(struct hv_guid type, struct h
"type {%02x%02x%02x%02x-%02x%02x-%02x%02x-%02x%02x%02x%02x%02x%02x%02x%02x}," "type {%02x%02x%02x%02x-%02x%02x-%02x%02x-%02x%02x%02x%02x%02x%02x%02x%02x},"
"id {%02x%02x%02x%02x-%02x%02x-%02x%02x-%02x%02x%02x%02x%02x%02x%02x%02x}", "id {%02x%02x%02x%02x-%02x%02x-%02x%02x-%02x%02x%02x%02x%02x%02x%02x%02x}",
&child_device_ctx->device, &child_device_ctx->device,
type.data[3], type.data[2], type.data[1], type.data[0], type->data[3], type->data[2], type->data[1], type->data[0],
type.data[5], type.data[4], type.data[7], type.data[6], type->data[5], type->data[4], type->data[7], type->data[6],
type.data[8], type.data[9], type.data[10], type.data[11], type->data[8], type->data[9], type->data[10], type->data[11],
type.data[12], type.data[13], type.data[14], type.data[15], type->data[12], type->data[13], type->data[14], type->data[15],
instance.data[3], instance.data[2], instance.data[1], instance.data[0], instance->data[3], instance->data[2], instance->data[1], instance->data[0],
instance.data[5], instance.data[4], instance.data[7], instance.data[6], instance->data[5], instance->data[4], instance->data[7], instance->data[6],
instance.data[8], instance.data[9], instance.data[10], instance.data[11], instance->data[8], instance->data[9], instance->data[10], instance->data[11],
instance.data[12], instance.data[13], instance.data[14], instance.data[15]); instance->data[12], instance->data[13], instance->data[14], instance->data[15]);
child_device_obj = &child_device_ctx->device_obj; child_device_obj = &child_device_ctx->device_obj;
child_device_obj->context = context; child_device_obj->context = context;
......
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