diff options
author | Joel Stanley <joel@jms.id.au> | 2015-04-30 16:02:03 +1000 |
---|---|---|
committer | Samuel Mendoza-Jonas <sam.mj@au1.ibm.com> | 2015-04-30 16:23:20 +1000 |
commit | 11996807cd4e79e35742aa4c4e19c6500141dfcc (patch) | |
tree | c87966770e0b78287ce15a96571f3ce807b434b4 | |
parent | 2598810aea74bf2f614d9e7b7cbb7c086a1fdc95 (diff) | |
download | talos-petitboot-11996807cd4e79e35742aa4c4e19c6500141dfcc.tar.gz talos-petitboot-11996807cd4e79e35742aa4c4e19c6500141dfcc.zip |
discover: Set OS Boot sensor with 0xa9
If we don't set all the things, we only get an assertion event, without
the state being asserted.
Despite the IPMI spec suggesting we only needed to set 0x10, aka the
assertion bits, testing reveals we want to set all of the things to
cause the BMC to assert the state.
Tested-by: Nick Bofferding <bofferdn@us.ibm.com>
Signed-off-by: Joel Stanley <joel@jms.id.au>
Signed-off-by: Samuel Mendoza-Jonas <sam.mj@au1.ibm.com>
-rw-r--r-- | discover/platform-powerpc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/discover/platform-powerpc.c b/discover/platform-powerpc.c index 4cc91fa..bda9368 100644 --- a/discover/platform-powerpc.c +++ b/discover/platform-powerpc.c @@ -802,7 +802,7 @@ static int set_ipmi_os_boot_sensor(struct platform_powerpc *platform) uint8_t resp[1]; uint8_t req[] = { 0x00, /* sensor number: os boot */ - 0x10, /* operation: set assertion bits */ + 0xA9, /* operation: set everything */ 0x00, /* sensor reading: none */ 0x40, /* assertion mask lsb: set state 6 */ 0x00, /* assertion mask msb: none */ |