[PATCH] discover: Set OS Boot sensor with 0xa9

Joel Stanley joel at jms.id.au
Thu Apr 30 16:02:03 AEST 2015


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 at us.ibm.com>
Signed-off-by: Joel Stanley <joel at jms.id.au>
---
 discover/platform-powerpc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

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 */
-- 
2.1.4



More information about the Petitboot mailing list