On Wed, Aug 10, 2016 at 02:25:30PM +0300, Jarkko Sakkinen wrote: > On Tue, Aug 09, 2016 at 03:34:55PM -0400, Nayna Jain wrote: > > Adds securityfs support for TPM2.0. > > This patch currently supports only binary_bios_measurements. > > > > Changelog v2: > > * Single tpm_of.c for reading both tpm and vtpm device tree values. > > * Some of the issues are fixed in Patch 1 itself. > > * Comments in tpm2.h give reference to the standard from where > > structs > > are taken. > > * Now, tpm_of.c has same code applied for both tpm and vtpm, so I think > > that now it is needed to have generic types rather than endian specific > > type. > > > > There are few preexisting issues as being mentioned in feedback and are not > > addressed in this patch. Reason being, I don't have much expertise of ACPI > > side as of now, > > and these changes will affect acpi,tpm,vtpm, all paths, so I would like to > > go slow > > and fix them as different patch later after better understanding. > > Hope this sounds ok to have them as different patch. > > > > Issues which are not addressed are as below: > > * tpm_eventlog.h still has #ifdef defined, for tpm_bios_log_setup() > > * tpm_bios_log_setup is still being called in tpm-chip register > > function. > > I do not understand your changelog entries. Please provide a nice one or > most two paragraph english language description and keep breakdowns and > changelogs in the cover letter. > > Commit message does not equal to a discussion forum and I do not have > any idea what feedback you are talking about... > > Not reviewing this further. This is just terrible.
Whoops, my email client did tricks for me. Please ignore the two other responses from me. /Jarkko ------------------------------------------------------------------------------ What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic patterns at an interface-level. Reveals which users, apps, and protocols are consuming the most bandwidth. Provides multi-vendor support for NetFlow, J-Flow, sFlow and other flows. Make informed decisions using capacity planning reports. http://sdm.link/zohodev2dev _______________________________________________ tpmdd-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/tpmdd-devel
