On Friday, June 14, 2013 12:32:27 AM Jiang Liu wrote: > Initialize acpi_dock_notifier_list from function acpi_dock_init() > instead of dock_add() to avoid initializing it multiple times. Well, makes sense. Thanks, Rafael > Signed-off-by: Jiang Liu <jiang.liu@xxxxxxxxxx> > Cc: Shaohua Li <shaohua.li@xxxxxxxxx> > Cc: Len Brown <lenb@xxxxxxxxxx> > Cc: "Rafael J. Wysocki" <rjw@xxxxxxx> > Cc: linux-acpi@xxxxxxxxxxxxxxx > Cc: linux-kernel@xxxxxxxxxxxxxxx > --- > drivers/acpi/dock.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/acpi/dock.c b/drivers/acpi/dock.c > index 50e38b7..8e578aa 100644 > --- a/drivers/acpi/dock.c > +++ b/drivers/acpi/dock.c > @@ -967,7 +967,6 @@ static int __init dock_add(acpi_handle handle) > spin_lock_init(&dock_station->dd_lock); > INIT_LIST_HEAD(&dock_station->sibling); > INIT_LIST_HEAD(&dock_station->hotplug_devices); > - ATOMIC_INIT_NOTIFIER_HEAD(&dock_notifier_list); > INIT_LIST_HEAD(&dock_station->dependent_devices); > > /* we want the dock device to send uevents */ > @@ -1038,6 +1037,7 @@ int __init acpi_dock_init(void) > return 0; > } > > + ATOMIC_INIT_NOTIFIER_HEAD(&dock_notifier_list); > register_acpi_bus_notifier(&dock_acpi_notifier); > pr_info(PREFIX "%s: %d docks/bays found\n", > ACPI_DOCK_DRIVER_DESCRIPTION, dock_station_count); > -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
