Hi Tom, On Tue, 16 Apr 2024 at 15:22, Tom Rini <tr...@konsulko.com> wrote:
> On Tue, Apr 16, 2024 at 12:00:07PM -0700, Raymond Mao wrote: > > > When MBEDTLS_LIB_CRYPTO is enabled, use the APIs of sha256 from > > hash shim layer instead. > > > > Signed-off-by: Raymond Mao <raymond....@linaro.org> > > --- > > lib/efi_loader/efi_tcg2.c | 9 +++++++++ > > 1 file changed, 9 insertions(+) > > Why can't we have the abstraction be at the level where we include one > library or the other so that the calling code doesn't change? > > Yes, we can do this - The only reason I added new hash APIs with "_mb" is just to avoid vendor drivers that are using hash functions switch to MbedTLS with this patch set (As a [RFC], I was going to control all effects within the EFI loader). But if you think a complete switching has more benefits for estimation, I can unify all APIs between on/off MbedTLS. Thanks and Regards, Raymond