From: RICARD Christophe <christophe.ricard-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: peterhuewe-Mmb7MZpHnFY@public.gmane.org
Cc: ashley-fm2HMyfA2y6tG0bUXCXiUA@public.gmane.org,
tpmdd-yWjUBOtONefk1uMJSBkQmQ@public.gmane.org,
tpmdd-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org,
christophe-h.ricard-qxv4g6HH51o@public.gmane.org,
jean-luc.blanc-qxv4g6HH51o@public.gmane.org,
benoit.houyere-qxv4g6HH51o@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org,
jarkko.sakkinen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org
Subject: Re: [PATCH v7 0/4] st33zp24 new architecture proposal and st33zp24 spi driver
Date: Tue, 10 Feb 2015 21:10:24 +0100 [thread overview]
Message-ID: <54DA65B0.3010809@gmail.com> (raw)
In-Reply-To: <1422826611-22942-1-git-send-email-christophe-h.ricard-qxv4g6HH51o@public.gmane.org>
Hi Peter,
Any news on this patchset ?
Best Regards
Christophe
Le 01/02/2015 22:36, Christophe Ricard a écrit :
> Hi,
>
> The following patchset:
> - propose a new architecture allowing to share a core st33zp24 data management
> layer with different phy (i2c & spi). For st33zp24 both phy have a proprietary transport
> protocol. Both are relying on the TCG TIS protocol. At the end, it simplifies the maintenance.
> - Add an spi phy allowing to support st33zp24 using with an SPI bus.
>
> The complete solution got tested in polling and interrupt mode successfully with i2c & spi phy.
> This patchset applies on top of Peter's tree https://github.com/PeterHuewe/linux-tpmdd.git for-james branch
> on top of:
> d4989d9f693b9502f9288da5db279c2f8c2e50be tpm/tpm_tis: Add missing ifdef CONFIG_ACPI for pnp_acpi_device
>
> I confirm also Jarkko Sakkinen's changes are working with this product with both phy's.
>
> - v2 takes into account feedbacks from Jason Gunthorpe.
> - v3 is reduced to 4 patches as 6 out of 10 got accepted for 3.20. Also compare to v2:
> * Fix build issue with patch v2 04/10 "Replace access to io_lpcpd from struct st33zp24_platform_data to tpm_stm_dev"
> * Fix link issue with patch v2 08/10 "Split tpm_i2c_tpm_st33 in 2 layers (core + phy)" when building as a module.
> The symbols wasn't exported in st33zp24.c.
> * Add missing MODULE_LICENSE in patch v2 09/10 "Add st33zp24 spi phy"
> * Fix node example in dts spi documentation in patch v2 10/10 "Add dts documentation for st33zp24 spi phy"
> * Fix typo on Jason Gunthorpe first name. Sorry for that :(...
> * Change contact email address as tpmsupport-qxv4g6HH51o@public.gmane.org is no more valid
> - v4 adds missing module_license in st33zp24
> - v5 includes as best as possible PeterHuewe comments.
> - v6 is more explicit about the spi buffer size and remove their buffer (tx_buf/rx_buf) dynamic allocation
> - v7 fix scripts/checkpatch.pl error
>
> Best Regards
> Christophe
>
> Christophe Ricard (4):
> tpm/tpm_i2c_stm_st33: Replace access to io_lpcpd from struct
> st33zp24_platform_data to tpm_stm_dev
> tpm/tpm_i2c_stm_st33: Split tpm_i2c_tpm_st33 in 2 layers (core + phy)
> tpm/st33zp24/spi: Add st33zp24 spi phy
> tpm/st33zp24/dts/st33zp24-spi: Add dts documentation for st33zp24 spi
> phy
>
> .../bindings/security/tpm/st33zp24-spi.txt | 34 +
> drivers/char/tpm/Kconfig | 11 +-
> drivers/char/tpm/Makefile | 2 +-
> drivers/char/tpm/st33zp24/Kconfig | 30 +
> drivers/char/tpm/st33zp24/Makefile | 12 +
> drivers/char/tpm/st33zp24/i2c.c | 276 +++++++
> drivers/char/tpm/st33zp24/spi.c | 392 +++++++++
> drivers/char/tpm/st33zp24/st33zp24.c | 688 ++++++++++++++++
> drivers/char/tpm/st33zp24/st33zp24.h | 37 +
> drivers/char/tpm/tpm_i2c_stm_st33.c | 911 ---------------------
> include/linux/platform_data/st33zp24.h | 28 +
> include/linux/platform_data/tpm_stm_st33.h | 39 -
> 12 files changed, 1499 insertions(+), 961 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/security/tpm/st33zp24-spi.txt
> create mode 100644 drivers/char/tpm/st33zp24/Kconfig
> create mode 100644 drivers/char/tpm/st33zp24/Makefile
> create mode 100644 drivers/char/tpm/st33zp24/i2c.c
> create mode 100644 drivers/char/tpm/st33zp24/spi.c
> create mode 100644 drivers/char/tpm/st33zp24/st33zp24.c
> create mode 100644 drivers/char/tpm/st33zp24/st33zp24.h
> delete mode 100644 drivers/char/tpm/tpm_i2c_stm_st33.c
> create mode 100644 include/linux/platform_data/st33zp24.h
> delete mode 100644 include/linux/platform_data/tpm_stm_st33.h
>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2015-02-10 20:10 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-02-01 21:36 [PATCH v7 0/4] st33zp24 new architecture proposal and st33zp24 spi driver Christophe Ricard
[not found] ` <1422826611-22942-1-git-send-email-christophe-h.ricard-qxv4g6HH51o@public.gmane.org>
2015-02-01 21:36 ` [PATCH v7 1/4] tpm/tpm_i2c_stm_st33: Replace access to io_lpcpd from struct st33zp24_platform_data to tpm_stm_dev Christophe Ricard
2015-02-01 21:36 ` [PATCH v7 2/4] tpm/tpm_i2c_stm_st33: Split tpm_i2c_tpm_st33 in 2 layers (core + phy) Christophe Ricard
2015-02-01 21:36 ` [PATCH v7 3/4] tpm/st33zp24/spi: Add st33zp24 spi phy Christophe Ricard
2015-02-01 21:36 ` [PATCH v7 4/4] tpm/st33zp24/dts/st33zp24-spi: Add dts documentation for " Christophe Ricard
2015-02-10 20:10 ` RICARD Christophe [this message]
[not found] ` <54DA65B0.3010809-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-02-10 20:36 ` [PATCH v7 0/4] st33zp24 new architecture proposal and st33zp24 spi driver Peter Hüwe
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=54DA65B0.3010809@gmail.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=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=jarkko.sakkinen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org \
--cc=jean-luc.blanc-qxv4g6HH51o@public.gmane.org \
--cc=jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org \
--cc=peterhuewe-Mmb7MZpHnFY@public.gmane.org \
--cc=tpmdd-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
--cc=tpmdd-yWjUBOtONefk1uMJSBkQmQ@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox