Commit ece7a36d authored by Takashi Iwai's avatar Takashi Iwai

ALSA: echoaudio: Use standard printk helpers

Convert with dev_err() and co from snd_printk(), etc.
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 2b96a7f1
...@@ -58,7 +58,8 @@ static int get_firmware(const struct firmware **fw_entry, ...@@ -58,7 +58,8 @@ static int get_firmware(const struct firmware **fw_entry,
snprintf(name, sizeof(name), "ea/%s", card_fw[fw_index].data); snprintf(name, sizeof(name), "ea/%s", card_fw[fw_index].data);
err = request_firmware(fw_entry, name, pci_device(chip)); err = request_firmware(fw_entry, name, pci_device(chip));
if (err < 0) if (err < 0)
snd_printk(KERN_ERR "get_firmware(): Firmware not available (%d)\n", err); dev_err(chip->card->dev,
"get_firmware(): Firmware not available (%d)\n", err);
#ifdef CONFIG_PM_SLEEP #ifdef CONFIG_PM_SLEEP
else else
chip->fw_cache[fw_index] = *fw_entry; chip->fw_cache[fw_index] = *fw_entry;
...@@ -563,7 +564,7 @@ static int init_engine(struct snd_pcm_substream *substream, ...@@ -563,7 +564,7 @@ static int init_engine(struct snd_pcm_substream *substream,
err = snd_pcm_lib_malloc_pages(substream, err = snd_pcm_lib_malloc_pages(substream,
params_buffer_bytes(hw_params)); params_buffer_bytes(hw_params));
if (err < 0) { if (err < 0) {
snd_printk(KERN_ERR "malloc_pages err=%d\n", err); dev_err(chip->card->dev, "malloc_pages err=%d\n", err);
spin_lock_irq(&chip->lock); spin_lock_irq(&chip->lock);
free_pipes(chip, pipe); free_pipes(chip, pipe);
spin_unlock_irq(&chip->lock); spin_unlock_irq(&chip->lock);
...@@ -1990,7 +1991,7 @@ static int snd_echo_create(struct snd_card *card, ...@@ -1990,7 +1991,7 @@ static int snd_echo_create(struct snd_card *card,
if ((chip->iores = request_mem_region(chip->dsp_registers_phys, sz, if ((chip->iores = request_mem_region(chip->dsp_registers_phys, sz,
ECHOCARD_NAME)) == NULL) { ECHOCARD_NAME)) == NULL) {
snd_echo_free(chip); snd_echo_free(chip);
snd_printk(KERN_ERR "cannot get memory region\n"); dev_err(chip->card->dev, "cannot get memory region\n");
return -EBUSY; return -EBUSY;
} }
chip->dsp_registers = (volatile u32 __iomem *) chip->dsp_registers = (volatile u32 __iomem *)
...@@ -1999,7 +2000,7 @@ static int snd_echo_create(struct snd_card *card, ...@@ -1999,7 +2000,7 @@ static int snd_echo_create(struct snd_card *card,
if (request_irq(pci->irq, snd_echo_interrupt, IRQF_SHARED, if (request_irq(pci->irq, snd_echo_interrupt, IRQF_SHARED,
KBUILD_MODNAME, chip)) { KBUILD_MODNAME, chip)) {
snd_echo_free(chip); snd_echo_free(chip);
snd_printk(KERN_ERR "cannot grab irq\n"); dev_err(chip->card->dev, "cannot grab irq\n");
return -EBUSY; return -EBUSY;
} }
chip->irq = pci->irq; chip->irq = pci->irq;
...@@ -2012,7 +2013,7 @@ static int snd_echo_create(struct snd_card *card, ...@@ -2012,7 +2013,7 @@ static int snd_echo_create(struct snd_card *card,
sizeof(struct comm_page), sizeof(struct comm_page),
&chip->commpage_dma_buf) < 0) { &chip->commpage_dma_buf) < 0) {
snd_echo_free(chip); snd_echo_free(chip);
snd_printk(KERN_ERR "cannot allocate the comm page\n"); dev_err(chip->card->dev, "cannot allocate the comm page\n");
return -ENOMEM; return -ENOMEM;
} }
chip->comm_page_phys = chip->commpage_dma_buf.addr; chip->comm_page_phys = chip->commpage_dma_buf.addr;
...@@ -2081,7 +2082,7 @@ static int snd_echo_probe(struct pci_dev *pci, ...@@ -2081,7 +2082,7 @@ static int snd_echo_probe(struct pci_dev *pci,
chip->dsp_registers_phys, chip->irq); chip->dsp_registers_phys, chip->irq);
if ((err = snd_echo_new_pcm(chip)) < 0) { if ((err = snd_echo_new_pcm(chip)) < 0) {
snd_printk(KERN_ERR "new pcm error %d\n", err); dev_err(chip->card->dev, "new pcm error %d\n", err);
snd_card_free(card); snd_card_free(card);
return err; return err;
} }
...@@ -2089,7 +2090,7 @@ static int snd_echo_probe(struct pci_dev *pci, ...@@ -2089,7 +2090,7 @@ static int snd_echo_probe(struct pci_dev *pci,
#ifdef ECHOCARD_HAS_MIDI #ifdef ECHOCARD_HAS_MIDI
if (chip->has_midi) { /* Some Mia's do not have midi */ if (chip->has_midi) { /* Some Mia's do not have midi */
if ((err = snd_echo_midi_create(card, chip)) < 0) { if ((err = snd_echo_midi_create(card, chip)) < 0) {
snd_printk(KERN_ERR "new midi error %d\n", err); dev_err(chip->card->dev, "new midi error %d\n", err);
snd_card_free(card); snd_card_free(card);
return err; return err;
} }
...@@ -2188,14 +2189,14 @@ static int snd_echo_probe(struct pci_dev *pci, ...@@ -2188,14 +2189,14 @@ static int snd_echo_probe(struct pci_dev *pci,
err = snd_card_register(card); err = snd_card_register(card);
if (err < 0) if (err < 0)
goto ctl_error; goto ctl_error;
snd_printk(KERN_INFO "Card registered: %s\n", card->longname); dev_info(card->dev, "Card registered: %s\n", card->longname);
pci_set_drvdata(pci, chip); pci_set_drvdata(pci, chip);
dev++; dev++;
return 0; return 0;
ctl_error: ctl_error:
snd_printk(KERN_ERR "new control error %d\n", err); dev_err(card->dev, "new control error %d\n", err);
snd_card_free(card); snd_card_free(card);
return err; return err;
} }
...@@ -2291,7 +2292,7 @@ static int snd_echo_resume(struct device *dev) ...@@ -2291,7 +2292,7 @@ static int snd_echo_resume(struct device *dev)
if (request_irq(pci->irq, snd_echo_interrupt, IRQF_SHARED, if (request_irq(pci->irq, snd_echo_interrupt, IRQF_SHARED,
KBUILD_MODNAME, chip)) { KBUILD_MODNAME, chip)) {
snd_echo_free(chip); snd_echo_free(chip);
snd_printk(KERN_ERR "cannot grab irq\n"); dev_err(chip->card->dev, "cannot grab irq\n");
return -EBUSY; return -EBUSY;
} }
chip->irq = pci->irq; chip->irq = pci->irq;
......
...@@ -53,7 +53,7 @@ static int wait_handshake(struct echoaudio *chip) ...@@ -53,7 +53,7 @@ static int wait_handshake(struct echoaudio *chip)
udelay(1); udelay(1);
} }
snd_printk(KERN_ERR "wait_handshake(): Timeout waiting for DSP\n"); dev_err(chip->card->dev, "wait_handshake(): Timeout waiting for DSP\n");
return -EBUSY; return -EBUSY;
} }
...@@ -149,7 +149,8 @@ static int read_sn(struct echoaudio *chip) ...@@ -149,7 +149,8 @@ static int read_sn(struct echoaudio *chip)
for (i = 0; i < 5; i++) { for (i = 0; i < 5; i++) {
if (read_dsp(chip, &sn[i])) { if (read_dsp(chip, &sn[i])) {
snd_printk(KERN_ERR "Failed to read serial number\n"); dev_err(chip->card->dev,
"Failed to read serial number\n");
return -EIO; return -EIO;
} }
} }
...@@ -184,7 +185,7 @@ static int load_asic_generic(struct echoaudio *chip, u32 cmd, short asic) ...@@ -184,7 +185,7 @@ static int load_asic_generic(struct echoaudio *chip, u32 cmd, short asic)
err = get_firmware(&fw, chip, asic); err = get_firmware(&fw, chip, asic);
if (err < 0) { if (err < 0) {
snd_printk(KERN_WARNING "Firmware not found !\n"); dev_warn(chip->card->dev, "Firmware not found !\n");
return err; return err;
} }
...@@ -247,7 +248,7 @@ static int install_resident_loader(struct echoaudio *chip) ...@@ -247,7 +248,7 @@ static int install_resident_loader(struct echoaudio *chip)
i = get_firmware(&fw, chip, FW_361_LOADER); i = get_firmware(&fw, chip, FW_361_LOADER);
if (i < 0) { if (i < 0) {
snd_printk(KERN_WARNING "Firmware not found !\n"); dev_warn(chip->card->dev, "Firmware not found !\n");
return i; return i;
} }
......
...@@ -221,7 +221,8 @@ static void snd_echo_midi_output_write(unsigned long data) ...@@ -221,7 +221,8 @@ static void snd_echo_midi_output_write(unsigned long data)
DE_MID(("Try to send %d bytes...\n", bytes)); DE_MID(("Try to send %d bytes...\n", bytes));
sent = write_midi(chip, buf, bytes); sent = write_midi(chip, buf, bytes);
if (sent < 0) { if (sent < 0) {
snd_printk(KERN_ERR "write_midi() error %d\n", sent); dev_err(chip->card->dev,
"write_midi() error %d\n", sent);
/* retry later */ /* retry later */
sent = 9000; sent = 9000;
chip->midi_full = 1; chip->midi_full = 1;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment