All of lore.kernel.org
 help / color / mirror / Atom feed
From: Miquel Raynal <miquel.raynal@bootlin.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 17/25] tpm: Export the open/close functions
Date: Wed, 7 Nov 2018 08:52:31 +0100	[thread overview]
Message-ID: <20181107085231.5fb99f27@xps13> (raw)
In-Reply-To: <20181106222142.94537-18-sjg@chromium.org>

Hi Simon,

Simon Glass <sjg@chromium.org> wrote on Tue,  6 Nov 2018 15:21:34 -0700:

> At present these functions are not accessible outside the TPM library, but
> in some cases we need to call them.

I was not aware, what is the use case? I don't get it.

> Export them in the header file and add
> a define for the SHA1 digest size.
> 
> Also adjust tpm_open() to call tpm_close() first so that the TPM is in a
> known state before opening (e.g. by a previous phase of U-Boot).
> 
> Signed-off-by: Simon Glass <sjg@chromium.org>
> ---
> 

[...]

> @@ -408,29 +435,12 @@ static int tpm_tis_lpc_open(struct udevice *dev)
>  		return ret;
>  	}
>  
> +	/* Certain TPMs need some delay here or they hang */
> +	udelay(10);
> +
>  	tpm_write_word(priv, TIS_STS_COMMAND_READY,
>  		       &regs[locality].tpm_status);

This is not in the commit message.

Perhaps, due to the nature of the changes, this patch would be best
split in 2 or 3?


Thanks,
Miquèl

  reply	other threads:[~2018-11-07  7:52 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-11-06 22:21 [U-Boot] [PATCH 00/25] sandbox: Changes and improvements to support verified boot Simon Glass
2018-11-06 22:21 ` [U-Boot] [PATCH 01/25] cros_ec: Use uint instead of u8 for parameters Simon Glass
2018-11-22 20:20   ` sjg at google.com
2018-11-06 22:21 ` [U-Boot] [PATCH 02/25] cros_ec: Add error logging on a few commands Simon Glass
2018-11-22 20:20   ` sjg at google.com
2018-11-06 22:21 ` [U-Boot] [PATCH 03/25] cros_ec: Fail if we cannot determine the flash burst size Simon Glass
2018-11-22 20:20   ` sjg at google.com
2018-11-06 22:21 ` [U-Boot] [PATCH 04/25] cros_ec: Align uclass data to a cache boundary Simon Glass
2018-11-22 20:20   ` sjg at google.com
2018-11-06 22:21 ` [U-Boot] [PATCH 05/25] cros_ec: Add new features for events and power Simon Glass
2018-11-22 20:20   ` sjg at google.com
2018-11-06 22:21 ` [U-Boot] [PATCH 06/25] sandbox: tpm: Allow debugging of data packages Simon Glass
2018-11-22 20:20   ` sjg at google.com
2018-11-06 22:21 ` [U-Boot] [PATCH 07/25] sandbox: log: Add a category for sandbox Simon Glass
2018-11-22 20:20   ` sjg at google.com
2018-11-06 22:21 ` [U-Boot] [PATCH 08/25] sandbox: Add a function to read a host file Simon Glass
2018-11-22 20:20   ` sjg at google.com
2018-11-06 22:21 ` [U-Boot] [PATCH 09/25] sandbox: cros_ec: exynos: Drop use of cros_ec_get_error() Simon Glass
2018-11-07  2:27   ` Minkyu Kang
2018-11-22 20:20     ` sjg at google.com
2018-11-06 22:21 ` [U-Boot] [PATCH 10/25] sandbox: Update some drivers to work in SPL/TPL Simon Glass
2018-11-22 20:20   ` sjg at google.com
2018-11-06 22:21 ` [U-Boot] [PATCH 11/25] spl: Support bootstage, log, hash and early malloc in TPL Simon Glass
2018-11-22 20:20   ` sjg at google.com
2018-11-06 22:21 ` [U-Boot] [PATCH 12/25] spl: Correct malloc debugging in board_init_r() Simon Glass
2018-11-07  6:22   ` Simon Goldschmidt
2018-11-07 17:09     ` Simon Glass
2018-11-06 22:21 ` [U-Boot] [PATCH 13/25] spl: lz4: Allow use of lz4 compression in SPL Simon Glass
2018-11-22 20:20   ` sjg at google.com
2018-11-06 22:21 ` [U-Boot] [PATCH 14/25] binman: Add a way to enable debugging from the build Simon Glass
2018-11-22 20:20   ` sjg at google.com
2018-11-06 22:21 ` [U-Boot] [PATCH 15/25] binman: Drop an unnecessary comma in blob handling Simon Glass
2018-11-22 20:20   ` sjg at google.com
2018-11-06 22:21 ` [U-Boot] [PATCH 16/25] binman: Set the pathname correctly for ELF files Simon Glass
2018-11-22 20:20   ` sjg at google.com
2018-11-06 22:21 ` [U-Boot] [PATCH 17/25] tpm: Export the open/close functions Simon Glass
2018-11-07  7:52   ` Miquel Raynal [this message]
2018-11-18 21:23     ` Simon Glass
2018-11-29 17:42       ` sjg at google.com
2018-11-06 22:21 ` [U-Boot] [PATCH 18/25] tpm: Convert to use a device parameter Simon Glass
2018-11-07  7:56   ` Miquel Raynal
2018-11-06 22:21 ` [U-Boot] [PATCH 19/25] video: Update video_set_default_colors() to support invert Simon Glass
2018-11-22 20:20   ` sjg at google.com
2018-11-06 22:21 ` [U-Boot] [PATCH 20/25] efi_loader: Don't enable in SPL/TPL by default Simon Glass
2018-11-14 13:28   ` Alexander Graf
2018-11-18 21:23     ` Simon Glass
2018-11-06 22:21 ` [U-Boot] [PATCH 21/25] string: Include the config header Simon Glass
2018-11-22 20:20   ` sjg at google.com
2018-11-06 22:21 ` [U-Boot] [PATCH 22/25] misc: Update read() and write() methods to return bytes xfered Simon Glass
2018-11-08 11:03   ` Patrick DELAUNAY
2018-11-22 20:20     ` sjg at google.com
2018-11-06 22:21 ` [U-Boot] [PATCH 23/25] test: sf: Add a simple SPI flash test Simon Glass
2018-11-22 20:20   ` sjg at google.com
2018-11-06 22:21 ` [U-Boot] [PATCH 24/25] sf: Add a method to obtain the block-protect setting Simon Glass
2018-11-22 20:20   ` sjg at google.com
2018-11-06 22:21 ` [U-Boot] [PATCH 25/25] mmc: Add hardware partition support Simon Glass
2018-11-07  7:38   ` Faiz Abbas
2018-11-18 21:23     ` 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=20181107085231.5fb99f27@xps13 \
    --to=miquel.raynal@bootlin.com \
    --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.