Hi Simon

On 11/08/2015 16:48, Simon Glass wrote:
Add driver model support to the TPM command and the TPM library. Both
support only a single TPM at present.

Signed-off-by: Simon Glass <s...@chromium.org>
---

  common/cmd_tpm.c | 26 ++++++++++++++++++++++++++
  include/tpm.h    |  2 +-
  lib/tpm.c        | 29 +++++++++++++++++++++++++----
  3 files changed, 52 insertions(+), 5 deletions(-)

diff --git a/common/cmd_tpm.c b/common/cmd_tpm.c
index 0294952..274e3cd 100644
--- a/common/cmd_tpm.c
+++ b/common/cmd_tpm.c
@@ -6,6 +6,7 @@
#include <common.h>
  #include <command.h>
+#include <dm.h>
  #include <malloc.h>
  #include <tpm.h>
  #include <asm/unaligned.h>
@@ -438,6 +439,21 @@ TPM_COMMAND_NO_ARG(tpm_force_clear)
  TPM_COMMAND_NO_ARG(tpm_physical_enable)
  TPM_COMMAND_NO_ARG(tpm_physical_disable)
+#ifdef CONFIG_DM_TPM
+static int get_tpm(struct udevice **devp)
+{
+       int rc;
+
+       rc = uclass_first_device(UCLASS_TPM, devp);
+       if (rc) {
+               printf("Could not find TPM (ret=%d)\n", rc);
+               return CMD_RET_FAILURE;
+       }
+
+       return 0;
+}
+#endif
+
  static int do_tpm_raw_transfer(cmd_tbl_t *cmdtp, int flag,
                int argc, char * const argv[])
  {
@@ -452,7 +468,17 @@ static int do_tpm_raw_transfer(cmd_tbl_t *cmdtp, int flag,
                return CMD_RET_FAILURE;
        }
+#ifdef CONFIG_DM_TPM
+       struct udevice *dev;
+
+       rc = get_tpm(&dev);
+       if (rc)
+               return rc;
+
+       rc = tis_xfer(dev, command, count, response, &response_length);
+#else
        rc = tis_sendrecv(command, count, response, &response_length);
+#endif
Why don't you retrieve the TPM dev inside a tis_sendrecv function defined only if DM_TPM is enabled ?
As we manage only 1 TPM per platform, it may simplify changes for now ?
        free(command);
        if (!rc) {
                puts("tpm response:\n");
diff --git a/include/tpm.h b/include/tpm.h
index 88aeba2..9a0f87c 100644
--- a/include/tpm.h
+++ b/include/tpm.h
@@ -157,7 +157,7 @@ enum tpm_return_code {
   *
   * @return 0 on success, non-0 on error.
   */
-uint32_t tpm_init(void);
+int tpm_init(void);
/**
   * Issue a TPM_Startup command.
diff --git a/lib/tpm.c b/lib/tpm.c
index d9789b0..9e2cc77 100644
--- a/lib/tpm.c
+++ b/lib/tpm.c
@@ -6,10 +6,11 @@
   */
#include <common.h>
-#include <stdarg.h>
-#include <u-boot/sha1.h>
+#include <dm.h>
+#include <tis.h>
  #include <tpm.h>
  #include <asm/unaligned.h>
+#include <u-boot/sha1.h>
/* Internal error of TPM command library */
  #define TPM_LIB_ERROR ((uint32_t)~0u)
@@ -240,8 +241,19 @@ static uint32_t tpm_sendrecv_command(const void *command,
                response = response_buffer;
                response_length = sizeof(response_buffer);
        }
+#ifdef CONFIG_DM_TPM
+       struct udevice *dev;
+       int ret;
+
+       ret = uclass_first_device(UCLASS_TPM, &dev);
+       if (ret)
+               return ret;
+       err = tis_xfer(dev, command, tpm_command_size(command),
+                      response, &response_length);
+#else
        err = tis_sendrecv(command, tpm_command_size(command),
                        response, &response_length);
+#endif
Same remark than above.
        if (err)
                return TPM_LIB_ERROR;
        if (size_ptr)
@@ -250,15 +262,24 @@ static uint32_t tpm_sendrecv_command(const void *command,
        return tpm_return_code(response);
  }
-uint32_t tpm_init(void)
+int tpm_init(void)
  {
-       uint32_t err;
+       int err;
+#ifdef CONFIG_DM_TPM
+       struct udevice *dev;
+
+       err = uclass_first_device(UCLASS_TPM, &dev);
+       if (err)
+               return err;
+       return tis_open(dev);
+#else
        err = tis_init();
        if (err)
                return err;
return tis_open();
+#endif
Same
  }
uint32_t tpm_startup(enum tpm_startup_type mode)
Best Regards
Christophe
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to