diff options
| author | Stewart Smith <stewart@linux.vnet.ibm.com> | 2016-06-20 17:51:13 +1000 |
|---|---|---|
| committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2016-06-20 17:51:13 +1000 |
| commit | 00e3e275344a42f6a682be72c88c015df87a0e28 (patch) | |
| tree | faade3c82e5bb09e9bd4e5331ebec130a2bb81ee /hw/npu-hw-procedures.c | |
| parent | 6865d6f04e7a5e12db2b365f899dc8a902fc304c (diff) | |
| download | skiboot-00e3e275344a42f6a682be72c88c015df87a0e28.zip skiboot-00e3e275344a42f6a682be72c88c015df87a0e28.tar.gz skiboot-00e3e275344a42f6a682be72c88c015df87a0e28.tar.bz2 | |
fwts: Add FWTS annotations for NPU errors
We also remove the NPUERR macros so that the FWTS parsing magic
can construct find the prlog statements.
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'hw/npu-hw-procedures.c')
| -rw-r--r-- | hw/npu-hw-procedures.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/hw/npu-hw-procedures.c b/hw/npu-hw-procedures.c index 4dbb4ba..cc3dcad 100644 --- a/hw/npu-hw-procedures.c +++ b/hw/npu-hw-procedures.c @@ -516,7 +516,7 @@ int64_t npu_dev_procedure_read(struct npu_dev_trap *trap, if (size != 4) { /* Short config reads are not supported */ - NPUDEVERR(dev, "Short read of procedure register\n"); + prlog(PR_ERR, "NPU%d: Short read of procedure register\n", dev->npu->phb.opal_id); return OPAL_PARAMETER; } @@ -538,8 +538,8 @@ int64_t npu_dev_procedure_read(struct npu_dev_trap *trap, break; default: - NPUDEVERR(dev, "Invalid vendor specific offset 0x%08x\n", - offset); + prlog(PR_ERR, "NPU%d: Invalid vendor specific offset 0x%08x\n", + dev->npu->phb.opal_id, offset); rc = OPAL_PARAMETER; } @@ -557,7 +557,8 @@ int64_t npu_dev_procedure_write(struct npu_dev_trap *trap, if (size != 4) { /* Short config writes are not supported */ - NPUDEVERR(dev, "Short read of procedure register\n"); + prlog(PR_ERR, "NPU%d: Short read of procedure register\n", + dev->npu->phb.opal_id); return OPAL_PARAMETER; } |