Re: [PATCH v1 12/12] tpm: TPM2 sysfs attributes

2014-09-26 Thread Jarkko Sakkinen
On Wed, Sep 24, 2014 at 02:46:27PM -0600, Jason Gunthorpe wrote: > > That would be 24*2 files only for pcrs... > > Some subsystems do just that.. > > $ ls /sys/class/infiniband/qib0/ports/1/sl2vl/ > 0 1 10 11 12 13 14 15 2 3 4 5 6 7 8 9 They use static structures in drivers/infini

Re: [PATCH v1 12/12] tpm: TPM2 sysfs attributes

2014-09-24 Thread Jason Gunthorpe
On Wed, Sep 24, 2014 at 10:35:42PM +0200, Peter Hüwe wrote: > Am Mittwoch, 24. September 2014, 22:19:38 schrieb Jason Gunthorpe: > > On Wed, Sep 24, 2014 at 10:02:34PM +0300, Jarkko Sakkinen wrote: > > > > The pcrs file never conformed to the sysfs rules, if TPM2 is getting a > > > > whole new file

Re: [PATCH v1 12/12] tpm: TPM2 sysfs attributes

2014-09-24 Thread Peter Hüwe
Am Mittwoch, 24. September 2014, 22:19:38 schrieb Jason Gunthorpe: > On Wed, Sep 24, 2014 at 10:02:34PM +0300, Jarkko Sakkinen wrote: > > > The pcrs file never conformed to the sysfs rules, if TPM2 is getting a > > > whole new file set, I wouldn't mind seeing it not include the > > > non-conformant

Re: [PATCH v1 12/12] tpm: TPM2 sysfs attributes

2014-09-24 Thread Jason Gunthorpe
On Wed, Sep 24, 2014 at 10:02:34PM +0300, Jarkko Sakkinen wrote: > > The pcrs file never conformed to the sysfs rules, if TPM2 is getting a > > whole new file set, I wouldn't mind seeing it not include the > > non-conformant ones. What do you think? > > I think that it's better to put extra focus

Re: [PATCH v1 12/12] tpm: TPM2 sysfs attributes

2014-09-24 Thread Jarkko Sakkinen
On Wed, Sep 24, 2014 at 11:13:38AM -0600, Jason Gunthorpe wrote: > On Wed, Sep 24, 2014 at 12:06:02PM +0300, Jarkko Sakkinen wrote: > > Added tpm2-sysfs.c that implements sysfs attributes for a TPM2 > > device. > > You need to document in Documentation every new sysfs that is added. > > The exist

Re: [PATCH v1 12/12] tpm: TPM2 sysfs attributes

2014-09-24 Thread Peter Hüwe
Am Mittwoch, 24. September 2014, 19:13:38 schrieb Jason Gunthorpe: > On Wed, Sep 24, 2014 at 12:06:02PM +0300, Jarkko Sakkinen wrote: > > Added tpm2-sysfs.c that implements sysfs attributes for a TPM2 > > device. > > You need to document in Documentation every new sysfs that is added. > > The exi

Re: [PATCH v1 12/12] tpm: TPM2 sysfs attributes

2014-09-24 Thread Jason Gunthorpe
On Wed, Sep 24, 2014 at 12:06:02PM +0300, Jarkko Sakkinen wrote: > Added tpm2-sysfs.c that implements sysfs attributes for a TPM2 > device. You need to document in Documentation every new sysfs that is added. The existing ones are not documented :( > +++ b/drivers/char/tpm/tpm-interface.c > @@ -