[PATCH 0/1] UBUNTU: Add hdaps_ec driver to support newer ThinkPads (Hard Drive Active Protection System)
stefan.bader at canonical.com
Mon Apr 27 07:43:34 UTC 2009
Tim Gardner wrote:
> Brad Figg wrote:
>> Please pull from :
>> git://kernel.ubuntu.com/bradf/ubuntu-karmic master
>> Bug: #297213
>> Hardy had two drivers which handled the HardDisk Active
>> Protection System (hdaps and hdaps_ec) which is present in IBM
>> ThinkPads. One of the two drivers was dropped when Intrepid
>> development began and due to that there are a number of ThinkPads
>> (newer models) for which this is not supported.
>> This patch brings back in the missing driver.
>> The code for this patch was taken from http://tpctl.sourceforge.net
>> and is the 0.40 release of December 16, 2008.
>> Brad Figg (1):
>> UBUNTU: Add hdaps_ec driver to support newer ThinkPads (Hard Drive
>> Active Protection System)
>> ubuntu/misc/Kconfig | 13 +
>> ubuntu/misc/Makefile | 2 +-
>> ubuntu/misc/hdaps_ec.c | 880 ++++++++++++++++++++++++++++++++++++++++++++++++
>> 3 files changed, 894 insertions(+), 1 deletions(-)
>> create mode 100644 ubuntu/misc/hdaps_ec.c
> How is this one better then drivers/hwmon/hdaps.c ? The in-kernel driver
> appears to contain a superset of DMI table information, i.e., supported
> platforms. Furthermore, it corrects some problems that exist in
> ubuntu/misc/hdaps_ec.c, such as I/O space registration, only loads if at
> least one DMI match is found, and uses in-kernel functionality for
> polled input devices.
> So, NAK from me unless you can convince me otherwise.
This might not be completely true anymore but at the time I did that driver for
Hardy LUM the following was:
- The in kernel driver could not do independent axis inversion which was
required for some models
- The driver from tpctl would use thinkpad_ec to channel acces to the embedded
controller, which allowed the other tool from that package concurrent access.
Upstreaming sounded always like "soonish" without anything happening. The
problem is/was the questionable origins (unclear). As we not necessarily need
the other part of tpctl, on option would be to update the in kernel driver to
support more models (do the additional inversion logic and merge the DMi
information). An interesting question there would be whether access to the ec
needs syncing with other stuff (like thinpad-acpi)
When all other means of communication fail, try words!
More information about the kernel-team