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: Jeremy Kerr <jk@ozlabs.org>

diff --git a/discover/platform-powerpc.c b/discover/platform-powerpc.c
index 4cc91fa2594eeac6c2b939ee50c4dc0821fe433a..bda9368c939bc955d37a5bb6c9b3ae60173a22c5 100644 (file)
--- 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 */
        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 */
                0x00, /* sensor reading: none */
                0x40, /* assertion mask lsb: set state 6 */
                0x00, /* assertion mask msb: none */