linux-kernel - Re: [Alsa-devel] add support for syba 7.1 surround sound card

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <519B3571.3070601@ladisch.de>
Date:	Tue, 21 May 2013 10:50:57 +0200
From:	Clemens Ladisch <clemens@...isch.de>
To:	good1.2guy@...il.com
CC:	linux-kernel@...r.kernel.org, alsa-devel@...a-project.org
Subject: Re: [Alsa-devel] add support for syba 7.1 surround sound card

good1.2guy@...il.com wrote:
> Summary: add support for syba 7.1 surround sound card
> Keywords: syba via VT1723 Tremor
> Kernel: 3.8.8-203.fc18.x86_64

What are these three lines for?

> add support for syba 7.1 surround sound card
> actually the card only has 6 dmas, so it is really only 5.1

The website shows a daughterboard.

> added code to force_rate setting to not fail EBUSY when the dmas are running.
> suggest using force_rate with pulseaudio to avoid loops with error logging.

This sounds as if it has nothing to do with this particular SYBA card
but affects all VT1723 cards.  Please move it into a separate patch.

And there should be no reason for a module parameter; if this is a bug
in the driver and/or in PulseAudio, it should be just fixed.

> Signed-off-by: psmith2004@...oo.com

Real names only.

Please read Documentation/SubmittingPatches.

> +static void syba_quirk_open(struct snd_ice1712 *ice)
> +{
> +	unsigned long flags;

It's common to have an empty line here.

> +	spin_lock_irqsave(&ice->reg_lock, flags);
> +	outb(0xff, ICEREG1724(ice, AC97_CFG));
> +	udelay(500);
> +	outb(ice->eeprom.data[ICE_EEP2_ACLINK], ICEREG1724(ice, AC97_CFG));
> +	spin_unlock_irqrestore(&ice->reg_lock, flags);
> +}

What is the reason for this quirk?
Is it possible that this might be needed for other cards using a AC'97
codec?


Regards,
Clemens
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/