Hi, pon., 14 mar 2022 o 18:17 Sunny Wang <sunny.w...@arm.com> napisaĆ(a): > > Looks good. Thanks much, Marcin. > > This will be helpful as http://wiki.macchiatobin.net/tiki-index.php has been > not accessible for quite a while. > > Reviewed-by: Sunny Wang <sunny.w...@arm.com> >
Do you have any feedback about the patch? Best regards, Marcin > Best Regards, > Sunny > -----Original Message----- > From: Marcin Wojtas <m...@semihalf.com> > Sent: 14 March 2022 12:06 > To: devel@edk2.groups.io > Cc: quic_llind...@quicinc.com; ardb+tianoc...@kernel.org; j...@semihalf.com; > g...@semihalf.com; Sunny Wang <sunny.w...@arm.com>; Marcin Wojtas > <m...@semihalf.com> > Subject: [edk2-platforms PATCH] Marvell/SolidRun: Rework Readme.md files > > From: Grzegorz Bernacki <g...@semihalf.com> > > This patch reworks Readme.md files of Marvell and SolidRun > platforms as follows: > * Add supported features list. > * Leave minmal EDK2 build command. > * Add 'NOTE' box about INCLUDE_TFTP_COMMAND build flag. > * Move full firmware image build/burn howtos to external wiki pages. > * Add links and update paragraphs. > > Signed-off-by: Marcin Wojtas <m...@semihalf.com> > > --- > In order to ease review/merge the patch is available in a public repository: > https://github.com/Semihalf/edk2-platforms/commits/marvell-howtos-upstream-r20220314 > > Platform/Marvell/Cn913xDb/Readme.md | 104 +++++++----------- > Platform/SolidRun/Armada80x0McBin/Readme.md | 113 ++++++++------------ > Platform/SolidRun/Cn913xCEx7Eval/Readme.md | 109 ++++++++----------- > 3 files changed, 124 insertions(+), 202 deletions(-) > > diff --git a/Platform/Marvell/Cn913xDb/Readme.md > b/Platform/Marvell/Cn913xDb/Readme.md > index ecdb78302b..0b2ab676f8 100644 > --- a/Platform/Marvell/Cn913xDb/Readme.md > +++ b/Platform/Marvell/Cn913xDb/Readme.md > @@ -5,93 +5,65 @@ > > > This is a port of 64-bit TianoCore EDK II firmware for the Marvell CN913x > Development Board. > > > > -# Building the firmware > > - > > -## Prepare toolchain (for cross-compilation only): > > - > > -1. Download the toolchain: > > - > > - ``` > > - wget > https://releases.linaro.org/components/toolchain/binaries/7.5-2019.12/aarch64-linux-gnu/gcc-linaro-7.5.0-2019.12-x86_64_aarch64-linux-gnu.tar.xz > > - ``` > > +# Supported features > > > > -1. After extracting, setup the path and compiler prefix to > GCC5\_AARCH64\_PREFIX variable: > > +Features supported in EDK2: > > > > - ``` > > - export > GCC5_AARCH64_PREFIX=<toolchain_path>/gcc-linaro-7.5.0-2019.12-x86_64_aarch64-linux-gnu/bin/aarch64-linux-gnu- > > - ``` > > +* 1x PCIE root complex > > +* Networking: > > + * 3x 10 GbE via SFP+ > > + * 2x 1 GbE RGMII via RJ45 > > +* 5x USB 2.0/3.0 > > +* 3x SATA > > +* 2x uSD > > +* 1x eMMC > > +* RTC > > +* SPI flash & memory-mapped variable storage access > > +* I2C > > +* GPIO > > > > -## Prepare prerequisites > > +Hardware description: > > > > -1. Create a new folder (directory) on your local development machine > > - for use as your workspace. This example uses `/work/git/tianocore`, > modify as > > - appropriate for your needs. > > +* ACPI (default) > > +* Device Tree > > > > - ``` > > - $ export WORKSPACE=/work/git/tianocore > > - $ mkdir -p $WORKSPACE > > - $ cd $WORKSPACE > > - ``` > > +Others: > > > > -1. Clone the Trusted Firmware repository: > > +* Signed capsule update > > +* X64 option ROM emulator > > > > - ``` > > - $ cd ${WORKSPACE} > > - $ git clone https://git.trustedfirmware.org/TF-A/trusted-firmware-a.git > > - ``` > > -1. Clone repository for auxiliary firmware on the SoC co-processors and > checkout to binaries-marvell-armada-SDK10.0.1.0: > > - > > - ``` > > - $ cd ${WORKSPACE} > > - $ git clone > https://github.com/MarvellEmbeddedProcessors/binaries-marvell.git > > - $ cd binaries-marvell/ > > - $ git checkout -b binaries-marvell-armada-SDK10.0.1.0 > origin/binaries-marvell-armada-SDK10.0.1.0 > > - ``` > > -1. Clone the DDR training code from: > > +# Building the firmware > > > > - ``` > > - $ cd ${WORKSPACE} > > - $ git clone > https://github.com/MarvellEmbeddedProcessors/mv-ddr-marvell.git > > - ``` > > ## Prepare EDKII environment: > > > > -Please follow instructions from "Obtaining source code" and "Manual > building" from the top level edk2-platforms > [Readme.md](https://github.com/tianocore/edk2-platforms#readme). > > +Please follow instructions from [Obtaining source > code](https://github.com/tianocore/edk2-platforms#obtaining-source-code) > > +and [Manual > building](https://github.com/tianocore/edk2-platforms#manual-building) from > the > > +top level edk2-platforms > [Readme.md](https://github.com/tianocore/edk2-platforms#readme). > > > > ## Build EDKII: > > > > -1. Use below build command: > > +Use below build command: > > > > - ``` > > - $ cd ${WORKSPACE} > > - $ build -a AARCH64 -t GCC5 -b RELEASE -D CN9132 -D INCLUDE_TFTP_COMMAND > -D CAPSULE_ENABLE -p Platform/Marvell/Cn913xDb/Cn913xDbA.dsc > > - ``` > > + ``` > > + $ build -a AARCH64 -t GCC5 -b RELEASE -D CN9132 -D CAPSULE_ENABLE -D > X64EMU_ENABLE -p Platform/Marvell/Cn913xDb/Cn913xDbA.dsc > > + ``` > > > > -## Build the final firmware image: > > +--- > > +**NOTE** > > > > -1. Set BL33 variable to path to EDK II output binary: > > +'-D INCLUDE_TFTP_COMMAND' is optional and can be added in order to enable > `tftp` command in UEFI Shell. > > > > - ``` > > - $ export > BL33=${WORKSPACE}/Build/Cn9132DbA-AARCH64/RELEASE_GCC5/FV/ARMADA_EFI.fd > > - ``` > > -1. Export SCP_BL2 variable: > > +--- > > > > - ``` > > - $ export SCP_BL2=${WORKSKPACE}/binaries-marvell/mrvl_scp_bl2.img > > - ``` > > -1. Export compiler variables (for cross-compilation only): > > +## Build the final firmware image: > > > > - ``` > > - $ export ARCH=arm64 > > - $ export > CROSS_COMPILE=<toolchain_path>/gcc-linaro-7.5.0-2019.12-x86_64_aarch64-linux-gnu/bin/aarch64-linux-gnu- > > - ``` > > -1. Build the image: > > +In addition to EDKII binary, the complete firmware image comprises the TF-A > and other components. > > +A complete build instruction can be found at [wiki > page](https://github.com/Semihalf/edk2-platforms/wiki/Build_firmware). > > > > - ``` > > - $ cd ${WORKSPACE}/trusted-firmware-a/ > > - $ make LOG_LEVEL=20 MV_DDR_PATH=${WORKSPACE}/mv-ddr-marvell CP_NUM=3 > PLAT=t9130 all fip mrvl_flash > > +## Burning the firmware > > > > - ``` > > -The firmware image `flash-image.bin` can be found in `build/t9130/release/` > directory. > > +Please follow instruction at [wiki > page](https://github.com/Semihalf/edk2-platforms/wiki/Burning_firmware) > > +to burn image to desired boot device. > > > > # ARM System Ready certification. > > > > diff --git a/Platform/SolidRun/Armada80x0McBin/Readme.md > b/Platform/SolidRun/Armada80x0McBin/Readme.md > index c63cf41b2d..8c514835c4 100644 > --- a/Platform/SolidRun/Armada80x0McBin/Readme.md > +++ b/Platform/SolidRun/Armada80x0McBin/Readme.md > @@ -3,94 +3,69 @@ > > > # Summary > > > > -This is a port of 64-bit TianoCore EDK II firmware for the SolidRun > MacchiatoBin platform based on the Marvell ARMADA 8040 SoC. > > +This is a port of 64-bit TianoCore EDK II firmware for the [SolidRun > MacchiatoBin Double > Shot](https://solidrun.atlassian.net/wiki/spaces/developer/pages/286655749/MACCHIATObin+Single+Double+Shot+Quick+Start+Guide) > > +platform based on the Marvell ARMADA 8040 SoC. > > > > -# Building the firmware > > +# Supported features > > > > -## Prepare toolchain (for cross-compilation only): > > +Features supported in EDK2: > > > > -1. Download the toolchain: > > +* 1x PCIE x4 > > +* Networking: > > + * 2x 10 GbE via SFP+ / RJ45 > > + * 1x 2500 Base-X via SFP+ > > + * 1x 1 GbE SGMII via RJ45 > > +* 1x USB 3.0 > > +* 2x USB 2.0 > > +* 3x SATA > > +* 1x uSD > > +* 1x eMMC > > +* RTC > > +* SPI flash & memory-mapped variable storage access > > +* GPIO > > > > - ``` > > - wget > https://releases.linaro.org/components/toolchain/binaries/7.5-2019.12/aarch64-linux-gnu/gcc-linaro-7.5.0-2019.12-x86_64_aarch64-linux-gnu.tar.xz > > - ``` > > +Hardware description: > > > > -1. After extracting, setup the path and compiler prefix to > GCC5\_AARCH64\_PREFIX variable: > > +* ACPI (default) > > +* Device Tree > > > > - ``` > > - export > GCC5_AARCH64_PREFIX=<toolchain_path>/gcc-linaro-7.5.0-2019.12-x86_64_aarch64-linux-gnu/bin/aarch64-linux-gnu- > > - ``` > > +Others: > > > > -## Prepare prerequisites > > +* Signed capsule update > > +* X64 option ROM emulator > > > > -1. Create a new folder (directory) on your local development machine > > - for use as your workspace. This example uses `/work/git/tianocore`, > modify as > > - appropriate for your needs. > > +# Building the firmware > > > > - ``` > > - $ export WORKSPACE=/work/git/tianocore > > - $ mkdir -p $WORKSPACE > > - $ cd $WORKSPACE > > - ``` > > +## Prepare EDKII environment: > > > > -1. Clone the Trusted Firmware repository: > > +Please follow instructions from [Obtaining source > code](https://github.com/tianocore/edk2-platforms#obtaining-source-code) > > +and [Manual > building](https://github.com/tianocore/edk2-platforms#manual-building) from > the > > +top level edk2-platforms > [Readme.md](https://github.com/tianocore/edk2-platforms#readme). > > > > - ``` > > - $ cd ${WORKSPACE} > > - $ git clone https://git.trustedfirmware.org/TF-A/trusted-firmware-a.git > > - ``` > > -1. Clone repository for auxiliary firmware on the SoC co-processors and > checkout to binaries-marvell-armada-SDK10.0.1.0: > > +## Build EDKII: > > > > - ``` > > - $ cd ${WORKSPACE} > > - $ git clone > https://github.com/MarvellEmbeddedProcessors/binaries-marvell.git > > - $ cd binaries-marvell/ > > - $ git checkout -b binaries-marvell-armada-SDK10.0.1.0 > origin/binaries-marvell-armada-SDK10.0.1.0 > > - ``` > > -1. Clone the DDR training code from: > > +Use below build command: > > > > - ``` > > - $ cd ${WORKSPACE} > > - $ git clone > https://github.com/MarvellEmbeddedProcessors/mv-ddr-marvell.git > > - ``` > > -## Prepare EDKII environment: > > + ``` > > + $ build -a AARCH64 -t GCC5 -b RELEASE -D X64EMU_ENABLE -p > Platform/SolidRun/Armada80x0McBin/Armada80x0McBin.dsc > > + ``` > > > > -Please follow instructions from "Obtaining source code" and "Manual > building" from the top level edk2-platforms > [Readme.md](https://github.com/tianocore/edk2-platforms#readme). > > +--- > > +**NOTE** > > > > -## Build EDKII: > > - > > -1. Use below build command: > > +'-D INCLUDE_TFTP_COMMAND' is optional and can be added in order to enable > `tftp` command in UEFI Shell. > > > > - ``` > > - $ cd ${WORKSPACE} > > - $ build -a AARCH64 -t GCC5 -b RELEASE -D INCLUDE_TFTP_COMMAND -D > X64EMU_ENABLE -p Platform/SolidRun/Armada80x0McBin/Armada80x0McBin.dsc > > - ``` > > +--- > > > > ## Build the final firmware image: > > > > -1. Set BL33 variable to path to EDK II output binary: > > - > > - ``` > > - $ export > BL33=${WORKSPACE}/Build/Armada80x0McBin-AARCH64/RELEASE_GCC5/FV/ARMADA_EFI.fd > > - ``` > > -1. Export SCP_BL2 variable: > > - > > - ``` > > - $ export SCP_BL2=${WORKSKPACE}/binaries-marvell/mrvl_scp_bl2.img > > - ``` > > -1. Export compiler variables (for cross-compilation only): > > - > > - ``` > > - $ export ARCH=arm64 > > - $ export > CROSS_COMPILE=<toolchain_path>/gcc-linaro-7.5.0-2019.12-x86_64_aarch64-linux-gnu/bin/aarch64-linux-gnu- > > - ``` > > -1. Build the image: > > - > > - ``` > > - $ cd ${WORKSPACE}/trusted-firmware-a/ > > - $ make LOG_LEVEL=20 MV_DDR_PATH=${WORKSPACE}/mv-ddr-marvell > PLAT=a80x0_mcbin all fip mrvl_flash > > - ``` > > -The firmware image `flash-image.bin` can be found in > `build/a80x0_mcbin/release/` directory. > > +In addition to EDKII binary, the complete firmware image comprises the TF-A > and other components. > > +A complete build instruction can be found at [wiki > page](https://github.com/Semihalf/edk2-platforms/wiki/Build_firmware). > > + > > +## Burning the firmware > > + > > +Please follow instruction at [wiki > page](https://github.com/Semihalf/edk2-platforms/wiki/Burning_firmware) > > +to burn image to desired boot device. > > > > # ARM System Ready certification. > > > > diff --git a/Platform/SolidRun/Cn913xCEx7Eval/Readme.md > b/Platform/SolidRun/Cn913xCEx7Eval/Readme.md > index 813e723b65..0db22b5175 100644 > --- a/Platform/SolidRun/Cn913xCEx7Eval/Readme.md > +++ b/Platform/SolidRun/Cn913xCEx7Eval/Readme.md > @@ -3,95 +3,70 @@ > > > # Summary > > > > -This is a port of 64-bit TianoCore EDK II firmware for the SolidRun CN913x > CEx7 Evaluation Board. > > +This is a port of 64-bit TianoCore EDK II firmware for the [SolidRun CN913x > CEx7 Evaluation > Board](https://solidrun.atlassian.net/wiki/spaces/developer/pages/197493948/CN9132+COM+EVK+Quick+Start+Guide). > > > > -# Building the firmware > > - > > -## Prepare toolchain (for cross-compilation only): > > - > > -1. Download the toolchain: > > - > > - ``` > > - wget > https://releases.linaro.org/components/toolchain/binaries/7.5-2019.12/aarch64-linux-gnu/gcc-linaro-7.5.0-2019.12-x86_64_aarch64-linux-gnu.tar.xz > > - ``` > > +# Supported features > > > > -1. After extracting, setup the path and compiler prefix to > GCC5\_AARCH64\_PREFIX variable: > > +Features supported in EDK2: > > > > - ``` > > - export > GCC5_AARCH64_PREFIX=<toolchain_path>/gcc-linaro-7.5.0-2019.12-x86_64_aarch64-linux-gnu/bin/aarch64-linux-gnu- > > - ``` > > +* 1x PCIE x4 + 6x PCIE x1 > > +* Networking: > > + * 1x 10 GbE via SFP+ > > + * 2x 5 GbE via RJ45 > > + * 1x 2500 Base-X via quad 1 Gbps switch > > + * 1x 1 GbE RGMII via RJ45 > > +* 2x USB 3.0 > > +* 4x USB 2.0 > > +* 2x SATA > > +* 1x uSD > > +* 1x eMMC > > +* RTC > > +* SPI flash & memory-mapped variable storage access > > +* I2C > > +* GPIO > > > > -## Prepare prerequisites > > +Hardware description: > > > > -1. Create a new folder (directory) on your local development machine > > - for use as your workspace. This example uses `/work/git/tianocore`, > modify as > > - appropriate for your needs. > > +* ACPI (default) > > +* Device Tree > > > > - ``` > > - $ export WORKSPACE=/work/git/tianocore > > - $ mkdir -p $WORKSPACE > > - $ cd $WORKSPACE > > - ``` > > +Others: > > > > -1. Clone the Trusted Firmware repository: > > +* Signed capsule update > > +* X64 option ROM emulator > > > > - ``` > > - $ cd ${WORKSPACE} > > - $ git clone https://git.trustedfirmware.org/TF-A/trusted-firmware-a.git > > - ``` > > -1. Clone repository for auxiliary firmware on the SoC co-processors and > checkout to binaries-marvell-armada-SDK10.0.1.0: > > - > > - ``` > > - $ cd ${WORKSPACE} > > - $ git clone > https://github.com/MarvellEmbeddedProcessors/binaries-marvell.git > > - $ cd binaries-marvell/ > > - $ git checkout -b binaries-marvell-armada-SDK10.0.1.0 > origin/binaries-marvell-armada-SDK10.0.1.0 > > - ``` > > -1. Clone the DDR training code from: > > +# Building the firmware > > > > - ``` > > - $ cd ${WORKSPACE} > > - $ git clone > https://github.com/MarvellEmbeddedProcessors/mv-ddr-marvell.git > > - ``` > > ## Prepare EDKII environment: > > > > -Please follow instructions from "Obtaining source code" and "Manual > building" from the top level edk2-platforms > [Readme.md](https://github.com/tianocore/edk2-platforms#readme). > > +Please follow instructions from [Obtaining source > code](https://github.com/tianocore/edk2-platforms#obtaining-source-code) > > +and [Manual > building](https://github.com/tianocore/edk2-platforms#manual-building) from > the > > +top level edk2-platforms > [Readme.md](https://github.com/tianocore/edk2-platforms#readme). > > > > ## Build EDKII: > > > > -1. Use below build command: > > +Use below build command: > > > > - ``` > > - $ cd ${WORKSPACE} > > - $ build -a AARCH64 -t GCC5 -b RELEASE -D INCLUDE_TFTP_COMMAND -D > CAPSULE_ENABLE -D X64EMU_ENABLE -p > Platform/SolidRun/Cn913xCEx7Eval/Cn913xCEx7Eval.dsc > > - ``` > > + ``` > > + build -a AARCH64 -t GCC5 -b RELEASE -D CAPSULE_ENABLE -D X64EMU_ENABLE -p > Platform/SolidRun/Cn913xCEx7Eval/Cn913xCEx7Eval.dsc > > + ``` > > > > -## Build the final firmware image: > > +--- > > +**NOTE** > > > > -1. Set BL33 variable to path to EDK II output binary: > > +'-D INCLUDE_TFTP_COMMAND' is optional and can be added in order to enable > `tftp` command in UEFI Shell. > > > > - ``` > > - $ export > BL33=${WORKSPACE}/Build/Cn913xCEx7Eval-AARCH64/RELEASE_GCC5/FV/ARMADA_EFI.fd > > - ``` > > -1. Export SCP_BL2 variable: > > +--- > > > > - ``` > > - $ export SCP_BL2=${WORKSKPACE}/binaries-marvell/mrvl_scp_bl2.img > > - ``` > > -1. Export compiler variables (for cross-compilation only): > > +## Build the final firmware image: > > > > - ``` > > - $ export ARCH=arm64 > > - $ export > CROSS_COMPILE=<toolchain_path>/gcc-linaro-7.5.0-2019.12-x86_64_aarch64-linux-gnu/bin/aarch64-linux-gnu- > > - ``` > > -1. Build the image: > > +In addition to EDKII binary, the complete firmware image comprises the TF-A > and other components. > > +A complete build instruction can be found at [wiki > page](https://github.com/Semihalf/edk2-platforms/wiki/Build_firmware). > > > > - ``` > > - $ cd ${WORKSPACE}/trusted-firmware-a/ > > - $ make LOG_LEVEL=20 MV_DDR_PATH=${WORKSPACE}/mv-ddr-marvell CP_NUM=3 > PLAT=t9130_cex7_eval all fip mrvl_flash > > +## Burning the firmware > > > > - ``` > > -The firmware image `flash-image.bin` can be found in > `build/t9130_cex7_eval/release/` directory. > > +Please follow instruction at [wiki > page](https://github.com/Semihalf/edk2-platforms/wiki/Burning_firmware) > > +to burn image to desired boot device. > > > > # ARM System Ready certification. > > > > -- > 2.29.0 > > IMPORTANT NOTICE: The contents of this email and any attachments are > confidential and may also be privileged. If you are not the intended > recipient, please notify the sender immediately and do not disclose the > contents to any other person, use it for any purpose, or store or copy the > information in any medium. Thank you. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#88479): https://edk2.groups.io/g/devel/message/88479 Mute This Topic: https://groups.io/mt/89771621/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-