qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] qemu-options.hx: Fix uncorrect description of "-serial"
@ 2024-03-05  1:30 steven.shen
  2024-03-05 13:21 ` Peter Maydell
  0 siblings, 1 reply; 2+ messages in thread
From: steven.shen @ 2024-03-05  1:30 UTC (permalink / raw)
  To: qemu-devel; +Cc: peter.maydell, steven.shen

Before v2.12, serial_hds used MAX_SERIAL_PORTS(4) for
resources of serials.The limitaion description of "-serial"
option: "This option can be used several times to simulate
up to 4 serial ports."
In latest qemu, serial_hds have been replaced by "Chardev **"
and now is dynamically allocated through "g_renew".
So the limitation description is not suitable now.
Update to "This option can be used several times to simulate
multiple serial ports." to avoid misleading.

Signed-off-by: Steven Shen <steven.shen@jaguarmicro.com>
---
 qemu-options.hx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/qemu-options.hx b/qemu-options.hx
index 9a47385c15..ac4a30fa83 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -4129,7 +4129,7 @@ SRST
     default device is ``vc`` in graphical mode and ``stdio`` in non
     graphical mode.
 
-    This option can be used several times to simulate up to 4 serial
+    This option can be used several times to simulate multiple serial
     ports.
 
     You can use ``-serial none`` to suppress the creation of default
-- 
2.36.0.windows.1



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

* Re: [PATCH] qemu-options.hx: Fix uncorrect description of "-serial"
  2024-03-05  1:30 [PATCH] qemu-options.hx: Fix uncorrect description of "-serial" steven.shen
@ 2024-03-05 13:21 ` Peter Maydell
  0 siblings, 0 replies; 2+ messages in thread
From: Peter Maydell @ 2024-03-05 13:21 UTC (permalink / raw)
  To: steven.shen@jaguarmicro.com; +Cc: qemu-devel

On Tue, 5 Mar 2024 at 01:31, steven.shen@jaguarmicro.com
<steven.shen@jaguarmicro.com> wrote:
>
> Before v2.12, serial_hds used MAX_SERIAL_PORTS(4) for
> resources of serials.The limitaion description of "-serial"
> option: "This option can be used several times to simulate
> up to 4 serial ports."
> In latest qemu, serial_hds have been replaced by "Chardev **"
> and now is dynamically allocated through "g_renew".
> So the limitation description is not suitable now.
> Update to "This option can be used several times to simulate
> multiple serial ports." to avoid misleading.
>
> Signed-off-by: Steven Shen <steven.shen@jaguarmicro.com>

Thanks for this documentation fix! I've applied it to target-arm.next
(with some tweaks to the commit message), so it should appear in
upstream git within a week or so.

-- PMM


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

end of thread, other threads:[~2024-03-05 13:22 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-03-05  1:30 [PATCH] qemu-options.hx: Fix uncorrect description of "-serial" steven.shen
2024-03-05 13:21 ` Peter Maydell

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).