linux-kernel - Re: [alsa-devel] [PATCH 3/4] Add Wolfson Microelectronics WM8776 codec ALSA driver

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <20120417170431.GT6652@opensource.wolfsonmicro.com>
Date:	Tue, 17 Apr 2012 18:04:31 +0100
From:	Mark Brown <broonie@...nsource.wolfsonmicro.com>
To:	Takashi Iwai <tiwai@...e.de>
Cc:	Ondrej Zary <linux@...nbow-software.org>,
	alsa-devel@...a-project.org, linux-kernel@...r.kernel.org
Subject: Re: [alsa-devel] [PATCH 3/4] Add Wolfson Microelectronics WM8776
 codec ALSA driver

On Tue, Apr 17, 2012 at 06:52:49PM +0200, Takashi Iwai wrote:
> Mark Brown wrote:

> > This isn't just adding something into a specific driver which fails at
> > abstraction, it's adding generic code.  If it were adding something to
> > the ice17xx driver then that'd be one thing but look at the subject line
> > and location of the file...  this stuff should be buried inside the
> > driver if it's too painful to make the driver sane.

> The codes in sound/i2c are mostly oly for ice1712/ice1724 drivers
> after all...  They could be used by others, but I don't think there
> will be any more at this point.

If they're specific to that driver we should make them specific to that
driver and make sure the pain is confined there.  We really don't want
to end up going back to the bad old days of having to do per-CPU/card
drivers for CODECs because nobody had thought to abstract this stuff,
that just makes everyone miserable.

Looking at these commits I'd not expect anyone to figure out that this
isn't how we want or expect people to add generic CODEC drivers.

Download attachment "signature.asc" of type "application/pgp-signature" (837 bytes)