Revert ACPI: ibm-acpi: make non-generic bay support optional

2007-01-11 Thread Linux Kernel Mailing List
Gitweb: 
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=d2fadbbbf0e42b842731da71864f222e7f119461
Commit: d2fadbbbf0e42b842731da71864f222e7f119461
Parent: 8f6d63adf8309a412bf2d3d2e49a85e519ebf57c
Author: Henrique de Moraes Holschuh [EMAIL PROTECTED]
AuthorDate: Thu Jan 11 02:58:15 2007 -0500
Committer:  Len Brown [EMAIL PROTECTED]
CommitDate: Thu Jan 11 02:58:15 2007 -0500

Revert ACPI: ibm-acpi: make non-generic bay support optional

This reverts commit 2df910b4c3edcce9a0c12394db6f5f4a6e69c712.

ACPI_BAY has not been merged into mainline yet, so the changes to ibm-acpi
related Kconfig entries that depend on ACPI_BAY were permanently disabling
ibm-acpi bay support.  This is a serious regression for ThinkPad users.

Signed-off-by: Henrique de Moraes Holschuh [EMAIL PROTECTED]
Signed-off-by: Len Brown [EMAIL PROTECTED]
---
 drivers/acpi/Kconfig|   11 ---
 drivers/acpi/ibm_acpi.c |   13 +
 2 files changed, 1 insertions(+), 23 deletions(-)

diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig
index 1639998..f4f000a 100644
--- a/drivers/acpi/Kconfig
+++ b/drivers/acpi/Kconfig
@@ -225,17 +225,6 @@ config ACPI_IBM_DOCK
 
  If you are not sure, say N here.
 
-config ACPI_IBM_BAY
-   bool Legacy Removable Bay Support
-   depends on ACPI_IBM
-   depends on ACPI_BAY=n
-   default n
-   ---help---
- Allows the ibm_acpi driver to handle removable bays.
- This support is obsoleted by CONFIG_ACPI_BAY.
-
- If you are not sure, say N here.
-
 config ACPI_TOSHIBA
tristate Toshiba Laptop Extras
depends on X86
diff --git a/drivers/acpi/ibm_acpi.c b/drivers/acpi/ibm_acpi.c
index b72d13d..c6144ca 100644
--- a/drivers/acpi/ibm_acpi.c
+++ b/drivers/acpi/ibm_acpi.c
@@ -157,7 +157,6 @@ IBM_HANDLE(dock, root, \\_SB.GDCK,/* X30, X31, 
X40 */
   \\_SB.PCI.ISA.SLCE,/* 570 */
 ); /* A21e,G4x,R30,R31,R32,R40,R40e,R50e */
 #endif
-#ifdef CONFIG_ACPI_IBM_BAY
 IBM_HANDLE(bay, root, \\_SB.PCI.IDE.SECN.MAST,   /* 570 */
   \\_SB.PCI0.IDE0.IDES.IDSM, /* 600e/x, 770e, 770x */
   \\_SB.PCI0.SATA.SCND.MSTR, /* T60, X60, Z60 */ 
@@ -175,7 +174,6 @@ IBM_HANDLE(bay2, root, \\_SB.PCI0.IDE0.PRIM.SLAV, /* A3x, 
R32 */
 IBM_HANDLE(bay2_ej, bay2, _EJ3,  /* 600e/x, 770e, A3x */
   _EJ0,  /* 770x */
 ); /* all others */
-#endif
 
 /* don't list other alternatives as we install a notify handler on the 570 */
 IBM_HANDLE(pci, root, \\_SB.PCI);/* 570 */
@@ -1042,7 +1040,6 @@ static int light_write(char *buf)
return 0;
 }
 
-#if defined(CONFIG_ACPI_IBM_DOCK) || defined(CONFIG_ACPI_IBM_BAY)
 static int _sta(acpi_handle handle)
 {
int status;
@@ -1052,7 +1049,7 @@ static int _sta(acpi_handle handle)
 
return status;
 }
-#endif
+
 #ifdef CONFIG_ACPI_IBM_DOCK
 #define dock_docked() (_sta(dock_handle)  1)
 
@@ -1118,7 +1115,6 @@ static void dock_notify(struct ibm_struct *ibm, u32 event)
 }
 #endif
 
-#ifdef CONFIG_ACPI_IBM_BAY
 static int bay_status_supported;
 static int bay_status2_supported;
 static int bay_eject_supported;
@@ -1194,7 +1190,6 @@ static void bay_notify(struct ibm_struct *ibm, u32 event)
 {
acpi_bus_generate_event(ibm-device, event, 0);
 }
-#endif
 
 static int cmos_read(char *p)
 {
@@ -2354,7 +2349,6 @@ static struct ibm_struct ibms[] = {
 .type = ACPI_SYSTEM_NOTIFY,
 },
 #endif
-#ifdef CONFIG_ACPI_IBM_BAY
{
 .name = bay,
 .init = bay_init,
@@ -2364,7 +2358,6 @@ static struct ibm_struct ibms[] = {
 .handle = bay_handle,
 .type = ACPI_SYSTEM_NOTIFY,
 },
-#endif
{
 .name = cmos,
 .read = cmos_read,
@@ -2650,9 +2643,7 @@ IBM_PARAM(light);
 #ifdef CONFIG_ACPI_IBM_DOCK
 IBM_PARAM(dock);
 #endif
-#ifdef CONFIG_ACPI_IBM_BAY
 IBM_PARAM(bay);
-#endif
 IBM_PARAM(cmos);
 IBM_PARAM(led);
 IBM_PARAM(beep);
@@ -2735,14 +2726,12 @@ static int __init acpi_ibm_init(void)
IBM_HANDLE_INIT(dock);
 #endif
IBM_HANDLE_INIT(pci);
-#ifdef CONFIG_ACPI_IBM_BAY
IBM_HANDLE_INIT(bay);
if (bay_handle)
IBM_HANDLE_INIT(bay_ej);
IBM_HANDLE_INIT(bay2);
if (bay2_handle)
IBM_HANDLE_INIT(bay2_ej);
-#endif
IBM_HANDLE_INIT(beep);
IBM_HANDLE_INIT(ecrd);
IBM_HANDLE_INIT(ecwr);
-
To unsubscribe from this list: send the line unsubscribe git-commits-head in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


ACPI: ibm-acpi: make non-generic bay support optional

2006-12-22 Thread Linux Kernel Mailing List
Gitweb: 
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=2df910b4c3edcce9a0c12394db6f5f4a6e69c712
Commit: 2df910b4c3edcce9a0c12394db6f5f4a6e69c712
Parent: e0298997acdba929e7f5b5987d305b67b50a3969
Author: Henrique de Moraes Holschuh [EMAIL PROTECTED]
AuthorDate: Sat Nov 25 16:35:09 2006 -0200
Committer:  Len Brown [EMAIL PROTECTED]
CommitDate: Thu Dec 7 01:38:45 2006 -0500

ACPI: ibm-acpi: make non-generic bay support optional

This patch makes it possible to disable ibm-acpi non-generic bay support,
as generic bay support already works well for a number of ThinkPads.

Signed-off-by: Henrique de Moraes Holschuh [EMAIL PROTECTED]
---
 drivers/acpi/Kconfig|   11 +++
 drivers/acpi/ibm_acpi.c |   13 -
 2 files changed, 23 insertions(+), 1 deletions(-)

diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig
index 59f9def..6f8c50e 100644
--- a/drivers/acpi/Kconfig
+++ b/drivers/acpi/Kconfig
@@ -224,6 +224,17 @@ config ACPI_IBM_DOCK
 
  If you are not sure, say N here.
 
+config ACPI_IBM_BAY
+   bool Legacy Removable Bay Support
+   depends on ACPI_IBM
+   depends on ACPI_BAY=n
+   default n
+   ---help---
+ Allows the ibm_acpi driver to handle removable bays.
+ This support is obsoleted by CONFIG_ACPI_BAY.
+
+ If you are not sure, say N here.
+
 config ACPI_TOSHIBA
tristate Toshiba Laptop Extras
depends on X86
diff --git a/drivers/acpi/ibm_acpi.c b/drivers/acpi/ibm_acpi.c
index 392abbb..fbb4970 100644
--- a/drivers/acpi/ibm_acpi.c
+++ b/drivers/acpi/ibm_acpi.c
@@ -172,6 +172,7 @@ IBM_HANDLE(dock, root, \\_SB.GDCK,/* X30, X31, 
X40 */
   \\_SB.PCI.ISA.SLCE,/* 570 */
 ); /* A21e,G4x,R30,R31,R32,R40,R40e,R50e */
 #endif
+#ifdef CONFIG_ACPI_IBM_BAY
 IBM_HANDLE(bay, root, \\_SB.PCI.IDE.SECN.MAST,   /* 570 */
   \\_SB.PCI0.IDE0.IDES.IDSM, /* 600e/x, 770e, 770x */
   \\_SB.PCI0.SATA.SCND.MSTR, /* T60, X60, Z60 */ 
@@ -189,6 +190,7 @@ IBM_HANDLE(bay2, root, \\_SB.PCI0.IDE0.PRIM.SLAV, /* A3x, 
R32 */
 IBM_HANDLE(bay2_ej, bay2, _EJ3,  /* 600e/x, 770e, A3x */
   _EJ0,  /* 770x */
 ); /* all others */
+#endif
 
 /* don't list other alternatives as we install a notify handler on the 570 */
 IBM_HANDLE(pci, root, \\_SB.PCI);/* 570 */
@@ -1051,6 +1053,7 @@ static int light_write(char *buf)
return 0;
 }
 
+#if defined(CONFIG_ACPI_IBM_DOCK) || defined(CONFIG_ACPI_IBM_BAY)
 static int _sta(acpi_handle handle)
 {
int status;
@@ -1060,7 +1063,7 @@ static int _sta(acpi_handle handle)
 
return status;
 }
-
+#endif
 #ifdef CONFIG_ACPI_IBM_DOCK
 #define dock_docked() (_sta(dock_handle)  1)
 
@@ -1126,6 +1129,7 @@ static void dock_notify(struct ibm_struct *ibm, u32 event)
 }
 #endif
 
+#ifdef CONFIG_ACPI_IBM_BAY
 static int bay_status_supported;
 static int bay_status2_supported;
 static int bay_eject_supported;
@@ -1201,6 +1205,7 @@ static void bay_notify(struct ibm_struct *ibm, u32 event)
 {
acpi_bus_generate_event(ibm-device, event, 0);
 }
+#endif
 
 static int cmos_read(char *p)
 {
@@ -2330,6 +2335,7 @@ static struct ibm_struct ibms[] = {
 .type = ACPI_SYSTEM_NOTIFY,
 },
 #endif
+#ifdef CONFIG_ACPI_IBM_BAY
{
 .name = bay,
 .init = bay_init,
@@ -2339,6 +2345,7 @@ static struct ibm_struct ibms[] = {
 .handle = bay_handle,
 .type = ACPI_SYSTEM_NOTIFY,
 },
+#endif
{
 .name = cmos,
 .read = cmos_read,
@@ -2624,7 +2631,9 @@ IBM_PARAM(light);
 #ifdef CONFIG_ACPI_IBM_DOCK
 IBM_PARAM(dock);
 #endif
+#ifdef CONFIG_ACPI_IBM_BAY
 IBM_PARAM(bay);
+#endif
 IBM_PARAM(cmos);
 IBM_PARAM(led);
 IBM_PARAM(beep);
@@ -2717,12 +2726,14 @@ static int __init acpi_ibm_init(void)
IBM_HANDLE_INIT(dock);
 #endif
IBM_HANDLE_INIT(pci);
+#ifdef CONFIG_ACPI_IBM_BAY
IBM_HANDLE_INIT(bay);
if (bay_handle)
IBM_HANDLE_INIT(bay_ej);
IBM_HANDLE_INIT(bay2);
if (bay2_handle)
IBM_HANDLE_INIT(bay2_ej);
+#endif
IBM_HANDLE_INIT(beep);
IBM_HANDLE_INIT(ecrd);
IBM_HANDLE_INIT(ecwr);
-
To unsubscribe from this list: send the line unsubscribe git-commits-head in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html