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>,
	qemu-ppc@nongnu.org, "Gerd Hoffmann" <kraxel@redhat.com>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Hervé Poussineau" <hpoussin@reactos.org>,
	"Aurelien Jarno" <aurelien@aurel32.net>,
	"Richard Henderson" <rth@twiddle.net>
Subject: [PATCH v5 20/20] audio: set default value for pcspk.iobase property
Date: Thu,  2 Jul 2020 15:25:25 +0200	[thread overview]
Message-ID: <20200702132525.6849-21-kraxel@redhat.com> (raw)
In-Reply-To: <20200702132525.6849-1-kraxel@redhat.com>

Allows dropping the explicit qdev_prop_set_uint32 call in pcspk_init.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
 include/hw/audio/pcspk.h | 6 +-----
 hw/audio/pcspk.c         | 2 +-
 2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/include/hw/audio/pcspk.h b/include/hw/audio/pcspk.h
index 8b485602675f..06cba00b8376 100644
--- a/include/hw/audio/pcspk.h
+++ b/include/hw/audio/pcspk.h
@@ -33,11 +33,7 @@
 
 static inline void pcspk_init(ISADevice *isadev, ISABus *bus, ISADevice *pit)
 {
-    DeviceState *dev;
-
-    dev = DEVICE(isadev);
-    qdev_prop_set_uint32(dev, "iobase", 0x61);
-    object_property_set_link(OBJECT(dev), OBJECT(pit), "pit", NULL);
+    object_property_set_link(OBJECT(isadev), OBJECT(pit), "pit", NULL);
     isa_realize_and_unref(isadev, bus, &error_fatal);
 }
 
diff --git a/hw/audio/pcspk.c b/hw/audio/pcspk.c
index 4c7e339ac2b5..ea539e7605a8 100644
--- a/hw/audio/pcspk.c
+++ b/hw/audio/pcspk.c
@@ -219,7 +219,7 @@ static const VMStateDescription vmstate_spk = {
 
 static Property pcspk_properties[] = {
     DEFINE_AUDIO_PROPERTIES(PCSpkState, card),
-    DEFINE_PROP_UINT32("iobase", PCSpkState, iobase,  -1),
+    DEFINE_PROP_UINT32("iobase", PCSpkState, iobase,  0x61),
     DEFINE_PROP_BOOL("migrate", PCSpkState, migrate,  true),
     DEFINE_PROP_END_OF_LIST(),
 };
-- 
2.18.4



      parent reply	other threads:[~2020-07-02 13:36 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-02 13:25 [PATCH v5 00/20] audio: deprecate -soundhw Gerd Hoffmann
2020-07-02 13:25 ` [PATCH v5 01/20] stubs: add isa_create_simple Gerd Hoffmann
2020-07-02 13:25 ` [PATCH v5 02/20] stubs: add pci_create_simple Gerd Hoffmann
2020-07-02 13:25 ` [PATCH v5 03/20] audio: add deprecated_register_soundhw Gerd Hoffmann
2020-07-02 13:25 ` [PATCH v5 04/20] audio: deprecate -soundhw ac97 Gerd Hoffmann
2020-07-02 13:25 ` [PATCH v5 05/20] audio: deprecate -soundhw es1370 Gerd Hoffmann
2020-07-02 13:25 ` [PATCH v5 06/20] audio: deprecate -soundhw adlib Gerd Hoffmann
2020-07-02 13:25 ` [PATCH v5 07/20] audio: deprecate -soundhw cs4231a Gerd Hoffmann
2020-07-02 13:25 ` [PATCH v5 08/20] audio: deprecate -soundhw gus Gerd Hoffmann
2020-07-02 13:25 ` [PATCH v5 09/20] audio: deprecate -soundhw sb16 Gerd Hoffmann
2020-07-02 13:25 ` [PATCH v5 10/20] audio: deprecate -soundhw hda Gerd Hoffmann
2020-07-02 13:25 ` [PATCH v5 11/20] pc_basic_device_init: pass PCMachineState Gerd Hoffmann
2020-07-02 13:44   ` Philippe Mathieu-Daudé
2020-07-02 13:25 ` [PATCH v5 12/20] pc_basic_device_init: drop has_pit arg Gerd Hoffmann
2020-07-02 13:25 ` [PATCH v5 13/20] pc_basic_device_init: drop no_vmport arg Gerd Hoffmann
2020-07-02 13:25 ` [PATCH v5 14/20] softmmu: initialize spice and audio earlier Gerd Hoffmann
2020-07-02 13:25 ` [PATCH v5 15/20] audio: rework pcspk_init() Gerd Hoffmann
2020-07-02 13:25 ` [PATCH v5 16/20] audio: create pcspk device early Gerd Hoffmann
2020-07-02 13:25 ` [PATCH v5 17/20] audio: deprecate -soundhw pcspk Gerd Hoffmann
2020-07-02 13:25 ` [PATCH v5 18/20] audio: add soundhw deprecation notice Gerd Hoffmann
2020-07-02 13:25 ` [PATCH v5 19/20] pcspk: update docs/system/target-i386-desc.rst.inc Gerd Hoffmann
2020-07-02 13:25 ` Gerd Hoffmann [this message]

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=20200702132525.6849-21-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.