[alsa-devel] [PATCH v1 1/5] ALSA: fm801: propagate TUNER_ONLY bit when autodetected
Takashi Iwai
tiwai at suse.de
Mon Dec 21 19:55:08 CET 2015
On Mon, 21 Dec 2015 18:09:52 +0100,
Andy Shevchenko wrote:
>
> The commit d7ba858a7f7a (ALSA: fm801: implement TEA575x tuner autodetection)
> brings autodetection to the driver. However the autodetection algorithm misses
> the TUNER_ONLY bit if it is supplied by the user.
>
> Thus, user gets weird messages and no card registered.
>
> snd_fm801 0000:0d:01.0: detected TEA575x radio type SF64-PCR
> snd_fm801 0000:0d:01.0: AC'97 interface is busy (1)
> snd_fm801 0000:0d:01.0: AC'97 interface is busy (1)
> ...
> snd_fm801 0000:0d:01.0: AC'97 0 does not respond - RESET
> snd_fm801 0000:0d:01.0: AC'97 interface is busy (1)
> snd_fm801 0000:0d:01.0: AC'97 interface is busy (1)
> snd_fm801 0000:0d:01.0: AC'97 0 access is not valid [0x0], removing mixer.
> snd_fm801: probe of 0000:0d:01.0 failed with error -5
>
> Do a copy of TUNER_ONLY bit to be applied after autodetection is done.
>
> Fixes: d7ba858a7f7a (ALSA: fm801: implement TEA575x tuner autodetection)
> Signed-off-by: Andy Shevchenko <andriy.shevchenko at linux.intel.com>
> Cc: Ondrej Zary <linux at rainbow-software.org>
Applied all five patches now. Thanks.
Takashi
> ---
> sound/pci/fm801.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/sound/pci/fm801.c b/sound/pci/fm801.c
> index 4c9d804..6114d81 100644
> --- a/sound/pci/fm801.c
> +++ b/sound/pci/fm801.c
> @@ -1269,6 +1269,8 @@ static int snd_fm801_create(struct snd_card *card,
> return -ENODEV;
> }
> } else if ((tea575x_tuner & TUNER_TYPE_MASK) == 0) {
> + unsigned int tuner_only = tea575x_tuner & TUNER_ONLY;
> +
> /* autodetect tuner connection */
> for (tea575x_tuner = 1; tea575x_tuner <= 3; tea575x_tuner++) {
> chip->tea575x_tuner = tea575x_tuner;
> @@ -1283,6 +1285,8 @@ static int snd_fm801_create(struct snd_card *card,
> dev_err(card->dev, "TEA575x radio not found\n");
> chip->tea575x_tuner = TUNER_DISABLED;
> }
> +
> + chip->tea575x_tuner |= tuner_only;
> }
> if (!(chip->tea575x_tuner & TUNER_DISABLED)) {
> strlcpy(chip->tea.card, get_tea575x_gpio(chip)->name,
> --
> 2.6.4
>
>
More information about the Alsa-devel
mailing list