diff options
author | Vasant Hegde <hegdevasant@linux.vnet.ibm.com> | 2020-10-19 17:14:08 +0530 |
---|---|---|
committer | Vasant Hegde <hegdevasant@linux.vnet.ibm.com> | 2020-10-22 14:30:50 +0530 |
commit | d3d0a349d48cfad5dba45cf78cb3b0c92155867e (patch) | |
tree | b1f4cf1695eb677f4f19c9865c506dc6cf04dab3 | |
parent | 9a094b60bfc56dbd56c6aeccede8532c0188dd10 (diff) | |
download | skiboot-d3d0a349d48cfad5dba45cf78cb3b0c92155867e.zip skiboot-d3d0a349d48cfad5dba45cf78cb3b0c92155867e.tar.gz skiboot-d3d0a349d48cfad5dba45cf78cb3b0c92155867e.tar.bz2 |
FSP/NVRAM: Do not assert in vNVRAM statistics call
[ Upstream commit 9ca8bf1bde56330075634bd3cb601d0f6ee90514 ]
`msg` is valid pointer here. I don't recall why I added assert here :-(
This is not correct. We shouldn't call assert here. Also we are not using
`msg`. Hence convert it to `__unused`.
Fixes: 19d4f98e ('FSP/NVRAM: Handle "get vNVRAM statistics" command')
Cc: skiboot-stable@lists.ozlabs.org # v5.4.x +
Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com>
Reviewed-by: Mahesh Salgaonkar <mahesh@linux.ibm.com>
Signed-off-by: Oliver O'Halloran <oohall@gmail.com>
Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com>
-rw-r--r-- | hw/fsp/fsp-nvram.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/hw/fsp/fsp-nvram.c b/hw/fsp/fsp-nvram.c index 1b4990f..4a583f9 100644 --- a/hw/fsp/fsp-nvram.c +++ b/hw/fsp/fsp-nvram.c @@ -310,12 +310,11 @@ static struct fsp_client fsp_nvram_client_rr = { .message = fsp_nvram_msg_rr, }; -static bool fsp_vnvram_msg(u32 cmd_sub_mod, struct fsp_msg *msg) +static bool fsp_vnvram_msg(u32 cmd_sub_mod, struct fsp_msg *msg __unused) { u32 cmd; struct fsp_msg *resp; - assert(msg == NULL); switch (cmd_sub_mod) { case FSP_CMD_GET_VNV_STATS: prlog(PR_DEBUG, |