public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH 3/3] acpi: remove unused LIST_HEAD(acpi_device_list)
       [not found] <1372864578-6925-1-git-send-email-haicheng.li@linux.intel.com>
@ 2013-07-03 15:16 ` Haicheng Li
  2013-07-03 21:24   ` Rafael J. Wysocki
  0 siblings, 1 reply; 5+ messages in thread
From: Haicheng Li @ 2013-07-03 15:16 UTC (permalink / raw)
  To: linux-kernel
  Cc: Haicheng Li, Haicheng Li, Len Brown, Rafael J. Wysocki,
	linux-acpi

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>
---
 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);
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH 3/3] acpi: remove unused LIST_HEAD(acpi_device_list)
  2013-07-03 15:16 ` [PATCH 3/3] acpi: remove unused LIST_HEAD(acpi_device_list) Haicheng Li
@ 2013-07-03 21:24   ` Rafael J. Wysocki
  2013-07-04  2:27     ` Haicheng Li
  0 siblings, 1 reply; 5+ messages in thread
From: Rafael J. Wysocki @ 2013-07-03 21:24 UTC (permalink / raw)
  To: Haicheng Li; +Cc: linux-kernel, Haicheng Li, Len Brown, linux-acpi

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

Apart from this, please send the *whole* patchset to linux-acpi.

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.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 3/3] acpi: remove unused LIST_HEAD(acpi_device_list)
  2013-07-03 21:24   ` Rafael J. Wysocki
@ 2013-07-04  2:27     ` Haicheng Li
  2013-07-04  4:07       ` [PATCH] " Haicheng Li
  0 siblings, 1 reply; 5+ messages in thread
From: Haicheng Li @ 2013-07-04  2:27 UTC (permalink / raw)
  To: Rafael J. Wysocki; +Cc: linux-kernel, Haicheng Li, Len Brown, linux-acpi

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.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH] acpi: remove unused LIST_HEAD(acpi_device_list)
  2013-07-04  2:27     ` Haicheng Li
@ 2013-07-04  4:07       ` Haicheng Li
  2013-07-04 12:50         ` Rafael J. Wysocki
  0 siblings, 1 reply; 5+ messages in thread
From: Haicheng Li @ 2013-07-04  4:07 UTC (permalink / raw)
  To: Rafael J. Wysocki, linux-acpi
  Cc: Len Brown, linux-kernel, Haicheng Li, Haicheng Li

acpi_device_list is not used by acpi code anymore, could be removed.

Signed-off-by: Haicheng Li <haicheng.li@linux.intel.com>
---
 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);
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] acpi: remove unused LIST_HEAD(acpi_device_list)
  2013-07-04  4:07       ` [PATCH] " Haicheng Li
@ 2013-07-04 12:50         ` Rafael J. Wysocki
  0 siblings, 0 replies; 5+ messages in thread
From: Rafael J. Wysocki @ 2013-07-04 12:50 UTC (permalink / raw)
  To: Haicheng Li; +Cc: linux-acpi, Len Brown, linux-kernel, Haicheng Li

On Thursday, July 04, 2013 12:07:11 PM Haicheng Li wrote:
> acpi_device_list is not used by acpi code anymore, could be removed.
> 
> Signed-off-by: Haicheng Li <haicheng.li@linux.intel.com>

OK, applied.

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.

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2013-07-04 12:41 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <1372864578-6925-1-git-send-email-haicheng.li@linux.intel.com>
2013-07-03 15:16 ` [PATCH 3/3] acpi: remove unused LIST_HEAD(acpi_device_list) Haicheng Li
2013-07-03 21:24   ` Rafael J. Wysocki
2013-07-04  2:27     ` Haicheng Li
2013-07-04  4:07       ` [PATCH] " Haicheng Li
2013-07-04 12:50         ` Rafael J. Wysocki

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox