diff options
| author | Liam Merwick <liam.merwick@oracle.com> | 2019-02-15 13:35:19 +0000 |
|---|---|---|
| committer | Stefan Berger <stefanb@linux.vnet.ibm.com> | 2019-02-24 14:46:14 -0500 |
| commit | cd38cc519b60bbe5dd8d85f6aaa72b2d7902930b (patch) | |
| tree | d2f280458b2d2173fff480d7944bffe5af8cd6ad /hw/tpm/trace-events | |
| parent | 37b55d67c0f001b20b7831db3f9f24f1d453e1de (diff) | |
tpm_tis: convert tpm_tis_show_buffer() to use trace event
cppcheck reports:
[hw/tpm/tpm_tis.c:113]: (warning) %d in format string (no. 2) requires 'int' but the argument type is 'unsigned int'
Rather than just converting the format specifier to use '%u", the
tpm_tis_show_buffer() function is converted to use trace points and
the two debug callers use the trace event infrastructure so that it's
available in production cases also and not just when DEBUG_TIS is enabled.
Signed-off-by: Liam Merwick <liam.merwick@oracle.com>
Reviewed-by: Stefan Berger <stefanb@linux.ibm.com>
Signed-off-by: Stefan Berger <stefanb@linux.ibm.com>
Diffstat (limited to 'hw/tpm/trace-events')
| -rw-r--r-- | hw/tpm/trace-events | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/hw/tpm/trace-events b/hw/tpm/trace-events index 920d32ad55..f45dcd2209 100644 --- a/hw/tpm/trace-events +++ b/hw/tpm/trace-events @@ -36,6 +36,7 @@ tpm_emulator_pre_save(void) "" tpm_emulator_inst_init(void) "" # hw/tpm/tpm_tis.c +tpm_tis_show_buffer(const char *direction, size_t len, const char *buf) "direction: %s len: %zu\nbuf: %s" tpm_tis_raise_irq(uint32_t irqmask) "Raising IRQ for flag 0x%08x" tpm_tis_new_active_locality(uint8_t locty) "Active locality is now %d" tpm_tis_abort(uint8_t locty) "New active locality is %d" |