diff options
| author | Cyril Bur <cyril.bur@au1.ibm.com> | 2017-06-08 17:30:05 +1000 |
|---|---|---|
| committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2017-06-15 13:37:37 +1000 |
| commit | 41ba89da0460ad792ee67986008a4e021599efa6 (patch) | |
| tree | 0bab09c1439371088f935046ef9305f09ba6db6d /hw/occ.c | |
| parent | 106557d9d1888765cf6fccca7fe0dcf2b8fbe7fb (diff) | |
| download | skiboot-41ba89da0460ad792ee67986008a4e021599efa6.zip skiboot-41ba89da0460ad792ee67986008a4e021599efa6.tar.gz skiboot-41ba89da0460ad792ee67986008a4e021599efa6.tar.bz2 | |
occ: Set return variable to correct value
When entering this section of code rc will be zero. If fsp_mkmsg() fails
the code responsible for printing an error message won't be set.
Resetting rc should allow for the error case to trigger if fsp_mkmsg
fails.
Fixes: 3c64f79ad2220d03a0af49d44f0548e08da54a44
Signed-off-by: Cyril Bur <cyril.bur@au1.ibm.com>
[stewart@linux.vnet.ibm.com: add Fixes]
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'hw/occ.c')
| -rw-r--r-- | hw/occ.c | 1 |
1 files changed, 1 insertions, 0 deletions
@@ -1083,6 +1083,7 @@ static void occ_do_load(u8 scope, u32 dbob_id __unused, u32 seq_id) return; if (proc_gen == proc_gen_p9) { + rc = -ENOMEM; /* OCC is pre-loaded in P9, so send SUCCESS to FSP */ rsp = fsp_mkmsg(FSP_CMD_LOAD_OCC_STAT, 2, 0, seq_id); if (rsp) |