Re: [PATCH v7 0/4] st33zp24 new architecture proposal and st33zp24 spi driver

2015-02-10 Thread Peter Hüwe
Hi Christophe,

 
 Any news on this patchset ?
 

Unfortunately not - I'm still lacking behind due to a big cold I caught.

I'll add it once I pushed the fixes for 3.20 out.
(still waiting on vicky's fixed patchset)

Peter
--
To unsubscribe from this list: send the line unsubscribe devicetree in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v7 0/4] st33zp24 new architecture proposal and st33zp24 spi driver

2015-02-10 Thread RICARD Christophe

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 tpmsupp...@st.com 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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH v7 0/4] st33zp24 new architecture proposal and st33zp24 spi driver

2015-02-01 Thread Christophe Ricard
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 tpmsupp...@st.com 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

-- 
2.1.0

--
To unsubscribe from this list: send the line unsubscribe devicetree in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html