Hi Jarkko,

After our discussion, please find an updated v3 serie allowing to
completely remove reference to the tpm_vendor_specific structure.

I have been able to test tpm_tis on a HP8200 Desktop machine.

v2 serie was dropped.

In v3:
- Merged 'tpm/tpm_atmel: drop remaining 'iobase' usage' from v2 and
'tpm: drop 'iobase' from struct tpm_vendor_specific' sent in v1
- Move patch tpm: Remove useless priv field in struct tpm_vendor_specific
at the end of the serie,
- Dropped codestyle/cleanup patches. May be send later...
(e.g: tpm/tpm_i2c_atmel: simplify patch to get tpm_chip from an i2c_client,
tpm/tpm_i2c_atmel: Few code style fixes)
- In patch 'tpm: drop 'irq' from struct tpm_vendor_specific', Renamed
TPM_CHIP_FLAG_USES_IRQ to TPM_CHIP_FLAG_IRQ
- Added patch 'tpm: drop 'locality' from struct tpm_vendor_specific'.

Please note: scripts/checkpatch.pl returns the following warning:
'WARNING: macros should not use a trailing semicolon
#165: FILE: drivers/char/tpm/tpm_atmel.h:41:
+#define atmel_getb(priv, offset) readb(priv->iobase + offset);'
The semicolon was already present in tpm_atmel.h at this place and concidered 
this
as a side fix to add in a future update (potententially with the future 
remaining
codestyle/cleanup serie). It does not generate any build errors on my side. I 
found
out because atmel_getb wasn't used in any place.

Best Regards
Christophe

Christophe Ricard (6):
  tpm: drop 'iobase' from struct tpm_vendor_specific
  tpm: drop 'irq' from struct tpm_vendor_specific
  tpm: drop 'read_queue' from struct tpm_vendor_specific
  tpm: drop 'locality' from struct tpm_vendor_specific
  tpm: Move tpm_vendor_specific data related with PTP specification to
    tpm_chip
  tpm: Remove useless priv field in struct tpm_vendor_specific

 drivers/char/tpm/st33zp24/i2c.c      |  15 ++-
 drivers/char/tpm/st33zp24/spi.c      |  15 ++-
 drivers/char/tpm/st33zp24/st33zp24.c | 113 +++++++----------
 drivers/char/tpm/st33zp24/st33zp24.h |  12 ++
 drivers/char/tpm/tpm-interface.c     |  52 ++++----
 drivers/char/tpm/tpm-sysfs.c         |  20 +--
 drivers/char/tpm/tpm.h               |  25 +---
 drivers/char/tpm/tpm2-cmd.c          |   2 +-
 drivers/char/tpm/tpm_atmel.c         |  32 +++--
 drivers/char/tpm/tpm_atmel.h         |  10 +-
 drivers/char/tpm/tpm_crb.c           |  12 +-
 drivers/char/tpm/tpm_i2c_atmel.c     |  23 ++--
 drivers/char/tpm/tpm_i2c_infineon.c  |  48 ++++----
 drivers/char/tpm/tpm_i2c_nuvoton.c   |  73 ++++++-----
 drivers/char/tpm/tpm_ibmvtpm.c       |  38 ++----
 drivers/char/tpm/tpm_nsc.c           |  48 ++++----
 drivers/char/tpm/tpm_tis.c           | 231 +++++++++++++++++++----------------
 drivers/char/tpm/xen-tpmfront.c      |  36 +++---
 18 files changed, 402 insertions(+), 403 deletions(-)

-- 
2.5.0


------------------------------------------------------------------------------
Transform Data into Opportunity.
Accelerate data analysis in your applications with
Intel Data Analytics Acceleration Library.
Click to learn more.
http://pubads.g.doubleclick.net/gampad/clk?id=278785471&iu=/4140
_______________________________________________
tpmdd-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/tpmdd-devel

Reply via email to