Push AUD_init down to devices
Now we can safely call AUD_init multiple times we can push it down to
individual audio devices, rather than having to pass it from the board
init.
Signed-off-by: Paul Brook <paul@codesourcery.com>
diff --git a/hw/ac97.c b/hw/ac97.c
index d9ef141..f2ae4b3 100644
--- a/hw/ac97.c
+++ b/hw/ac97.c
@@ -1308,8 +1308,9 @@
mixer_reset (s);
}
-int ac97_init (PCIBus *bus, AudioState *audio)
+int ac97_init (PCIBus *bus)
{
+ AudioState *audio = AUD_init();
PCIAC97LinkState *d;
AC97LinkState *s;
uint8_t *c;
@@ -1319,11 +1320,6 @@
return -1;
}
- if (!audio) {
- AUD_log ("ac97", "No audio state\n");
- return -1;
- }
-
d = (PCIAC97LinkState *) pci_register_device (bus, "AC97",
sizeof (PCIAC97LinkState),
-1, NULL, NULL);
diff --git a/hw/adlib.c b/hw/adlib.c
index 9f8f6f2..076c1a0 100644
--- a/hw/adlib.c
+++ b/hw/adlib.c
@@ -277,16 +277,12 @@
AUD_remove_card (&s->card);
}
-int Adlib_init (AudioState *audio, qemu_irq *pic)
+int Adlib_init (qemu_irq *pic)
{
+ AudioState *audio = AUD_init();
AdlibState *s = &glob_adlib;
struct audsettings as;
- if (!audio) {
- dolog ("No audio state\n");
- return -1;
- }
-
#ifdef HAS_YMF262
if (YMF262Init (1, 14318180, conf.freq)) {
dolog ("YMF262Init %d failed\n", conf.freq);
diff --git a/hw/audiodev.h b/hw/audiodev.h
index 5f4a211..39a729b 100644
--- a/hw/audiodev.h
+++ b/hw/audiodev.h
@@ -1,17 +1,17 @@
/* es1370.c */
-int es1370_init (PCIBus *bus, AudioState *s);
+int es1370_init(PCIBus *bus);
/* sb16.c */
-int SB16_init (AudioState *s, qemu_irq *pic);
+int SB16_init(qemu_irq *pic);
/* adlib.c */
-int Adlib_init (AudioState *s, qemu_irq *pic);
+int Adlib_init(qemu_irq *pic);
/* gus.c */
-int GUS_init (AudioState *s, qemu_irq *pic);
+int GUS_init(qemu_irq *pic);
/* ac97.c */
-int ac97_init (PCIBus *buf, AudioState *s);
+int ac97_init(PCIBus *buf);
/* cs4231a.c */
-int cs4231a_init (AudioState *s, qemu_irq *pic);
+int cs4231a_init(qemu_irq *pic);
diff --git a/hw/cs4231a.c b/hw/cs4231a.c
index 25ad409..4212ed1 100644
--- a/hw/cs4231a.c
+++ b/hw/cs4231a.c
@@ -636,16 +636,12 @@
return 0;
}
-int cs4231a_init (AudioState *audio, qemu_irq *pic)
+int cs4231a_init (qemu_irq *pic)
{
+ AudioState *audio = AUD_init();
int i;
CSState *s;
- if (!audio) {
- lerr ("No audio state\n");
- return -1;
- }
-
s = qemu_mallocz (sizeof (*s));
s->pic = pic;
diff --git a/hw/devices.h b/hw/devices.h
index 4d4a489..7bf3063 100644
--- a/hw/devices.h
+++ b/hw/devices.h
@@ -17,9 +17,8 @@
ADS7846State *ads7846_init(qemu_irq penirq);
/* tsc210x.c */
-uWireSlave *tsc2102_init(qemu_irq pint, AudioState *audio);
-uWireSlave *tsc2301_init(qemu_irq penirq, qemu_irq kbirq,
- qemu_irq dav, AudioState *audio);
+uWireSlave *tsc2102_init(qemu_irq pint);
+uWireSlave *tsc2301_init(qemu_irq penirq, qemu_irq kbirq, qemu_irq dav);
I2SCodec *tsc210x_codec(uWireSlave *chip);
uint32_t tsc210x_txrx(void *opaque, uint32_t value, int len);
void tsc210x_set_transform(uWireSlave *chip,
diff --git a/hw/es1370.c b/hw/es1370.c
index 50f5a55..99ee208 100644
--- a/hw/es1370.c
+++ b/hw/es1370.c
@@ -1005,8 +1005,9 @@
es1370_reset (s);
}
-int es1370_init (PCIBus *bus, AudioState *audio)
+int es1370_init (PCIBus *bus)
{
+ AudioState *audio = AUD_init();
PCIES1370State *d;
ES1370State *s;
uint8_t *c;
@@ -1016,11 +1017,6 @@
return -1;
}
- if (!audio) {
- dolog ("No audio state\n");
- return -1;
- }
-
d = (PCIES1370State *) pci_register_device (bus, "ES1370",
sizeof (PCIES1370State),
-1, NULL, NULL);
diff --git a/hw/gus.c b/hw/gus.c
index 070aa99..44f3e25 100644
--- a/hw/gus.c
+++ b/hw/gus.c
@@ -250,16 +250,12 @@
return 0;
}
-int GUS_init (AudioState *audio, qemu_irq *pic)
+int GUS_init (qemu_irq *pic)
{
+ AudioState *audio = AUD_init();
GUSState *s;
struct audsettings as;
- if (!audio) {
- dolog ("No audio state\n");
- return -1;
- }
-
s = qemu_mallocz (sizeof (*s));
AUD_register_card (audio, "gus", &s->card);
diff --git a/hw/i2c.h b/hw/i2c.h
index fea2ee1..870a084 100644
--- a/hw/i2c.h
+++ b/hw/i2c.h
@@ -60,7 +60,7 @@
void max7310_gpio_out_set(i2c_slave *i2c, int line, qemu_irq handler);
/* wm8750.c */
-i2c_slave *wm8750_init(i2c_bus *bus, AudioState *audio);
+i2c_slave *wm8750_init(i2c_bus *bus);
void wm8750_reset(i2c_slave *i2c);
void wm8750_data_req_set(i2c_slave *i2c,
void (*data_req)(void *, int, int), void *opaque);
diff --git a/hw/mips_jazz.c b/hw/mips_jazz.c
index 3734aa9..5ea4bba 100644
--- a/hw/mips_jazz.c
+++ b/hw/mips_jazz.c
@@ -104,13 +104,10 @@
}
if (audio_enabled) {
- AudioState *s;
-
- s = AUD_init();
for (c = soundhw; c->name; ++c) {
if (c->enabled) {
if (c->isa) {
- c->init.init_isa(s, pic);
+ c->init.init_isa(pic);
}
}
}
diff --git a/hw/mips_malta.c b/hw/mips_malta.c
index 015c16b..083e732 100644
--- a/hw/mips_malta.c
+++ b/hw/mips_malta.c
@@ -469,12 +469,9 @@
}
if (audio_enabled) {
- AudioState *s;
-
- s = AUD_init ();
for (c = soundhw; c->name; ++c) {
if (c->enabled) {
- c->init.init_pci (pci_bus, s);
+ c->init.init_pci(pci_bus);
}
}
}
diff --git a/hw/musicpal.c b/hw/musicpal.c
index 0c47e5d..b1cece4 100644
--- a/hw/musicpal.c
+++ b/hw/musicpal.c
@@ -422,13 +422,10 @@
static i2c_interface *musicpal_audio_init(qemu_irq irq)
{
- AudioState *audio;
musicpal_audio_state *s;
i2c_interface *i2c;
int iomemtype;
- audio = AUD_init();
-
s = qemu_mallocz(sizeof(musicpal_audio_state));
s->irq = irq;
@@ -436,7 +433,7 @@
i2c->bus = i2c_init_bus();
i2c->current_addr = -1;
- s->wm = wm8750_init(i2c->bus, audio);
+ s->wm = wm8750_init(i2c->bus);
if (!s->wm)
return NULL;
i2c_set_slave_address(s->wm, MP_WM_ADDR);
diff --git a/hw/nseries.c b/hw/nseries.c
index 55df20e..8f90d53 100644
--- a/hw/nseries.c
+++ b/hw/nseries.c
@@ -252,7 +252,7 @@
qemu_irq kbirq = omap2_gpio_in_get(s->cpu->gpif, N800_TSC_KP_IRQ_GPIO)[0];
qemu_irq dav = omap2_gpio_in_get(s->cpu->gpif, N800_TSC_TS_GPIO)[0];
- s->ts.chip = tsc2301_init(penirq, kbirq, dav, 0);
+ s->ts.chip = tsc2301_init(penirq, kbirq, dav);
s->ts.opaque = s->ts.chip->opaque;
s->ts.txrx = tsc210x_txrx;
diff --git a/hw/omap2.c b/hw/omap2.c
index 1cbbf49..82ea618 100644
--- a/hw/omap2.c
+++ b/hw/omap2.c
@@ -1973,7 +1973,6 @@
omap_eac_reset(s);
#ifdef HAS_AUDIO
- /* TODO: do AUD_init globally for machine */
AUD_register_card(AUD_init(), "OMAP EAC", &s->codec.card);
iomemtype = cpu_register_io_memory(0, omap_eac_readfn,
diff --git a/hw/palm.c b/hw/palm.c
index 7c01682..785ddf1 100644
--- a/hw/palm.c
+++ b/hw/palm.c
@@ -93,14 +93,8 @@
static void palmte_microwire_setup(struct omap_mpu_state_s *cpu)
{
uWireSlave *tsc;
- AudioState *audio = 0;
-#ifdef HAS_AUDIO
- audio = AUD_init();
-#endif
-
- tsc = tsc2102_init(omap_gpio_in_get(cpu->gpio)[PALMTE_PINTDAV_GPIO],
- audio);
+ tsc = tsc2102_init(omap_gpio_in_get(cpu->gpio)[PALMTE_PINTDAV_GPIO]);
omap_uwire_attach(cpu->microwire, tsc, 0);
omap_mcbsp_i2s_attach(cpu->mcbsp1, tsc210x_codec(tsc));
diff --git a/hw/pc.c b/hw/pc.c
index fe9d644..c81a7a8 100644
--- a/hw/pc.c
+++ b/hw/pc.c
@@ -788,17 +788,13 @@
}
if (audio_enabled) {
- AudioState *s;
-
- s = AUD_init ();
for (c = soundhw; c->name; ++c) {
if (c->enabled) {
if (c->isa) {
- c->init.init_isa (s, pic);
- }
- else {
+ c->init.init_isa(pic);
+ } else {
if (pci_bus) {
- c->init.init_pci (pci_bus, s);
+ c->init.init_pci(pci_bus);
}
}
}
diff --git a/hw/pc.h b/hw/pc.h
index 50e6c39..7567d28 100644
--- a/hw/pc.h
+++ b/hw/pc.h
@@ -119,7 +119,7 @@
/* pcspk.c */
void pcspk_init(PITState *);
-int pcspk_audio_init(AudioState *, qemu_irq *pic);
+int pcspk_audio_init(qemu_irq *pic);
/* piix_pci.c */
PCIBus *i440fx_init(PCIDevice **pi440fx_state, qemu_irq *pic);
diff --git a/hw/pcspk.c b/hw/pcspk.c
index ec1d0c6..0dad161 100644
--- a/hw/pcspk.c
+++ b/hw/pcspk.c
@@ -96,15 +96,12 @@
}
}
-int pcspk_audio_init(AudioState *audio, qemu_irq *pic)
+int pcspk_audio_init(qemu_irq *pic)
{
+ AudioState *audio = AUD_init();
PCSpkState *s = &pcspk_state;
struct audsettings as = {PCSPK_SAMPLE_RATE, 1, AUD_FMT_U8, 0};
- if (!audio) {
- AUD_log(s_spk, "No audio state\n");
- return -1;
- }
AUD_register_card(audio, s_spk, &s->card);
s->voice = AUD_open_out(&s->card, s->voice, s_spk, s, pcspk_callback, &as);
diff --git a/hw/ppc_prep.c b/hw/ppc_prep.c
index 020b5b7..e078211 100644
--- a/hw/ppc_prep.c
+++ b/hw/ppc_prep.c
@@ -696,7 +696,6 @@
}
i8042_init(i8259[1], i8259[12], 0x60);
DMA_init(1);
- // AUD_init();
// SB16_init();
for(i = 0; i < MAX_FD; i++) {
diff --git a/hw/sb16.c b/hw/sb16.c
index 14f3e3d..32e6a28 100644
--- a/hw/sb16.c
+++ b/hw/sb16.c
@@ -1398,18 +1398,14 @@
return 0;
}
-int SB16_init (AudioState *audio, qemu_irq *pic)
+int SB16_init (qemu_irq *pic)
{
+ AudioState *audio = AUD_init();
SB16State *s;
int i;
static const uint8_t dsp_write_ports[] = {0x6, 0xc};
static const uint8_t dsp_read_ports[] = {0x6, 0xa, 0xc, 0xd, 0xe, 0xf};
- if (!audio) {
- dolog ("No audio state\n");
- return -1;
- }
-
s = qemu_mallocz (sizeof (*s));
s->cmd = -1;
diff --git a/hw/spitz.c b/hw/spitz.c
index e53a81d..805af35 100644
--- a/hw/spitz.c
+++ b/hw/spitz.c
@@ -740,12 +740,10 @@
i2c_bus *bus = pxa2xx_i2c_bus(cpu->i2c[0]);
#ifdef HAS_AUDIO
- AudioState *audio;
i2c_slave *wm;
- audio = AUD_init();
/* Attach a WM8750 to the bus */
- wm = wm8750_init(bus, audio);
+ wm = wm8750_init(bus);
spitz_wm8750_addr(wm, 0, 0);
pxa2xx_gpio_out_set(cpu->gpio, SPITZ_GPIO_WM,
diff --git a/hw/tsc210x.c b/hw/tsc210x.c
index 76e36d6..316156d 100644
--- a/hw/tsc210x.c
+++ b/hw/tsc210x.c
@@ -684,8 +684,7 @@
"wrong value written into Audio 1\n");
#endif
tsc2102_audio_rate_update(s);
- if (s->audio)
- tsc2102_audio_output_update(s);
+ tsc2102_audio_output_update(s);
return;
case 0x01:
@@ -729,8 +728,7 @@
"wrong value written into Power\n");
#endif
tsc2102_audio_rate_update(s);
- if (s->audio)
- tsc2102_audio_output_update(s);
+ tsc2102_audio_output_update(s);
return;
case 0x06: /* Audio Control 3 */
@@ -741,8 +739,7 @@
fprintf(stderr, "tsc2102_audio_register_write: "
"wrong value written into Audio 3\n");
#endif
- if (s->audio)
- tsc2102_audio_output_update(s);
+ tsc2102_audio_output_update(s);
return;
case 0x07: /* LCH_BASS_BOOST_N0 */
@@ -1105,7 +1102,7 @@
return 0;
}
-uWireSlave *tsc2102_init(qemu_irq pint, AudioState *audio)
+uWireSlave *tsc2102_init(qemu_irq pint)
{
TSC210xState *s;
@@ -1120,7 +1117,7 @@
s->pint = pint;
s->model = 0x2102;
s->name = "tsc2102";
- s->audio = audio;
+ s->audio = AUD_init();;
s->tr[0] = 0;
s->tr[1] = 1;
@@ -1146,8 +1143,7 @@
qemu_add_mouse_event_handler(tsc210x_touchscreen_event, s, 1,
"QEMU TSC2102-driven Touchscreen");
- if (s->audio)
- AUD_register_card(s->audio, s->name, &s->card);
+ AUD_register_card(s->audio, s->name, &s->card);
qemu_register_reset((void *) tsc210x_reset, s);
register_savevm(s->name, -1, 0,
@@ -1156,8 +1152,7 @@
return &s->chip;
}
-uWireSlave *tsc2301_init(qemu_irq penirq, qemu_irq kbirq,
- qemu_irq dav, AudioState *audio)
+uWireSlave *tsc2301_init(qemu_irq penirq, qemu_irq kbirq, qemu_irq dav)
{
TSC210xState *s;
@@ -1174,7 +1169,7 @@
s->davint = dav;
s->model = 0x2301;
s->name = "tsc2301";
- s->audio = audio;
+ s->audio = AUD_init();
s->tr[0] = 0;
s->tr[1] = 1;
@@ -1200,8 +1195,7 @@
qemu_add_mouse_event_handler(tsc210x_touchscreen_event, s, 1,
"QEMU TSC2301-driven Touchscreen");
- if (s->audio)
- AUD_register_card(s->audio, s->name, &s->card);
+ AUD_register_card(s->audio, s->name, &s->card);
qemu_register_reset((void *) tsc210x_reset, s);
register_savevm(s->name, -1, 0, tsc210x_save, tsc210x_load, s);
diff --git a/hw/wm8750.c b/hw/wm8750.c
index 7086a79..3a428c6 100644
--- a/hw/wm8750.c
+++ b/hw/wm8750.c
@@ -645,8 +645,9 @@
return 0;
}
-i2c_slave *wm8750_init(i2c_bus *bus, AudioState *audio)
+i2c_slave *wm8750_init(i2c_bus *bus)
{
+ AudioState *audio = AUD_init();
WM8750State *s = (WM8750State *)
i2c_slave_init(bus, 0, sizeof(WM8750State));
s->i2c.event = wm8750_event;
diff --git a/sysemu.h b/sysemu.h
index 9bb9fbc..25a7c32 100644
--- a/sysemu.h
+++ b/sysemu.h
@@ -241,8 +241,8 @@
int enabled;
int isa;
union {
- int (*init_isa) (AudioState *s, qemu_irq *pic);
- int (*init_pci) (PCIBus *bus, AudioState *s);
+ int (*init_isa) (qemu_irq *pic);
+ int (*init_pci) (PCIBus *bus);
} init;
};
