[PATCH] efi_loader: Fix Kconfig for EFI_TCG2 protocol

Ilias Apalodimas ilias.apalodimas at linaro.org
Tue May 11 09:38:20 CEST 2021


On Tue, May 11, 2021 at 09:25:04AM +0200, Heinrich Schuchardt wrote:
> On 10.05.21 20:26, Ilias Apalodimas wrote:
> > EFI_TCG2 depends not only on TPMv2 but also on the underlying algorithms.
> > So add the missing SHA1, SHA256, SHA384 and SHA512 we currently support
> > as depenmdencies
> 
> Thanks for looking into the dependencies.
> 
> %s/depenmdencies/dependencies/
> 
> >
> > Signed-off-by: Ilias Apalodimas <ilias.apalodimas at linaro.org>
> > ---
> >  lib/efi_loader/Kconfig | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/lib/efi_loader/Kconfig b/lib/efi_loader/Kconfig
> > index 0b99d7c7749b..6decdee6aa3e 100644
> > --- a/lib/efi_loader/Kconfig
> > +++ b/lib/efi_loader/Kconfig
> > @@ -301,6 +301,7 @@ config EFI_RNG_PROTOCOL
> >  config EFI_TCG2_PROTOCOL
> >  	bool "EFI_TCG2_PROTOCOL support"
> >  	depends on TPM_V2
> 
> Should we add default y?

We could, the memory 'wasted' for the eventlog is 4kb so I don't think
that's too much

> 
> > +	depends on SHA1 && SHA256 && SHA384 && SHA512
> 
> Would select make sense here?
> 
> select SHA1
> select SHA256
> select SHA512_ALGO
> select SHA384
> select SHA512

Yea ti would, I can change it if you prefer it.

Cheers
/Ilias
> 
> Best regards
> 
> Heinrich
> 
> >  	help
> >  	  Provide a EFI_TCG2_PROTOCOL implementation using the TPM hardware
> >  	  of the platform.
> >
> 


More information about the U-Boot mailing list