diff mbox series

[RFC] docs/system: clean up code escape for riscv virt platform

Message ID 20220905163939.1599368-1-alex.bennee@linaro.org
State Accepted
Commit 0c2d4671916333e5b66fd923279fb6fb62315bed
Headers show
Series [RFC] docs/system: clean up code escape for riscv virt platform | expand

Commit Message

Alex Bennée Sept. 5, 2022, 4:39 p.m. UTC
The example code is rendered slightly mangled due to missing code
block. Properly escape the code block and add shell prompt and qemu to
fit in with the other examples on the page.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
 docs/system/riscv/virt.rst | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

Comments

Alistair Francis Sept. 6, 2022, 11:37 a.m. UTC | #1
On Mon, Sep 5, 2022 at 6:39 PM Alex Bennée <alex.bennee@linaro.org> wrote:
>
> The example code is rendered slightly mangled due to missing code
> block. Properly escape the code block and add shell prompt and qemu to
> fit in with the other examples on the page.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>

Reviewed-by: Alistair Francis <alistair.francis@wdc.com>

Alistair

> ---
>  docs/system/riscv/virt.rst | 13 +++++++++----
>  1 file changed, 9 insertions(+), 4 deletions(-)
>
> diff --git a/docs/system/riscv/virt.rst b/docs/system/riscv/virt.rst
> index f8ecec95f3..4b16e41d7f 100644
> --- a/docs/system/riscv/virt.rst
> +++ b/docs/system/riscv/virt.rst
> @@ -168,14 +168,19 @@ Enabling TPM
>
>  A TPM device can be connected to the virt board by following the steps below.
>
> -First launch the TPM emulator
> +First launch the TPM emulator:
>
> -    swtpm socket --tpm2 -t -d --tpmstate dir=/tmp/tpm \
> +.. code-block:: bash
> +
> +  $ swtpm socket --tpm2 -t -d --tpmstate dir=/tmp/tpm \
>          --ctrl type=unixio,path=swtpm-sock
>
> -Then launch QEMU with:
> +Then launch QEMU with some additional arguments to link a TPM device to the backend:
> +
> +.. code-block:: bash
>
> -    ...
> +  $ qemu-system-riscv64 \
> +    ... other args .... \
>      -chardev socket,id=chrtpm,path=swtpm-sock \
>      -tpmdev emulator,id=tpm0,chardev=chrtpm \
>      -device tpm-tis-device,tpmdev=tpm0
> --
> 2.34.1
>
>
Alex Bennée Sept. 6, 2022, 12:26 p.m. UTC | #2
Alistair Francis <alistair23@gmail.com> writes:

> On Mon, Sep 5, 2022 at 6:39 PM Alex Bennée <alex.bennee@linaro.org> wrote:
>>
>> The example code is rendered slightly mangled due to missing code
>> block. Properly escape the code block and add shell prompt and qemu to
>> fit in with the other examples on the page.
>>
>> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
>
> Reviewed-by: Alistair Francis <alistair.francis@wdc.com>

Are you going to queue via your tree?
Alistair Francis Sept. 8, 2022, 9:19 a.m. UTC | #3
On Tue, Sep 6, 2022 at 2:26 PM Alex Bennée <alex.bennee@linaro.org> wrote:
>
>
> Alistair Francis <alistair23@gmail.com> writes:
>
> > On Mon, Sep 5, 2022 at 6:39 PM Alex Bennée <alex.bennee@linaro.org> wrote:
> >>
> >> The example code is rendered slightly mangled due to missing code
> >> block. Properly escape the code block and add shell prompt and qemu to
> >> fit in with the other examples on the page.
> >>
> >> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> >
> > Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
>
> Are you going to queue via your tree?

Yep! It's applied now, was just catching up on the first 7.2 PR

Alistair

>
> --
> Alex Bennée
diff mbox series

Patch

diff --git a/docs/system/riscv/virt.rst b/docs/system/riscv/virt.rst
index f8ecec95f3..4b16e41d7f 100644
--- a/docs/system/riscv/virt.rst
+++ b/docs/system/riscv/virt.rst
@@ -168,14 +168,19 @@  Enabling TPM
 
 A TPM device can be connected to the virt board by following the steps below.
 
-First launch the TPM emulator
+First launch the TPM emulator:
 
-    swtpm socket --tpm2 -t -d --tpmstate dir=/tmp/tpm \
+.. code-block:: bash
+
+  $ swtpm socket --tpm2 -t -d --tpmstate dir=/tmp/tpm \
         --ctrl type=unixio,path=swtpm-sock
 
-Then launch QEMU with:
+Then launch QEMU with some additional arguments to link a TPM device to the backend:
+
+.. code-block:: bash
 
-    ...
+  $ qemu-system-riscv64 \
+    ... other args .... \
     -chardev socket,id=chrtpm,path=swtpm-sock \
     -tpmdev emulator,id=tpm0,chardev=chrtpm \
     -device tpm-tis-device,tpmdev=tpm0