diff options
author | Joel Stanley <joel@jms.id.au> | 2014-09-11 15:43:50 +0930 |
---|---|---|
committer | Benjamin Herrenschmidt <benh@kernel.crashing.org> | 2014-10-01 14:23:06 +1000 |
commit | ae5fb8512e8ecfc1f1b24f5f89ef373dc4502844 (patch) | |
tree | a890b2891db1628f51229821c7ce4a49e60ef18d /include | |
parent | d0521e8074012f368033879b1b6e09c89e41b379 (diff) | |
download | skiboot-ae5fb8512e8ecfc1f1b24f5f89ef373dc4502844.zip skiboot-ae5fb8512e8ecfc1f1b24f5f89ef373dc4502844.tar.gz skiboot-ae5fb8512e8ecfc1f1b24f5f89ef373dc4502844.tar.bz2 |
ipmi/power: Update chassis control command
Linux sends us a 0 when shutting down. This means we don't need to pass
the u64 to the IPMI driver. Add a check that the value is what we expect
in case Linux changes it's behaviour in the future.
When rebooting, we should send the BMC a HARD_RESET command (0x03), not
POWER_CYCLE (0x02).
While we are here, trim some whitespace and drop opal from the IPMI
function name for readability.
Signed-off-by: Joel Stanley <joel@jms.id.au>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Diffstat (limited to 'include')
-rw-r--r-- | include/ipmi.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/ipmi.h b/include/ipmi.h index 643dbf4..3bf343f 100644 --- a/include/ipmi.h +++ b/include/ipmi.h @@ -135,8 +135,8 @@ int ipmi_queue_msg(struct ipmi_msg *msg); /* Process a completed message */ void ipmi_cmd_done(struct ipmi_msg *msg); -/* Change the power state of the P8 */ -int64_t ipmi_opal_chassis_control(uint64_t request); +/* 28.3 Chassis Control Command. Changes the power state of the P8. */ +int ipmi_chassis_control(uint8_t request); /* Register a backend with the ipmi core. Currently we only support one. */ void ipmi_register_backend(struct ipmi_backend *backend); |