[PATCH 2/2 v4] efi: Add basic EFI_TCG2_PROTOCOL support

Ilias Apalodimas ilias.apalodimas at linaro.org
Thu Nov 12 20:10:59 CET 2020


Hi Heinrich, 


[...]
> > +		return EFI_SUCCESS;
> > +
> > +	tpm_ver = tpm_get_version(dev);
> > +	if (tpm_ver != TPM_V2) {
> > +		log_warning("Only TPMv2 supported for EFI_TCG2_PROTOCOL");
> 
> The message should end with \n.
> 
> This message becomes superfluous if you correct platform_get_tpm2_device().
> 

platform_get_tpm2_device() is used in EFI calls, won't that break printing from 
EFI apps?

> > +		return EFI_SUCCESS;
> > +	}
> > +
> > +	ret = efi_add_protocol(efi_root, &efi_guid_tcg2_protocol,
> > +			       (void *)&efi_tcg2_protocol);
> > +	if (ret != EFI_SUCCESS)
> > +		log_err("Cannot install EFI_TCG2_PROTOCOL");
> 
> The message should end with \n.
> 
> Best regards
> 
> Heinrich
> 
> > +
> > +	return ret;
> > +}
> >
> 

Cheers
/Ilias


More information about the U-Boot mailing list