Hi Rebecca,<br /><br />Thank you for your patch.<br /><br />It functionally looks good however I noticed one small grammatical error to fix, noted below.<br /><br />Reviewed-by: Chris Jones <christopher.jones@arm.com><br /><br /><br />Thanks,<br />Chris<br /><br />
<blockquote>The repo with the Visual Studio support no longer exists.<br />fiptool from the prebuilt_tools repo doesn't work due to a missing<br />dependency on libcrypto.so.1.0.0, so tell users to build it from the<br />trusted-firmware-a repo instead.<br />There's a newer version of fvp-uefi.zip that was released in 2020.<br /><br />Signed-off-by: Rebecca Cran <rebecca@bsdio.com><br />---<br />Platform/ARM/Readme.md | 20 ++++++++++----------<br />1 file changed, 10 insertions(+), 10 deletions(-)<br /><br />diff --git a/Platform/ARM/Readme.md b/Platform/ARM/Readme.md<br />index e1a405b700..afc9ad3646 100644<br />--- a/Platform/ARM/Readme.md<br />+++ b/Platform/ARM/Readme.md<br />@@ -8,10 +8,8 @@ can be found here:<br />=0D<br />##Requirements=0D<br />- A 32-bit or 64-bit Linux host machine.=0D<br />-- Visual Studio is not officially supported, experimental support can be f=<br />ound here:=0D<br />-[<a href="https://git.linaro.org/people/leif.lindholm/edk2.git/log/?h=3Daarch64-vs" target="_blank" rel="noopener">https://git.linaro.org/people/leif.lindholm/edk2.git/log/?h=3Daarch64-vs</a>]=<br />=0D<br />=0D<br />-# Build EDK2 Tianocore=0D<br />+# Build EDK2 TianoCore=0D<br />=0D<br />`build -a AARCH64 -p Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc =<br />-t GCC5`=0D<br />=0D<br />@@ -26,7 +24,7 @@ prebuilt edk2 image.<br />=0D<br />We will also rely on the "run_model" script that comes with the prebuilts,=<br />it=0D<br />is entirely possible to run the model without this but would require quite=<br />a bit=0D</blockquote>
[CHRIS] This does not make sense, instead I suggest:<br />"of knowledge regarding the arguments <strong>of the</strong> ARM fastmodel (documentation can be f=<br />ound here:=0D".<br />Note: while this was not introduced by you it would be nice to fix now.<br />
<blockquote>-of knowledge regarding the areguments ARM fastmodel (documentation can be =<br />found here:=0D<br />+of knowledge regarding the arguments ARM fastmodel (documentation can be f=<br />ound here:=0D</blockquote>
<br />
<blockquote>[<a href="https://developer.arm.com/docs/100966/1101/programming-reference-for-base=" target="_blank" rel="noopener">https://developer.arm.com/docs/100966/1101/programming-reference-for-base=</a><br />-fvps/base-platform-revc-features])=0D<br />however the manual set of the FVP is outside the scope of this document. I=<br />f you are interested=0D<br />please consult the documentation.=0D<br />@@ -40,16 +38,18 @@ the binaries in the same directory.<br />- Select Armv8-A Base Platform FVP based on Fast Models 11.4=0D<br />- It has a click through license but is free.=0D<br />=0D<br />-2. Download the 18.10 Linaro ARM Landing Team release for FVP booting UEFI=<br />=0D<br />-<a href="https://releases.linaro.org/members/arm/platforms/18.10/fvp-uefi.zip=0D" target="_blank" rel="noopener">https://releases.linaro.org/members/arm/platforms/18.10/fvp-uefi.zip=0D</a><br />+2. Download the 20.01 Linaro ARM Landing Team release for FVP booting UEFI=<br />=0D<br />+<a href="https://releases.linaro.org/members/arm/platforms/20.01/fvp-uefi.zip=0D" target="_blank" rel="noopener">https://releases.linaro.org/members/arm/platforms/20.01/fvp-uefi.zip=0D</a><br />=0D<br />-3. Download the prebuilt fiptool from <a href="https://git.linaro.org/landing-teams=" target="_blank" rel="noopener">https://git.linaro.org/landing-teams=</a><br />/working/arm/prebuilt_tools.git=0D<br />+3. Clone the trusted firmware repo from <a href="https://git.trustedfirmware.org/TF=" target="_blank" rel="noopener">https://git.trustedfirmware.org/TF=</a><br />-A/trusted-firmware-a.git=0D<br />=0D<br />-4. Update the fip.bin image from fvp-uefi.zip by running the following com=<br />mand:=0D<br />+4. Build fiptool: `make -C trusted-firmware-a/tools/fiptool`=0D<br />=0D<br />- `fiptool update --nt-fw=3D[path to binary built above] fip.bin`=0D<br />+5. Update the fip.bin image from fvp-uefi.zip by running the following com=<br />mand:=0D<br />=0D<br />-5. Execute the FVP run_model.sh script from fvp-uefi.zip and provide a pat=<br />h to the FVP binaries=0D<br />+ `./trusted-firmware-a/tools/fiptool/fiptool update --nt-fw=3D[path to bin=<br />ary built above] fip.bin`=0D<br />+=0D<br />+6. Execute the FVP run_model.sh script from fvp-uefi.zip and provide a pat=<br />h to the FVP binaries=0D<br />downloaded in step 1):=0D<br />=0D<br />`MODEL=3D[path to FVP binary] ./run_model.sh`=0D<br />--=20<br />2.31.1</blockquote>


 <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/75234">View/Reply Online (#75234)</a> |    |  <a target="_blank" href="https://groups.io/mt/82806791/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>