<html xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0cm;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
span.EmailStyle19
{mso-style-type:personal-reply;
font-family:"Calibri",sans-serif;
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;}
@page WordSection1
{size:612.0pt 792.0pt;
margin:72.0pt 72.0pt 72.0pt 72.0pt;}
div.WordSection1
{page:WordSection1;}
--></style>
</head>
<body lang="EN-GB" link="blue" vlink="purple" style="word-wrap:break-word">
<div class="WordSection1">
<p class="MsoNormal"><span style="mso-fareast-language:EN-US">Hi ilias,<o:p></o:p></span></p>
<p class="MsoNormal"><span style="mso-fareast-language:EN-US"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="mso-fareast-language:EN-US">Thank you for this series.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="mso-fareast-language:EN-US"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="mso-fareast-language:EN-US">Pushed as afb0ce7e9756..89f6170d6686<o:p></o:p></span></p>
<p class="MsoNormal"><span style="mso-fareast-language:EN-US"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="mso-fareast-language:EN-US">Regards,<o:p></o:p></span></p>
<p class="MsoNormal"><span style="mso-fareast-language:EN-US"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="mso-fareast-language:EN-US">Sami Mujawar<o:p></o:p></span></p>
<p class="MsoNormal"><span style="mso-fareast-language:EN-US"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="mso-fareast-language:EN-US"><o:p> </o:p></span></p>
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0cm 0cm 0cm">
<p class="MsoNormal" style="margin-bottom:12.0pt"><b><span style="font-size:12.0pt;color:black">From:
</span></b><span style="font-size:12.0pt;color:black">Ilias Apalodimas <ilias.apalodimas@linaro.org><br>
<b>Date: </b>Saturday, 13 March 2021 at 18:32<br>
<b>To: </b>devel@edk2.groups.io <devel@edk2.groups.io>, Sami Mujawar <Sami.Mujawar@arm.com>, Pierre Gondois <Pierre.Gondois@arm.com><br>
<b>Cc: </b>ardb+tianocore@kernel.org <ardb+tianocore@kernel.org>, sughosh.ganu@linaro.org <sughosh.ganu@linaro.org>, leif@nuviainc.com <leif@nuviainc.com>, Ilias Apalodimas <ilias.apalodimas@linaro.org><br>
<b>Subject: </b>[PATCH 0/3 v7] Add support for running StandaloneMm as OP-TEE TA<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal" style="margin-bottom:12.0pt">Hi,<br>
This is v7 of [1]<br>
<br>
Changes since V6:<br>
- Added param[in,out] in comments<br>
- Fixed an error on Maintainers entry <br>
<br>
Changes since V5:<br>
- Addressed coding style fixes from Pierre<br>
- Removed redundant checks in memory allocation and block usage<br>
- Removed unused COMPRESSION_TOOL_GUID<br>
- Renamed the files and directories following Leif's sugestion<br>
- Added Sami and myself as maintainers<br>
<br>
Changes since V4:<br>
- More coding stule fixes proposed by Sami, which Ecc or Patchcheck didn't<br>
report.<br>
- Adding missing error handling in InitializeFvAndVariableStoreHeaders().<br>
An allocation wasn't properly checked for success<br>
<br>
Changes since V3:<br>
- Coding style fixes proposed by Sami<br>
- Fixed all reported PatchCheck errors<br>
- Added overflow checks on the base aaddress allocated for EFI variables.<br>
The size of the partition is user defined (via Pcd's) and the memory layout<br>
and allocation address depends on OP-TEE. So let's make sure we won't overflow<br>
when calculating the 3 partitions needed for FTW<br>
- Switched some PcdGet/Set32 to 64 to accomodate 64-bit addressing<br>
- Removed some duplicate entries in Platform/StMMRpmb/PlatformStandaloneMm.dsc<br>
- Added reviewed-by tags on patch 2/2<br>
<br>
Changes since V2:<br>
- Allocate a dynamic number of pages based on the Pcd values instead<br>
of a static number<br>
- Clean up unused structs in header file<br>
- Added checks in OpTeeRpmbFvbGetBlockSize and handle NumLba=0<br>
<br>
Changes since V1:<br>
Some enhancements made by Ilias to the Optee Rpmb driver<br>
<br>
[1] <a href="https://edk2.groups.io/g/devel/message/66483?p=,,,20,0,0,0::Created,,ilias+apalodimas,20,2,0,77703661">
https://edk2.groups.io/g/devel/message/66483?p=,,,20,0,0,0::Created,,ilias+apalodimas,20,2,0,77703661</a><br>
<br>
Ilias Apalodimas (3):<br>
Drivers/OpTeeRpmb: Add an OP-TEE backed RPMB driver<br>
StMMRpmb: Add support for building StandaloneMm image for OP-TEE<br>
Maintainers: Add maintainers for StandAloneMM and RPMB driver<br>
<br>
Drivers/OpTee/OpteeRpmbPkg/FixupPcd.c | 81 ++<br>
Drivers/OpTee/OpteeRpmbPkg/FixupPcd.inf | 43 +<br>
Drivers/OpTee/OpteeRpmbPkg/OpTeeRpmbFv.inf | 58 ++<br>
Drivers/OpTee/OpteeRpmbPkg/OpTeeRpmbFvb.c | 861 ++++++++++++++++++<br>
Drivers/OpTee/OpteeRpmbPkg/OpTeeRpmbFvb.h | 51 ++<br>
Maintainers.txt | 10 +<br>
.../PlatformStandaloneMmRpmb.dsc | 162 ++++<br>
.../PlatformStandaloneMmRpmb.fdf | 111 +++<br>
8 files changed, 1377 insertions(+)<br>
create mode 100644 Drivers/OpTee/OpteeRpmbPkg/FixupPcd.c<br>
create mode 100644 Drivers/OpTee/OpteeRpmbPkg/FixupPcd.inf<br>
create mode 100644 Drivers/OpTee/OpteeRpmbPkg/OpTeeRpmbFv.inf<br>
create mode 100644 Drivers/OpTee/OpteeRpmbPkg/OpTeeRpmbFvb.c<br>
create mode 100644 Drivers/OpTee/OpteeRpmbPkg/OpTeeRpmbFvb.h<br>
create mode 100644 Platform/StandaloneMm/PlatformStandaloneMmPkg/PlatformStandaloneMmRpmb.dsc<br>
create mode 100644 Platform/StandaloneMm/PlatformStandaloneMmPkg/PlatformStandaloneMmRpmb.fdf<br>
<br>
-- <br>
2.30.1<o:p></o:p></p>
</div>
</div>
</body>
</html>
<div width="1" style="color:white;clear:both">_._,_._,_</div> <hr> Groups.io Links:<p> You receive all messages sent to this group. <p> <a target="_blank" href="https://edk2.groups.io/g/devel/message/73418">View/Reply Online (#73418)</a> | | <a target="_blank" href="https://groups.io/mt/81309166/1813853">Mute This Topic</a> | <a href="https://edk2.groups.io/g/devel/post">New Topic</a><br> <a href="https://edk2.groups.io/g/devel/editsub/1813853">Your Subscription</a> | <a href="mailto:devel+owner@edk2.groups.io">Contact Group Owner</a> | <a href="https://edk2.groups.io/g/devel/unsub">Unsubscribe</a> [edk2-devel-archive@redhat.com]<br> <div width="1" style="color:white;clear:both">_._,_._,_</div>