All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] make ibm_acpi work right with notify handlers.
@ 2006-10-25 14:29 Alexey Starikovskiy
  2006-10-27  5:34 ` Len Brown
  0 siblings, 1 reply; 2+ messages in thread
From: Alexey Starikovskiy @ 2006-10-25 14:29 UTC (permalink / raw)
  To: Linux ACPI, Brown, Len, Borislav Deianov

[-- Attachment #1: Type: text/plain, Size: 173 bytes --]

Hi,

There is a bug in ibm_acpi module, which prevents its clean removal.
This patch fixes the problem by moving setting notify_installed to the right place.

Thanks,
Alex.

[-- Attachment #2: ibm_acpi-do_notify_remove.patch --]
[-- Type: text/x-patch, Size: 675 bytes --]

make unload of ibm_acpi right.

From:  Alexey Starikovskiy <alexey.y.starikovskiy@intel.com>


---

 drivers/acpi/ibm_acpi.c |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/drivers/acpi/ibm_acpi.c b/drivers/acpi/ibm_acpi.c
index 003a987..719a320 100644
--- a/drivers/acpi/ibm_acpi.c
+++ b/drivers/acpi/ibm_acpi.c
@@ -1805,7 +1805,7 @@ static int __init setup_notify(struct ib
 		       ibm->name, status);
 		return -ENODEV;
 	}
-
+	ibm->notify_installed = 1;
 	return 0;
 }
 
@@ -1882,7 +1882,6 @@ static int __init ibm_init(struct ibm_st
 		ret = setup_notify(ibm);
 		if (ret < 0)
 			return ret;
-		ibm->notify_installed = 1;
 	}
 
 	return 0;

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

end of thread, other threads:[~2006-10-27  5:34 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-10-25 14:29 [PATCH] make ibm_acpi work right with notify handlers Alexey Starikovskiy
2006-10-27  5:34 ` Len Brown

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.