diff options
author | Ananth N Mavinakayanahalli <ananth@in.ibm.com> | 2014-07-31 13:04:53 +0530 |
---|---|---|
committer | Benjamin Herrenschmidt <benh@kernel.crashing.org> | 2014-08-08 16:07:26 +1000 |
commit | aa1f3190cdea6287b8c1f84644bea7e48d09d25a (patch) | |
tree | 76428e92bfc97f74ad955ac9fd6cc0b0223b8c88 | |
parent | 068219597da3610e35afde3c75b0965fc3578545 (diff) | |
download | talos-skiboot-aa1f3190cdea6287b8c1f84644bea7e48d09d25a.tar.gz talos-skiboot-aa1f3190cdea6287b8c1f84644bea7e48d09d25a.zip |
FSP: Make mbx error messages more verbose
... helps figure out what the cause of the failure was.
Signed-off-by: Ananth N Mavinakayanahalli <ananth@in.ibm.com>
Acked-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
-rw-r--r-- | hw/fsp/fsp.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/hw/fsp/fsp.c b/hw/fsp/fsp.c index 75ef2a09..b87c8b36 100644 --- a/hw/fsp/fsp.c +++ b/hw/fsp/fsp.c @@ -1351,9 +1351,10 @@ static void fsp_handle_incoming(struct fsp *fsp) __fsp_drop_incoming(fsp); return; } - - if (!cmdclass->busy || list_empty(&cmdclass->msgq)) { - prerror("FSP #%d: Got orphan response !\n", fsp->index); + + if (!cmdclass->busy || list_empty(&cmdclass->msgq)) { + prerror("FSP #%d: Got orphan response! w0 = 0x%08x w1 = 0x%08x\n", + fsp->index, w0, w1); __fsp_drop_incoming(fsp); return; } @@ -1364,8 +1365,8 @@ static void fsp_handle_incoming(struct fsp *fsp) (req->word0 & 0xff) != (w0 & 0xff) || (req->word1 & 0xff) != (w1 & 0x7f)) { __fsp_drop_incoming(fsp); - prerror("FSP #%d: Response doesn't match pending msg\n", - fsp->index); + prerror("FSP #%d: Response doesn't match pending msg. w0 = 0x%08x w1 = 0x%08x\n", + fsp->index, w0, w1); return; } else { u64 resetbit = ~fsp_get_class_bit(req->word0 & 0xff); |