On Thu, Oct 04, 2018 at 11:45:30AM +0000, Winkler, Tomas wrote: > There is no API change, in that sense. > The exported API is in tpm-interface.c int tpm_pcr_extend(struct tpm_chip *chip, int pcr_idx, const u8 *hash) > that is used is outside of the tpm reminds the same, only the open coded implementation of tpm2_pcr_extned has moved to > tpm2-cmd.c, This code is not called out of tpm module. > Please review the code again. I did now revisit this and you are right that my choice of word was not exactly correct. I apologize for that. The patch introduces API that we would take away and that does make much sense. The best way to sort things out is to just fix the warnings and leave the TPM 2.0 part open coded inside tpm_pcr_extend(). The rationale for this is to avoid unnecessary mainline changes when ever possible (which is bad for backporting for stable kernels). > Thanks > Tomas /Jarkko
