<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>
</head>
<body dir="ltr">
<div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Helvetica,sans-serif;" dir="ltr">
<p style="margin-top:0;margin-bottom:0">Hi Ben,</p>
<p style="margin-top:0;margin-bottom:0"><br>
</p>
<p style="margin-top:0;margin-bottom:0">Adding the 'generic' board it's an interesting option as a starting point to develop/porting to new boards.</p>
<p style="margin-top:0;margin-bottom:0"><br>
</p>
<p style="margin-top:0;margin-bottom:0">I'll try your suggestion to incorporate the changes from dsc and fdf files to my current working Tianocore coreboot payload.</p>
<p style="margin-top:0;margin-bottom:0">I'll report back the results!</p>
<p style="margin-top:0;margin-bottom:0"><br>
</p>
<p style="margin-top:0;margin-bottom:0">Thanks!</p>
<p style="margin-top:0;margin-bottom:0">Jorge<br>
</p>
<p style="margin-top:0;margin-bottom:0"><br>
</p>
<div id="Signature">
<div id="divtagdefaultwrapper" dir="ltr" style="font-size: 12pt; color: rgb(0, 0, 0); font-family: Calibri, Helvetica, sans-serif, "EmojiFont", "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols;">
<br>
</div>
</div>
</div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>De:</b> You, Benjamin <benjamin.you@intel.com><br>
<b>Enviado:</b> jueves, 20 de septiembre de 2018 10:21:55<br>
<b>Para:</b> Jorge Fernandez Monteagudo; coreboot@coreboot.org<br>
<b>Asunto:</b> RE: [coreboot] Tianocore and TPM</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">
<div class="PlainText">Hi Jorge,<br>
<br>
You could use UEFI Payload's .dsc and .fdf files as a reference and modify the TianoCore CorebootPayload's .dsc and .fdf files accordingly for those TPM related modules.<br>
<br>
UEFI Payload is under development (in staging area) and hasn't reached the quality standard required by EDKII master.
<br>
<br>
On CustomizationSample/Boards, yes it is required. However, a board's content may be trivial (as in the Qemu folder). Per your suggestion, probably we can add a board named "generic" that has all the minimalized settings so user won't have to create a new one
 if the "generic" one meets the needs.<br>
<br>
Thanks!<br>
<br>
- ben<br>
<br>
From: Jorge Fernandez Monteagudo [<a href="mailto:jorgefm@cirsa.com">mailto:jorgefm@cirsa.com</a>]
<br>
Sent: Thursday, September 20, 2018 3:24 PM<br>
To: You, Benjamin <benjamin.you@intel.com>; coreboot@coreboot.org<br>
Subject: Re: Tianocore and TPM<br>
<br>
Hi Ben!<br>
<br>
Thanks for the info! I have one question. Have I to implement a CustomizationSample/Boards for my board? With the current<br>
tianocore payload I don't have to implement nothing to have a working UEFI...<br>
<br>
Thanks!<br>
Jorge<br>
<br>
<br>
________________________________________<br>
De: You, Benjamin <benjamin.you@intel.com><br>
Enviado: jueves, 20 de septiembre de 2018 3:42:33<br>
Para: Jorge Fernandez Monteagudo; coreboot@coreboot.org<br>
Asunto: RE: Tianocore and TPM <br>
 <br>
Hi, <br>
<br>
Another note is on the use of NULL|UefiPayloadPkg/Library/Tpm2InstanceLib/Tpm2InstanceLib.inf. This lib is not fully populated right now.<br>
<br>
Please consider using  NULL|SecurityPkg/Library/Tpm2DeviceLibDTpm/Tpm2InstanceLibDTpm.inf depending on your needs.<br>
<br>
Thanks,<br>
<br>
- ben<br>
<br>
From: coreboot [<a href="mailto:coreboot-bounces@coreboot.org">mailto:coreboot-bounces@coreboot.org</a>] On Behalf Of You, Benjamin<br>
Sent: Thursday, September 20, 2018 8:44 AM<br>
To: Jorge Fernandez Monteagudo <jorgefm@cirsa.com>; coreboot@coreboot.org<br>
Subject: Re: [coreboot] Tianocore and TPM<br>
<br>
Hi Jorge,<br>
<br>
The staging UEFI Payload project (<a href="https://github.com/tianocore/edk2-staging/tree/UEFIPayload">https://github.com/tianocore/edk2-staging/tree/UEFIPayload</a>) has TPM support (although turned off by default, and using "FTPM" as the name (which needs
 to be fixed)).<br>
<br>
Please have a look at UefiPayloadPkgIA32X64.dsc for the components under tag "$(FTPM_ENABLE)". These components mainly do the measuring of firmware components and log the results.<br>
<br>
Also there is parsing logic in Library/PlatformInfoParseLib/ParseLib.c that parses TPM info in ACPI table passed from Coreboot. (This logic hasn't been sufficiently verified as this is still a "staging" project).<br>
<br>
You might have a try. Please let us know if you see any bugs / problems in these. You might also use the EDKII mailing list for discussing issues with the UEFI Payload.<br>
<br>
Thanks,<br>
<br>
- ben<br>
<br>
<br>
<br>
From: coreboot [<a href="mailto:coreboot-bounces@coreboot.org">mailto:coreboot-bounces@coreboot.org</a>] On Behalf Of Jorge Fernandez Monteagudo<br>
Sent: Wednesday, September 19, 2018 5:24 PM<br>
To: coreboot@coreboot.org<br>
Subject: [coreboot] Tianocore and TPM<br>
<br>
Hi all!<br>
<br>
I'm trying to enable the TPM2 support in the tianocore payload. The TPM2 device is working, because I've enabled the DEBUG_TPM and coreboot reports is up. I guess that I have to modify the 'CorebootPayloadPkgIa32X64.dsc' file to enable the TPM support but there
 are so many dependencies. Anybody has a working tianocore payload with TPM?<br>
<br>
Thanks!<br>
<br>
</div>
</span></font></div>
</body>
</html>