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 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.