public inbox for devicetree@vger.kernel.org
 help / color / mirror / Atom feed
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

  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