[1/3] docs/platform: Update QEMU parameter for fw_payload

Message ID	1592906212-13711-1-git-send-email-bmeng.cn@gmail.com
State	Superseded
Headers	show Return-Path: <opensbi-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org> From: Bin Meng <bmeng.cn@gmail.com> To: Anup Patel <Anup.Patel@wdc.com>, opensbi@lists.infradead.org Subject: [PATCH 1/3] docs/platform: Update QEMU parameter for fw_payload Date: Tue, 23 Jun 2020 02:56:50 -0700 Message-Id: <1592906212-13711-1-git-send-email-bmeng.cn@gmail.com> summary: Content analysis details: (-0.2 points) pts rule name description ---- ---------------------- -------------------------------------------------- -0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at https://www.dnswl.org/, no trust [2607:f8b0:4864:20:0:0:0:62c listed in] [list.dnswl.org] -0.0 SPF_PASS SPF: sender matches SPF record 0.0 SPF_HELO_NONE SPF: HELO does not publish an SPF Record 0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail provider [bmeng.cn[at]gmail.com] -0.1 DKIM_VALID Message has at least one valid DKIM or DK signature -0.1 DKIM_VALID_EF Message has a valid DKIM or DK signature from envelope-from domain -0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from author's domain 0.1 DKIM_SIGNED Message has a DKIM or DK signature, not necessarily valid Precedence: list Cc: Bin Meng <bin.meng@windriver.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "opensbi" <opensbi-bounces@lists.infradead.org> Errors-To: opensbi-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org
Series	[1/3] docs/platform: Update QEMU parameter for fw_payload \| expand [1/3] docs/platform: Update QEMU parameter for fw_payload [2/3] config.mk: Update QEMU run command for generic and sifive fu540 platforms [3/3] docs/platform: sifive_fu540: Update U-Boot instructions

Message ID

1592906212-13711-1-git-send-email-bmeng.cn@gmail.com

State

Superseded

Headers

From: Bin Meng <bmeng.cn@gmail.com>
To: Anup Patel <Anup.Patel@wdc.com>,
	opensbi@lists.infradead.org
Subject: [PATCH 1/3] docs/platform: Update QEMU parameter for fw_payload
Date: Tue, 23 Jun 2020 02:56:50 -0700
Message-Id: <1592906212-13711-1-git-send-email-bmeng.cn@gmail.com>
Precedence: list
Cc: Bin Meng <bin.meng@windriver.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Sender: "opensbi" <opensbi-bounces@lists.infradead.org>
Errors-To: opensbi-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org

Series

[1/3] docs/platform: Update QEMU parameter for fw_payload | expand

Commit Message

Bin Meng June 23, 2020, 9:56 a.m. UTC

From: Bin Meng <bin.meng@windriver.com>

Since QEMU v5.1, if there is no "-bios" option provided, the default
OpenSBI firmware will be loaded by QEMU as the BIOS automatically,
hence to load fw_payload type image, we should explicitly pass the
"-bios" option to QEMU.

Signed-off-by: Bin Meng <bin.meng@windriver.com>
---

 docs/platform/qemu_virt.md | 12 ++++++------
 docs/platform/spike.md     |  4 ++--
 2 files changed, 8 insertions(+), 8 deletions(-)

Comments

Atish Patra June 25, 2020, 5:30 p.m. UTC | #1

On Wed, Jun 24, 2020 at 11:48 PM Bin Meng <bmeng.cn@gmail.com> wrote:
>
> From: Bin Meng <bin.meng@windriver.com>
>
> Since QEMU v5.1, if there is no "-bios" option provided, the default
> OpenSBI firmware will be loaded by QEMU as the BIOS automatically,
> hence to load fw_payload type image, we should explicitly pass the
> "-bios" option to QEMU.
>
> Signed-off-by: Bin Meng <bin.meng@windriver.com>
> ---
>
>  docs/platform/qemu_virt.md | 12 ++++++------
>  docs/platform/spike.md     |  4 ++--
>  2 files changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/docs/platform/qemu_virt.md b/docs/platform/qemu_virt.md
> index 2196e5a..2777148 100644
> --- a/docs/platform/qemu_virt.md
> +++ b/docs/platform/qemu_virt.md
> @@ -28,7 +28,7 @@ make PLATFORM=generic
>  Run:
>  ```
>  qemu-system-riscv64 -M virt -m 256M -nographic \
> -       -kernel build/platform/generic/firmware/fw_payload.bin
> +       -bios build/platform/generic/firmware/fw_payload.bin
>  ```
>
>  **U-Boot Payload**
> @@ -44,7 +44,7 @@ make PLATFORM=generic FW_PAYLOAD_PATH=<uboot_build_directory>/u-boot.bin
>  Run:
>  ```
>  qemu-system-riscv64 -M virt -m 256M -nographic \
> -       -kernel build/platform/generic/firmware/fw_payload.elf
> +       -bios build/platform/generic/firmware/fw_payload.elf
>  ```
>  or
>  ```
> @@ -66,7 +66,7 @@ make PLATFORM=generic FW_PAYLOAD_PATH=<linux_build_directory>/arch/riscv/boot/Im
>  Run:
>  ```
>  qemu-system-riscv64 -M virt -m 256M -nographic \
> -       -kernel build/platform/generic/firmware/fw_payload.elf \
> +       -bios build/platform/generic/firmware/fw_payload.elf \
>         -drive file=<path_to_linux_rootfs>,format=raw,id=hd0 \
>         -device virtio-blk-device,drive=hd0 \
>         -append "root=/dev/vda rw console=ttyS0"
> @@ -95,7 +95,7 @@ make PLATFORM=generic PLATFORM_RISCV_XLEN=32
>  Run:
>  ```
>  qemu-system-riscv32 -M virt -m 256M -nographic \
> -       -kernel build/platform/generic/firmware/fw_payload.bin
> +       -bios build/platform/generic/firmware/fw_payload.bin
>  ```
>
>  **U-Boot Payload**
> @@ -111,7 +111,7 @@ make PLATFORM=generic PLATFORM_RISCV_XLEN=32 FW_PAYLOAD_PATH=<uboot_build_direct
>  Run:
>  ```
>  qemu-system-riscv32 -M virt -m 256M -nographic \
> -       -kernel build/platform/generic/firmware/fw_payload.elf
> +       -bios build/platform/generic/firmware/fw_payload.elf
>  ```
>  or
>  ```
> @@ -133,7 +133,7 @@ make PLATFORM=generic PLATFORM_RISCV_XLEN=32 FW_PAYLOAD_PATH=<linux_build_direct
>  Run:
>  ```
>  qemu-system-riscv32 -M virt -m 256M -nographic \
> -       -kernel build/platform/generic/firmware/fw_payload.elf \
> +       -bios build/platform/generic/firmware/fw_payload.elf \
>         -drive file=<path_to_linux_rootfs>,format=raw,id=hd0 \
>         -device virtio-blk-device,drive=hd0 \
>         -append "root=/dev/vda rw console=ttyS0"
> diff --git a/docs/platform/spike.md b/docs/platform/spike.md
> index 8d6b422..79e2eb2 100644
> --- a/docs/platform/spike.md
> +++ b/docs/platform/spike.md
> @@ -59,7 +59,7 @@ make PLATFORM=generic
>  Run:
>  ```
>  qemu-system-riscv64 -M spike -m 256M -nographic \
> -       -kernel build/platform/generic/firmware/fw_payload.elf
> +       -bios build/platform/generic/firmware/fw_payload.elf
>  ```
>
>  **Linux Kernel Payload**
> @@ -75,7 +75,7 @@ make PLATFORM=generic FW_PAYLOAD_PATH=<linux_build_directory>/arch/riscv/boot/Im
>  Run:
>  ```
>  qemu-system-riscv64 -M spike -m 256M -nographic \
> -       -kernel build/platform/generic/firmware/fw_payload.elf \
> +       -bios build/platform/generic/firmware/fw_payload.elf \
>         -initrd <path_to_cpio_ramdisk> \
>         -append "root=/dev/ram rw console=hvc0 earlycon=sbi"
>  ```
> --
> 2.7.4
>
>
> --
> opensbi mailing list
> opensbi@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/opensbi


Reviewed-by: Atish Patra <atish.patra@wdc.com>

diff --git a/docs/platform/qemu_virt.md b/docs/platform/qemu_virt.md
index 2196e5a..2777148 100644
--- a/docs/platform/qemu_virt.md
+++ b/docs/platform/qemu_virt.md
@@ -28,7 +28,7 @@  make PLATFORM=generic
 Run:
 ```
 qemu-system-riscv64 -M virt -m 256M -nographic \
-	-kernel build/platform/generic/firmware/fw_payload.bin
+	-bios build/platform/generic/firmware/fw_payload.bin
 ```
 
 **U-Boot Payload**
@@ -44,7 +44,7 @@  make PLATFORM=generic FW_PAYLOAD_PATH=<uboot_build_directory>/u-boot.bin
 Run:
 ```
 qemu-system-riscv64 -M virt -m 256M -nographic \
-	-kernel build/platform/generic/firmware/fw_payload.elf
+	-bios build/platform/generic/firmware/fw_payload.elf
 ```
 or
 ```
@@ -66,7 +66,7 @@  make PLATFORM=generic FW_PAYLOAD_PATH=<linux_build_directory>/arch/riscv/boot/Im
 Run:
 ```
 qemu-system-riscv64 -M virt -m 256M -nographic \
-	-kernel build/platform/generic/firmware/fw_payload.elf \
+	-bios build/platform/generic/firmware/fw_payload.elf \
 	-drive file=<path_to_linux_rootfs>,format=raw,id=hd0 \
 	-device virtio-blk-device,drive=hd0 \
 	-append "root=/dev/vda rw console=ttyS0"
@@ -95,7 +95,7 @@  make PLATFORM=generic PLATFORM_RISCV_XLEN=32
 Run:
 ```
 qemu-system-riscv32 -M virt -m 256M -nographic \
-	-kernel build/platform/generic/firmware/fw_payload.bin
+	-bios build/platform/generic/firmware/fw_payload.bin
 ```
 
 **U-Boot Payload**
@@ -111,7 +111,7 @@  make PLATFORM=generic PLATFORM_RISCV_XLEN=32 FW_PAYLOAD_PATH=<uboot_build_direct
 Run:
 ```
 qemu-system-riscv32 -M virt -m 256M -nographic \
-	-kernel build/platform/generic/firmware/fw_payload.elf
+	-bios build/platform/generic/firmware/fw_payload.elf
 ```
 or
 ```
@@ -133,7 +133,7 @@  make PLATFORM=generic PLATFORM_RISCV_XLEN=32 FW_PAYLOAD_PATH=<linux_build_direct
 Run:
 ```
 qemu-system-riscv32 -M virt -m 256M -nographic \
-	-kernel build/platform/generic/firmware/fw_payload.elf \
+	-bios build/platform/generic/firmware/fw_payload.elf \
 	-drive file=<path_to_linux_rootfs>,format=raw,id=hd0 \
 	-device virtio-blk-device,drive=hd0 \
 	-append "root=/dev/vda rw console=ttyS0"
diff --git a/docs/platform/spike.md b/docs/platform/spike.md
index 8d6b422..79e2eb2 100644
--- a/docs/platform/spike.md
+++ b/docs/platform/spike.md
@@ -59,7 +59,7 @@  make PLATFORM=generic
 Run:
 ```
 qemu-system-riscv64 -M spike -m 256M -nographic \
-	-kernel build/platform/generic/firmware/fw_payload.elf
+	-bios build/platform/generic/firmware/fw_payload.elf
 ```
 
 **Linux Kernel Payload**
@@ -75,7 +75,7 @@  make PLATFORM=generic FW_PAYLOAD_PATH=<linux_build_directory>/arch/riscv/boot/Im
 Run:
 ```
 qemu-system-riscv64 -M spike -m 256M -nographic \
-	-kernel build/platform/generic/firmware/fw_payload.elf \
+	-bios build/platform/generic/firmware/fw_payload.elf \
 	-initrd <path_to_cpio_ramdisk> \
 	-append "root=/dev/ram rw console=hvc0 earlycon=sbi"
 ```

[1/3] docs/platform: Update QEMU parameter for fw_payload

Commit Message

Comments

Patch