From: Ilias Apalodimas <ilias.apalodimas@linaro.org>
To: Simon Glass <sjg@chromium.org>
Cc: U-Boot Mailing List <u-boot@lists.denx.de>
Subject: Re: [PATCH 1/8] tpm: Export the TPM-version functions
Date: Tue, 7 Jun 2022 11:28:44 +0300 [thread overview]
Message-ID: <Yp8MPKjs/eTy+ENY@hera> (raw)
In-Reply-To: <20220301001125.1554442-2-sjg@chromium.org>
Hi Simon,
Apologies the late reply. I wasn't cc'ed on those and missed them in my
mailbox.
On Mon, Feb 28, 2022 at 05:11:18PM -0700, Simon Glass wrote:
> These functions should really be available outside the TPM code, so that
> other callers can find out which version the TPM is. Rename them to have
> a tpm_ prefix() and add them to the header file.
>
> Signed-off-by: Simon Glass <sjg@chromium.org>
> ---
>
> include/tpm_api.h | 10 ++++++
> lib/tpm_api.c | 92 +++++++++++++++++++++--------------------------
> 2 files changed, 51 insertions(+), 51 deletions(-)
>
> diff --git a/include/tpm_api.h b/include/tpm_api.h
> index ef45b43a8f..11aa14eb79 100644
> --- a/include/tpm_api.h
> +++ b/include/tpm_api.h
> @@ -319,4 +319,14 @@ u32 tpm_write_lock(struct udevice *dev, u32 index);
> */
> u32 tpm_resume(struct udevice *dev);
>
> +static inline bool tpm_is_v1(struct udevice *dev)
> +{
> + return IS_ENABLED(CONFIG_TPM_V1) && tpm_get_version(dev) == TPM_V1;
> +}
> +
> +static inline bool tpm_is_v2(struct udevice *dev)
> +{
> + return IS_ENABLED(CONFIG_TPM_V2) && tpm_get_version(dev) == TPM_V2;
> +}
> +
> #endif /* __TPM_API_H */
> diff --git a/lib/tpm_api.c b/lib/tpm_api.c
> index 4c662640a9..4ac4612c81 100644
> --- a/lib/tpm_api.c
> +++ b/lib/tpm_api.c
> @@ -11,21 +11,11 @@
> #include <tpm-v2.h>
> #include <tpm_api.h>
>
> -static bool is_tpm1(struct udevice *dev)
> -{
> - return IS_ENABLED(CONFIG_TPM_V1) && tpm_get_version(dev) == TPM_V1;
> -}
> -
> -static bool is_tpm2(struct udevice *dev)
> -{
> - return IS_ENABLED(CONFIG_TPM_V2) && tpm_get_version(dev) == TPM_V2;
> -}
> -
> u32 tpm_startup(struct udevice *dev, enum tpm_startup_type mode)
> {
> - if (is_tpm1(dev)) {
> + if (tpm_is_v1(dev)) {
> return tpm1_startup(dev, mode);
> - } else if (is_tpm2(dev)) {
> + } else if (tpm_is_v2(dev)) {
> enum tpm2_startup_types type;
>
> switch (mode) {
> @@ -47,9 +37,9 @@ u32 tpm_startup(struct udevice *dev, enum tpm_startup_type mode)
>
> u32 tpm_resume(struct udevice *dev)
> {
> - if (is_tpm1(dev))
> + if (tpm_is_v1(dev))
> return tpm1_startup(dev, TPM_ST_STATE);
> - else if (is_tpm2(dev))
> + else if (tpm_is_v2(dev))
> return tpm2_startup(dev, TPM2_SU_STATE);
> else
> return -ENOSYS;
> @@ -57,9 +47,9 @@ u32 tpm_resume(struct udevice *dev)
>
> u32 tpm_self_test_full(struct udevice *dev)
> {
> - if (is_tpm1(dev))
> + if (tpm_is_v1(dev))
> return tpm1_self_test_full(dev);
> - else if (is_tpm2(dev))
> + else if (tpm_is_v2(dev))
> return tpm2_self_test(dev, TPMI_YES);
> else
> return -ENOSYS;
> @@ -67,9 +57,9 @@ u32 tpm_self_test_full(struct udevice *dev)
>
> u32 tpm_continue_self_test(struct udevice *dev)
> {
> - if (is_tpm1(dev))
> + if (tpm_is_v1(dev))
> return tpm1_continue_self_test(dev);
> - else if (is_tpm2(dev))
> + else if (tpm_is_v2(dev))
> return tpm2_self_test(dev, TPMI_NO);
> else
> return -ENOSYS;
> @@ -86,7 +76,7 @@ u32 tpm_clear_and_reenable(struct udevice *dev)
> return ret;
> }
>
> - if (is_tpm1(dev)) {
> + if (tpm_is_v1(dev)) {
> ret = tpm1_physical_enable(dev);
> if (ret != TPM_SUCCESS) {
> log_err("TPM: Can't set enabled state\n");
> @@ -105,9 +95,9 @@ u32 tpm_clear_and_reenable(struct udevice *dev)
>
> u32 tpm_nv_enable_locking(struct udevice *dev)
> {
> - if (is_tpm1(dev))
> + if (tpm_is_v1(dev))
> return tpm1_nv_define_space(dev, TPM_NV_INDEX_LOCK, 0, 0);
> - else if (is_tpm2(dev))
> + else if (tpm_is_v2(dev))
> return -ENOSYS;
> else
> return -ENOSYS;
> @@ -115,9 +105,9 @@ u32 tpm_nv_enable_locking(struct udevice *dev)
>
> u32 tpm_nv_read_value(struct udevice *dev, u32 index, void *data, u32 count)
> {
> - if (is_tpm1(dev))
> + if (tpm_is_v1(dev))
> return tpm1_nv_read_value(dev, index, data, count);
> - else if (is_tpm2(dev))
> + else if (tpm_is_v2(dev))
> return tpm2_nv_read_value(dev, index, data, count);
> else
> return -ENOSYS;
> @@ -126,9 +116,9 @@ u32 tpm_nv_read_value(struct udevice *dev, u32 index, void *data, u32 count)
> u32 tpm_nv_write_value(struct udevice *dev, u32 index, const void *data,
> u32 count)
> {
> - if (is_tpm1(dev))
> + if (tpm_is_v1(dev))
> return tpm1_nv_write_value(dev, index, data, count);
> - else if (is_tpm2(dev))
> + else if (tpm_is_v2(dev))
> return tpm2_nv_write_value(dev, index, data, count);
> else
> return -ENOSYS;
> @@ -141,9 +131,9 @@ u32 tpm_set_global_lock(struct udevice *dev)
>
> u32 tpm_write_lock(struct udevice *dev, u32 index)
> {
> - if (is_tpm1(dev))
> + if (tpm_is_v1(dev))
> return -ENOSYS;
> - else if (is_tpm2(dev))
> + else if (tpm_is_v2(dev))
> return tpm2_write_lock(dev, index);
> else
> return -ENOSYS;
> @@ -152,9 +142,9 @@ u32 tpm_write_lock(struct udevice *dev, u32 index)
> u32 tpm_pcr_extend(struct udevice *dev, u32 index, const void *in_digest,
> void *out_digest)
> {
> - if (is_tpm1(dev))
> + if (tpm_is_v1(dev))
> return tpm1_extend(dev, index, in_digest, out_digest);
> - else if (is_tpm2(dev))
> + else if (tpm_is_v2(dev))
> return tpm2_pcr_extend(dev, index, TPM2_ALG_SHA256, in_digest,
> TPM2_DIGEST_LEN);
> else
> @@ -163,9 +153,9 @@ u32 tpm_pcr_extend(struct udevice *dev, u32 index, const void *in_digest,
>
> u32 tpm_pcr_read(struct udevice *dev, u32 index, void *data, size_t count)
> {
> - if (is_tpm1(dev))
> + if (tpm_is_v1(dev))
> return tpm1_pcr_read(dev, index, data, count);
> - else if (is_tpm2(dev))
> + else if (tpm_is_v2(dev))
> return -ENOSYS;
> else
> return -ENOSYS;
> @@ -173,14 +163,14 @@ u32 tpm_pcr_read(struct udevice *dev, u32 index, void *data, size_t count)
>
> u32 tpm_tsc_physical_presence(struct udevice *dev, u16 presence)
> {
> - if (is_tpm1(dev))
> + if (tpm_is_v1(dev))
> return tpm1_tsc_physical_presence(dev, presence);
>
> /*
> * Nothing to do on TPM2 for this; use platform hierarchy availability
> * instead.
> */
> - else if (is_tpm2(dev))
> + else if (tpm_is_v2(dev))
> return 0;
> else
> return -ENOSYS;
> @@ -188,11 +178,11 @@ u32 tpm_tsc_physical_presence(struct udevice *dev, u16 presence)
>
> u32 tpm_finalise_physical_presence(struct udevice *dev)
> {
> - if (is_tpm1(dev))
> + if (tpm_is_v1(dev))
> return tpm1_finalise_physical_presence(dev);
>
> /* Nothing needs to be done with tpm2 */
> - else if (is_tpm2(dev))
> + else if (tpm_is_v2(dev))
> return 0;
> else
> return -ENOSYS;
> @@ -200,9 +190,9 @@ u32 tpm_finalise_physical_presence(struct udevice *dev)
>
> u32 tpm_read_pubek(struct udevice *dev, void *data, size_t count)
> {
> - if (is_tpm1(dev))
> + if (tpm_is_v1(dev))
> return tpm1_read_pubek(dev, data, count);
> - else if (is_tpm2(dev))
> + else if (tpm_is_v2(dev))
> return -ENOSYS; /* not implemented yet */
> else
> return -ENOSYS;
> @@ -210,9 +200,9 @@ u32 tpm_read_pubek(struct udevice *dev, void *data, size_t count)
>
> u32 tpm_force_clear(struct udevice *dev)
> {
> - if (is_tpm1(dev))
> + if (tpm_is_v1(dev))
> return tpm1_force_clear(dev);
> - else if (is_tpm2(dev))
> + else if (tpm_is_v2(dev))
> return tpm2_clear(dev, TPM2_RH_PLATFORM, NULL, 0);
> else
> return -ENOSYS;
> @@ -220,11 +210,11 @@ u32 tpm_force_clear(struct udevice *dev)
>
> u32 tpm_physical_enable(struct udevice *dev)
> {
> - if (is_tpm1(dev))
> + if (tpm_is_v1(dev))
> return tpm1_physical_enable(dev);
>
> /* Nothing needs to be done with tpm2 */
> - else if (is_tpm2(dev))
> + else if (tpm_is_v2(dev))
> return 0;
> else
> return -ENOSYS;
> @@ -232,11 +222,11 @@ u32 tpm_physical_enable(struct udevice *dev)
>
> u32 tpm_physical_disable(struct udevice *dev)
> {
> - if (is_tpm1(dev))
> + if (tpm_is_v1(dev))
> return tpm1_physical_disable(dev);
>
> /* Nothing needs to be done with tpm2 */
> - else if (is_tpm2(dev))
> + else if (tpm_is_v2(dev))
> return 0;
> else
> return -ENOSYS;
> @@ -244,10 +234,10 @@ u32 tpm_physical_disable(struct udevice *dev)
>
> u32 tpm_physical_set_deactivated(struct udevice *dev, u8 state)
> {
> - if (is_tpm1(dev))
> + if (tpm_is_v1(dev))
> return tpm1_physical_set_deactivated(dev, state);
> /* Nothing needs to be done with tpm2 */
> - else if (is_tpm2(dev))
> + else if (tpm_is_v2(dev))
> return 0;
> else
> return -ENOSYS;
> @@ -256,9 +246,9 @@ u32 tpm_physical_set_deactivated(struct udevice *dev, u8 state)
> u32 tpm_get_capability(struct udevice *dev, u32 cap_area, u32 sub_cap,
> void *cap, size_t count)
> {
> - if (is_tpm1(dev))
> + if (tpm_is_v1(dev))
> return tpm1_get_capability(dev, cap_area, sub_cap, cap, count);
> - else if (is_tpm2(dev))
> + else if (tpm_is_v2(dev))
> return tpm2_get_capability(dev, cap_area, sub_cap, cap, count);
> else
> return -ENOSYS;
> @@ -266,9 +256,9 @@ u32 tpm_get_capability(struct udevice *dev, u32 cap_area, u32 sub_cap,
>
> u32 tpm_get_permissions(struct udevice *dev, u32 index, u32 *perm)
> {
> - if (is_tpm1(dev))
> + if (tpm_is_v1(dev))
> return tpm1_get_permissions(dev, index, perm);
> - else if (is_tpm2(dev))
> + else if (tpm_is_v2(dev))
> return -ENOSYS; /* not implemented yet */
> else
> return -ENOSYS;
> @@ -276,9 +266,9 @@ u32 tpm_get_permissions(struct udevice *dev, u32 index, u32 *perm)
>
> u32 tpm_get_random(struct udevice *dev, void *data, u32 count)
> {
> - if (is_tpm1(dev))
> + if (tpm_is_v1(dev))
> return tpm1_get_random(dev, data, count);
> - else if (is_tpm2(dev))
> + else if (tpm_is_v2(dev))
> return -ENOSYS; /* not implemented yet */
> else
> return -ENOSYS;
> --
> 2.35.1.574.g5d30c73bfb-goog
>
Reviewed-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>
next prev parent reply other threads:[~2022-06-07 8:28 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-03-01 0:11 [PATCH 0/8] tpm: Various minor fixes and enhancements Simon Glass
2022-03-01 0:11 ` [PATCH 1/8] tpm: Export the TPM-version functions Simon Glass
2022-06-07 8:28 ` Ilias Apalodimas [this message]
2022-03-01 0:11 ` [PATCH 2/8] tpm: Require a digest source when extending the PCR Simon Glass
2022-06-07 8:42 ` Ilias Apalodimas
2022-08-14 23:29 ` Simon Glass
2022-03-01 0:11 ` [PATCH 3/8] tpm: Correct the permissions command in TPMv1 Simon Glass
2022-06-07 8:44 ` Ilias Apalodimas
2022-08-14 23:29 ` Simon Glass
2022-03-01 0:11 ` [PATCH 4/8] tpm: Correct the define-space command in TPMv2 Simon Glass
2022-06-07 8:46 ` Ilias Apalodimas
2022-08-14 23:29 ` Simon Glass
2022-03-01 0:11 ` [PATCH 5/8] tpm: sandbox: Allow init of TPM in a different phase Simon Glass
2022-06-07 8:48 ` Ilias Apalodimas
2022-03-01 0:11 ` [PATCH 6/8] tpm: Allow reporting the internal state Simon Glass
2022-03-01 0:11 ` [PATCH 7/8] tpm: Implement state command for Cr50 Simon Glass
2022-06-07 8:54 ` Ilias Apalodimas
2022-08-14 23:29 ` Simon Glass
2022-03-01 0:11 ` [PATCH 8/8] tpm: Allow commiting non-volatile data Simon Glass
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Yp8MPKjs/eTy+ENY@hera \
--to=ilias.apalodimas@linaro.org \
--cc=sjg@chromium.org \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.