public inbox for docs@lists.yoctoproject.org
 help / color / mirror / Atom feed
* [docs] [PATCH v2] qemu.rst: audio: reference to Command-Line options
@ 2022-11-10  5:53 Atanas Bunchev
  2022-11-10 10:17 ` Quentin Schulz
  0 siblings, 1 reply; 3+ messages in thread
From: Atanas Bunchev @ 2022-11-10  5:53 UTC (permalink / raw)
  To: docs; +Cc: Atanas Bunchev

The previous statement can cause confusion, because this is the
first time when the `audio` argument is mentioned.

Signed-off-by: Atanas Bunchev <atanas.bunchev@konsulko.com>
---
 documentation/dev-manual/qemu.rst | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/documentation/dev-manual/qemu.rst b/documentation/dev-manual/qemu.rst
index 5a4a82ce6..d9b9f69ab 100644
--- a/documentation/dev-manual/qemu.rst
+++ b/documentation/dev-manual/qemu.rst
@@ -125,8 +125,9 @@ available. Follow these general steps to run QEMU:
    -  This example specifies to boot an :term:`Initramfs` image and to
       enable audio in QEMU. For this case, ``runqemu`` sets the internal
       variable ``FSTYPE`` to ``cpio.gz``. Also, for audio to be enabled,
-      an appropriate driver must be installed (see the previous
-      description for the ``audio`` option for more information).
+      an appropriate driver must be installed (see the ``audio`` option
+      in :ref:`dev-manual/qemu:\`\`runqemu\`\` command-line options`
+      for more information).
       ::
 
          $ runqemu qemux86-64 ramfs audio
-- 
2.37.4



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [docs] [PATCH v2] qemu.rst: audio: reference to Command-Line options
  2022-11-10  5:53 [docs] [PATCH v2] qemu.rst: audio: reference to Command-Line options Atanas Bunchev
@ 2022-11-10 10:17 ` Quentin Schulz
  2022-11-10 13:07   ` Michael Opdenacker
  0 siblings, 1 reply; 3+ messages in thread
From: Quentin Schulz @ 2022-11-10 10:17 UTC (permalink / raw)
  To: Atanas Bunchev, docs

Hi Atanas,

On 11/10/22 06:53, Atanas Bunchev wrote:
> The previous statement can cause confusion, because this is the
> first time when the `audio` argument is mentioned.
> 
> Signed-off-by: Atanas Bunchev <atanas.bunchev@konsulko.com>

Reviewed-by: Quentin Schulz <foss+yocto@0leil.net>

Thanks,
Quentin


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [docs] [PATCH v2] qemu.rst: audio: reference to Command-Line options
  2022-11-10 10:17 ` Quentin Schulz
@ 2022-11-10 13:07   ` Michael Opdenacker
  0 siblings, 0 replies; 3+ messages in thread
From: Michael Opdenacker @ 2022-11-10 13:07 UTC (permalink / raw)
  To: quentin.schulz, Atanas Bunchev; +Cc: docs

Hi Atanas

On 11/10/22 11:17, Quentin Schulz via lists.yoctoproject.org wrote:
> Hi Atanas,
>
> On 11/10/22 06:53, Atanas Bunchev wrote:
>> The previous statement can cause confusion, because this is the
>> first time when the `audio` argument is mentioned.
>>
>> Signed-off-by: Atanas Bunchev <atanas.bunchev@konsulko.com>
>
> Reviewed-by: Quentin Schulz <foss+yocto@0leil.net>


Reviewed-by: Michael Opdenacker <michael.opdenacker@bootlin.com>
and merged into "master-next".

Many thanks for the patch!
Cheers
Michael.

-- 
Michael Opdenacker, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-11-10 13:07 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-11-10  5:53 [docs] [PATCH v2] qemu.rst: audio: reference to Command-Line options Atanas Bunchev
2022-11-10 10:17 ` Quentin Schulz
2022-11-10 13:07   ` Michael Opdenacker

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox