Notes: 0.40 2008-12-16 --------------------- - thinkpad_ec: Added a "force_io=1" module parameter, to let it load on recent ThinkPad models (e.g., T400 and T500). This is a kludge to work around an ACPI DSDT which claims the ports we need. - tp_smapi: Don't restore battery charging thresholds when resuming from suspend-to-RAM. - When building, use /lib/modules/*/build instead of /lib/modules/*/src and give a clearer error message if that's missing.
would be nice if you could fix bug #215967 as it is part of that package and uses existing USE flag as an optional switch to install it.
Created attachment 177719 [details] Proposed ebuild for tp_smapi-0.40 This is modified from the 0.39 ebuild. It compiles and runs properly for me with the hdaps use flag on amd64. I run a T400 that needs the force_io=1 module parameter introduced in 0.40.
Created attachment 177733 [details] tp_smapi-0.40, EAPI2 Not sure if we still need RESTRICT="userpriv" It might be also possible to check for CONFIG_DMI* (?) in .config before doing DEPEND. It should not strictly depend on it.
Created attachment 179758 [details] tp_smapi-0.40, EAPI2 the patch from bug #219285 applied
I have a 2.6.28 kernel. I installed tp_smapi with USE="hdaps". however, in the init file /etc/init.d/hdapsd, there were the following lines if [[ ! -e /sys/block/${DISK}/queue/protect ]] ; then eerror "No protect entry for ${DISK}!" eerror "Make sure your kernel is patched with the blk_freeze patch" return 1 fi Obviously, Kernel 2.6.28 doesn't neet that check. In fact, it produces errors and does not allow, hdapsd to start. hdapsd works on my system, if I start it with hdapsd -d sda -s 12 -a. It is just that check in /etc/init.d/hdapsd which does not work. So, I commented the above lines out. Disk-shock protection now works on my system.
Yes, you already reported in the bug #166166. Please stop spamming. You should use my overlay, the attach #175470 in the bug above or suggest your own.
This is not intendet as spam. The problem is, that I don't know which ebuild installs the file in /etc/init.d/hdapsd I do not speak of a test in an ebuild file. But of a test in a config file, which prevents the hdapsd daemon to be executed. However, I don't know which ebuild does install the /etc/init.d/hdapsd file....
another point: Looking at the source-code of tha hdapsd drivers from the Kernel and tp_smapi shows, that they updated the kernel driver somewhat. The Kernel driver now seems to support more thinkpads than the hdaps module from tp_smapi. Although the kernel driver still uses the sys interface. Furthermore, even if the hdaps module is loaded before tp_smapi (as suggested in the readme), tp_smapi seems to refuse to work with the new kernel driver. Since it seems that the hdaps module from kernel supports more thinkpads now, I would suggest, that the hdapsd driver from tp_smapi is updated a bit. At least it should support the same models as the driver from kernel.
Created attachment 183118 [details] tp_smapi-0.40 Don't provide udev rule any more. It's provided by app-laptop/hdapsd
(In reply to comment #8) hdapsd.init file belongs to app-laptop/hdapsd, you can verify it by running: equery b /etc/init.d/hdapsd hdaps use flag is optional and allows you to choose which HDAPS module (kernel or tp_smapi's hacked version) to use. I suggest you to contact upstream if have any suggestions on improvement any of them. And thanks for the note. I have updated ebuilds to make that option clearer.
app-laptop/tp_smapi-0.40 is now in the tree with a fix for bug #219285.