* [Qemu-devel] [PATCH] misc: fix naming scheme of compatiblity arrays
@ 2019-07-08 20:49 Marc-André Lureau
2019-07-09 8:47 ` Stefano Garzarella
2019-08-21 8:29 ` [Qemu-devel] [Qemu-trivial] " Laurent Vivier
0 siblings, 2 replies; 3+ messages in thread
From: Marc-André Lureau @ 2019-07-08 20:49 UTC (permalink / raw)
To: qemu-devel
Cc: peter.maydell, Eduardo Habkost, Juan Quintela, qemu-trivial,
Michael S. Tsirkin, Dr. David Alan Gilbert,
Marc-André Lureau, Paolo Bonzini, Richard Henderson
Since merge 31ed41889e6e13699871040fe089a2884dca46cb ("Merge
remote-tracking branch
'remotes/elmarco/tags/machine-props-pull-request' into staging"), the
compat arrays are in lowercase.
Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
---
docs/devel/migration.rst | 2 +-
hw/i386/pc_piix.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/docs/devel/migration.rst b/docs/devel/migration.rst
index 220059679a..f7668ae389 100644
--- a/docs/devel/migration.rst
+++ b/docs/devel/migration.rst
@@ -314,7 +314,7 @@ For example:
a) Add a new property using ``DEFINE_PROP_BOOL`` - e.g. support-foo and
default it to true.
- b) Add an entry to the ``HW_COMPAT_`` for the previous version that sets
+ b) Add an entry to the ``hw_compat_`` for the previous version that sets
the property to false.
c) Add a static bool support_foo function that tests the property.
d) Add a subsection with a .needed set to the support_foo function
diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c
index c2280c72ef..6a0e29de7c 100644
--- a/hw/i386/pc_piix.c
+++ b/hw/i386/pc_piix.c
@@ -313,7 +313,7 @@ else {
* pc_compat_*() functions that run on machine-init time and
* change global QEMU state are deprecated. Please don't create
* one, and implement any pc-*-2.4 (and newer) compat code in
- * HW_COMPAT_*, PC_COMPAT_*, or * pc_*_machine_options().
+ * hw_compat_*, pc_compat_*, or * pc_*_machine_options().
*/
static void pc_compat_2_3_fn(MachineState *machine)
--
2.22.0.214.g8dca754b1e
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] [PATCH] misc: fix naming scheme of compatiblity arrays
2019-07-08 20:49 [Qemu-devel] [PATCH] misc: fix naming scheme of compatiblity arrays Marc-André Lureau
@ 2019-07-09 8:47 ` Stefano Garzarella
2019-08-21 8:29 ` [Qemu-devel] [Qemu-trivial] " Laurent Vivier
1 sibling, 0 replies; 3+ messages in thread
From: Stefano Garzarella @ 2019-07-09 8:47 UTC (permalink / raw)
To: Marc-André Lureau
Cc: peter.maydell, Eduardo Habkost, Juan Quintela, qemu-trivial,
Michael S. Tsirkin, qemu-devel, Dr. David Alan Gilbert,
Paolo Bonzini, Richard Henderson
On Tue, Jul 09, 2019 at 12:49:09AM +0400, Marc-André Lureau wrote:
> Since merge 31ed41889e6e13699871040fe089a2884dca46cb ("Merge
> remote-tracking branch
> 'remotes/elmarco/tags/machine-props-pull-request' into staging"), the
> compat arrays are in lowercase.
>
> Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
> ---
> docs/devel/migration.rst | 2 +-
> hw/i386/pc_piix.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
Reviewed-by: Stefano Garzarella <sgarzare@redhat.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] [Qemu-trivial] [PATCH] misc: fix naming scheme of compatiblity arrays
2019-07-08 20:49 [Qemu-devel] [PATCH] misc: fix naming scheme of compatiblity arrays Marc-André Lureau
2019-07-09 8:47 ` Stefano Garzarella
@ 2019-08-21 8:29 ` Laurent Vivier
1 sibling, 0 replies; 3+ messages in thread
From: Laurent Vivier @ 2019-08-21 8:29 UTC (permalink / raw)
To: Marc-André Lureau, qemu-devel
Cc: peter.maydell, Eduardo Habkost, Michael S. Tsirkin, qemu-trivial,
Juan Quintela, Dr. David Alan Gilbert, Paolo Bonzini,
Richard Henderson
Le 08/07/2019 à 22:49, Marc-André Lureau a écrit :
> Since merge 31ed41889e6e13699871040fe089a2884dca46cb ("Merge
> remote-tracking branch
> 'remotes/elmarco/tags/machine-props-pull-request' into staging"), the
> compat arrays are in lowercase.
>
> Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
> ---
> docs/devel/migration.rst | 2 +-
> hw/i386/pc_piix.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/docs/devel/migration.rst b/docs/devel/migration.rst
> index 220059679a..f7668ae389 100644
> --- a/docs/devel/migration.rst
> +++ b/docs/devel/migration.rst
> @@ -314,7 +314,7 @@ For example:
>
> a) Add a new property using ``DEFINE_PROP_BOOL`` - e.g. support-foo and
> default it to true.
> - b) Add an entry to the ``HW_COMPAT_`` for the previous version that sets
> + b) Add an entry to the ``hw_compat_`` for the previous version that sets
> the property to false.
> c) Add a static bool support_foo function that tests the property.
> d) Add a subsection with a .needed set to the support_foo function
> diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c
> index c2280c72ef..6a0e29de7c 100644
> --- a/hw/i386/pc_piix.c
> +++ b/hw/i386/pc_piix.c
> @@ -313,7 +313,7 @@ else {
> * pc_compat_*() functions that run on machine-init time and
> * change global QEMU state are deprecated. Please don't create
> * one, and implement any pc-*-2.4 (and newer) compat code in
> - * HW_COMPAT_*, PC_COMPAT_*, or * pc_*_machine_options().
> + * hw_compat_*, pc_compat_*, or * pc_*_machine_options().
> */
>
> static void pc_compat_2_3_fn(MachineState *machine)
>
Applied to my trivial-patches branch.
Thanks,
Laurent
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-08-21 8:30 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-07-08 20:49 [Qemu-devel] [PATCH] misc: fix naming scheme of compatiblity arrays Marc-André Lureau
2019-07-09 8:47 ` Stefano Garzarella
2019-08-21 8:29 ` [Qemu-devel] [Qemu-trivial] " Laurent Vivier
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.