* Frans Pop <[EMAIL PROTECTED]> [2008-08-10 21:30]:
> I've added support for multiple configuration files and have already
> tested that upgrading/downgrading qcontrol will work correctly.
> 
> New version is available at http://people.debian.org/~fjp/qcontrol/.

/etc/qcontrol/ts209.lua contains;
        116, "restart_button",
        408, "media_button")

116 is KEY_POWER, 408 is KEY_RESTART.  This is what the kernel used
for these buttons in the past.  But a patch got accepted a few days
ago to change them to something that makes more sense: KEY_RESTART
(408) for restart_button and KEY_COPY (133) for media_button.  The
Debian kernel (2.6.26) already has this patch.

So this should be:
        408, "restart_button",
        133, "media_button")

Same goes for the ts409.lua file.

/etc/qcontrol/ts409.lua defines temp_low and temp_high.  This is what
the TS-209 uses but the TS0-409 uses a generic temp function.  So
these two (temp_low, temp_high) can be removed from ts409.lua.
However, it needs the following line:

function temp( temp )
        print("ts409 temperature:", temp)
end

Otherwise it won't work.

BTW, I wondered about something the other day: why does
/usr/lib/finish-install.d/21qcontrol modify /etc/modules rather than
/usr/lib/base-installer.d/10qcontrol?

changelog: +    - Add support for QNAP TS-409.
This should close the bug report.

qcontrol-0.4.2/debian/README.Debian
+Because of the missing daemon mode, the functionality to monitor temperature
+and control fan speed (TS-209 only) have been disabled in the config file.

This should mention TS-409.

> Can you please test if this works for the TS-409 (both in D-I and for an
> installed system) before I upload?

After adding
function temp( temp )
        print("ts409 temperature:", temp)
end
to ts409.lua, it works correctly, both in d-i and the installed
system.

Thanks a lot!
-- 
Martin Michlmayr
http://www.cyrius.com/



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to