From: Christophe Ricard <christophe.ricard-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: jarkko.sakkinen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org
Cc: jean-luc.blanc-qxv4g6HH51o@public.gmane.org,
ashley-fm2HMyfA2y6tG0bUXCXiUA@public.gmane.org,
tpmdd-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org,
christophe-h.ricard-qxv4g6HH51o@public.gmane.org,
benoit.houyere-qxv4g6HH51o@public.gmane.org
Subject: [PATCH v7 0/7] Rework of tpm_tis to share common logic accross phy's (lpc/spi/-i2c-)
Date: Wed, 4 May 2016 21:06:21 +0200 [thread overview]
Message-ID: <1462388788-2410-1-git-send-email-christophe-h.ricard@st.com> (raw)
Hi Jarkko,
This serie is including common work from Infineon (Peter Huewe & Alexander Steffen) and ST.
We propose a common low level API used by a core TIS interface allowing to share
some protocols behavior between physical layers (lpc, spi, i2c...) or drivers.
To easy the group review, i only send 10 patches showing how we rework the tpm_tis driver
keeping existing proprietary logic (itpm workaround, irq for tpm_tis ("lpc")).
Finally we are adding spi support based on TCG PTP specification. This additional driver
support the SPI bit protocol including optional flow control.
Reworked tpm_tis got validated on a HP 8200 machine with an Infineon TPM
tpm_tis_spi got validated using Minnowboard Max as well as Raspberry Pi.
v2:
- Add Rob Herring acks on devicetree specific patch
- Renamed priv_data tpm_tis_data
- Moved data_expect_val & data_expect_mask in tpm_tis_data
- Moved "hal" (e.g: tpm_write_xxx/tpm_read_xxx) in tpm_tis_core.h
- Reduced list of the exported functions from tpm_tis_core
- Make tpm_tis ops internal to tpm_tis_core
v3:
- Merge tpm_class_lowlevel and tpm_tis_phy_ops
- Changed the way phy specific structure looked like in order to use container_of
- Cleanup patches with rebase mistakes and build verify them...
- Moved "Add include guards in tpm.h" at first position
v4:
- Move patch "Use read/write_bytes for drivers without more specialized methods"
before spi phy patch and rework so that functions are no more inlined and available
from tpm_tis_core.
v5:
- Squashed several patch together. (2, 3, 4).
- Renammed tpm_tis_lpc_data --> tpm_tis_tcg_data.
- Change tpm_tis_read_xxx/tpm_tis_write_xx parameters to take a struct tpm_tis_data* instead of a struct tpm_chip*.
- Manage itpm workaround internally to the tpm_tis_core and add a TIS flag (TPM_TIS_ITPM_POSSIBLE).
- Several fixes in the spi phy.
- Use chip->ops->cancel(chip) instead of tpm_tis_ready out of tpm_tis_core.
- Reworked Makefile
v6:
- Fix rebase mistaked in patch 2 and 3.
- Remove post_probe handler from tpm_tis_phy_ops in "Split tpm_tis driver into a core and TCG TIS compliant phy"
v7:
- Merge patch 7 in patch 8 as requested by Jarkko.
You can check this work on:
https://github.com/cricard13/linux-tpmdd tpm_tis_core_v7
Christophe Ricard (7):
tpm: Add include guards in tpm.h
tpm: tpm_tis: Share common data between phys
tpm_tis: Introduce intermediate layer for TPM access
devicetree: Add infineon to vendor-prefix.txt
devicetree: Add Trusted Computing Group to vendor-prefix.txt
tpm/tpm_tis: Split tpm_tis driver into a core and TCG TIS compliant
phy
tpm/tpm_tis_spi: Add support for spi phy
.../bindings/security/tpm/tpm_tis_spi.txt | 24 +
.../devicetree/bindings/vendor-prefixes.txt | 2 +
drivers/char/tpm/Kconfig | 19 +
drivers/char/tpm/Makefile | 2 +
drivers/char/tpm/tpm.h | 5 +
drivers/char/tpm/tpm_tis.c | 851 ++------------------
drivers/char/tpm/tpm_tis_core.c | 860 +++++++++++++++++++++
drivers/char/tpm/tpm_tis_core.h | 153 ++++
drivers/char/tpm/tpm_tis_spi.c | 272 +++++++
9 files changed, 1400 insertions(+), 788 deletions(-)
create mode 100644 Documentation/devicetree/bindings/security/tpm/tpm_tis_spi.txt
create mode 100644 drivers/char/tpm/tpm_tis_core.c
create mode 100644 drivers/char/tpm/tpm_tis_core.h
create mode 100644 drivers/char/tpm/tpm_tis_spi.c
--
2.1.4
------------------------------------------------------------------------------
Find and fix application performance issues faster with Applications Manager
Applications Manager provides deep performance insights into multiple tiers of
your business applications. It resolves application problems quickly and
reduces your MTTR. Get your free trial!
https://ad.doubleclick.net/ddm/clk/302982198;130105516;z
next reply other threads:[~2016-05-04 19:06 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-04 19:06 Christophe Ricard [this message]
[not found] ` <1462388788-2410-1-git-send-email-christophe-h.ricard-qxv4g6HH51o@public.gmane.org>
2016-05-04 19:06 ` [PATCH v7 1/7] tpm: Add include guards in tpm.h Christophe Ricard
2016-05-04 19:06 ` [PATCH v7 2/7] tpm: tpm_tis: Share common data between phys Christophe Ricard
[not found] ` <1462388788-2410-3-git-send-email-christophe-h.ricard-qxv4g6HH51o@public.gmane.org>
2016-05-09 13:23 ` Jarkko Sakkinen
[not found] ` <20160509132312.GA21366-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2016-05-13 8:34 ` Christophe Henri RICARD
2016-05-13 13:28 ` Jarkko Sakkinen
2016-05-04 19:06 ` [PATCH v7 3/7] tpm_tis: Introduce intermediate layer for TPM access Christophe Ricard
[not found] ` <1462388788-2410-4-git-send-email-christophe-h.ricard-qxv4g6HH51o@public.gmane.org>
2016-05-13 12:54 ` Jarkko Sakkinen
2016-05-04 19:06 ` [PATCH v7 4/7] devicetree: Add infineon to vendor-prefix.txt Christophe Ricard
2016-05-04 19:06 ` [PATCH v7 5/7] devicetree: Add Trusted Computing Group " Christophe Ricard
2016-05-04 19:06 ` [PATCH v7 6/7] tpm/tpm_tis: Split tpm_tis driver into a core and TCG TIS compliant phy Christophe Ricard
2016-05-04 19:06 ` [PATCH v7 7/7] tpm/tpm_tis_spi: Add support for spi phy Christophe Ricard
[not found] ` <1462388788-2410-8-git-send-email-christophe-h.ricard-qxv4g6HH51o@public.gmane.org>
2016-05-13 13:13 ` Jarkko Sakkinen
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=1462388788-2410-1-git-send-email-christophe-h.ricard@st.com \
--to=christophe.ricard-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
--cc=ashley-fm2HMyfA2y6tG0bUXCXiUA@public.gmane.org \
--cc=benoit.houyere-qxv4g6HH51o@public.gmane.org \
--cc=christophe-h.ricard-qxv4g6HH51o@public.gmane.org \
--cc=jarkko.sakkinen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org \
--cc=jean-luc.blanc-qxv4g6HH51o@public.gmane.org \
--cc=tpmdd-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
/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.