Staging: hv: remove OnChildDeviceCreate vmbus_driver callback

It's only ever set to one function, so just call that function instead.

Cc: Haiyang Zhang <haiyangz@microsoft.com>
Cc: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

diff --git a/drivers/staging/hv/channel_mgmt.c b/drivers/staging/hv/channel_mgmt.c
index ae830f2e4e3a81097d1326b39ae9519cc2eccdfe..3105fd1afe7dac3db95ec5aca47c805adb6ca9fd 100644 (file)
--- a/drivers/staging/hv/channel_mgmt.c
+++ b/drivers/staging/hv/channel_mgmt.c
@@ -355,7 +355,7 @@ static void vmbus_process_offer(void *context)
         * We need to set the DeviceObject field before calling
         * VmbusChildDeviceAdd()
         */
-       newchannel->device_obj = VmbusChildDeviceCreate(
+       newchannel->device_obj = vmbus_child_device_create(
                &newchannel->offermsg.offer.InterfaceType,
                &newchannel->offermsg.offer.InterfaceInstance,
                newchannel);

diff --git a/drivers/staging/hv/vmbus.c b/drivers/staging/hv/vmbus.c
index e6462c6774d16bdbf31bb6527f63b71fd672eb06..163c743ea476595ea4b61ba941cc506e23977223 100644 (file)
--- a/drivers/staging/hv/vmbus.c
+++ b/drivers/staging/hv/vmbus.c
@@ -60,19 +60,6 @@ static void VmbusGetChannelOffers(void)
        vmbus_request_offers();
 }
 
-/*
- * VmbusCreateChildDevice - Creates the child device on the bus that represents the channel offer
- */
-struct hv_device *VmbusChildDeviceCreate(struct hv_guid *DeviceType,
-                                        struct hv_guid *DeviceInstance,
-                                        struct vmbus_channel *channel)
-{
-       struct vmbus_driver *vmbusDriver = (struct vmbus_driver *)gDriver;
-
-       return vmbusDriver->OnChildDeviceCreate(DeviceType, DeviceInstance,
-                                               channel);
-}
-
 /*
  * VmbusChildDeviceAdd - Registers the child device with the vmbus
  */

diff --git a/drivers/staging/hv/vmbus_api.h b/drivers/staging/hv/vmbus_api.h
index 2af42e55007633e1c8b94051bf3634017a8d07d2..e26365c0e7984cebfc19f132f55ef32e84ccc24e 100644 (file)
--- a/drivers/staging/hv/vmbus_api.h
+++ b/drivers/staging/hv/vmbus_api.h
@@ -122,9 +122,6 @@ struct vmbus_driver {
        struct hv_driver Base;
 
        /* Set by the caller */
-       struct hv_device * (*OnChildDeviceCreate)(struct hv_guid *DeviceType,
-                                               struct hv_guid *DeviceInstance,
-                                               struct vmbus_channel *channel);
        void (*OnChildDeviceDestroy)(struct hv_device *device);
        int (*OnChildDeviceAdd)(struct hv_device *RootDevice,
                                struct hv_device *ChildDevice);

diff --git a/drivers/staging/hv/vmbus_drv.c b/drivers/staging/hv/vmbus_drv.c
index 096587592e2329ead005b264c75e988689852d90..224e8440b8187549fda65f0685bf82bd1ffb2e90 100644 (file)
--- a/drivers/staging/hv/vmbus_drv.c
+++ b/drivers/staging/hv/vmbus_drv.c
@@ -69,9 +69,6 @@ static irqreturn_t vmbus_isr(int irq, void *dev_id);
 static void vmbus_device_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,
-                                                  struct vmbus_channel *channel);
 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);
@@ -301,7 +298,6 @@ static int vmbus_bus_init(int (*drv_init)(struct hv_driver *drv))
         * Set this up to allow lower layer to callback to add/remove child
         * devices on the bus
         */
-       vmbus_drv_obj->OnChildDeviceCreate = vmbus_child_device_create;
        vmbus_drv_obj->OnChildDeviceDestroy = vmbus_child_device_destroy;
        vmbus_drv_obj->OnChildDeviceAdd = vmbus_child_device_register;
        vmbus_drv_obj->OnChildDeviceRemove = vmbus_child_device_unregister;
@@ -492,9 +488,9 @@ EXPORT_SYMBOL(vmbus_child_driver_unregister);
  * vmbus_child_device_create - 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,
-                                                  struct vmbus_channel *channel)
+struct hv_device *vmbus_child_device_create(struct hv_guid *type,
+                                           struct hv_guid *instance,
+                                           struct vmbus_channel *channel)
 {
        struct vm_device *child_device_ctx;
        struct hv_device *child_device_obj;

diff --git a/drivers/staging/hv/vmbus_private.h b/drivers/staging/hv/vmbus_private.h
index 09eaec964b305b1fb42a399f62bf251a55d50544..d07d832edc01ed7ff89abb159d082778c1f92acb 100644 (file)
--- a/drivers/staging/hv/vmbus_private.h
+++ b/drivers/staging/hv/vmbus_private.h
@@ -102,7 +102,7 @@ extern struct VMBUS_CONNECTION gVmbusConnection;
 
 /* General vmbus interface */
 
-struct hv_device *VmbusChildDeviceCreate(struct hv_guid *deviceType,
+struct hv_device *vmbus_child_device_create(struct hv_guid *deviceType,
                                         struct hv_guid *deviceInstance,
                                         struct vmbus_channel *channel);