[PATCH v2 1/1] tpm: clear state post probing

[Ilias Apalodimas]

On Mon, Nov 15, 2021 at 08:30:06PM +0100, Heinrich Schuchardt wrote:
> Before we can start measuring the TPM must be cleared. Do this in the
> post_probe() method of the uclass.
> 
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
> ---
> v2:
> 	tpm_startup2() is not available on all boards.
> 	tpm_startup() takes care of translating the call.
> ---
>  drivers/tpm/tpm-uclass.c | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 
> diff --git a/drivers/tpm/tpm-uclass.c b/drivers/tpm/tpm-uclass.c
> index f67fe1019b..abd9ce35e8 100644
> --- a/drivers/tpm/tpm-uclass.c
> +++ b/drivers/tpm/tpm-uclass.c
> @@ -11,6 +11,7 @@
>  #include <log.h>
>  #include <linux/delay.h>
>  #include <linux/unaligned/be_byteshift.h>
> +#include <tpm_api.h>
>  #include <tpm-v1.h>
>  #include <tpm-v2.h>
>  #include "tpm_internal.h"
> @@ -136,6 +137,17 @@ int tpm_xfer(struct udevice *dev, const uint8_t *sendbuf, size_t send_size,
>  	return 0;
>  }
>  
> +static int dm_tpm_post_probe(struct udevice *dev)
> +{
> +	/*
> +	 * Clearing the TPM state is only possible once after a hard reset.
> +	 * As we do not know if the TPM has been cleared by a prior boot stage
> +	 * ignore the return value here.
> +	 */
> +	tpm_startup(dev, TPM_ST_CLEAR);
> +	return 0;
> +}
> +
>  UCLASS_DRIVER(tpm) = {
>  	.id		= UCLASS_TPM,
>  	.name		= "tpm",
> @@ -143,5 +155,6 @@ UCLASS_DRIVER(tpm) = {
>  #if CONFIG_IS_ENABLED(OF_REAL)
>  	.post_bind	= dm_scan_fdt_dev,
>  #endif
> +	.post_probe	= dm_tpm_post_probe,
>  	.per_device_auto	= sizeof(struct tpm_chip_priv),
>  };
> -- 
> 2.32.0
> 

Reviewed-by: Ilias Apalodimas <ilias.apalodimas at linaro.org>