[lkml]   [2009]   [Aug]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    Patch in this message
    Subject[patch 27/74] thinkpad-acpi: disable broken bay and dock subdrivers
    2.6.30-stable review patch.  If anyone has any objections, please let us know.


    From: Henrique de Moraes Holschuh <>

    commit 550e7fd8afb7664ae7cedb398c407694e2bf7d3c upstream.

    Currently, the ThinkPad-ACPI bay and dock drivers are completely
    broken, and cause a NULL pointer derreference in kernel mode (and,
    therefore, an OOPS) when they try to issue events (i.e. on dock,
    undock, bay ejection, etc).

    OTOH, the standard ACPI dock driver can handle the hotplug bays and
    docks of the ThinkPads just fine (including batteries) as of 2.6.27.
    In fact, it does a much better job of it than thinkpad-acpi ever did.

    It is just not worth the hassle to find a way to fix this crap without
    breaking the (deprecated) thinkpad-acpi dock/bay ABI. This is old,
    deprecated code that sees little testing or use.

    As a quick fix suitable for -stable backports, mark the thinkpad-acpi
    bay and dock subdrivers as BROKEN in Kconfig. The dead code will be
    removed by a later patch.

    This fixes bugzilla #13669, and should be applied to 2.6.27 and later.

    Signed-off-by: Henrique de Moraes Holschuh <>
    Reported-by: Joerg Platte <>
    Signed-off-by: Len Brown <>
    Signed-off-by: Greg Kroah-Hartman <>

    drivers/platform/x86/Kconfig | 4 +++-
    1 file changed, 3 insertions(+), 1 deletion(-)

    --- a/drivers/platform/x86/Kconfig
    +++ b/drivers/platform/x86/Kconfig
    @@ -265,6 +265,7 @@ config THINKPAD_ACPI_DOCK
    bool "Legacy Docking Station Support"
    depends on THINKPAD_ACPI
    depends on ACPI_DOCK=n
    + depends on BROKEN
    default n
    Allows the thinkpad_acpi driver to handle docking station events.
    @@ -278,7 +279,8 @@ config THINKPAD_ACPI_DOCK
    bool "Legacy Removable Bay Support"
    depends on THINKPAD_ACPI
    - default y
    + depends on BROKEN
    + default n
    Allows the thinkpad_acpi driver to handle removable bays. It will
    electrically disable the device in the bay, and also generate

     \ /
      Last update: 2009-08-13 22:03    [W:0.022 / U:44.088 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site