Staging: hv: Get rid of the forward declaration for vmbus_probe
authorK. Y. Srinivasan <kys@microsoft.com>
Tue, 15 Mar 2011 22:03:39 +0000 (15:03 -0700)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 5 Apr 2011 04:47:29 +0000 (21:47 -0700)
Get rid of the forward declaration of vmbus_probe by moving
the code around.

Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
Signed-off-by: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Mike Sterling <mike.sterling@microsoft.com>
Signed-off-by: Abhishek Kane <v-abkane@microsoft.com>
Signed-off-by: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/hv/vmbus_drv.c

index 5dcfd9854921f4b05eb347cbc1f0f4e22839d2c1..8153a4d493d5e40044fd2a624d8d6fe1f1d66216 100644 (file)
@@ -47,7 +47,6 @@ struct hv_bus {
        struct tasklet_struct event_dpc;
 };
 
-static int vmbus_probe(struct device *device);
 static int vmbus_remove(struct device *device);
 static void vmbus_shutdown(struct device *device);
 
@@ -196,6 +195,59 @@ static int vmbus_match(struct device *device, struct device_driver *driver)
        return match;
 }
 
+
+/*
+ * vmbus_probe_failed_cb - Callback when a driver probe failed in vmbus_probe()
+ *
+ * We need a callback because we cannot invoked device_unregister() inside
+ * vmbus_probe() since vmbus_probe() may be invoked inside device_register()
+ * i.e. we cannot call device_unregister() inside device_register()
+ */
+static void vmbus_probe_failed_cb(struct work_struct *context)
+{
+       struct hv_device *device_ctx = (struct hv_device *)context;
+
+       /*
+        * Kick off the process of unregistering the device.
+        * This will call vmbus_remove() and eventually vmbus_device_release()
+        */
+       device_unregister(&device_ctx->device);
+
+       /* put_device(&device_ctx->device); */
+}
+
+/*
+ * vmbus_probe - Add the new vmbus's child device
+ */
+static int vmbus_probe(struct device *child_device)
+{
+       int ret = 0;
+       struct hv_driver *drv =
+                       drv_to_hv_drv(child_device->driver);
+       struct hv_device *dev = device_to_hv_device(child_device);
+
+       /* Let the specific open-source driver handles the probe if it can */
+       if (drv->driver.probe) {
+               ret = dev->probe_error =
+               drv->driver.probe(child_device);
+               if (ret != 0) {
+                       DPRINT_ERR(VMBUS_DRV, "probe() failed for device %s "
+                                  "(%p) on driver %s (%d)...",
+                                  dev_name(child_device), child_device,
+                                  child_device->driver->name, ret);
+
+                       INIT_WORK(&dev->probe_failed_work_item,
+                                 vmbus_probe_failed_cb);
+                       schedule_work(&dev->probe_failed_work_item);
+               }
+       } else {
+               DPRINT_ERR(VMBUS_DRV, "probe() method not set for driver - %s",
+                          child_device->driver->name);
+               ret = -1;
+       }
+       return ret;
+}
+
 /* The one and only one */
 static struct hv_bus  hv_bus = {
        .bus.name =             "vmbus",
@@ -724,59 +776,6 @@ void vmbus_child_device_unregister(struct hv_device *device_obj)
                    &device_obj->device);
 }
 
-
-/*
- * vmbus_probe_failed_cb - Callback when a driver probe failed in vmbus_probe()
- *
- * We need a callback because we cannot invoked device_unregister() inside
- * vmbus_probe() since vmbus_probe() may be invoked inside device_register()
- * i.e. we cannot call device_unregister() inside device_register()
- */
-static void vmbus_probe_failed_cb(struct work_struct *context)
-{
-       struct hv_device *device_ctx = (struct hv_device *)context;
-
-       /*
-        * Kick off the process of unregistering the device.
-        * This will call vmbus_remove() and eventually vmbus_device_release()
-        */
-       device_unregister(&device_ctx->device);
-
-       /* put_device(&device_ctx->device); */
-}
-
-/*
- * vmbus_probe - Add the new vmbus's child device
- */
-static int vmbus_probe(struct device *child_device)
-{
-       int ret = 0;
-       struct hv_driver *drv =
-                       drv_to_hv_drv(child_device->driver);
-       struct hv_device *dev = device_to_hv_device(child_device);
-
-       /* Let the specific open-source driver handles the probe if it can */
-       if (drv->driver.probe) {
-               ret = dev->probe_error =
-               drv->driver.probe(child_device);
-               if (ret != 0) {
-                       DPRINT_ERR(VMBUS_DRV, "probe() failed for device %s "
-                                  "(%p) on driver %s (%d)...",
-                                  dev_name(child_device), child_device,
-                                  child_device->driver->name, ret);
-
-                       INIT_WORK(&dev->probe_failed_work_item,
-                                 vmbus_probe_failed_cb);
-                       schedule_work(&dev->probe_failed_work_item);
-               }
-       } else {
-               DPRINT_ERR(VMBUS_DRV, "probe() method not set for driver - %s",
-                          child_device->driver->name);
-               ret = -1;
-       }
-       return ret;
-}
-
 /*
  * vmbus_remove - Remove a vmbus device
  */