diff mbox series

[v2,1/3] ipmi: Fix SEL get/set time commands

Message ID 1503265602-9349-2-git-send-email-minyard@acm.org
State Superseded
Headers show
Series [v2,1/3] ipmi: Fix SEL get/set time commands | expand

Commit Message

Corey Minyard Aug. 20, 2017, 9:46 p.m. UTC
From: Corey Minyard <cminyard@mvista.com>


The minimum message size was on the wrong commands, for getting
the time it's zero and for setting the time it's 6.

Signed-off-by: Corey Minyard <cminyard@mvista.com>

---
 hw/ipmi/ipmi_bmc_sim.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

-- 
2.7.4

Comments

Cédric Le Goater Aug. 21, 2017, 7:28 a.m. UTC | #1
On 08/20/2017 11:46 PM, minyard@acm.org wrote:
> From: Corey Minyard <cminyard@mvista.com>

> 

> The minimum message size was on the wrong commands, for getting

> the time it's zero and for setting the time it's 6.

> 

> Signed-off-by: Corey Minyard <cminyard@mvista.com>


Reviewed-by: Cédric Le Goater <clg@kaod.org>

> ---

>  hw/ipmi/ipmi_bmc_sim.c | 4 ++--

>  1 file changed, 2 insertions(+), 2 deletions(-)

> 

> diff --git a/hw/ipmi/ipmi_bmc_sim.c b/hw/ipmi/ipmi_bmc_sim.c

> index 277c28c..cc068f2 100644

> --- a/hw/ipmi/ipmi_bmc_sim.c

> +++ b/hw/ipmi/ipmi_bmc_sim.c

> @@ -1802,8 +1802,8 @@ static const IPMICmdHandler storage_cmds[] = {

>      [IPMI_CMD_GET_SEL_ENTRY] = { get_sel_entry, 8 },

>      [IPMI_CMD_ADD_SEL_ENTRY] = { add_sel_entry, 18 },

>      [IPMI_CMD_CLEAR_SEL] = { clear_sel, 8 },

> -    [IPMI_CMD_GET_SEL_TIME] = { get_sel_time, 6 },

> -    [IPMI_CMD_SET_SEL_TIME] = { set_sel_time },

> +    [IPMI_CMD_GET_SEL_TIME] = { get_sel_time },

> +    [IPMI_CMD_SET_SEL_TIME] = { set_sel_time, 6 },

>  };

>  

>  static const IPMINetfn storage_netfn = {

>
diff mbox series

Patch

diff --git a/hw/ipmi/ipmi_bmc_sim.c b/hw/ipmi/ipmi_bmc_sim.c
index 277c28c..cc068f2 100644
--- a/hw/ipmi/ipmi_bmc_sim.c
+++ b/hw/ipmi/ipmi_bmc_sim.c
@@ -1802,8 +1802,8 @@  static const IPMICmdHandler storage_cmds[] = {
     [IPMI_CMD_GET_SEL_ENTRY] = { get_sel_entry, 8 },
     [IPMI_CMD_ADD_SEL_ENTRY] = { add_sel_entry, 18 },
     [IPMI_CMD_CLEAR_SEL] = { clear_sel, 8 },
-    [IPMI_CMD_GET_SEL_TIME] = { get_sel_time, 6 },
-    [IPMI_CMD_SET_SEL_TIME] = { set_sel_time },
+    [IPMI_CMD_GET_SEL_TIME] = { get_sel_time },
+    [IPMI_CMD_SET_SEL_TIME] = { set_sel_time, 6 },
 };
 
 static const IPMINetfn storage_netfn = {