tpm: Tidy up logging in tpm-common.c
authorSimon Glass <sjg@chromium.org>
Mon, 1 Oct 2018 18:22:28 +0000 (12:22 -0600)
committerSimon Glass <sjg@chromium.org>
Tue, 9 Oct 2018 10:40:27 +0000 (04:40 -0600)
At present this file uses logging but it should use the new macros. Update
it and add a log message for an error.

Signed-off-by: Simon Glass <sjg@chromium.org>
lib/tpm-common.c

index 43b530865a0c4e7c01d32d66f5f43fe1702afe83..a440639cec302fc23d7a0fc31f2f82e2ab9f12ed 100644 (file)
@@ -4,6 +4,8 @@
  * Coypright (c) 2013 Guntermann & Drunck GmbH
  */
 
+#define LOG_CATEGORY UCLASS_TPM
+
 #include <common.h>
 #include <dm.h>
 #include <asm/unaligned.h>
@@ -110,6 +112,8 @@ int unpack_byte_string(const u8 *str, size_t size, const char *format, ...)
 
                if (offset + length > size) {
                        va_end(args);
+                       log_err("Failed to read: size=%d, offset=%x, len=%x\n",
+                               size, offset, length);
                        return -1;
                }
 
@@ -176,10 +180,10 @@ u32 tpm_sendrecv_command(const void *command, void *response, size_t *size_ptr)
 
        ret = tpm_return_code(response);
 
-       log(LOGC_NONE, LOGL_DEBUG, "TPM response [ret:%d]: ", ret);
+       log_debug("TPM response [ret:%d]: ", ret);
        for (i = 0; i < response_length; i++)
-               log(LOGC_NONE, LOGL_DEBUG, "%02x ", ((u8 *)response)[i]);
-       log(LOGC_NONE, LOGL_DEBUG, "\n");
+               log_debug("%02x ", ((u8 *)response)[i]);
+       log_debug("\n");
 
        return ret;
 }