All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gerd Hoffmann <kraxel@redhat.com>
To: qemu-devel@nongnu.org
Cc: "Aleksandar Rikalo" <aleksandar.rikalo@syrmia.com>,
	"Daniel P. Berrangé" <berrange@redhat.com>,
	"Eduardo Habkost" <ehabkost@redhat.com>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	libvir-list@redhat.com,
	"Aleksandar Markovic" <aleksandar.qemu.devel@gmail.com>,
	"Hervé Poussineau" <hpoussin@reactos.org>,
	qemu-ppc@nongnu.org, "Gerd Hoffmann" <kraxel@redhat.com>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Aurelien Jarno" <aurelien@aurel32.net>,
	"Richard Henderson" <rth@twiddle.net>
Subject: [PULL 03/20] audio: add deprecated_register_soundhw
Date: Mon,  6 Jul 2020 21:29:04 +0200	[thread overview]
Message-ID: <20200706192921.1154-4-kraxel@redhat.com> (raw)
In-Reply-To: <20200706192921.1154-1-kraxel@redhat.com>

Add helper function for -soundhw deprecation.  It can replace the
simple init functions which just call {isa,pci}_create_simple()
with a hardcoded type.  It also prints a deprecation message.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Message-id: 20200702132525.6849-4-kraxel@redhat.com
---
 include/hw/audio/soundhw.h |  2 ++
 hw/audio/soundhw.c         | 24 +++++++++++++++++++++++-
 2 files changed, 25 insertions(+), 1 deletion(-)

diff --git a/include/hw/audio/soundhw.h b/include/hw/audio/soundhw.h
index c8eef8241846..f09a297854af 100644
--- a/include/hw/audio/soundhw.h
+++ b/include/hw/audio/soundhw.h
@@ -6,6 +6,8 @@ void isa_register_soundhw(const char *name, const char *descr,
 
 void pci_register_soundhw(const char *name, const char *descr,
                           int (*init_pci)(PCIBus *bus));
+void deprecated_register_soundhw(const char *name, const char *descr,
+                                 int isa, const char *typename);
 
 void soundhw_init(void);
 void select_soundhw(const char *optarg);
diff --git a/hw/audio/soundhw.c b/hw/audio/soundhw.c
index c750473c8f0c..173b674ff53a 100644
--- a/hw/audio/soundhw.c
+++ b/hw/audio/soundhw.c
@@ -22,6 +22,7 @@
  * THE SOFTWARE.
  */
 #include "qemu/osdep.h"
+#include "qemu/option.h"
 #include "qemu/help_option.h"
 #include "qemu/error-report.h"
 #include "qom/object.h"
@@ -32,6 +33,7 @@
 struct soundhw {
     const char *name;
     const char *descr;
+    const char *typename;
     int enabled;
     int isa;
     union {
@@ -65,6 +67,17 @@ void pci_register_soundhw(const char *name, const char *descr,
     soundhw_count++;
 }
 
+void deprecated_register_soundhw(const char *name, const char *descr,
+                                 int isa, const char *typename)
+{
+    assert(soundhw_count < ARRAY_SIZE(soundhw) - 1);
+    soundhw[soundhw_count].name = name;
+    soundhw[soundhw_count].descr = descr;
+    soundhw[soundhw_count].isa = isa;
+    soundhw[soundhw_count].typename = typename;
+    soundhw_count++;
+}
+
 void select_soundhw(const char *optarg)
 {
     struct soundhw *c;
@@ -136,7 +149,16 @@ void soundhw_init(void)
 
     for (c = soundhw; c->name; ++c) {
         if (c->enabled) {
-            if (c->isa) {
+            if (c->typename) {
+                warn_report("'-soundhw %s' is deprecated, "
+                            "please use '-device %s' instead",
+                            c->name, c->typename);
+                if (c->isa) {
+                    isa_create_simple(isa_bus, c->typename);
+                } else {
+                    pci_create_simple(pci_bus, -1, c->typename);
+                }
+            } else if (c->isa) {
                 if (!isa_bus) {
                     error_report("ISA bus not available for %s", c->name);
                     exit(1);
-- 
2.18.4



  parent reply	other threads:[~2020-07-06 19:33 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-06 19:29 [PULL 00/20] Audio 20200706 patches Gerd Hoffmann
2020-07-06 19:29 ` [PULL 01/20] stubs: add isa_create_simple Gerd Hoffmann
2020-07-06 19:29 ` [PULL 02/20] stubs: add pci_create_simple Gerd Hoffmann
2020-07-06 19:29 ` Gerd Hoffmann [this message]
2020-07-06 19:29 ` [PULL 04/20] audio: deprecate -soundhw ac97 Gerd Hoffmann
2020-07-06 19:29 ` [PULL 05/20] audio: deprecate -soundhw es1370 Gerd Hoffmann
2020-07-06 19:29 ` [PULL 06/20] audio: deprecate -soundhw adlib Gerd Hoffmann
2020-07-06 19:29 ` [PULL 07/20] audio: deprecate -soundhw cs4231a Gerd Hoffmann
2020-07-06 19:29 ` [PULL 08/20] audio: deprecate -soundhw gus Gerd Hoffmann
2020-07-06 19:29 ` [PULL 09/20] audio: deprecate -soundhw sb16 Gerd Hoffmann
2020-07-06 19:29 ` [PULL 10/20] audio: deprecate -soundhw hda Gerd Hoffmann
2020-07-06 19:29 ` [PULL 11/20] pc_basic_device_init: pass PCMachineState Gerd Hoffmann
2020-07-06 19:29 ` [PULL 12/20] pc_basic_device_init: drop has_pit arg Gerd Hoffmann
2020-07-06 19:29 ` [PULL 13/20] pc_basic_device_init: drop no_vmport arg Gerd Hoffmann
2020-07-06 19:29 ` [PULL 14/20] softmmu: initialize spice and audio earlier Gerd Hoffmann
2020-07-06 19:29 ` [PULL 15/20] audio: rework pcspk_init() Gerd Hoffmann
2020-07-06 19:29 ` [PULL 16/20] audio: create pcspk device early Gerd Hoffmann
2020-07-06 19:29 ` [PULL 17/20] audio: deprecate -soundhw pcspk Gerd Hoffmann
2020-07-06 19:29 ` [PULL 18/20] audio: add soundhw deprecation notice Gerd Hoffmann
2020-07-06 19:29 ` [PULL 19/20] pcspk: update docs/system/target-i386-desc.rst.inc Gerd Hoffmann
2020-07-06 19:29 ` [PULL 20/20] audio: set default value for pcspk.iobase property Gerd Hoffmann
2020-07-08 20:38 ` [PULL 00/20] Audio 20200706 patches Peter Maydell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200706192921.1154-4-kraxel@redhat.com \
    --to=kraxel@redhat.com \
    --cc=aleksandar.qemu.devel@gmail.com \
    --cc=aleksandar.rikalo@syrmia.com \
    --cc=aurelien@aurel32.net \
    --cc=berrange@redhat.com \
    --cc=ehabkost@redhat.com \
    --cc=hpoussin@reactos.org \
    --cc=libvir-list@redhat.com \
    --cc=mst@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@nongnu.org \
    --cc=rth@twiddle.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.