hw/audio: Cleanups around QOM style

[PATCH 0/5] hw/audio: Cleanups around QOM style

Philippe Mathieu-Daudé posted 5 patches 1 year, 2 months ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20230220131837.26292-1-philmd@linaro.org

Maintainers: Gerd Hoffmann <kraxel@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Richard Henderson <richard.henderson@linaro.org>, Eduardo Habkost <eduardo@habkost.net>, "Michael S. Tsirkin" <mst@redhat.com>, Marcel Apfelbaum <marcel.apfelbaum@gmail.com>, "Hervé Poussineau" <hpoussin@reactos.org>, Aleksandar Rikalo <aleksandar.rikalo@syrmia.com>, "Philippe Mathieu-Daudé" <philmd@linaro.org>

hw/audio/ac97.c          |  2 +-
hw/audio/es1370.c        | 10 +++++-----
hw/audio/hda-codec.c     |  7 +++----
hw/i386/pc.c             |  3 ++-
hw/isa/i82378.c          |  5 ++++-
hw/mips/jazz.c           |  5 ++++-
include/hw/audio/pcspk.h | 10 ----------
7 files changed, 19 insertions(+), 23 deletions(-)

Expand all Fold all

[PATCH 0/5] hw/audio: Cleanups around QOM style

Posted by Philippe Mathieu-Daudé 1 year, 2 months ago

- Use QOM's OBJECT_DECLARE*TYPE() to declare typedef / macros
- Use QOM cast macros instead of container_of()
- Open-code QDev helpers used few times

Philippe Mathieu-Daudé (5):
  hw/audio/hda-codec: Avoid forward-declaring HDAAudioState
  hw/audio/es1370: Avoid forward-declaring ES1370State
  hw/audio/es1370: Replace container_of() by ES1370() QOM cast macro
  hw/audio/ac97: Replace container_of() by AC97() QOM cast macro
  hw/audio/pcspk: Open-code pcspk_init()

 hw/audio/ac97.c          |  2 +-
 hw/audio/es1370.c        | 10 +++++-----
 hw/audio/hda-codec.c     |  7 +++----
 hw/i386/pc.c             |  3 ++-
 hw/isa/i82378.c          |  5 ++++-
 hw/mips/jazz.c           |  5 ++++-
 include/hw/audio/pcspk.h | 10 ----------
 7 files changed, 19 insertions(+), 23 deletions(-)

-- 
2.38.1