| Message ID | 1371141152-9468-7-git-send-email-jiang.liu@huawei.com (mailing list archive) |
|---|---|
| State | New, archived |
| Delegated to: | Bjorn Helgaas |
| Headers | show |
On Friday, June 14, 2013 12:32:29 AM Jiang Liu wrote: > Mark all initialization functions with __init to reduce memory > consumption at runtime. Again, this is a *cleanup*, not a fix. Please don't mix cleanups with fixes, especially with ones you want to go into 3.10, because the cleanups aren't 3.10 material for sure. Thanks, Rafael > Signed-off-by: Jiang Liu <jiang.liu@huawei.com> > Cc: Shaohua Li <shaohua.li@intel.com> > Cc: Len Brown <lenb@kernel.org> > Cc: "Rafael J. Wysocki" <rjw@sisk.pl> > Cc: linux-acpi@vger.kernel.org > Cc: linux-kernel@vger.kernel.org > --- > drivers/acpi/dock.c | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/drivers/acpi/dock.c b/drivers/acpi/dock.c > index cd2d5df..da3314d 100644 > --- a/drivers/acpi/dock.c > +++ b/drivers/acpi/dock.c > @@ -94,7 +94,7 @@ struct dock_dependent_device { > * > * Add the dependent device to the dock's dependent device list. > */ > -static int > +static int __init > add_dock_dependent_device(struct dock_station *ds, acpi_handle handle) > { > struct dock_dependent_device *dd; > @@ -192,7 +192,7 @@ static int is_dock(acpi_handle handle) > return 1; > } > > -static int is_ejectable(acpi_handle handle) > +static int __init is_ejectable(acpi_handle handle) > { > acpi_status status; > acpi_handle tmp; > @@ -203,7 +203,7 @@ static int is_ejectable(acpi_handle handle) > return 1; > } > > -static int is_ata(acpi_handle handle) > +static int __init is_ata(acpi_handle handle) > { > acpi_handle tmp; > > @@ -216,7 +216,7 @@ static int is_ata(acpi_handle handle) > return 0; > } > > -static int is_battery(acpi_handle handle) > +static int __init is_battery(acpi_handle handle) > { > struct acpi_device_info *info; > int ret = 1; > @@ -232,7 +232,7 @@ static int is_battery(acpi_handle handle) > return ret; > } > > -static int is_ejectable_bay(acpi_handle handle) > +static int __init is_ejectable_bay(acpi_handle handle) > { > acpi_handle phandle; > > @@ -809,7 +809,7 @@ static struct notifier_block dock_acpi_notifier = { > * check to see if an object has an _EJD method. If it does, then it > * will see if it is dependent on the dock station. > */ > -static acpi_status > +static acpi_status __init > find_dock_devices(acpi_handle handle, u32 lvl, void *context, void **rv) > { > acpi_status status; >
diff --git a/drivers/acpi/dock.c b/drivers/acpi/dock.c index cd2d5df..da3314d 100644 --- a/drivers/acpi/dock.c +++ b/drivers/acpi/dock.c @@ -94,7 +94,7 @@ struct dock_dependent_device { * * Add the dependent device to the dock's dependent device list. */ -static int +static int __init add_dock_dependent_device(struct dock_station *ds, acpi_handle handle) { struct dock_dependent_device *dd; @@ -192,7 +192,7 @@ static int is_dock(acpi_handle handle) return 1; } -static int is_ejectable(acpi_handle handle) +static int __init is_ejectable(acpi_handle handle) { acpi_status status; acpi_handle tmp; @@ -203,7 +203,7 @@ static int is_ejectable(acpi_handle handle) return 1; } -static int is_ata(acpi_handle handle) +static int __init is_ata(acpi_handle handle) { acpi_handle tmp; @@ -216,7 +216,7 @@ static int is_ata(acpi_handle handle) return 0; } -static int is_battery(acpi_handle handle) +static int __init is_battery(acpi_handle handle) { struct acpi_device_info *info; int ret = 1; @@ -232,7 +232,7 @@ static int is_battery(acpi_handle handle) return ret; } -static int is_ejectable_bay(acpi_handle handle) +static int __init is_ejectable_bay(acpi_handle handle) { acpi_handle phandle; @@ -809,7 +809,7 @@ static struct notifier_block dock_acpi_notifier = { * check to see if an object has an _EJD method. If it does, then it * will see if it is dependent on the dock station. */ -static acpi_status +static acpi_status __init find_dock_devices(acpi_handle handle, u32 lvl, void *context, void **rv) { acpi_status status;
Mark all initialization functions with __init to reduce memory consumption at runtime. Signed-off-by: Jiang Liu <jiang.liu@huawei.com> Cc: Shaohua Li <shaohua.li@intel.com> Cc: Len Brown <lenb@kernel.org> Cc: "Rafael J. Wysocki" <rjw@sisk.pl> Cc: linux-acpi@vger.kernel.org Cc: linux-kernel@vger.kernel.org --- drivers/acpi/dock.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-)