On Thu, 15 Mar 2007 12:17:21 -0700 Chris Wedgwood <cw@xxxxxxxx> wrote: > On Thu, Mar 15, 2007 at 02:51:14PM -0300, Henrique de Moraes Holschuh wrote: > > > This patch allows for ibm-acpi to coexist (with diminished > > functionality) with other drivers like ACPI_BAY. > > Given the ACP_IBM_BAY implementation is more complete (or seems to be, > please comment if that isn't the case) we should probably actually > make sure that is the *preferred* code used (on suitable hardware) at > run time surely? I haven't followed the ibm_acpi development lately, but when I first wrote the bay driver, it had a couple features that ibm_acpi didn't have, and then of course, is missing some it does have. What used to be the case is that if bay was inserted after the laptop was booted, the ibm_acpi driver would not capture the events because it would only install it's notifier at driver load time on bay devices that existed. The second case is that if you are using the generic dock driver (which is more funtional than the ibm_acpi dock feature), for certain ibm laptops such as the X60, you need to have undock events sent to the bay device so that it can alert user space to unmount cd's etc (since the bay is on the ultrabase). > > That way distributions can build both options and on IBM/Lenovo > hardware it will use the IBM ACPI code and otherwise will use the > generic ACPI BAY code? I think for now this is a good idea - the ibm_acpi solution should be preferred over the generic bay driver till it is more fully developed. I don't know how to ensure this is done wrt load order though. > > Perhaps the IBM_ACPI_BAY code should go away and any missing > functionality provided there should be merged into ACPI_BAY? > I think this is a good idea. It would certainly be less confusing for users. too late for this release though, perhaps this can be targetted at 2.6.22/23? - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
