Re: [PATCH 3/3] acpi: remove unused LIST_HEAD(acpi_device_list)

Haicheng Li
2013-07-04T02:27:58Z

On Wed, Jul 03, 2013 at 11:24:03PM +0200, Rafael J. Wysocki wrote:
> On Wednesday, July 03, 2013 11:16:18 PM Haicheng Li wrote:
> > Cc: Len Brown <lenb@kernel.org>
> > Cc: "Rafael J. Wysocki" <rjw@sisk.pl>
> > Cc: linux-acpi@vger.kernel.org
> > Signed-off-by: Haicheng Li <haicheng.li@linux.intel.com>
>
> -ENOCHANGELOG

Ah, just thought the patch is so trivial a cleanup, should be self-explainable:).

> Apart from this, please send the *whole* patchset to linux-acpi.
Sorry for the confusion. This is actually a standalone patch. I just noticed this
piece of code is not used by acpi anymore and can be removed cleanly.

If needed, I can resend it as a standalone patch.

> Thanks,
> Rafael
>
>
> > ---
> > drivers/acpi/scan.c | 1 -
> > 1 file changed, 1 deletion(-)
> >
> > diff --git a/drivers/acpi/scan.c b/drivers/acpi/scan.c
> > index 27da630..fcc0cdc 100644
> > --- a/drivers/acpi/scan.c
> > +++ b/drivers/acpi/scan.c
> > @@ -29,7 +29,6 @@ extern struct acpi_device *acpi_root;
> >
> > static const char *dummy_hid = "device";
> >
> > -static LIST_HEAD(acpi_device_list);
> > static LIST_HEAD(acpi_bus_id_list);
> > static DEFINE_MUTEX(acpi_scan_lock);
> > static LIST_HEAD(acpi_scan_handlers_list);
> >
> --
> I speak only for myself.
> Rafael J. Wysocki, Intel Open Source Technology Center.