[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