On Thu, Nov 24, 2016 at 03:57:23PM +0200, Jarkko Sakkinen wrote: > I manually added the changes to: > > tpm: replace dynamically allocated bios_dir with a static array
For this patch.. Could drop 'int rc' from tpm1_chip_register, but it will come back in a later patch Could dump TPM_NUM_EVENT_LOG_FILES and just use ARRAY_SIZE(chip->bios_dir) Now the the stub for tpm_bios_log_setup can properly return -ENODEV This is no good at this point in the series - we need the ENODEV detection in tpm_chip_register() from the 'Fix handle of missing event log' moved into this patch, because it now returns ENODEV due to sercurityfs The commit 'tpm: vtpm_proxy: Do not access host's event log' still needs a proper commit message - it doesn't match what the patch is doing at all. If you are going to be editing the patches I'd suggest squashing all the bug fix ones with proper credit so it at least makes sense to read.. Jason