mirror of
https://github.com/torvalds/linux.git
synced 2024-11-10 22:21:40 +00:00
c5e58c4545
The previous cleanup with devres may lead to the incorrect release
orders at the probe error handling due to the devres's nature. Until
we register the card, snd_card_free() has to be called at first for
releasing the stuff properly when the driver tries to manage and
release the stuff via card->private_free().
This patch fixes it by calling snd_card_free() manually on the error
from the probe callback.
Fixes: 5bff69b364 ("ALSA: cs46xx: Allocate resources with device-managed APIs")
Cc: <stable@vger.kernel.org>
Reported-and-tested-by: Jan Engelhardt <jengelh@inai.de>
Link: https://lore.kernel.org/r/p2p1s96o-746-74p4-s95-61qo1p7782pn@vanv.qr
Link: https://lore.kernel.org/r/20220705152336.350-1-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de>
143 lines
3.6 KiB
C
143 lines
3.6 KiB
C
// SPDX-License-Identifier: GPL-2.0-or-later
|
|
/*
|
|
* The driver for the Cirrus Logic's Sound Fusion CS46XX based soundcards
|
|
* Copyright (c) by Jaroslav Kysela <perex@perex.cz>
|
|
*/
|
|
|
|
/*
|
|
NOTES:
|
|
- sometimes the sound is metallic and sibilant, unloading and
|
|
reloading the module may solve this.
|
|
*/
|
|
|
|
#include <linux/pci.h>
|
|
#include <linux/time.h>
|
|
#include <linux/init.h>
|
|
#include <linux/module.h>
|
|
#include <sound/core.h>
|
|
#include "cs46xx.h"
|
|
#include <sound/initval.h>
|
|
|
|
MODULE_AUTHOR("Jaroslav Kysela <perex@perex.cz>");
|
|
MODULE_DESCRIPTION("Cirrus Logic Sound Fusion CS46XX");
|
|
MODULE_LICENSE("GPL");
|
|
|
|
static int index[SNDRV_CARDS] = SNDRV_DEFAULT_IDX; /* Index 0-MAX */
|
|
static char *id[SNDRV_CARDS] = SNDRV_DEFAULT_STR; /* ID for this card */
|
|
static bool enable[SNDRV_CARDS] = SNDRV_DEFAULT_ENABLE_PNP; /* Enable this card */
|
|
static bool external_amp[SNDRV_CARDS];
|
|
static bool thinkpad[SNDRV_CARDS];
|
|
static bool mmap_valid[SNDRV_CARDS] = {[0 ... (SNDRV_CARDS - 1)] = 1};
|
|
|
|
module_param_array(index, int, NULL, 0444);
|
|
MODULE_PARM_DESC(index, "Index value for the CS46xx soundcard.");
|
|
module_param_array(id, charp, NULL, 0444);
|
|
MODULE_PARM_DESC(id, "ID string for the CS46xx soundcard.");
|
|
module_param_array(enable, bool, NULL, 0444);
|
|
MODULE_PARM_DESC(enable, "Enable CS46xx soundcard.");
|
|
module_param_array(external_amp, bool, NULL, 0444);
|
|
MODULE_PARM_DESC(external_amp, "Force to enable external amplifier.");
|
|
module_param_array(thinkpad, bool, NULL, 0444);
|
|
MODULE_PARM_DESC(thinkpad, "Force to enable Thinkpad's CLKRUN control.");
|
|
module_param_array(mmap_valid, bool, NULL, 0444);
|
|
MODULE_PARM_DESC(mmap_valid, "Support OSS mmap.");
|
|
|
|
static const struct pci_device_id snd_cs46xx_ids[] = {
|
|
{ PCI_VDEVICE(CIRRUS, 0x6001), 0, }, /* CS4280 */
|
|
{ PCI_VDEVICE(CIRRUS, 0x6003), 0, }, /* CS4612 */
|
|
{ PCI_VDEVICE(CIRRUS, 0x6004), 0, }, /* CS4615 */
|
|
{ 0, }
|
|
};
|
|
|
|
MODULE_DEVICE_TABLE(pci, snd_cs46xx_ids);
|
|
|
|
static int snd_card_cs46xx_probe(struct pci_dev *pci,
|
|
const struct pci_device_id *pci_id)
|
|
{
|
|
static int dev;
|
|
struct snd_card *card;
|
|
struct snd_cs46xx *chip;
|
|
int err;
|
|
|
|
if (dev >= SNDRV_CARDS)
|
|
return -ENODEV;
|
|
if (!enable[dev]) {
|
|
dev++;
|
|
return -ENOENT;
|
|
}
|
|
|
|
err = snd_devm_card_new(&pci->dev, index[dev], id[dev], THIS_MODULE,
|
|
sizeof(*chip), &card);
|
|
if (err < 0)
|
|
return err;
|
|
chip = card->private_data;
|
|
err = snd_cs46xx_create(card, pci,
|
|
external_amp[dev], thinkpad[dev]);
|
|
if (err < 0)
|
|
goto error;
|
|
card->private_data = chip;
|
|
chip->accept_valid = mmap_valid[dev];
|
|
err = snd_cs46xx_pcm(chip, 0);
|
|
if (err < 0)
|
|
goto error;
|
|
#ifdef CONFIG_SND_CS46XX_NEW_DSP
|
|
err = snd_cs46xx_pcm_rear(chip, 1);
|
|
if (err < 0)
|
|
goto error;
|
|
err = snd_cs46xx_pcm_iec958(chip, 2);
|
|
if (err < 0)
|
|
goto error;
|
|
#endif
|
|
err = snd_cs46xx_mixer(chip, 2);
|
|
if (err < 0)
|
|
goto error;
|
|
#ifdef CONFIG_SND_CS46XX_NEW_DSP
|
|
if (chip->nr_ac97_codecs ==2) {
|
|
err = snd_cs46xx_pcm_center_lfe(chip, 3);
|
|
if (err < 0)
|
|
goto error;
|
|
}
|
|
#endif
|
|
err = snd_cs46xx_midi(chip, 0);
|
|
if (err < 0)
|
|
goto error;
|
|
err = snd_cs46xx_start_dsp(chip);
|
|
if (err < 0)
|
|
goto error;
|
|
|
|
snd_cs46xx_gameport(chip);
|
|
|
|
strcpy(card->driver, "CS46xx");
|
|
strcpy(card->shortname, "Sound Fusion CS46xx");
|
|
sprintf(card->longname, "%s at 0x%lx/0x%lx, irq %i",
|
|
card->shortname,
|
|
chip->ba0_addr,
|
|
chip->ba1_addr,
|
|
chip->irq);
|
|
|
|
err = snd_card_register(card);
|
|
if (err < 0)
|
|
goto error;
|
|
|
|
pci_set_drvdata(pci, card);
|
|
dev++;
|
|
return 0;
|
|
|
|
error:
|
|
snd_card_free(card);
|
|
return err;
|
|
}
|
|
|
|
static struct pci_driver cs46xx_driver = {
|
|
.name = KBUILD_MODNAME,
|
|
.id_table = snd_cs46xx_ids,
|
|
.probe = snd_card_cs46xx_probe,
|
|
#ifdef CONFIG_PM_SLEEP
|
|
.driver = {
|
|
.pm = &snd_cs46xx_pm,
|
|
},
|
|
#endif
|
|
};
|
|
|
|
module_pci_driver(cs46xx_driver);
|