ALSA: hda - Drop hda_bus_template for snd_hda_bus_new()

Instead of copying from the given template, let the caller fills the
fields after creation.  This simplifies the code after all.

Signed-off-by: Takashi Iwai <tiwai@suse.de>

diff --git a/sound/pci/hda/hda_codec.c b/sound/pci/hda/hda_codec.c
index 2fe86d2e1b09dfc6f5d8f095a3fc3ad4792680f5..215bf048c668760463495d549aa9413be660e630 100644 (file)
--- a/sound/pci/hda/hda_codec.c
+++ b/sound/pci/hda/hda_codec.c
@@ -861,14 +861,12 @@ static int snd_hda_bus_dev_disconnect(struct snd_device *device)
 /**
  * snd_hda_bus_new - create a HDA bus
  * @card: the card entry
- * @temp: the template for hda_bus information
  * @busp: the pointer to store the created bus instance
  *
  * Returns 0 if successful, or a negative error code.
  */
 int snd_hda_bus_new(struct snd_card *card,
-                             const struct hda_bus_template *temp,
-                             struct hda_bus **busp)
+                   struct hda_bus **busp)
 {
        struct hda_bus *bus;
        int err;
@@ -877,11 +875,6 @@ int snd_hda_bus_new(struct snd_card *card,
                .dev_free = snd_hda_bus_dev_free,
        };
 
-       if (snd_BUG_ON(!temp))
-               return -EINVAL;
-       if (snd_BUG_ON(!temp->ops.command || !temp->ops.get_response))
-               return -EINVAL;
-
        if (busp)
                *busp = NULL;
 
@@ -892,12 +885,6 @@ int snd_hda_bus_new(struct snd_card *card,
        }
 
        bus->card = card;
-       bus->private_data = temp->private_data;
-       bus->pci = temp->pci;
-       bus->modelname = temp->modelname;
-       bus->power_save = temp->power_save;
-       bus->ops = temp->ops;
-
        mutex_init(&bus->cmd_mutex);
        mutex_init(&bus->prepare_mutex);
        INIT_LIST_HEAD(&bus->codec_list);

diff --git a/sound/pci/hda/hda_codec.h b/sound/pci/hda/hda_codec.h
index 9c8820f21f948ffb98bea96a0804d6e9b80e1aa8..5a6594884069244668d4ccabba64834e6fa0db08 100644 (file)
--- a/sound/pci/hda/hda_codec.h
+++ b/sound/pci/hda/hda_codec.h
@@ -101,15 +101,6 @@ struct hda_bus_ops {
 #endif
 };
 
-/* template to pass to the bus constructor */
-struct hda_bus_template {
-       void *private_data;
-       struct pci_dev *pci;
-       const char *modelname;
-       int *power_save;
-       struct hda_bus_ops ops;
-};
-
 /*
  * codec bus
  *
@@ -119,7 +110,6 @@ struct hda_bus_template {
 struct hda_bus {
        struct snd_card *card;
 
-       /* copied from template */
        void *private_data;
        struct pci_dev *pci;
        const char *modelname;
@@ -420,8 +410,7 @@ enum {
 /*
  * constructors
  */
-int snd_hda_bus_new(struct snd_card *card, const struct hda_bus_template *temp,
-                   struct hda_bus **busp);
+int snd_hda_bus_new(struct snd_card *card, struct hda_bus **busp);
 int snd_hda_codec_new(struct hda_bus *bus, unsigned int codec_addr,
                      struct hda_codec **codecp);
 int snd_hda_codec_configure(struct hda_codec *codec);

diff --git a/sound/pci/hda/hda_controller.c b/sound/pci/hda/hda_controller.c
index dfcb5e929f9fc9643701eb33f04aa8efd7f101d3..31ff8b55f3867b1f594d2a32187eda2f8a345599 100644 (file)
--- a/sound/pci/hda/hda_controller.c
+++ b/sound/pci/hda/hda_controller.c
@@ -1815,39 +1815,45 @@ static int get_jackpoll_interval(struct azx *chip)
        return j;
 }
 
+static struct hda_bus_ops bus_ops = {
+       .command = azx_send_cmd,
+       .get_response = azx_get_response,
+       .attach_pcm = azx_attach_pcm_stream,
+       .bus_reset = azx_bus_reset,
+#ifdef CONFIG_PM
+       .pm_notify = azx_power_notify,
+#endif
+#ifdef CONFIG_SND_HDA_DSP_LOADER
+       .load_dsp_prepare = azx_load_dsp_prepare,
+       .load_dsp_trigger = azx_load_dsp_trigger,
+       .load_dsp_cleanup = azx_load_dsp_cleanup,
+#endif
+};
+
 /* Codec initialization */
 int azx_codec_create(struct azx *chip, const char *model,
                     unsigned int max_slots,
                     int *power_save_to)
 {
-       struct hda_bus_template bus_temp;
+       struct hda_bus *bus;
        int c, codecs, err;
 
-       memset(&bus_temp, 0, sizeof(bus_temp));
-       bus_temp.private_data = chip;
-       bus_temp.modelname = model;
-       bus_temp.pci = chip->pci;
-       bus_temp.ops.command = azx_send_cmd;
-       bus_temp.ops.get_response = azx_get_response;
-       bus_temp.ops.attach_pcm = azx_attach_pcm_stream;
-       bus_temp.ops.bus_reset = azx_bus_reset;
-#ifdef CONFIG_PM
-       bus_temp.power_save = power_save_to;
-       bus_temp.ops.pm_notify = azx_power_notify;
-#endif
-#ifdef CONFIG_SND_HDA_DSP_LOADER
-       bus_temp.ops.load_dsp_prepare = azx_load_dsp_prepare;
-       bus_temp.ops.load_dsp_trigger = azx_load_dsp_trigger;
-       bus_temp.ops.load_dsp_cleanup = azx_load_dsp_cleanup;
-#endif
-
-       err = snd_hda_bus_new(chip->card, &bus_temp, &chip->bus);
+       err = snd_hda_bus_new(chip->card, &bus);
        if (err < 0)
                return err;
 
+       chip->bus = bus;
+       bus->private_data = chip;
+       bus->pci = chip->pci;
+       bus->modelname = model;
+       bus->ops = bus_ops;
+#ifdef CONFIG_PM
+       bus->power_save = power_save_to;
+#endif
+
        if (chip->driver_caps & AZX_DCAPS_RIRB_DELAY) {
                dev_dbg(chip->card->dev, "Enable delay in RIRB handling\n");
-               chip->bus->needs_damn_long_delay = 1;
+               bus->needs_damn_long_delay = 1;
        }
 
        codecs = 0;
@@ -1883,15 +1889,15 @@ int azx_codec_create(struct azx *chip, const char *model,
         */
        if (chip->driver_caps & AZX_DCAPS_SYNC_WRITE) {
                dev_dbg(chip->card->dev, "Enable sync_write for stable communication\n");
-               chip->bus->sync_write = 1;
-               chip->bus->allow_bus_reset = 1;
+               bus->sync_write = 1;
+               bus->allow_bus_reset = 1;
        }
 
        /* Then create codec instances */
        for (c = 0; c < max_slots; c++) {
                if ((chip->codec_mask & (1 << c)) & chip->codec_probe_mask) {
                        struct hda_codec *codec;
-                       err = snd_hda_codec_new(chip->bus, c, &codec);
+                       err = snd_hda_codec_new(bus, c, &codec);
                        if (err < 0)
                                continue;
                        codec->jackpoll_interval = get_jackpoll_interval(chip);