ACPI: dock: dock_add - hoist up platform_device_register_simple()
authorAlex Chiang <achiang@hp.com>
Mon, 19 Oct 2009 21:14:40 +0000 (15:14 -0600)
committerLen Brown <len.brown@intel.com>
Wed, 16 Dec 2009 08:03:11 +0000 (03:03 -0500)
Move the call to platform_device_register_simple so that we do it
before allocating and initializing our struct dock_station.

Signed-off-by: Alex Chiang <achiang@hp.com>
Acked-by: Shaohua Li <shaohua.li@intel.com>
Signed-off-by: Len Brown <len.brown@intel.com>
drivers/acpi/dock.c

index 0a1221854a2548e728888ad56bdcfb63ee478541..3bd89823402a2cdf07dfd712c1ca7148f280e642 100644 (file)
@@ -948,11 +948,18 @@ static int dock_add(acpi_handle handle)
        struct dock_station *dock_station;
        struct platform_device *dock_device;
 
+       dock_device =
+               platform_device_register_simple("dock",
+                       dock_station_count, NULL, 0);
+       if (IS_ERR(dock_device))
+               return PTR_ERR(dock_device);
+
        /* allocate & initialize the dock_station private data */
        dock_station = kzalloc(sizeof(*dock_station), GFP_KERNEL);
        if (!dock_station)
                return -ENOMEM;
        dock_station->handle = handle;
+       dock_station->dock_device = dock_device;
        dock_station->last_dock_time = jiffies - HZ;
        INIT_LIST_HEAD(&dock_station->dependent_devices);
        INIT_LIST_HEAD(&dock_station->hotplug_devices);
@@ -961,15 +968,6 @@ static int dock_add(acpi_handle handle)
        mutex_init(&dock_station->hp_lock);
        ATOMIC_INIT_NOTIFIER_HEAD(&dock_notifier_list);
 
-       /* initialize platform device stuff */
-       dock_station->dock_device =
-               platform_device_register_simple("dock",
-                       dock_station_count, NULL, 0);
-       dock_device = dock_station->dock_device;
-       if (IS_ERR(dock_device)) {
-               ret = PTR_ERR(dock_device);
-               goto out;
-       }
        platform_device_add_data(dock_device, &dock_station,
                sizeof(struct dock_station *));