* [PATCH] docs/devel/qom: Fix some editing errors
@ 2026-05-20 11:15 J. Neuschäfer
2026-05-20 12:20 ` Markus Armbruster
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: J. Neuschäfer @ 2026-05-20 11:15 UTC (permalink / raw)
To: qemu-devel
Cc: Paolo Bonzini, Daniel P. Berrangé, Pierrick Bouvier,
J. Neuschäfer
"devices have an two additional [...]" -> "devices have two additional",
and also a case of "were" vs. "where".
Signed-off-by: J. Neuschäfer <j.neuschaefer@9elements.com>
---
docs/devel/qom.rst | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/docs/devel/qom.rst b/docs/devel/qom.rst
index 5870745ba27b108726da0756aa2b90109d5e0bd6..935bb1004a7839c8e963985e765c866d5b980c53 100644
--- a/docs/devel/qom.rst
+++ b/docs/devel/qom.rst
@@ -435,14 +435,14 @@ OBJECT_DEFINE_ABSTRACT_TYPE() macro can be used instead:
Device Life-cycle
=================
-As class initialisation cannot fail devices have an two additional
+As class initialisation cannot fail, devices have two additional
methods to handle the creation of dynamic devices. The ``realize``
function is called with ``Error **`` pointer which should be set if
the device cannot complete its setup. Otherwise on successful
completion of the ``realize`` method the device object is added to the
QOM tree and made visible to the guest.
-The reverse function is ``unrealize`` and should be were clean-up
+The reverse function is ``unrealize`` and should be where clean-up
code lives to tidy up after the system is done with the device.
All devices can be instantiated by C code, however only some can
---
base-commit: e89049b3ba5f1f0468bc0d294173345597514a1b
change-id: 20260520-qom-76537da673b6
Best regards,
--
J. Neuschäfer <j.neuschaefer@9elements.com>
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] docs/devel/qom: Fix some editing errors
2026-05-20 11:15 [PATCH] docs/devel/qom: Fix some editing errors J. Neuschäfer
@ 2026-05-20 12:20 ` Markus Armbruster
2026-05-22 18:06 ` Pierrick Bouvier
2026-05-25 15:23 ` Pierrick Bouvier
2 siblings, 0 replies; 4+ messages in thread
From: Markus Armbruster @ 2026-05-20 12:20 UTC (permalink / raw)
To: J. Neuschäfer
Cc: qemu-devel, Paolo Bonzini, Daniel P. Berrangé,
Pierrick Bouvier, qemu-trivial
J. Neuschäfer <j.neuschaefer@9elements.com> writes:
> "devices have an two additional [...]" -> "devices have two additional",
> and also a case of "were" vs. "where".
>
> Signed-off-by: J. Neuschäfer <j.neuschaefer@9elements.com>
> ---
> docs/devel/qom.rst | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/docs/devel/qom.rst b/docs/devel/qom.rst
> index 5870745ba27b108726da0756aa2b90109d5e0bd6..935bb1004a7839c8e963985e765c866d5b980c53 100644
> --- a/docs/devel/qom.rst
> +++ b/docs/devel/qom.rst
> @@ -435,14 +435,14 @@ OBJECT_DEFINE_ABSTRACT_TYPE() macro can be used instead:
> Device Life-cycle
> =================
>
> -As class initialisation cannot fail devices have an two additional
> +As class initialisation cannot fail, devices have two additional
> methods to handle the creation of dynamic devices. The ``realize``
> function is called with ``Error **`` pointer which should be set if
> the device cannot complete its setup. Otherwise on successful
> completion of the ``realize`` method the device object is added to the
> QOM tree and made visible to the guest.
>
> -The reverse function is ``unrealize`` and should be were clean-up
> +The reverse function is ``unrealize`` and should be where clean-up
> code lives to tidy up after the system is done with the device.
>
> All devices can be instantiated by C code, however only some can
>
> ---
> base-commit: e89049b3ba5f1f0468bc0d294173345597514a1b
> change-id: 20260520-qom-76537da673b6
>
> Best regards,
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Perhaps this could go via qemu-trivial (cc'ed).
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] docs/devel/qom: Fix some editing errors
2026-05-20 11:15 [PATCH] docs/devel/qom: Fix some editing errors J. Neuschäfer
2026-05-20 12:20 ` Markus Armbruster
@ 2026-05-22 18:06 ` Pierrick Bouvier
2026-05-25 15:23 ` Pierrick Bouvier
2 siblings, 0 replies; 4+ messages in thread
From: Pierrick Bouvier @ 2026-05-22 18:06 UTC (permalink / raw)
To: J. Neuschäfer, qemu-devel; +Cc: Paolo Bonzini, Daniel P. Berrangé
On 5/20/2026 4:15 AM, J. Neuschäfer wrote:
> "devices have an two additional [...]" -> "devices have two additional",
> and also a case of "were" vs. "where".
>
> Signed-off-by: J. Neuschäfer <j.neuschaefer@9elements.com>
> ---
> docs/devel/qom.rst | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/docs/devel/qom.rst b/docs/devel/qom.rst
> index 5870745ba27b108726da0756aa2b90109d5e0bd6..935bb1004a7839c8e963985e765c866d5b980c53 100644
> --- a/docs/devel/qom.rst
> +++ b/docs/devel/qom.rst
> @@ -435,14 +435,14 @@ OBJECT_DEFINE_ABSTRACT_TYPE() macro can be used instead:
> Device Life-cycle
> =================
>
> -As class initialisation cannot fail devices have an two additional
> +As class initialisation cannot fail, devices have two additional
> methods to handle the creation of dynamic devices. The ``realize``
> function is called with ``Error **`` pointer which should be set if
> the device cannot complete its setup. Otherwise on successful
> completion of the ``realize`` method the device object is added to the
> QOM tree and made visible to the guest.
>
> -The reverse function is ``unrealize`` and should be were clean-up
> +The reverse function is ``unrealize`` and should be where clean-up
> code lives to tidy up after the system is done with the device.
>
> All devices can be instantiated by C code, however only some can
>
> ---
> base-commit: e89049b3ba5f1f0468bc0d294173345597514a1b
> change-id: 20260520-qom-76537da673b6
>
> Best regards,
Thanks,
I'll pull this through docs subsystem.
Reviewed-by: Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com>
Regards,
Pierrick
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] docs/devel/qom: Fix some editing errors
2026-05-20 11:15 [PATCH] docs/devel/qom: Fix some editing errors J. Neuschäfer
2026-05-20 12:20 ` Markus Armbruster
2026-05-22 18:06 ` Pierrick Bouvier
@ 2026-05-25 15:23 ` Pierrick Bouvier
2 siblings, 0 replies; 4+ messages in thread
From: Pierrick Bouvier @ 2026-05-25 15:23 UTC (permalink / raw)
To: J. Neuschäfer, qemu-devel; +Cc: Paolo Bonzini, Daniel P. Berrangé
On 5/20/2026 4:15 AM, J. Neuschäfer wrote:
> "devices have an two additional [...]" -> "devices have two additional",
> and also a case of "were" vs. "where".
>
> Signed-off-by: J. Neuschäfer <j.neuschaefer@9elements.com>
> ---
> docs/devel/qom.rst | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/docs/devel/qom.rst b/docs/devel/qom.rst
> index 5870745ba27b108726da0756aa2b90109d5e0bd6..935bb1004a7839c8e963985e765c866d5b980c53 100644
> --- a/docs/devel/qom.rst
> +++ b/docs/devel/qom.rst
> @@ -435,14 +435,14 @@ OBJECT_DEFINE_ABSTRACT_TYPE() macro can be used instead:
> Device Life-cycle
> =================
>
> -As class initialisation cannot fail devices have an two additional
> +As class initialisation cannot fail, devices have two additional
> methods to handle the creation of dynamic devices. The ``realize``
> function is called with ``Error **`` pointer which should be set if
> the device cannot complete its setup. Otherwise on successful
> completion of the ``realize`` method the device object is added to the
> QOM tree and made visible to the guest.
>
> -The reverse function is ``unrealize`` and should be were clean-up
> +The reverse function is ``unrealize`` and should be where clean-up
> code lives to tidy up after the system is done with the device.
>
> All devices can be instantiated by C code, however only some can
>
> ---
> base-commit: e89049b3ba5f1f0468bc0d294173345597514a1b
> change-id: 20260520-qom-76537da673b6
>
> Best regards,
This was merged into master (cbf877d67a812be17a9ce404a589e1bdf722c1f6).
Thank you for your contribution!
Regards,
Pierrick
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2026-05-25 15:24 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-05-20 11:15 [PATCH] docs/devel/qom: Fix some editing errors J. Neuschäfer
2026-05-20 12:20 ` Markus Armbruster
2026-05-22 18:06 ` Pierrick Bouvier
2026-05-25 15:23 ` Pierrick Bouvier
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.