dm: Convert TPM drivers to driver model
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v2 Latest17 of 28 patches 0 repliesNew
- v13 of 25 patches 0 repliesSuperseded
17 of 28 patches are in this archive. The cover letter announces 28.
Patches in v2 No applies recorded yet
Reviews trailers per patch
13 of 17 patches have a Reviewed-by or Acked-by.
| Patch | Christophe Ricard |
|---|---|
| 3/28 | A-b |
| 4/28 | A-b |
| 6/28 | |
| 6/28 | |
| 12/28 | A-b |
| 15/28 | A-b |
| 16/28 | A-b |
| 17/28 | A-b |
| 18/28 | A-b |
| 20/28 | |
| 20/28 | |
| 21/28 | A-b |
| 22/28 | A-b |
| 23/28 | A-b |
| 25/28 | A-b |
| 26/28 | A-b |
| 27/28 | A-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [U-Boot] [PATCH v2 00/28] dm: Convert TPM drivers to driver model
Hi Simon,
I like your updates but i have few comments:
- I believe tpm_tis_i2c should be renamed as tpm_tis_infineon to be aligned with Linux naming.
Also as there is no standard for i2c for tpm, every vendor does have their own low level transport protocol.
I believe this is confusing.
- I think atmel tpm driver can be easily move to driver model with this new tpm class. I would cleanup some few
#ifdef and make the code even clearer. May be i can try to send an update when sending ST33ZP24 TPM driver ?
Other than that, i looks pretty good.
Thanks a lot.
Christophe
Quoted 123 lines · Simon Glass
This series adds driver model support for Trusted Platform Modules (TPMs). These have a very simple interface and are configured via the device tree.
Two bus types are supported at present: I2C and LPC (Intel Low-Pin-Count).
Most drivers and users are converted over to driver model. The exception is the Atmel TPM and its users.
The I2C driver has been cleaned up and simplified. It was ported from Linux and was pretty hard to follow. This series includes patches to unify the code, remove duplicated data structures and drop unnecessary indirection.
Also this series enables the TPM on all Chromebooks supported by upstream U-Boot (snow, spring, nyan-big, pit, pi, link, panther) since some did not have it fully enabled.
As before, the 'tpm' command can be used to implement TPM functionality. In addition a 'tpmtest' command provides some basic TPM tests taken from Chrome OS U-Boot. These are fairly rudimentary but are useful if you know what you are doing.
Changes in v2:
- Add a 'TPM' menu inKconfig
- Add new patch with a 'tpmtest' command
- Add new patch with functions to access flags and permissions
- Add tpm device tree node for panther also
- Correct error checking code intpm_sendrecv_command()
- Correct error-checking code inreport_return_code()
- Enable 'tpmtest' command for nyan
- Enhance uclass to support timeouts with new send()/recv() methods
- Implement theget_desc()method
- Includetpm.hheader file instead oftis.h
- Move the timeout definitions into a newtpm_internal.hheader
- Update cover letter to mention panther and the tpmtest command.
- Update driver for the new uclass interface
- Update driver for v2 TPM uclass which supports timeouts
- Usetpm_prefix instead oftis_for TPM functionsSimon Glass (28):
tpm: Remove old pre-driver-model I2C code
tpm: Drop two unused options
tpm: AddKconfigoptions for TPMs
tpm: Convert board config TPM options toKconfig
tpm: Convert drivers to use SPDX
tpm: Move the I2C TPM code into one file
tpm:tpm_tis_i2c: Drop unnecessary methods
tpm:tpm_tis_i2c: Drop structtpm_vendor_specific
tpm:tpm_tis_i2c: Merge structtpm_devintotpm_chip
tpm:tpm_tis_i2c: Merge struct tpm intotpm_chip
tpm:tpm_tis_i2c: Move definitions into the header file
tpm:tpm_tis_i2c: Simplify init code
tpm:tpm_tis_i2c: Use a consistenttpm_tis_i2c_prefix
tpm:tpm_tis_i2c: Tidy up delays
dm: tpm: Add a uclass for Trusted Platform Modules
dm: tpm: Convert the TPM command and library to driver model
dm: i2c: Add a command to adjust the offset length
tpm: Report tpm errors on the command line
dm: tpm: sandbox: Convert TPM driver to driver model
tpm: Check thatparse_byte_string()has data to parse
exynos: x86: dts: Add tpm nodes to the device tree for Chrome OS
devices
dm: tpm: Convert I2C driver to driver model
dm: tpm: Convert LPC driver to driver model
tpm: Add a 'tpm info' command
tpm: Add functions to access flags and permissions
dm: tpm: Add a 'tpmtest' command
tpm: Enable 'tpmtest' command for Chrome OS boards with TPMs
tegra: nyan: Enable TPM command and driverREADME | 6 - arch/arm/dts/exynos5250-snow.dts | 9 + arch/arm/dts/exynos5250-spring.dts | 8 + arch/arm/dts/exynos5420-peach-pit.dts | 6 +- arch/arm/dts/exynos5800-peach-pi.dts | 6 +- arch/sandbox/dts/sandbox.dts | 4 + arch/x86/dts/chromebook_link.dts | 5 + arch/x86/dts/chromebox_panther.dts | 5 + common/Kconfig | 22 + common/Makefile | 1 + common/cmd_i2c.c | 33 + common/cmd_tpm.c | 100 ++- common/cmd_tpm_test.c | 565 +++++++++++++++++ configs/chromebook_link_defconfig | 5 + configs/chromebox_panther_defconfig | 5 + .../controlcenterd_36BIT_SDCARD_DEVELOP_defconfig | 4 + configs/controlcenterd_36BIT_SDCARD_defconfig | 4 + .../controlcenterd_TRAILBLAZER_DEVELOP_defconfig | 4 + configs/controlcenterd_TRAILBLAZER_defconfig | 4 + configs/coreboot-x86_defconfig | 5 + configs/nyan-big_defconfig | 5 + configs/peach-pi_defconfig | 5 + configs/peach-pit_defconfig | 5 + configs/sandbox_defconfig | 4 + configs/snow_defconfig | 5 + configs/spring_defconfig | 5 + drivers/tpm/Kconfig | 69 ++ drivers/tpm/Makefile | 4 +- drivers/tpm/tpm-uclass.c | 133 ++++ drivers/tpm/tpm.c | 694 --------------------- drivers/tpm/tpm_atmel_twi.c | 15 +- drivers/tpm/tpm_internal.h | 287 +++++++++ drivers/tpm/tpm_tis_i2c.c | 572 ++++++++--------- drivers/tpm/{tpm_private.h => tpm_tis_i2c.h} | 88 +-- drivers/tpm/tpm_tis_lpc.c | 290 ++++----- drivers/tpm/tpm_tis_sandbox.c | 57 +- include/configs/controlcenterd.h | 7 - include/configs/efi-x86.h | 2 - include/configs/exynos5-common.h | 7 - include/configs/x86-common.h | 2 - include/dm/uclass-id.h | 1 + include/fdtdec.h | 2 - include/tis.h | 3 + include/tpm.h | 243 +++++++- lib/Kconfig | 10 + lib/fdtdec.c | 2 - lib/tpm.c | 82 ++- 47 files changed, 2102 insertions(+), 1298 deletions(-)create mode 100644
common/cmd_tpm_test.c
create mode 100644drivers/tpm/tpm-uclass.cdelete mode 100644drivers/tpm/tpm.c
create mode 100644drivers/tpm/tpm_internal.hrenamedrivers/tpm/{tpm_private.h=>tpm_tis_i2c.h} (54%)